Hi Simon, On Wed, Aug 26, 2015 at 9:29 AM, Bin Meng <bmeng...@gmail.com> wrote: > Hi Joe, > > On Wed, Aug 26, 2015 at 2:43 AM, Joe Hershberger > <joe.hershber...@gmail.com> wrote: >> Hi Bin, >> >> On Tue, Aug 25, 2015 at 2:22 AM, Bin Meng <bmeng...@gmail.com> wrote: >>> With driver model, board_eth_init() or cpu_eth_init() is not needed. >>> Remove the call to these in eth_common_init(). >> >> I'm pretty sure Simon needed this when he ported some allwinner board >> originally. >> >> 3bc427006ac8d0661169ed771b3cac7e86f960e8 (dm: net: Use existing >> Ethernet init for driver model) >> > > I think my patch does not break Simon's. My patch only comments out > the call to board_eth_init() or cpu_eth_init() which is not needed for > driver model. Other stuff in eth_common_init() is still there. In > fact, my patch series also needs phy_init() call (required by pch_gbe > driver). >
Would you confirm this does not break your boards? Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot