[PATCH v1] soc: qcom: do not disable the iface clock in probe

[PATCH v1] soc: qcom: do not disable the iface clock in probe

[Kumar Gala]

On Aug 15, 2014, at 12:41 AM, Srinivas Kandagatla <srinivas.kandagatla@linaro.org> wrote:

> The use case here is when we have a bootconsole which is printing the
> characters on serial console and gsbi driver comes up after some time.
> As gsbi driver disables the clock in probe the bootconsole locks up.
> 
> This patch fixes the problem by disabling the clock in platform remove
> rather than in probe.
> 
> Tested-by: Linus Walleij <linus.walleij@linaro.org>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
> Changes since RFC:
> 	- none, just a subject line change.

Srini,

We need some changes to the commit message for this to get into v3.17.

Probably some references to:

http://www.spinics.net/lists/linux-arm-msm/msg10589.html

Adding Kevin to provide details on what else should be in commit message w/regards to the regression we introduced.

- k

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

[PATCH v1] soc: qcom: do not disable the iface clock in probe

[Kevin Hilman]

On Tue, Sep 23, 2014 at 10:42 AM, Kumar Gala <galak@codeaurora.org> wrote:
>
> On Aug 15, 2014, at 12:41 AM, Srinivas Kandagatla <srinivas.kandagatla@linaro.org> wrote:
>
>> The use case here is when we have a bootconsole which is printing the
>> characters on serial console and gsbi driver comes up after some time.
>> As gsbi driver disables the clock in probe the bootconsole locks up.
>>
>> This patch fixes the problem by disabling the clock in platform remove
>> rather than in probe.
>>
>> Tested-by: Linus Walleij <linus.walleij@linaro.org>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> Changes since RFC:
>>       - none, just a subject line change.
>
> Srini,
>
> We need some changes to the commit message for this to get into v3.17.
>
> Probably some references to:
>
> http://www.spinics.net/lists/linux-arm-msm/msg10589.html
>
> Adding Kevin to provide details on what else should be in commit message w/regards to the regression we introduced.

At this point of the cycle, we're rather strict about what qualifies
as a fix.  It should be fixing a specific regression.  Looking at the
above link, it looks like it is, so could you re-spin this with
changelog, summarizing the regression and how this fixes the
regression, and we should still be able to get it in for v3.17.

Also, it looks like the original was only posted to the msm list.  For
future reference, for patches that need to go via the arm-soc
maintainers, make sure to Cc arm at kernel.org.

Thanks,

Kevin

[PATCH v1] soc: qcom: do not disable the iface clock in probe

[Srinivas Kandagatla]

On 23/09/14 18:42, Kumar Gala wrote:
>
> On Aug 15, 2014, at 12:41 AM, Srinivas Kandagatla <srinivas.kandagatla@linaro.org> wrote:
>
>> The use case here is when we have a bootconsole which is printing the
>> characters on serial console and gsbi driver comes up after some time.
>> As gsbi driver disables the clock in probe the bootconsole locks up.
>>
>> This patch fixes the problem by disabling the clock in platform remove
>> rather than in probe.
>>
>> Tested-by: Linus Walleij <linus.walleij@linaro.org>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> Changes since RFC:
>> 	- none, just a subject line change.
>
> Srini,
>
> We need some changes to the commit message for this to get into v3.17.
>
Sure.. I will send v2 with change log referring to 
http://www.spinics.net/lists/linux-arm-msm/msg10589.html


--srini
> Probably some references to:
>
> http://www.spinics.net/lists/linux-arm-msm/msg10589.html
>
> Adding Kevin to provide details on what else should be in commit message w/regards to the regression we introduced.
>
> - k
>