All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: linux-arm-kernel@lists.infradead.org
Cc: Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	Ezequiel Garcia <ezequiel.garcia@free-electrons.com>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	Boris BREZILLON <boris.brezillon@free-electrons.com>,
	Lior Amsalem <alior@marvell.com>,
	Tawfik Bayouk <tawfik@marvell.com>,
	Nadav Haklai <nadavh@marvell.com>,
	linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/4] Add standby support for the recent mvebu SoCs
Date: Sat, 25 Jul 2015 17:20:07 +0200	[thread overview]
Message-ID: <55B3A927.6070906@free-electrons.com> (raw)
In-Reply-To: <1435684740-24912-1-git-send-email-gregory.clement@free-electrons.com>

Hi,

On 30/06/2015 19:18, Gregory CLEMENT wrote:
> Hi,
> 
> Until now only few mvebu v7 based board supported suspend to ram. This
> suspend to ram mode was unusual because it involved shutting down the
> SoC and relied on a PIC to wake up the system.
> 
> However, most of the recent mvebu SoCs can support the standby
> mode. Unlike for the suspend to ram, nothing special have to be done
> for these SoCs. In this mode the SoCs go in idle mode (but they remain
> powered up) and the devices enter in suspend mode. The support itself
> was added in the patch 2.
> 
> In order to wake-up the interrupt controller driver have been
> updated. As in standby mode the interrupt controller is not shutdown,
> any interrupt can be a wake-up source. So the GIC (patch 3) now used
> the flags IRQCHIP_SKIP_SET_WAKE and IRQCHIP_MASK_ON_SUSPEND.
> 
> A wake up source is supposed to work in suspend _and_ in standby mode
> but for the mvebu SoCs, no interrupt can wake up the system. The last
> patch warns the user about it.
> 
> The first patch is a clean-up found while working on this series
> 
> This series was applied on top of Thomas' series "ARM: mvebu: add
> suspend to RAM support for Armada 38x":
> http://thread.gmane.org/gmane.linux.ports.arm.kernel/420458
> 
> It has been either using rtcwake or by setting the serial line as a
> wake-up source through the sysfs interface.


I applied the series on mvebu/soc

Thanks,

Gregory

> 
> Thanks,
> 
> Gregory
> 
> 
> Gregory CLEMENT (4):
>   ARM: mvebu: Use __init for the PM initialization functions
>   ARM: mvebu: Add standby support
>   ARM: mvebu: Allow using the GIC for wakeup in standby mode
>   ARM: mvebu: Warn about the wake-up sources not taken into account in
>     suspend
> 
>  arch/arm/mach-mvebu/board-v7.c |  7 +++++
>  arch/arm/mach-mvebu/common.h   |  5 ++--
>  arch/arm/mach-mvebu/pm-board.c | 19 +++++++++-----
>  arch/arm/mach-mvebu/pm.c       | 58 +++++++++++++++++++++++++++++++++++++-----
>  4 files changed, 73 insertions(+), 16 deletions(-)
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/4] Add standby support for the recent mvebu SoCs
Date: Sat, 25 Jul 2015 17:20:07 +0200	[thread overview]
Message-ID: <55B3A927.6070906@free-electrons.com> (raw)
In-Reply-To: <1435684740-24912-1-git-send-email-gregory.clement@free-electrons.com>

Hi,

On 30/06/2015 19:18, Gregory CLEMENT wrote:
> Hi,
> 
> Until now only few mvebu v7 based board supported suspend to ram. This
> suspend to ram mode was unusual because it involved shutting down the
> SoC and relied on a PIC to wake up the system.
> 
> However, most of the recent mvebu SoCs can support the standby
> mode. Unlike for the suspend to ram, nothing special have to be done
> for these SoCs. In this mode the SoCs go in idle mode (but they remain
> powered up) and the devices enter in suspend mode. The support itself
> was added in the patch 2.
> 
> In order to wake-up the interrupt controller driver have been
> updated. As in standby mode the interrupt controller is not shutdown,
> any interrupt can be a wake-up source. So the GIC (patch 3) now used
> the flags IRQCHIP_SKIP_SET_WAKE and IRQCHIP_MASK_ON_SUSPEND.
> 
> A wake up source is supposed to work in suspend _and_ in standby mode
> but for the mvebu SoCs, no interrupt can wake up the system. The last
> patch warns the user about it.
> 
> The first patch is a clean-up found while working on this series
> 
> This series was applied on top of Thomas' series "ARM: mvebu: add
> suspend to RAM support for Armada 38x":
> http://thread.gmane.org/gmane.linux.ports.arm.kernel/420458
> 
> It has been either using rtcwake or by setting the serial line as a
> wake-up source through the sysfs interface.


