[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