Hi Krzysztof,
On 2018-08-10 13:39, Krzysztof Kozlowski wrote:
> On 10 August 2018 at 10:04, Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> wrote:
>> Regulators, which are marked as 'on-in-suspend' seems to be critical for
>> board operation, thus they must not be disabled anytime. This can be
>> only assured by marking them as 'always-on', because otherwise some
>> actions of their clients might result in turning them off. This patch
>> restores suspend/resume operation on Peach-Pit Chromebook board. It
>> partially reverts 'always-on' property removal done by the commit
>> mentioned in the Fixes tag.
>>
>> Fixes: 665c441eea3d ("ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards")
>> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
>> ---
>> This partial revert restores suspend/resume operation on Linux v4.7
>> release. To make suspend/resume working on current linux-next (tested
>> on next-20180806), one has to additionally revert following commits:
>>
>> 925ffff2ea8b "Input: cros_ec_keyb - remove check before calling pm_wakeup_event"
>> 38ba34a43dbc "Input: cros_ec_keyb - mark cros_ec_keyb driver as wake enabled device."
>> 47b7de2f6c18 "mmc: dw_mmc: fix falling from idmac to PIO mode when dw_mci_reset occurs"
>> 2695578b896a "net: usbnet: fix potential deadlock on 32bit hosts"
>>
>> I'm investigating those issues and separate reports/fixes will be
>> posted for them.
>>
>> Best regards
>> Marek Szyprowski
>> ---
>> arch/arm/boot/dts/exynos5420-peach-pit.dts | 3 +++
>> arch/arm/boot/dts/exynos5800-peach-pi.dts | 3 +++
>> 2 files changed, 6 insertions(+)
>>
> Makes sense. While looking around this, in
> arch/arm/boot/dts/exynos4412-midas.dtsi two more regulators follow
> this pattern (LDO15, LDO16). Maybe these are one of reasons why S2R
> stopped working on Trats2?
They should be fixed too, but they are not the source of the s2r issues.
I've checked and S2R works on Trats2 when DEVfreq is disabled, so it is
yet another issue caused by a missing suspend opp in devfreq...
See https://www.spinics.net/lists/linux-samsung-soc/msg56602.html thread.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
