[U-Boot] [PATCH 1/2] net: Sort Makefile labels

Michal Simek monstr at monstr.eu
Tue Feb 10 10:09:29 CET 2009


Dear Wolfgang,

> Dear Michal Simek,
> 
> In message <49913792.9060008 at monstr.eu> you wrote:
>>>>>  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.
> 
> Well, that's IMO for the PPC4xx custodian to decide.
> 
> Stefan, what do you think?
> 
>> I would like to describe what happen. I sent to mailing list two patches. One
>> with Makefile sort and second with LL_TEMAC. First patch just sort some labels
>> in drivers/net/Makefile. Wolfgang sent that he don't like it and he reject this
>> patch. I haven't wanted to sort any Makefile labels I just wanted to add
>> LL_TEMAC driver. Makefile sort was not my point.
> 
> I'm aware of this. But this is how contributing to U-Boot works : you
> just want to add a tiny  piece  of  code  here,  and  you  will  find
> yourself  involved  in some bigger cleanup all over the place. That's
> what happens to me all day, and to many others,  so  please  help  to
> carry that burdon.  Thanks.

Yes I know that's why I sent sort Makefile patch in that pack.

Regards,
Michal


> Best regards,
> 
> Wolfgang Denk
> 


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854


More information about the U-Boot mailing list