Subject: [PATCH v6 04/13] clk: qcom: gdsc: Manage clocks with !CONFIG_PM

From: [email-masked] (Rajendra Nayak)
Date: Wed, 29 Jul 2015 10:07:04 +0530

On 07/29/2015 06:34 AM, Stephen Boyd wrote:
> On 07/23/2015 01:35 AM, Rajendra Nayak wrote:
>> []..
>>
>>>> +
>>>> +#ifndef CONFIG_PM
>>>> +static void enable_clock(struct device *dev, const char *con_id)
>>>> +{
>>>> +    struct clk *clk;
>>>> +
>>>> +    clk = clk_get(dev, con_id);
>>>> +    if (!IS_ERR(clk)) {
>>>> +        clk_prepare_enable(clk);
>>>> +        clk_put(clk);
>>>> +    }
>>>> +}
>>>> +
>>>> +static void disable_clock(struct device *dev, const char *con_id)
>>>> +{
>>>> +    struct clk *clk;
>>>> +
>>>> +    clk = clk_get(dev, con_id);
>>>> +    if (!IS_ERR(clk)) {
>>>> +        clk_disable_unprepare(clk);
>>>> +        clk_put(clk);
>>>> +    }
>>>> +}
>>>
>>> Is there a reason why this whole patch isn't generic code? I recall some
>>> discussion but I forgot now and there isn't any mention of why this
>>> isn't generic code in the commit text.
>>
>> If by generic code, you mean using PM clocks, then this thread should
>> give some context..
>> http://www.spinics.net/lists/arm-kernel/msg414072.html
>
> Sorry, I read the thread and I tried to understand what was going on but
> I'm still lost. Can you clarify further in the commit text somehow?

So I can add this in the commit text, if it seems fine

"The use of pm_clk_add_notifier() isn't appropriate here since we need
to only manage clocks with valid power domain associations done via
DT, instead of what pm_clk_add_notifier() does, which is manage clocks
for all on SoC/off SoC devices associating all of them to a dummy power 
domain instead"

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation