Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support

From: Dave Gerlach
Date: Thu May 18 2017 - 10:20:23 EST


Tony,
On 05/16/2017 10:49 AM, Dave Gerlach wrote:
On 05/16/2017 10:47 AM, Tony Lindgren wrote:
* Dave Gerlach <d-gerlach@xxxxxx> [170329 11:52]:
Hi Tony,
On 03/29/2017 12:13 PM, Tony Lindgren wrote:
* Dave Gerlach <d-gerlach@xxxxxx> [170328 19:00]:
Hi,
This series contains the remaining code to enable suspend to mem and
standby on
am335x and am437x platforms. It depends on the ti-emif-sram series sent
here [1].

Looks good to me in general. To remove the dependency, can some
of this be merged separately without breaking things before [1]?
Maybe by disabling some of the PM states until [1] is in place?

Thanks for the feedback. Actually, patch 7 is the only patch with a
dependency to [1]. The rest of the patches just add platform PM support
needed for suspend but won't do much of anything until patch 7 actually
enables the suspend states, so there is no reason why patches 1-6 and patch
8 can't be merged now.

I'm applying patches 1-5 into omap-for-v4.13/soc. Patch 6 needs
ack from Russell.


Thanks. I will resend patches 6-8 as a separate series.

On second thought, it's probably best to keep patches 4 and 5 as part of the new series as well for review purposes, as they are not added to the build until patch 6 but also make use of the asm-offsets macros introduced in patch 6. I will send them along with patch 6-8 resend, making the new series patches 4-8.

Regards,
Dave


Regards,
Dave

Regards,

Tony