| Message ID | 20170718175604.11735-3-andrew.smirnov@gmail.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
Where is patch 1? > Cc: cphealy@gmail.com > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Nikita Yushchenko <nikita.yoush@cogentembedded.com> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: devicetree@vger.kernel.org > Acked-by: Rob Herring <robh@kernel.org> > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > --- > .../devicetree/bindings/mfd/zii,rave-sp.txt | 39 ++++++++++++++++++++++ > 1 file changed, 39 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > > diff --git a/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > new file mode 100644 > index 000000000000..61d9105ded07 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > @@ -0,0 +1,39 @@ > +Zodiac Inflight Innovations RAVE Supervisory Processor > + > +RAVE Supervisory Processor communicates with SoC over UART. It is > +expected that its device-tree node is specified as a child of a node Nit: Device Tree Once fixed, please apply my: For my own reference: Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org> > +corresponding to UART controller used for communication. > + > +Required parent device properties: > + > + - compatible: Should be one of: > + - "zii,rave-sp-niu" > + - "zii,rave-sp-mezz" > + - "zii,rave-sp-esb" > + - "zii,rave-sp-rdu1" > + - "zii,rave-sp-rdu2" > + > + - current-speed: Should be set to baud rate SP device is using > + > +RAVE SP consists of the following sub-devices: > + > +Device Description > +------ ----------- > +rave-sp-wdt : Watchdog > +rave-sp-nvmem : Interface to onborad EEPROM > +rave-sp-backlight : Display backlight > +rave-sp-hwmon : Interface to onboard hardware sensors > +rave-sp-leds : Interface to onboard LEDs > +rave-sp-input : Interface to onboard power button > + > +Example of usage: > + > + rdu { > + compatible = "zii,rave-sp-rdu2"; > + current-speed = <1000000>; > + > + watchdog { > + compatible = "zii,rave-sp-watchdog"; > + }; > + }; > +
On Thu, Jul 20, 2017 at 2:04 AM, Lee Jones <lee.jones@linaro.org> wrote: > Where is patch 1? > lkml.kernel.org/r/20170718175604.11735-2-andrew.smirnov@gmail.com I dropped you from CC list to spare you the e-mail traffic, but I can add you back if you'd prefer that. >> Cc: cphealy@gmail.com >> Cc: Lucas Stach <l.stach@pengutronix.de> >> Cc: Nikita Yushchenko <nikita.yoush@cogentembedded.com> >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: Mark Rutland <mark.rutland@arm.com> >> Cc: devicetree@vger.kernel.org >> Acked-by: Rob Herring <robh@kernel.org> >> Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> >> --- >> .../devicetree/bindings/mfd/zii,rave-sp.txt | 39 ++++++++++++++++++++++ >> 1 file changed, 39 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mfd/zii,rave-sp.txt >> >> diff --git a/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt >> new file mode 100644 >> index 000000000000..61d9105ded07 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt >> @@ -0,0 +1,39 @@ >> +Zodiac Inflight Innovations RAVE Supervisory Processor >> + >> +RAVE Supervisory Processor communicates with SoC over UART. It is >> +expected that its device-tree node is specified as a child of a node > > Nit: Device Tree > > Once fixed, please apply my: > > For my own reference: > Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org> > OK. Will do. Thanks, Andrey Smirnov -- 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 Thu, 20 Jul 2017, Andrey Smirnov wrote: > On Thu, Jul 20, 2017 at 2:04 AM, Lee Jones <lee.jones@linaro.org> wrote: > > Where is patch 1? > > > > lkml.kernel.org/r/20170718175604.11735-2-andrew.smirnov@gmail.com > > I dropped you from CC list to spare you the e-mail traffic, but I can > add you back if you'd prefer that. It's always handy to see the C code which handles these bindings. Also, it looks like there is an MFD change in that patch, so I should have been Cc'ed anyway. > >> Cc: cphealy@gmail.com > >> Cc: Lucas Stach <l.stach@pengutronix.de> > >> Cc: Nikita Yushchenko <nikita.yoush@cogentembedded.com> > >> Cc: Rob Herring <robh+dt@kernel.org> > >> Cc: Mark Rutland <mark.rutland@arm.com> > >> Cc: devicetree@vger.kernel.org > >> Acked-by: Rob Herring <robh@kernel.org> > >> Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > >> --- > >> .../devicetree/bindings/mfd/zii,rave-sp.txt | 39 ++++++++++++++++++++++ > >> 1 file changed, 39 insertions(+) > >> create mode 100644 Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > >> > >> diff --git a/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > >> new file mode 100644 > >> index 000000000000..61d9105ded07 > >> --- /dev/null > >> +++ b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt > >> @@ -0,0 +1,39 @@ > >> +Zodiac Inflight Innovations RAVE Supervisory Processor > >> + > >> +RAVE Supervisory Processor communicates with SoC over UART. It is > >> +expected that its device-tree node is specified as a child of a node > > > > Nit: Device Tree > > > > Once fixed, please apply my: > > > > For my own reference: > > Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org> > > > > OK. Will do. > > Thanks, > Andrey Smirnov
On Tue, Jul 25, 2017 at 2:14 AM, Lee Jones <lee.jones@linaro.org> wrote: > On Thu, 20 Jul 2017, Andrey Smirnov wrote: > >> On Thu, Jul 20, 2017 at 2:04 AM, Lee Jones <lee.jones@linaro.org> wrote: >> > Where is patch 1? >> > >> >> lkml.kernel.org/r/20170718175604.11735-2-andrew.smirnov@gmail.com >> >> I dropped you from CC list to spare you the e-mail traffic, but I can >> add you back if you'd prefer that. > > It's always handy to see the C code which handles these bindings. > > Also, it looks like there is an MFD change in that patch, so I should > have been Cc'ed anyway. > OK, I'll add you back on CC list for future versions of the patchset. Thanks, Andrey Smirnov -- 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
diff --git a/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt new file mode 100644 index 000000000000..61d9105ded07 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/zii,rave-sp.txt @@ -0,0 +1,39 @@ +Zodiac Inflight Innovations RAVE Supervisory Processor + +RAVE Supervisory Processor communicates with SoC over UART. It is +expected that its device-tree node is specified as a child of a node +corresponding to UART controller used for communication. + +Required parent device properties: + + - compatible: Should be one of: + - "zii,rave-sp-niu" + - "zii,rave-sp-mezz" + - "zii,rave-sp-esb" + - "zii,rave-sp-rdu1" + - "zii,rave-sp-rdu2" + + - current-speed: Should be set to baud rate SP device is using + +RAVE SP consists of the following sub-devices: + +Device Description +------ ----------- +rave-sp-wdt : Watchdog +rave-sp-nvmem : Interface to onborad EEPROM +rave-sp-backlight : Display backlight +rave-sp-hwmon : Interface to onboard hardware sensors +rave-sp-leds : Interface to onboard LEDs +rave-sp-input : Interface to onboard power button + +Example of usage: + + rdu { + compatible = "zii,rave-sp-rdu2"; + current-speed = <1000000>; + + watchdog { + compatible = "zii,rave-sp-watchdog"; + }; + }; +