Hi Shinya, On Sat, Jun 7, 2008 at 9:16 AM, Shinya Kuribayashi <[EMAIL PROTECTED]> wrote: > net: Conditional COBJS inclusino of remainings >
s/inclusino/inclusion/ "remainings" isn't a word. Please use something like "remaining Ethernet controllers" > From: Shinya Kuribayashi <[EMAIL PROTECTED]> > > Signed-off-by: Shinya Kuribayashi <[EMAIL PROTECTED]> > --- > > drivers/net/3c589.c | 5 ----- > drivers/net/Makefile | 26 +++++++++++++------------- > drivers/net/cs8900.c | 8 -------- > drivers/net/dc2114x.c | 6 ------ > drivers/net/dm9000x.c | 3 --- > drivers/net/enc28j60.c | 3 --- > drivers/net/ks8695eth.c | 4 ---- > drivers/net/macb.c | 5 ----- > drivers/net/netconsole.c | 5 ----- > drivers/net/pcnet.c | 4 ---- > drivers/net/plb2800_eth.c | 6 ------ > drivers/net/s3c4510b_eth.c | 5 ----- > drivers/net/tsi108_eth.c | 5 ----- > drivers/net/uli526x.c | 4 ---- > 14 files changed, 13 insertions(+), 76 deletions(-) > > > diff --git a/drivers/net/3c589.c b/drivers/net/3c589.c > index 3f1e770..0cf8dff 100644 > --- a/drivers/net/3c589.c > +++ b/drivers/net/3c589.c > @@ -26,8 +26,6 @@ > #include <command.h> > #include <net.h> > > -#ifdef CONFIG_DRIVER_3C589 > - > #include "3c589.h" > > > @@ -514,6 +512,3 @@ int eth_send(volatile void *packet, int length) { > > return length; > } > - > - > -#endif /* CONFIG_DRIVER_3C589 */ > diff --git a/drivers/net/Makefile b/drivers/net/Makefile > index ca30952..84be288 100644 > --- a/drivers/net/Makefile > +++ b/drivers/net/Makefile > @@ -25,21 +25,21 @@ include $(TOPDIR)/config.mk > > LIB := $(obj)libnet.a > > -COBJS-y += 3c589.o > +COBJS-$(CONFIG_DRIVER_3C589) += 3c589.o > COBJS-$(CONFIG_BCM570x) += bcm570x.o bcm570x_autoneg.o 5701rls.o > COBJS-$(CONFIG_BFIN_MAC) += bfin_mac.o > -COBJS-y += cs8900.o > -COBJS-y += dc2114x.o > -COBJS-y += dm9000x.o > +COBJS-$(CONFIG_DRIVER_CS8900) += cs8900.o > +COBJS-$(CONFIG_TULIP) += dc2114x.o > +COBJS-$(CONFIG_DRIVER_DM9000) += dm9000x.o > COBJS-$(CONFIG_E1000) += e1000.o > COBJS-$(CONFIG_EEPRO100) += eepro100.o > -COBJS-y += enc28j60.o > +COBJS-$(CONFIG_ENC28J60) += enc28j60.o > COBJS-$(CONFIG_FSLDMAFEC) += fsl_mcdmafec.o > COBJS-$(CONFIG_GRETH) += greth.o > COBJS-$(CONFIG_INCA_IP_SWITCH) += inca-ip_sw.o > -COBJS-y += ks8695eth.o > +COBJS-$(CONFIG_DRIVER_KS8695ETH) += ks8695eth.o > COBJS-$(CONFIG_DRIVER_LAN91C96) += lan91c96.o > -COBJS-y += macb.o > +COBJS-$(CONFIG_MACB) += macb.o > COBJS-$(CONFIG_MCFFEC) += mcffec.o > COBJS-$(CONFIG_NATSEMI) += natsemi.o > ifeq ($(CONFIG_DRIVER_NE2000),y) > @@ -47,22 +47,22 @@ COBJS-y += ne2000.o > COBJS-$(CONFIG_DRIVER_AX88796L) += ax88796.o > endif > COBJS-$(CONFIG_DRIVER_NETARMETH) += netarm_eth.o > -COBJS-y += netconsole.o > +COBJS-$(CONFIG_NETCONSOLE) += netconsole.o > COBJS-$(CONFIG_DRIVER_NS7520_ETHERNET) += ns7520_eth.o > COBJS-$(CONFIG_NS8382X) += ns8382x.o > COBJS-$(CONFIG_DRIVER_NS9750_ETHERNET) += ns9750_eth.o > -COBJS-y += pcnet.o > -COBJS-y += plb2800_eth.o > +COBJS-$(CONFIG_PCNET) += pcnet.o > +COBJS-$(CONFIG_PLB2800_ETHER) += plb2800_eth.o > COBJS-$(CONFIG_DRIVER_RTL8019) += rtl8019.o > COBJS-$(CONFIG_RTL8139) += rtl8139.o > COBJS-$(CONFIG_RTL8169) += rtl8169.o > -COBJS-y += s3c4510b_eth.o > +COBJS-$(CONFIG_DRIVER_S3C4510_ETH) += s3c4510b_eth.o > COBJS-$(CONFIG_DRIVER_SMC91111) += smc91111.o > COBJS-$(CONFIG_DRIVER_SMC911X) += smc911x.o > COBJS-$(CONFIG_TIGON3) += tigon3.o bcm570x_autoneg.o 5701rls.o > COBJS-$(CONFIG_TSEC_ENET) += tsec.o > -COBJS-y += tsi108_eth.o > -COBJS-y += uli526x.o > +COBJS-$(CONFIG_TSI108_ETH) += tsi108_eth.o > +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 > diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c > index 458b517..ae1983a 100644 > --- a/drivers/net/cs8900.c > +++ b/drivers/net/cs8900.c > @@ -41,10 +41,6 @@ > #include "cs8900.h" > #include <net.h> > > -#ifdef CONFIG_DRIVER_CS8900 > - > -#if defined(CONFIG_CMD_NET) > - > #undef DEBUG > > /* packet page register access functions */ > @@ -315,7 +311,3 @@ int cs8900_e2prom_write(unsigned char addr, unsigned > short value) > > return 0; > } > - > -#endif /* CONFIG_CMD_NET */ > - > -#endif /* CONFIG_DRIVER_CS8900 */ > diff --git a/drivers/net/dc2114x.c b/drivers/net/dc2114x.c > index 1d728d8..8117239 100644 > --- a/drivers/net/dc2114x.c > +++ b/drivers/net/dc2114x.c > @@ -19,10 +19,6 @@ > */ > > #include <common.h> > - > -#if defined(CONFIG_CMD_NET) \ > - && defined(CONFIG_NET_MULTI) && defined(CONFIG_TULIP) > - > #include <malloc.h> > #include <net.h> > #include <pci.h> > @@ -766,5 +762,3 @@ static void update_srom(struct eth_device *dev, bd_t > *bis) > } > } > #endif /* UPDATE_SROM */ > - > -#endif > diff --git a/drivers/net/dm9000x.c b/drivers/net/dm9000x.c > index 68901cd..cb21925 100644 > --- a/drivers/net/dm9000x.c > +++ b/drivers/net/dm9000x.c > @@ -47,8 +47,6 @@ TODO: Homerun NIC and longrun NIC are not functional, only > internal at the > #include <net.h> > #include <asm/io.h> > > -#ifdef CONFIG_DRIVER_DM9000 > - > #include "dm9000x.h" > This patch no longer applies due to other changes to the DM9000 driver. Please re-base against the net repo and resubmit. <snip> thanks for doing all this Makefile fixing! It's been on my to-do list but you got there first. All others apply cleanly, BTW. regards, Ben ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users