[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