On Thu, Mar 22, 2018 at 5:49 AM, Calvin Johnson calvin.johnson@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