[PATCH v2 00/12] clk/exynos convert clock IDs to macros

Andrzej Hajda a.hajda at samsung.com
Thu Sep 26 08:21:52 EDT 2013


On 09/16/2013 09:37 PM, Mike Turquette wrote:
> Quoting Sylwester Nawrocki (2013-09-16 07:33:45)
>> On 09/12/2013 12:50 PM, Kukjin Kim wrote:
>>> Well...I'm not sure changing to use macro is better or not at this moment...
>> I think it is. ;) It's really less error prone to have symbolic names instead
>> of the plain numbers. The clock ids are defined in one place and it is more
>> clear what a clock is by looking at symbolic name. It's especially annoying 
>> to use plain numbers where are are many clock in a single node, like 10 or 
>> more. What are you main concerns with this ?
> I think having the symbolic names is a win for readability.
>
> Regards,
> Mike
>
>> --
>> Regards,
>> Sylwester
Hi Kukjin,

Any thoughts/decisions regarding this patchset ?

Regards
Andrzej




More information about the linux-arm-kernel mailing list