Re: [PATCH v2 net-next 02/13] net: dsa: felix: set proper link speed in felix_phylink_mac_config

[Florian Fainelli]

On 5/30/2020 4:51 AM, Vladimir Oltean wrote:
> From: Vladimir Oltean <vladimir.olt...@nxp.com>
> 
> state->speed holds a value of 10, 100, 1000 or 2500, but
> SYS_MAC_FC_CFG_FC_LINK_SPEE and DEV_CLOCK_CFG_LINK_SPEED expect a value
> in the range 0, 1, 2 or 3.
> 
> Even truncated to 2 bits, we are still writing incorrect values to the
> registers, but for some reason Felix still works.
> 
> On Seville (which we're introducing now), however, we need to set
> correct values for the link speed into the MAC registers. Do that now.
> 
> Signed-off-by: Vladimir Oltean <vladimir.olt...@nxp.com>

Reviewed-by: Florian Fainelli <f.faine...@gmail.com>
-- 
Florian