[PATCH u-boot-net v3 14/14] net: phy: don't require PHY interface mode during PHY creation

Vladimir Oltean olteanv at gmail.com
Wed Apr 6 15:38:00 CEST 2022


On Tue, Mar 29, 2022 at 10:08:45PM +0200, Marek Behún wrote:
> From: Marek Behún <marek.behun at nic.cz>
> 
> Currently we require PHY interface mode to be known when
> finding/creating the PHY - the functions
>   * phy_device_create()
>   * create_phy_by_mask()
>   * search_for_existing_phy()
>   * get_phy_device_by_mask()
>   * phy_find_by_mask() (this is the only one global)
> all require the interface parameter, but the only thing done with it is
> that it is assigned to phydev->interface.
> 
> This makes it impossible to find a PHY device without overwriting the
> set mode.
> 
> Since the interface mode is not used during .probe() and should be used
> at first in .config(), drop the interface parameter from these
> functions. Make the default value of phydev->interface (in
> phy_device_create()) to be PHY_INTERFACE_MODE_NA. Move the interface
> parameter to phy_connect_dev(), where it should be.
> 
> Change all occurrences treewide. In occurrences where we don't call
> phy_connect_dev() for some reason (they only configure the PHY without
> connecting it to an ethernet controller), set
>   phydev->interface = value from phy_find_by_mask call.
> 
> Signed-off-by: Marek Behún <marek.behun at nic.cz>
> Reviewed-by: Ramon Fried <rfried.dev at gmail.com>
> ---

Reviewed-by: Vladimir Oltean <vladimir.oltean at nxp.com>


More information about the U-Boot mailing list