| Message ID | 20180301165527.22274-2-andrew.smirnov@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Am Donnerstag, den 01.03.2018, 08:55 -0800 schrieb Andrey Smirnov: > Add Device Tree bindings for RAVE SP input drvier - an MFD cell of > parent RAVE SP driver (documented in > Documentation/devicetree/bindings/mfd/zii,rave-sp.txt). > > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > Cc: linux-input@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > > Cc: Rob Herring <robh+dt@kernel.org> > > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: devicetree@vger.kernel.org > > Cc: Guenter Roeck <linux@roeck-us.net> > > Cc: Chris Healy <cphealy@gmail.com> > > Cc: Lucas Stach <l.stach@pengutronix.de> > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Acked-by: Lucas Stach <l.stach@pengutronix.de> > --- > .../bindings/input/zii,rave-sp-pwrbutton.txt | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt > > diff --git a/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt > new file mode 100644 > index 000000000000..e14e12d27772 > --- /dev/null > +++ b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt > @@ -0,0 +1,24 @@ > +Zodiac Inflight Innovations RAVE Supervisory Processor Power Button Bindings > + > +RAVE SP input device is a "MFD cell" device corresponding to power > +button functionality of RAVE Supervisory Processor. It is expected > +that its Device Tree node is specified as a child of the node > +corresponding to the parent RAVE SP device (as documented in > +Documentation/devicetree/bindings/mfd/zii,rave-sp.txt) > + > +Required properties: > + > +- compatible: Should be "zii,rave-sp-pwrbutton" > + > +Example: > + > > + rave-sp { > > + compatible = "zii,rave-sp-rdu1"; > > + current-speed = <38400>; > + > > + pwrbutton { > > + compatible = "zii,rave-sp-pwrbutton"; > > + status = "okay"; > > + }; > > + } > + -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Andrey, On Thu, Mar 1, 2018 at 1:55 PM, Andrey Smirnov <andrew.smirnov@gmail.com> wrote: l > +++ b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt > @@ -0,0 +1,24 @@ > +Zodiac Inflight Innovations RAVE Supervisory Processor Power Button Bindings > + > +RAVE SP input device is a "MFD cell" device corresponding to power > +button functionality of RAVE Supervisory Processor. It is expected > +that its Device Tree node is specified as a child of the node > +corresponding to the parent RAVE SP device (as documented in > +Documentation/devicetree/bindings/mfd/zii,rave-sp.txt) > + > +Required properties: > + > +- compatible: Should be "zii,rave-sp-pwrbutton" > + > +Example: > + > + rave-sp { > + compatible = "zii,rave-sp-rdu1"; > + current-speed = <38400>; > + > + pwrbutton { > + compatible = "zii,rave-sp-pwrbutton"; > + status = "okay"; In the dts patch review Shawn said: "The okay status is to flip the state of devices that are initially disabled. I think it's unnecessary for the case here." so please remove the status line. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Mar 6, 2018 at 6:17 AM, Fabio Estevam <festevam@gmail.com> wrote: > Hi Andrey, > > On Thu, Mar 1, 2018 at 1:55 PM, Andrey Smirnov <andrew.smirnov@gmail.com> wrote: > l >> +++ b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt >> @@ -0,0 +1,24 @@ >> +Zodiac Inflight Innovations RAVE Supervisory Processor Power Button Bindings >> + >> +RAVE SP input device is a "MFD cell" device corresponding to power >> +button functionality of RAVE Supervisory Processor. It is expected >> +that its Device Tree node is specified as a child of the node >> +corresponding to the parent RAVE SP device (as documented in >> +Documentation/devicetree/bindings/mfd/zii,rave-sp.txt) >> + >> +Required properties: >> + >> +- compatible: Should be "zii,rave-sp-pwrbutton" >> + >> +Example: >> + >> + rave-sp { >> + compatible = "zii,rave-sp-rdu1"; >> + current-speed = <38400>; >> + >> + pwrbutton { >> + compatible = "zii,rave-sp-pwrbutton"; >> + status = "okay"; > > In the dts patch review Shawn said: > > "The okay status is to flip the state of devices that are initially > disabled. I think it's unnecessary for the case here." > > so please remove the status line. Sure, will do in v3. Thanks, Andrey Smirnov -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt new file mode 100644 index 000000000000..e14e12d27772 --- /dev/null +++ b/Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt @@ -0,0 +1,24 @@ +Zodiac Inflight Innovations RAVE Supervisory Processor Power Button Bindings + +RAVE SP input device is a "MFD cell" device corresponding to power +button functionality of RAVE Supervisory Processor. It is expected +that its Device Tree node is specified as a child of the node +corresponding to the parent RAVE SP device (as documented in +Documentation/devicetree/bindings/mfd/zii,rave-sp.txt) + +Required properties: + +- compatible: Should be "zii,rave-sp-pwrbutton" + +Example: + + rave-sp { + compatible = "zii,rave-sp-rdu1"; + current-speed = <38400>; + + pwrbutton { + compatible = "zii,rave-sp-pwrbutton"; + status = "okay"; + }; + } +
Add Device Tree bindings for RAVE SP input drvier - an MFD cell of parent RAVE SP driver (documented in Documentation/devicetree/bindings/mfd/zii,rave-sp.txt). Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: linux-input@vger.kernel.org Cc: linux-kernel@vger.kernel.org Cc: Rob Herring <robh+dt@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: devicetree@vger.kernel.org Cc: Guenter Roeck <linux@roeck-us.net> Cc: Chris Healy <cphealy@gmail.com> Cc: Lucas Stach <l.stach@pengutronix.de> Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> --- .../bindings/input/zii,rave-sp-pwrbutton.txt | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/zii,rave-sp-pwrbutton.txt