I applied the series on mvebu/soc

Thanks,

Gregory

> 
> Thanks,
> 
> Gregory
> 
> 
> Gregory CLEMENT (4):
>   ARM: mvebu: Use __init for the PM initialization functions
>   ARM: mvebu: Add standby support
>   ARM: mvebu: Allow using the GIC for wakeup in standby mode
>   ARM: mvebu: Warn about the wake-up sources not taken into account in
>     suspend
> 
>  arch/arm/mach-mvebu/board-v7.c |  7 +++++
>  arch/arm/mach-mvebu/common.h   |  5 ++--
>  arch/arm/mach-mvebu/pm-board.c | 19 +++++++++-----
>  arch/arm/mach-mvebu/pm.c       | 58 +++++++++++++++++++++++++++++++++++++-----
>  4 files changed, 73 insertions(+), 16 deletions(-)
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  parent reply	other threads:[~2015-07-25 15:20 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-30 17:18 [PATCH v2 0/4] Add standby support for the recent mvebu SoCs Gregory CLEMENT
2015-06-30 17:18 ` Gregory CLEMENT
2015-06-30 17:18 ` [PATCH v2 1/4] ARM: mvebu: Use __init for the PM initialization functions Gregory CLEMENT
2015-06-30 17:18   ` Gregory CLEMENT
2015-06-30 17:18   ` Gregory CLEMENT
2015-07-01 15:18   ` Thomas Petazzoni
2015-07-01 15:18     ` Thomas Petazzoni
2015-06-30 17:18 ` [PATCH v2 2/4] ARM: mvebu: Add standby support Gregory CLEMENT
2015-06-30 17:18   ` Gregory CLEMENT
2015-07-01 15:47   ` Thomas Petazzoni
2015-07-01 15:47     ` Thomas Petazzoni
2015-07-03 11:39     ` Gregory CLEMENT
2015-07-03 11:39       ` Gregory CLEMENT
2015-07-03 12:17       ` Thomas Petazzoni
2015-07-03 12:17         ` Thomas Petazzoni
2015-07-03 12:21         ` Gregory CLEMENT
2015-07-03 12:21           ` Gregory CLEMENT
2015-07-03 12:33           ` Thomas Petazzoni
2015-07-03 12:33             ` Thomas Petazzoni
2015-06-30 17:18 ` [PATCH v2 3/4] ARM: mvebu: Allow using the GIC for wakeup in standby mode Gregory CLEMENT
2015-06-30 17:18   ` Gregory CLEMENT
2015-07-01 15:54   ` Thomas Petazzoni
2015-07-01 15:54     ` Thomas Petazzoni
2015-07-03  7:18     ` Gregory CLEMENT
2015-07-03  7:18       ` Gregory CLEMENT
2015-07-27 11:02     ` Sudeep Holla
2015-07-27 11:02       ` Sudeep Holla
2015-07-28  9:42       ` Gregory CLEMENT
2015-07-28  9:42         ` Gregory CLEMENT
2015-07-01 16:05   ` Sudeep Holla
2015-07-01 16:05     ` Sudeep Holla
2015-07-01 16:56     ` Geert Uytterhoeven
2015-07-01 16:56       ` Geert Uytterhoeven
2015-06-30 17:19 ` [PATCH v2 4/4] ARM: mvebu: Warn about the wake-up sources not taken into account in suspend Gregory CLEMENT
2015-06-30 17:19   ` Gregory CLEMENT
2015-07-01 16:04   ` Thomas Petazzoni
2015-07-01 16:04     ` Thomas Petazzoni
2015-07-03  9:55     ` Gregory CLEMENT
2015-07-03  9:55       ` Gregory CLEMENT
2015-07-25 15:20 ` Gregory CLEMENT [this message]
2015-07-25 15:20   ` [PATCH v2 0/4] Add standby support for the recent mvebu SoCs Gregory CLEMENT

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=55B3A927.6070906@free-electrons.com \
    --to=gregory.clement@free-electrons.com \
    --cc=alior@marvell.com \
    --cc=andrew@lunn.ch \
    --cc=boris.brezillon@free-electrons.com \
    --cc=ezequiel.garcia@free-electrons.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=maxime.ripard@free-electrons.com \
    --cc=nadavh@marvell.com \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=tawfik@marvell.com \
    --cc=thomas.petazzoni@free-electrons.com \
    /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.