Re: [PATCH v2 net-next] net: dsa: Allow only up to two HSR HW offloaded ports for KSZ9477

From: Andrew Lunn
Date: Wed Jun 19 2024 - 10:27:50 EST


> + /* KSZ9477 can only perform HSR offloading for up to two ports */
> + if (hweight8(dev->hsr_ports) >= 2) {
> + NL_SET_ERR_MSG_MOD(extack,
> + "Cannot offload more than two ports - use software HSR");

Bit of a nit pick. 'use' suggests it is a directive, you need to
changing the configuration to make it work. 'using' would indicate
nothing needs changing, it has decided to use software HSR for you.

Other than that:

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>

Andrew