[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