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

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sun May 24 18:12:12 CEST 2009


>>>  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?
the stuct name need to be less generic

Best Regards,
J.


More information about the U-Boot mailing list