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

Andrzej Hajda a.hajda at samsung.com
Mon Oct 28 10:31:47 EDT 2013


Ping

Regards
Andrzej

On 10/07/2013 10:46 AM, Andrzej Hajda wrote:
> On 10/06/2013 10:27 PM, Mike Turquette wrote:
>> Quoting Kukjin Kim (2013-10-04 19:16:56)
>>> Mike Turquette wrote:
>>>> Quoting Kukjin Kim (2013-09-30 03:11:41)
>>>>> 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.
>>>>>>
>>>>> Hmm...since Mike who is a maintainer for ccf agrees with Sylwester's
>>>> opinion, so I agree. Actually, I had no strong objection on this but still
>>>> I'm not sure using macro is really better because sometimes the name
>>>> doesn't give readability really I think though.
>>>>> Mike, can you give me your ack on clk stuff?
>>>> Yes, but I was thinking to take this change through the clk tree. The
>>>> new branch based on -rc3 will be published tonight.
>>>>
>>>> Let me know if it causes problems for you for me to take it.
>>>>
>>> Uhm, I think dt changes can make ugly conflicts because of spread dt changing?
>> Sure, go ahead and take the patches then.
>>
>> Acked-by: Mike Turquette <mturquette at linaro.org>
>>
>> Regards,
>> Mike
> 
> I can create patchset based on different git branch if neccessary.
> 
> Regards
> Andrzej
>>> - Kukjin
> 




More information about the linux-arm-kernel mailing list