Hi Ahmad. On Wed, Aug 22, 2018 at 03:14:57PM +0200, Ahmad Fatoum wrote: > Hello Sam, > > On 08/21/2018 09:46 PM, Sam Ravnborg wrote: > > But macb.txt or the general bindigns do not mention > > a mdio child. > > And the Linux macb_main.c do not support a "mdio" node either. > > So my best guess is that the "mdio" support in macb was accidently > > ported over from fec-imx when adding DT support. > [snip] > > Considering the above I will do the following: > > a) Remove "mdio" support from macb.c > > While submitting a macb patch to linux-netdev, I was asked [1] to add > support for a "mdio" subtree in the macb binding. I intend to submit > it when net-next opens for inclusion into Linux v4.20. > That way, macb would look at phys in the list of its children as well > as the children of its "mdio" child. > > Just a heads up to avoid removing functionality that would've to be > reinstated later on... > > [1]: https://www.spinics.net/lists/netdev/msg519196.html Thanks for bringing this to my attention. I will leave the code as-is. Sam _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox