OK. I'll look at it this week. Thanks, Michal
> Hi Michal, > > You submitted this so long ago that this driver should be in the tree > right now. Unfortunately, I didn't give a thorough enough review the > first time and I think it would be wrong to pull it in right now without > addressing some more things. If you take care of this, I promise to put > it in the front of the queue for the next release: > > mon...@monstr.eu wrote: >> From: Michal Simek <mon...@monstr.eu> >> >> --- >> > This '---' line ensures that nothing below it (including your SOB) goes > in the commit. Please remove it, add a meaningful commit message and > move the next few lines below the next '---' line. Also, the "version > 2" in the title should be in square brackets so it get stripped from the > commit title. >> v2: Coding style cleanup, use debug() instead of printf >> in debug messages >> >> v1: Initial version >> >> Signed-off-by: Michal Simek <mon...@monstr.eu> >> --- >> drivers/net/Makefile | 1 + >> drivers/net/xilinx_ll_temac.c | 540 >> +++++++++++++++++++++++++++++++++++++++++ >> 2 files changed, 541 insertions(+), 0 deletions(-) >> create mode 100644 drivers/net/xilinx_ll_temac.c >> >> diff --git a/drivers/net/Makefile b/drivers/net/Makefile >> index 631336a..1e80698 100644 >> --- a/drivers/net/Makefile >> +++ b/drivers/net/Makefile >> @@ -68,6 +68,7 @@ COBJS-$(CONFIG_ULI526X) += uli526x.o >> COBJS-$(CONFIG_VSC7385_ENET) += vsc7385.o >> COBJS-$(CONFIG_XILINX_EMAC) += xilinx_emac.o >> COBJS-$(CONFIG_XILINX_EMACLITE) += xilinx_emaclite.o >> +COBJS-$(CONFIG_XILINX_LL_TEMAC) += xilinx_ll_temac.o >> COBJS-$(CONFIG_SH_ETHER) += sh_eth.o >> >> > This doesn't apply any more. Not your fault :) > > > I get the following whitespace errors: > > Applying net: Add Xilinx LL Temac driver > .dotest/patch:415: trailing whitespace. > > .dotest/patch:419: trailing whitespace. > > .dotest/patch:424: trailing whitespace. > > > All of the above mentioned issues are ones that I could easily deal > with, but one thing that really does need to change is that you need to > use the CONFIG_NET_MULTI API. In other words, your driver should have a > single initialize() function (prototyped in include/netdev.h), and an > eth_device struct that gets registered. All your access functions > (eth_init, eth_send, eth_recv etc.) will be static and pointed to by the > eth_device struct. Most drivers are already this way. > > Other than that, it's fine. Sorry again for taking so very long. > > regards, > Ben > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
