Message ID | 20170531061457.23744-5-o.rempel@pengutronix.de |
---|---|
State | Changes Requested, archived |
Headers | show |
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?
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?
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. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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.
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
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(+)