[U-Boot] [PATCH v5 4/4] ARM: SAMSUNG: support sdhci controller

Minkyu Kang promsoft at gmail.com
Fri Apr 13 10:27:30 CEST 2012


On 13 April 2012 15:30, Graeme Russ <graeme.russ at gmail.com> wrote:
> Hi Prabhakar,
>
> On Fri, Apr 13, 2012 at 4:16 PM, Prabhakar Lad
> <prabhakar.csengg at gmail.com> wrote:
>> Hi Jaehoon,
>>
>
> [snip]
>
>>    There is no consistency for the #define  CONFIG_S5P_SDHCI,
>>    few have been assigned to value 1, And looking at the patchset
>>    the value of it is not used if I am not wrong, depending or defined
>>    or not some file is built. Best way could be not to assign any value
>>    for such cases, just  #define CONFIG_S5P_SDHCI would make
>>    sense. It would be good If you also do the same for other CONFIG'S
>>    defined.
>
> All boolean defines should be #define'd with no value
>
> Patches should be submitted to cleanup any that are currently defined with
> a value. Such patches should be marked 'cosmetic' and be distinctly
> seperate from any patches that are adding new code, prefferable prior to
> the patches addding new defines so each file always has a consistent style
>
> Regards,

Yes, right.
Need patch for cleanup.

Jaehoon,
Please update this patch.

I'll post cleanup patch for other defines.

Thanks.
Minkyu Kang.
-- 
from. prom.
www.promsoft.net


More information about the U-Boot mailing list