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]

 



On Tue, Aug 07, 2018 at 09:40:12AM +0200, Krzysztof Kozlowski wrote:
> [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)?

Yes please, that makes the easiest to handle.

thanks,

greg k-h
--
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