From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 10 Feb 2009 09:39:27 +0100 Subject: [U-Boot] [PATCH 1/2] net: Sort Makefile labels In-Reply-To: <49913792.9060008@monstr.eu> References: <1231154714-15799-1-git-send-email-monstr@monstr.eu> <4990A841.3000705@gmail.com> <49913792.9060008@monstr.eu> Message-ID: <200902100939.27809.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tuesday 10 February 2009, Michal Simek wrote: > >>> +++ b/drivers/net/Makefile > >>> @@ -26,6 +26,7 @@ include $(TOPDIR)/config.mk > >>> LIB := $(obj)libnet.a > >>> > >>> COBJS-$(CONFIG_DRIVER_3C589) += 3c589.o > >>> +COBJS-$(CONFIG_PPC4xx_EMAC) += 4xx_enet.o > >>> COBJS-$(CONFIG_DRIVER_AX88180) += ax88180.o > >> > >> What makes you think that CONFIG_PPC4xx_EMAC has a good place between > >> CONFIG_DRIVER_3C589 and CONFIG_DRIVER_AX88180? > > From my point of view make me more sense to rename 4xx_enec.c file to > correspond with driver name. I mean if the config is CONFIG_PPC4xx_EMAC the > filename should be ppc4xx_emac.c or in second case CONFIG_4XX_ENET for > current 4xx_enet.c file. From these two choices I like the first one. Yes, I also think it makes sense to rename this driver to "ppc4xx_emac.c". If nobody objects I'll send a patch doing this shortly. Best regards, Stefan ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================