Re: [U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-09 Thread Ilya Yanok
Hello All, Sorry guys, it looks like I posted the wrong version. Please discard [v4] patch. I'll post [v5] version soon. Regards, Ilya. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-09 Thread Ilya Yanok
Hi Wolfgang, Wolfgang Denk wrote: +u32 TX_JUMBO_CNT; +u32 TX_BYTE_CNT; +} __attribute__((packed)); >>> Why is this packed? All fields are 32-bit aligned already. >>> >> Generally, I prefer declaring such structures with ((packed)) regardless >>

Re: [U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-09 Thread Wolfgang Denk
Dear Ilya, In message <49902403.2010...@emcraft.com> you wrote: > ... > >> +u32 TX_JUMBO_CNT; > >> +u32 TX_BYTE_CNT; > >> +} __attribute__((packed)); > >> > > Why is this packed? All fields are 32-bit aligned already. > > Generally, I prefer declaring such structures with ((packed))

Re: [U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-09 Thread Ilya Yanok
Hi Ben, thanks for review. Ben Warren wrote: >> @@ -69,6 +70,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_SH_ETHER) += sh_eth.o >

Re: [U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-08 Thread Ben Warren
Hi Ilya, This looks much better. Ilya Yanok wrote: > Driver for Dave DNET ethernet controller (used on Dave/DENX > QongEVB-LITE board). > > Signed-off-by: Ilya Yanok > > --- > drivers/net/Makefile |2 + > drivers/net/dnet.c | 396 > ++

[U-Boot] [PATCH 2/3][v3] dnet: driver for Dave DNET ethernet controller

2009-02-07 Thread Ilya Yanok
Driver for Dave DNET ethernet controller (used on Dave/DENX QongEVB-LITE board). Signed-off-by: Ilya Yanok --- drivers/net/Makefile |2 + drivers/net/dnet.c | 396 ++ drivers/net/dnet.h | 166 + include/netdev.h |