All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Dave Gerlach <d-gerlach@ti.com>
Cc: Santosh Shilimkar <ssantosh@kernel.org>,
	Russell King <linux@armlinux.org.uk>,
	linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
	linux-kernel@vger.kernel.org, Keerthy J <j-keerthy@ti.com>
Subject: Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support
Date: Thu, 18 May 2017 07:59:24 -0700	[thread overview]
Message-ID: <20170518145923.GB10472@atomide.com> (raw)
In-Reply-To: <0bf4723f-dd94-bbbf-21a8-2ec60dff58cb@ti.com>

* Dave Gerlach <d-gerlach@ti.com> [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@ti.com> [170329 11:52]:
> > > > Hi Tony,
> > > > On 03/29/2017 12:13 PM, Tony Lindgren wrote:
> > > > > * Dave Gerlach <d-gerlach@ti.com> [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

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support
Date: Thu, 18 May 2017 07:59:24 -0700	[thread overview]
Message-ID: <20170518145923.GB10472@atomide.com> (raw)
In-Reply-To: <0bf4723f-dd94-bbbf-21a8-2ec60dff58cb@ti.com>

* Dave Gerlach <d-gerlach@ti.com> [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@ti.com> [170329 11:52]:
> > > > Hi Tony,
> > > > On 03/29/2017 12:13 PM, Tony Lindgren wrote:
> > > > > * Dave Gerlach <d-gerlach@ti.com> [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

  reply	other threads:[~2017-05-18 14:59 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-29  1:57 [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Dave Gerlach
2017-03-29  1:57 ` Dave Gerlach
2017-03-29  1:57 ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 1/8] ARM: OMAP: Wakeupgen: Add context save/restore for AM43XX Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 2/8] ARM: OMAP2+: timer: Add suspend-resume callbacks for clkevent device Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 3/8] ARM: OMAP2+: pm: Remove __init from omap_pm_clkdms_setup Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 4/8] ARM: OMAP2+: Introduce low-level suspend code for AM33XX Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 5/8] ARM: OMAP2+: Introduce low-level suspend code for AM43XX Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57 ` [PATCH 6/8] ARM: OMAP2+: pm33xx-core: Add platform code needed for PM Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-03-29  1:57   ` Dave Gerlach
2017-04-04 16:19   ` Tony Lindgren
2017-04-04 16:19     ` Tony Lindgren
2017-04-04 16:19     ` Tony Lindgren
2017-03-29  1:58 ` [PATCH 7/8] soc: ti: Add pm33xx driver for basic suspend support Dave Gerlach
2017-03-29  1:58   ` Dave Gerlach
2017-03-29  1:58   ` Dave Gerlach
2017-03-29  1:58 ` [PATCH 8/8] ARM: OMAP2+: Create dummy platform_device for pm33xx Dave Gerlach
2017-03-29  1:58   ` Dave Gerlach
2017-03-29  1:58   ` Dave Gerlach
2017-03-29 17:13 ` [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Tony Lindgren
2017-03-29 17:13   ` Tony Lindgren
2017-03-29 18:50   ` Dave Gerlach
2017-03-29 18:50     ` Dave Gerlach
2017-03-29 18:50     ` Dave Gerlach
2017-05-16 15:47     ` Tony Lindgren
2017-05-16 15:47       ` Tony Lindgren
2017-05-16 15:49       ` Dave Gerlach
2017-05-16 15:49         ` Dave Gerlach
2017-05-16 15:49         ` Dave Gerlach
2017-05-18 14:19         ` Dave Gerlach
2017-05-18 14:19           ` Dave Gerlach
2017-05-18 14:19           ` Dave Gerlach
2017-05-18 14:59           ` Tony Lindgren [this message]
2017-05-18 14:59             ` Tony Lindgren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170518145923.GB10472@atomide.com \
    --to=tony@atomide.com \
    --cc=d-gerlach@ti.com \
    --cc=j-keerthy@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=ssantosh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.