Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support
From: Tony Lindgren
Date: Thu May 18 2017 - 10:59:46 EST
* Dave Gerlach <d-gerlach@xxxxxx> [170518 07:23]:
> 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.
OK dropping 4 and 5 today.
Regards,
Tony