Re: [PATCH] RFC: net: phy: of phys probe/reset issue

2020-10-28 Thread Florian Fainelli
On 10/28/2020 12:32 PM, Grygorii Strashko wrote: > hi Andrew, > > On 23/10/2020 23:10, Andrew Lunn wrote: >>> Yes there is: have your Ethernet PHY compatible string be of the form >>> "ethernet." and then there is no need for such hacking. >>> of_get_phy_id() will parse that compatible

Re: [PATCH] RFC: net: phy: of phys probe/reset issue

2020-10-28 Thread Grygorii Strashko
hi Andrew, On 23/10/2020 23:10, Andrew Lunn wrote: Yes there is: have your Ethernet PHY compatible string be of the form "ethernet." and then there is no need for such hacking. of_get_phy_id() will parse that compatible and that will trigger of_mdiobus_register_phy() to take the phy_devi

Re: [PATCH] RFC: net: phy: of phys probe/reset issue

2020-10-23 Thread Andrew Lunn
> Yes there is: have your Ethernet PHY compatible string be of the form > "ethernet." and then there is no need for such hacking. > of_get_phy_id() will parse that compatible and that will trigger > of_mdiobus_register_phy() to take the phy_device_create() path. Yep. That does seem like th

Re: [PATCH] RFC: net: phy: of phys probe/reset issue

2020-10-23 Thread Florian Fainelli
On 10/23/20 10:47 AM, Grygorii Strashko wrote: > Hi All, > > The main intention of this mail is to trigger discussion to find a proper > solution. All code is hackish and based on v5.9. > > Problem statement: > > There is an issue observed with MDIO OF PHYs discover/reset sequence in > case PHY

[PATCH] RFC: net: phy: of phys probe/reset issue

2020-10-23 Thread Grygorii Strashko
Hi All, The main intention of this mail is to trigger discussion to find a proper solution. All code is hackish and based on v5.9. Problem statement: There is an issue observed with MDIO OF PHYs discover/reset sequence in case PHY has reset line with default state is (1). In this case, when Linu