Re: [PATCH net-next v6 2/2] net: dsa: mv88e6xxx: add support for credit based shaper

[Paolo Abeni]

On 6/4/26 3:44 PM, Cedric Jehasse via B4 Relay wrote:
> @@ -5679,6 +5686,27 @@ static const struct mv88e6xxx_ops mv88e6393x_ops = {
>  	.tcam_ops = &mv88e6393_tcam_ops,
>  };
>  
> +static const struct mv88e6xxx_qav_info mv88e6352_qav_info = {
> +	.rate_unit = 32,
> +	.rate_mask = GENMASK(14, 0),
> +	.hilimit_mask = GENMASK(14, 0),
> +	.queue_mask = GENMASK(3, 0),
> +};
> +
> +static const struct mv88e6xxx_qav_info mv88e6341_qav_info = {
> +	.rate_unit = 64,
> +	.rate_mask = GENMASK(15, 0),
> +	.hilimit_mask = GENMASK(14, 0),
> +	.queue_mask = GENMASK(3, 0),
> +};
> +
> +static const struct mv88e6xxx_qav_info mv88e6390_qav_info = {
> +	.rate_unit = 64,
> +	.rate_mask = GENMASK(15, 0),
> +	.hilimit_mask = GENMASK(13, 0),
> +	.queue_mask = GENMASK(7, 0),
> +};

Sashiko mentions that the above masks are not consistent with the switch
datasheet.

I'm quite surprised the LLM could check such fine grain detail, could
you please double check if the comment makes sense:

https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260604-net-next-mv88e6xxx-cbs-v6-0-b68c01142d1e%40luminex.be

?

Thanks,

Paolo