Hi Oleksij,
On Fri, 2024-04-05 at 11:52 +0200, Oleksij Rempel wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you
> know the content is safe
>
> I tested ETS support on KSZ9893, so it should work other KSZ989X
> variants too.
>
> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
nitpick: For the source code I infer ets support is support for all
switches except ksz8x family(KSZ8795/4, KSZ8765 and KSZ8830).
Instead of .tc_ets_supported, can we add in the ksz_tc_setup_qdisc_ets(
)
like
if( ksz8_family())
return -EOPNOTSUPP;
Otherwise
Acked-by: Arun Ramadoss <arun.ramadoss@microchip.com>
> ---
> drivers/net/dsa/microchip/ksz_common.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/net/dsa/microchip/ksz_common.c
> b/drivers/net/dsa/microchip/ksz_common.c
> index 840b17b8507e1..dc96931e62da8 100644
> --- a/drivers/net/dsa/microchip/ksz_common.c
> +++ b/drivers/net/dsa/microchip/ksz_common.c
> @@ -1378,6 +1378,7 @@ const struct ksz_chip_data ksz_switch_chips[] =
> {
> .port_nirqs = 2,
> .num_tx_queues = 4,
> .num_ipvs = 8,
> + .tc_ets_supported = true,
> .ops = &ksz9477_dev_ops,
> .mib_names = ksz9477_mib_names,
> .mib_cnt = ARRAY_SIZE(ksz9477_mib_names),
> @@ -1411,6 +1412,7 @@ const struct ksz_chip_data ksz_switch_chips[] =
> {
> .port_nirqs = 2,
> .num_tx_queues = 4,
> .num_ipvs = 8,
> + .tc_ets_supported = true,
> .ops = &ksz9477_dev_ops,
> .mib_names = ksz9477_mib_names,
> .mib_cnt = ARRAY_SIZE(ksz9477_mib_names),
> @@ -1442,6 +1444,7 @@ const struct ksz_chip_data ksz_switch_chips[] =
> {
> .port_nirqs = 2,
> .num_tx_queues = 4,
> .num_ipvs = 8,
> + .tc_ets_supported = true,
> .ops = &ksz9477_dev_ops,
> .mib_names = ksz9477_mib_names,
> .mib_cnt = ARRAY_SIZE(ksz9477_mib_names),
> --
> 2.39.2
>