[U-Boot] [PATCH 1/2 v3] net: introduce MDIO DM class for MDIO devices
Alex Marginean
alexm.osslist at gmail.com
Tue Jun 11 07:17:43 UTC 2019
+Ken,
Hi Joe,
On 6/10/2019 11:25 PM, Joe Hershberger wrote:
> On Mon, Jun 3, 2019 at 11:11 AM Alex Marginean <alexm.osslist at gmail.com> wrote:
>>
>> Adds UCLASS_MDIO DM class supporting MDIO buses that are probed as
>> stand-alone devices. Useful in particular for systems that support
>> DM_ETH and have a stand-alone MDIO hardware block shared by multiple
>> Ethernet interfaces.
>>
>> Signed-off-by: Alex Marginean <alexm.osslist at gmail.com>
>> ---
>>
>> Changes in v2:
>> - fixed several comments using wrong API names
>> - dropped dm_ from names of internal functions that don't use udevice *
>> - fixed UCLASS driver name
>> - added missing mdio_unregister in dm_mdio_pre_remove
>> - added a comment on why spaces in names aren't ok
>> - added a comment on how static mdio_read/_write/_reset functions
>> are used
>> Changes in v3:
>> - none
>
>
> Not sure if you already noticed this [1] or not, but there may be
> something there that you want to incorporate or maybe not.
>
> Cheers,
> -Joe
>
> [1] - https://patchwork.ozlabs.org/patch/939726/
>
I didn't notice it, thanks for pointing it out!
Apart from the obvious overlap of adding UCLASS_MDIO and code like
_post_probe they seem to deal with different needs.
Ken, can you please take a look at the patch I sent? It has a
wrapper over phy_connect, but provides no helpers on how the caller
would get the PHY ADDR. Do you want to try pulling the API you add on
top of the patch I sent, or do you want me to try? It looks like it
would work with minimal effort.
Thank you!
Alex
More information about the U-Boot
mailing list