Messages in this thread | | | From | Andrey Smirnov <> | Date | Tue, 6 Mar 2018 17:51:29 -0800 | Subject | Re: [PATCH v2 2/2] dt-bindings: input: Add binding for RAVE SP input driver |
| |
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
| |