On 13 April 2012 15:30, Graeme Russ <graeme.r...@gmail.com> wrote:
> Hi Prabhakar,
>
> On Fri, Apr 13, 2012 at 4:16 PM, Prabhakar Lad
> <prabhakar.cse...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to