Re: [PATCH 2/2] ARM: dts: exynos: Mark 1GHz CPU OPP as suspend OPP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[1On 7 August 2018 at 09:11, Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> wrote:
> Hi Krzysztof,
>
> On 2018-08-07 09:02, Krzysztof Kozlowski wrote:
>> On 3 August 2018 at 08:51, Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> wrote:
>>> 1GHz CPU OPP is the default boot value for the Exynos5250 SOC, so mark it
>>> as suspend OPP. This fixes suspend/resume on Samsung Exynos5250 Snow
>>> Chomebook, which was broken since switching to generic cpufreq-dt driver
>>> in v4.3.
>>>
>>> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
>>> ---
>>>   arch/arm/boot/dts/exynos5250.dtsi | 1 +
>>>   1 file changed, 1 insertion(+)
>> Looks okay. I think this should be backported to stable. The
>> operating-points-v2 bindings should be supported since v4.2 (after
>> looking as Documentation/devicetree/bindings/power/opp.txt).
>>
>> How about adding cc-stable then?
>
> I can add cc-stable, but I wonder which tags should be used in this case.
> Only patch 2/2 is the real bugfix, while patch 1/2 is just a prerequisite
> for it...

Hi Greg,

We have a stable-candidate fix [1] which depends on first patch in the
same series [2]. They were not applied yet so we wonder how we should
mark them in commit? Just add cc-stable for both of them (fix and
prerequisite)?

Best regards,
Krzysztof

[1] fix: https://patchwork.kernel.org/patch/10554603
[2] prerequisite: https://patchwork.kernel.org/patch/10554607
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux