Re: [PATCH net-next V5 4/5] net: lan743x: Migrate phylib to phylink

[Maxime Chevallier <maxime.chevallier@bootlin.com>]

Hello Raju,

On Wed, 4 Sep 2024 14:36:44 +0530
Raju Lakkaraju <Raju.Lakkaraju@microchip.com> wrote:

> Migrate phy support from phylib to phylink.
> 
> Signed-off-by: Raju Lakkaraju <Raju.Lakkaraju@microchip.com>
> ---
> Change List:                                                                    
> ============                                                                    
> V4 -> V5:                                                                       
>   - Remove the fixed_phy_unregister( ) function. Not require                    
> V3 -> V4:                                                                       
>   - Add the Fixed-link by include the Russell King patch
>   - Change lan743x_phylink_create( ) argument from netdev to adapter
> V2 -> V3:                                                                       
>   - Remove the unwanted parens in each of these if() sub-blocks                 
>   - Replace "to_net_dev(config->dev)" with "netdev".                            
>   - Add GMII_ID/RGMII_TXID/RGMII_RXID in supported_interfaces                   
>   - Fix the lan743x_phy_handle_exists( ) return type
> V1 -> V2:                                                                       
>   - Split the PHYLINK and SFP changes in 2 different patch series                                                                  

Besides what Andrew pointed out, I don't see any obvious problem.

With Andrew's comments taken into account,

Reviewed-by: Maxime Chevallier <maxime.chevallier@bootlin.com>

Thanks,

Maxime