On Mon, 02 Jul 2007 08:30:09 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote:
> > --- a/drivers/net/Kconfig > > +++ b/drivers/net/Kconfig > > @@ -2264,6 +2264,16 @@ config TSI108_ETH > > To compile this driver as a module, choose M here: the module > > will be called tsi108_eth. > > > > +config GELIC_NET > > + tristate "PS3 Gigabit Ethernet driver" > > + depends on PPC_PS3 > > + help > > + This driver supports the network device on the PS3 game > > + console. This driver has built-in support for Ethernet. > > + > > + To compile this driver as a module, choose M here: the > > + module will be called ps3_gelic. > > + > > config GIANFAR > > tristate "Gianfar Ethernet" > > depends on 85xx || 83xx || PPC_86xx > > --- a/drivers/net/Makefile > > +++ b/drivers/net/Makefile > > @@ -60,6 +60,8 @@ obj-$(CONFIG_TIGON3) += tg3.o > > obj-$(CONFIG_BNX2) += bnx2.o > > spidernet-y += spider_net.o spider_net_ethtool.o > > obj-$(CONFIG_SPIDER_NET) += spidernet.o sungem_phy.o > > +obj-$(CONFIG_GELIC_NET) += ps3_gelic.o > > +ps3_gelic-objs += gelic_net.o > > obj-$(CONFIG_TC35815) += tc35815.o > > obj-$(CONFIG_SKGE) += skge.o > > obj-$(CONFIG_SKY2) += sky2.o > > --- /dev/null > > +++ b/drivers/net/gelic_net.c > > The patch does not reflect the new name... The driver's module name is 'ps3_gelic.ko' and the souce file names are 'gelic_net.{c,h}'. As Geoff Levand said in the mail dated June 16, we have plan to add wireless support and its file name would be gelic_wireless.c. If this support added, the make file would be: obj-$(CONFIG_GELIC_NET) += ps3_gelic.o gelic-$(CONFIG_GELIC_WIRELESS) += gelic_wireless.o ps3_gelic-objs += gelic_net.o $(gelic-y) To avoid from changing the module name after adding wireless support, I submitted with that name. If the make file were like: obj-$(CONFIG_GELIC_NET) += gelic_net.o obj-$(CONFIG_GELIC_WIRELESS) += gelic_wireless.o then we would get two modules gelic_net.ko and gelic_wireless.ko, which we don't want. -- Masakazu MOKUNO - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html