Re: [PATCH 0/2] net: phy: Make phy_interface_is_sgmii|rgmii a switch statement

2023-04-13 Thread Nishanth Menon
On 13:07-20230413, Nishanth Menon wrote: > Originally reported by Tom[1], turned out to be that recent commit > 75d28899e3e9 ("net: phy: Synchronize PHY interface modes with Linux") > reordered the enum definitions which in turn broke the range checks. > > we are left with two options: > a) check

[PATCH 0/2] net: phy: Make phy_interface_is_sgmii|rgmii a switch statement

2023-04-13 Thread Nishanth Menon
Originally reported by Tom[1], turned out to be that recent commit 75d28899e3e9 ("net: phy: Synchronize PHY interface modes with Linux") reordered the enum definitions which in turn broke the range checks. we are left with two options: a) check against explicit values to help reuse as much as