[U-Boot] [PATCHv3 3/5] omap3: remove typedefs for configuration structs

Dirk Behme dirk.behme at googlemail.com
Sun May 24 18:30:59 CEST 2009


Dear Jean-Christophe,

Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>  void beagle_identify(void)
>>>>  {
>>>> -	gpio_t *gpio6_base = (gpio_t *)OMAP34XX_GPIO6_BASE;
>>>> +	struct gpio *gpio6_base = (struct gpio *)OMAP34XX_GPIO6_BASE;
>>> please use a namespace name gpio is too much generic
>>> and it's the same for the other struct sms, pm etc...
>>> omap_ will be better as the omap4 will certanly use the same or near IP
>> But OMAP4 most probably will not use a file board/omap3/beagle/beagle.c.
>>
>> And using "omap_" would be wrong if your argument is OMAP4. Then it  
>> needs at least to be "omap3_".
>>
>> And I can't see how a local variable could pollute the namespace, if  
>> this is what you mean.
> where did I speak about var name?

Nowhere, sorry. I talk about it. Please replace string 'local 
variable' by 'local struct' above if you think it fits better.

> the stuct name need to be less generic

Why? Please see my arguments why I think it's fine as done in 
Matthias' patch.

Best regards

Dirk



More information about the U-Boot mailing list