Subject: Re: [PATCH v3 net-next 2/6] net: phy: realtek: add get_rate_matching() for rtl822xb PHYs

From: "Russell King (Oracle)" <[email-masked]>
Date: Tue, 2 Apr 2024 20:24:44 +0100

On Tue, Apr 02, 2024 at 07:58:44AM +0200, Eric Woudstra wrote:
> +	switch (val & RTL822X_VND1_SERDES_OPTION_MODE_MASK) {
> +	case RTL822X_VND1_SERDES_OPTION_MODE_2500BASEX:
> +		return RATE_MATCH_PAUSE;
> +	/* case RTL822X_VND1_SERDES_OPTION_MODE_2500BASEX_SGMII: */
> +	default:
> +		return RATE_MATCH_NONE;
> +	}

Nit: Is this actually useful to have as a switch statement?

In any case,

Reviewed-by: Russell King (Oracle) <[email-masked]>

Thanks!

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!