[U-Boot] [PATCH V2 4/5] net: fec: sharing MDIO for two enet controllers

Joe Hershberger joe.hershberger at ni.com
Thu Mar 22 18:22:50 UTC 2018


On Thu, Mar 22, 2018 at 5:49 AM, Calvin Johnson <calvin.johnson at nxp.com> wrote:
> Hi Joe,
>
>> >>>
>> >>> No, I think we can do this with adding new DM MDIO similar to DM PHY
>> which
>> >>> recently done. May be some sort of efforts but it is permanent.
>> >> We do not have that driver now, so could we first have this patch? When
>> >> DM MDIO ready, this piece code could be removed then?
>> >
>> > ie. up to Joe. Honestly this macro become removed in future, my point
>> > here is why we need to maintain dead macro instead of adding proper
>> > maintainable stuff. I'm pretty sure adding DM_MDIO is straight forward
>> > and as of now just add what we need and rest will implement future.
>> > You may become victim to others to move DM_ETH as soon as possible :)
>>
>> It would be ideal if you wanted to implement DM MDIO, but I can also
>> appreciate that this is not already there for you to use. As I
>> commented when I acked this, the I'm OK with this approach at this
>> time due to the state of the DM support in eth.
>>
>> As a side note, maybe moving other boards that use this NIC to DM_ETH
>> and removing non-DM support would be a better cleanup to start with.
>
> Is someone already working on DM MDIO?
> Are there any patches already submitted for this? I couldn't find any.

I have a rough start on it, but I'm not happy with it so far.

-Joe


More information about the U-Boot mailing list