lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 14 Oct 2007 15:43:25 -0600 From: "Grant Likely" <grant.likely@...retlab.ca> To: "Domen Puncer" <domen.puncer@...argo.com> Cc: galak@...nel.crashing.org, jgarzik@...ox.com, linuxppc-dev@...abs.org, tnt@...tnt.com, netdev@...r.kernel.org Subject: Re: [PATCH v3 3/4] FEC mpc52xx: the driver On 10/14/07, Domen Puncer <domen.puncer@...argo.com> wrote: > Driver for ethernet on mpc5200/mpc5200b SoCs (FEC). > > > Signed-off-by: Domen Puncer <domen.puncer@...argo.com> Looks quite good to me. Comments below... > > --- > drivers/net/Kconfig | 1 > drivers/net/Makefile | 1 > drivers/net/fec_mpc52xx/Kconfig | 15 > drivers/net/fec_mpc52xx/Makefile | 2 > drivers/net/fec_mpc52xx/fec.c | 1098 +++++++++++++++++++++++++++++++++++++++ > drivers/net/fec_mpc52xx/fec.h | 313 +++++++++++ > 6 files changed, 1430 insertions(+) > > Index: linux.git/drivers/net/Kconfig > =================================================================== > --- linux.git.orig/drivers/net/Kconfig > +++ linux.git/drivers/net/Kconfig > @@ -1894,6 +1894,7 @@ config NE_H8300 > controller on the Renesas H8/300 processor. > > source "drivers/net/fec_8xx/Kconfig" > +source "drivers/net/fec_mpc52xx/Kconfig" Personally, I'd just add the mpc52xx items to drivers/net/Kconfig; but I won't make a fuss either way. =================================================================== > --- linux.git.orig/drivers/net/Makefile > +++ linux.git/drivers/net/Makefile > @@ -210,6 +210,7 @@ obj-$(CONFIG_SMC911X) += smc911x.o > obj-$(CONFIG_BFIN_MAC) += bfin_mac.o > obj-$(CONFIG_DM9000) += dm9000.o > obj-$(CONFIG_FEC_8XX) += fec_8xx/ > +obj-$(CONFIG_FEC_MPC52xx) += fec_mpc52xx/ Ditto here; Heck, we're only talking about 3 files here. I'd probably put all three into drivers/net... or even merge the 2 .c and one .h files into a single file. But, again, it's not a big deal. > obj-$(CONFIG_PASEMI_MAC) += pasemi_mac.o > obj-$(CONFIG_MLX4_CORE) += mlx4/ > > Index: linux.git/drivers/net/fec_mpc52xx/Kconfig > =================================================================== > --- /dev/null > +++ linux.git/drivers/net/fec_mpc52xx/Kconfig > @@ -0,0 +1,15 @@ > +menu "MPC5200 Networking Options" > + depends PPC_MPC52xx && NET_ETHERNET Drop the menu > + > +config FEC_MPC52xx > + tristate "FEC driver" > + depends on NET_ETHERNET Drop "NET_ETHERNET"; if you're here, then NET_ETHERNET *is* selected. Add "PPC_MPC52xx" > + select PPC_BESTCOMM > + select PPC_BESTCOMM_FEC > + select CRC32 > + select PHYLIB > + ---help--- > + This option enables support for the MPC5200's on-chip > + Fast Ethernet Controller > + > +endmenu > Index: linux.git/drivers/net/fec_mpc52xx/Makefile > Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. grant.likely@...retlab.ca (403) 399-0195 - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists