Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1653860 > unrolled thread

[PATCH v2 0/6] iMX6: Implement power off over standby signal

Started byOleksij Rempel <o.rempel@pengutronix.de>
First post2017-05-31 08:20 +0200
Last post2017-05-31 17:00 +0200
Articles 6 — 4 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v2 0/6] iMX6: Implement power off over standby signal Oleksij Rempel <o.rempel@pengutronix.de> - 2017-05-31 08:20 +0200
    [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property Oleksij Rempel <o.rempel@pengutronix.de> - 2017-05-31 08:20 +0200
      Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff  property Mark Brown <broonie@kernel.org> - 2017-05-31 16:50 +0200
        Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff  property Oleksij Rempel <ore@pengutronix.de> - 2017-05-31 17:00 +0200
          Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff  property Mark Brown <broonie@kernel.org> - 2017-05-31 18:00 +0200
      Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff  property Mark Rutland <mark.rutland@arm.com> - 2017-05-31 17:00 +0200

#1653860 — [PATCH v2 0/6] iMX6: Implement power off over standby signal

FromOleksij Rempel <o.rempel@pengutronix.de>
Date2017-05-31 08:20 +0200
Subject[PATCH v2 0/6] iMX6: Implement power off over standby signal
Message-ID<tN597-16T-3@gated-at.bofh.it>
v2:
 - export pm_power_off_prepare to fix pfuze100-regulator as module compilation.

Oleksij Rempel (6):
  ARM: imx6q: provide documentation for new fsl,pmic_stby_poweroff
    property
  ARM: imx6: register pm_power_off handler if "fsl,pmic_stby_poweroff"
    is set
  kernel/reboot.c: export pm_power_off_prepare
  regulator: pfuze100: add fsl,pmic_stby_poweroff property
  regulator: pfuze100-regulator: provide pm_power_off_prepare handler
  ARM: dts: imx6: RIoTboard provide Standby to PowerOff option

 .../devicetree/bindings/clock/imx6q-clock.txt      |  4 ++
 .../devicetree/bindings/regulator/pfuze100.txt     |  4 ++
 arch/arm/boot/dts/imx6dl-riotboard.dts             |  5 ++
 arch/arm/mach-imx/pm-imx6.c                        | 30 +++++++++
 drivers/regulator/pfuze100-regulator.c             | 76 ++++++++++++++++++++++
 kernel/reboot.c                                    |  1 +
 6 files changed, 120 insertions(+)

Cc: kernel@pengutronix.de
Cc: linux-arm-kernel@lists.infradead.org
Cc: Michael Turquette <mturquette@baylibre.com>
Cc: Stephen Boyd <sboyd@codeaurora.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: linux-clk@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org

-- 
2.11.0

[toc] | [next] | [standalone]

#1653864 — [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property

FromOleksij Rempel <o.rempel@pengutronix.de>
Date2017-05-31 08:20 +0200
Subject[PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property
Message-ID<tN598-16T-37@gated-at.bofh.it>
In reply to#1653860
Document the new optional "fsl,pmic_stby_poweroff" property.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Cc: kernel@pengutronix.de
Cc: linux-arm-kernel@lists.infradead.org
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: linux-kernel@vger.kernel.org
Cc: devicetree@vger.kernel.org
---
 Documentation/devicetree/bindings/regulator/pfuze100.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Documentation/devicetree/bindings/regulator/pfuze100.txt
index 444c47831a40..216519c4a50d 100644
--- a/Documentation/devicetree/bindings/regulator/pfuze100.txt
+++ b/Documentation/devicetree/bindings/regulator/pfuze100.txt
@@ -4,6 +4,10 @@ Required properties:
 - compatible: "fsl,pfuze100", "fsl,pfuze200", "fsl,pfuze3000"
 - reg: I2C slave address
 
+Optional properties:
+- fsl,pmic_stby_poweroff: if present, configure the PMIC to shutdown all
+  power rails when PMIC_STBY_REQ line is asserted during the power off sequence.
+
 Required child node:
 - regulators: This is the list of child nodes that specify the regulator
   initialization data for defined regulators. Please refer to below doc
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1654319 — Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property

FromMark Brown <broonie@kernel.org>
Date2017-05-31 16:50 +0200
SubjectRe: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property
Message-ID<tNd6G-67G-7@gated-at.bofh.it>
In reply to#1653864

[Multipart message — attachments visible in raw view] — view raw

On Wed, May 31, 2017 at 08:14:55AM +0200, Oleksij Rempel wrote:
> Document the new optional "fsl,pmic_stby_poweroff" property.

I'm missing both the cover letter and the rest of the series, what's
going on with dependencies here?

[toc] | [prev] | [next] | [standalone]

#1654323 — Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property

FromOleksij Rempel <ore@pengutronix.de>
Date2017-05-31 17:00 +0200
SubjectRe: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property
Message-ID<tNdgl-6b3-1@gated-at.bofh.it>
In reply to#1654319
On Wed, May 31, 2017 at 03:44:38PM +0100, Mark Brown wrote:
> On Wed, May 31, 2017 at 08:14:55AM +0200, Oleksij Rempel wrote:
> > Document the new optional "fsl,pmic_stby_poweroff" property.
> 
> I'm missing both the cover letter and the rest of the series, what's
> going on with dependencies here?

Hmm... I probably needed to add same list of maintainers to all patches.
I will fix it next time. The patches are here:
https://patchwork.kernel.org/bundle/olerem/poweroff/

Should I resend it with complete Cc list?

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

[toc] | [prev] | [next] | [standalone]

#1654376 — Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property

FromMark Brown <broonie@kernel.org>
Date2017-05-31 18:00 +0200
SubjectRe: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property
Message-ID<tNecq-6Ly-9@gated-at.bofh.it>
In reply to#1654323

[Multipart message — attachments visible in raw view] — view raw

On Wed, May 31, 2017 at 04:55:36PM +0200, Oleksij Rempel wrote:
> On Wed, May 31, 2017 at 03:44:38PM +0100, Mark Brown wrote:

> > I'm missing both the cover letter and the rest of the series, what's
> > going on with dependencies here?

> Hmm... I probably needed to add same list of maintainers to all patches.
> I will fix it next time. The patches are here:
> https://patchwork.kernel.org/bundle/olerem/poweroff/

> Should I resend it with complete Cc list?

Like I say from my point of view I need to know what the dependencies
are somehow.

[toc] | [prev] | [next] | [standalone]

#1654329 — Re: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property

FromMark Rutland <mark.rutland@arm.com>
Date2017-05-31 17:00 +0200
SubjectRe: [PATCH v2 4/6] regulator: pfuze100: add fsl,pmic_stby_poweroff property
Message-ID<tNdgm-6b3-23@gated-at.bofh.it>
In reply to#1653864
On Wed, May 31, 2017 at 08:14:55AM +0200, Oleksij Rempel wrote:
> Document the new optional "fsl,pmic_stby_poweroff" property.
> 
> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> Cc: kernel@pengutronix.de
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: linux-kernel@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> ---
>  Documentation/devicetree/bindings/regulator/pfuze100.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Documentation/devicetree/bindings/regulator/pfuze100.txt
> index 444c47831a40..216519c4a50d 100644
> --- a/Documentation/devicetree/bindings/regulator/pfuze100.txt
> +++ b/Documentation/devicetree/bindings/regulator/pfuze100.txt
> @@ -4,6 +4,10 @@ Required properties:
>  - compatible: "fsl,pfuze100", "fsl,pfuze200", "fsl,pfuze3000"
>  - reg: I2C slave address
>  
> +Optional properties:
> +- fsl,pmic_stby_poweroff: if present, configure the PMIC to shutdown all
> +  power rails when PMIC_STBY_REQ line is asserted during the power off sequence.

As with patch 1, s/_/-/ please.

Please also explain why/why this is needed.

Thanks,
Mark.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web