* [net] net: dsa: felix: convert TAS link speed based on phylink speed
@ 2020-09-24 1:57 Xiaoliang Yang
2020-09-25 3:02 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Xiaoliang Yang @ 2020-09-24 1:57 UTC (permalink / raw)
To: xiaoliang.yang_1, davem, netdev, linux-kernel, allan.nielsen,
joergen.andreasen, UNGLinuxDriver, alexandru.marginean, po.liu,
claudiu.manoil, vladimir.oltean, leoyang.li
state->speed holds a value of 10, 100, 1000 or 2500, but
QSYS_TAG_CONFIG_LINK_SPEED expects a value of 0, 1, 2, 3. So convert the
speed to a proper value.
Fixes: de143c0e274b ("net: dsa: felix: Configure Time-Aware Scheduler via taprio offload")
Signed-off-by: Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
drivers/net/dsa/ocelot/felix_vsc9959.c | 22 +++++++++++++++++++++-
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/drivers/net/dsa/ocelot/felix_vsc9959.c b/drivers/net/dsa/ocelot/felix_vsc9959.c
index 6855c94256f8..36db631a55e6 100644
--- a/drivers/net/dsa/ocelot/felix_vsc9959.c
+++ b/drivers/net/dsa/ocelot/felix_vsc9959.c
@@ -1284,8 +1284,28 @@ void vsc9959_mdio_bus_free(struct ocelot *ocelot)
static void vsc9959_sched_speed_set(struct ocelot *ocelot, int port,
u32 speed)
{
+ u8 tas_speed;
+
+ switch (speed) {
+ case SPEED_10:
+ tas_speed = OCELOT_SPEED_10;
+ break;
+ case SPEED_100:
+ tas_speed = OCELOT_SPEED_100;
+ break;
+ case SPEED_1000:
+ tas_speed = OCELOT_SPEED_1000;
+ break;
+ case SPEED_2500:
+ tas_speed = OCELOT_SPEED_2500;
+ break;
+ default:
+ tas_speed = OCELOT_SPEED_1000;
+ break;
+ }
+
ocelot_rmw_rix(ocelot,
- QSYS_TAG_CONFIG_LINK_SPEED(speed),
+ QSYS_TAG_CONFIG_LINK_SPEED(tas_speed),
QSYS_TAG_CONFIG_LINK_SPEED_M,
QSYS_TAG_CONFIG, port);
}
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [net] net: dsa: felix: convert TAS link speed based on phylink speed
2020-09-24 1:57 [net] net: dsa: felix: convert TAS link speed based on phylink speed Xiaoliang Yang
@ 2020-09-25 3:02 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2020-09-25 3:02 UTC (permalink / raw)
To: xiaoliang.yang_1
Cc: netdev, linux-kernel, allan.nielsen, joergen.andreasen,
UNGLinuxDriver, alexandru.marginean, po.liu, claudiu.manoil,
vladimir.oltean, leoyang.li
From: Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
Date: Thu, 24 Sep 2020 09:57:46 +0800
> state->speed holds a value of 10, 100, 1000 or 2500, but
> QSYS_TAG_CONFIG_LINK_SPEED expects a value of 0, 1, 2, 3. So convert the
> speed to a proper value.
>
> Fixes: de143c0e274b ("net: dsa: felix: Configure Time-Aware Scheduler via taprio offload")
> Signed-off-by: Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
> Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Applied and queued up for -stable.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-25 3:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-24 1:57 [net] net: dsa: felix: convert TAS link speed based on phylink speed Xiaoliang Yang
2020-09-25 3:02 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).