| Message ID | 1461339219-15255-2-git-send-email-tthayer@opensource.altera.com |
|---|---|
| State | New |
| Headers | show |
On Fri, Apr 22, 2016 at 10:33:29AM -0500, tthayer@opensource.altera.com wrote: > From: Thor Thayer <tthayer@opensource.altera.com> > > The Altera Arria10 Devkit System Resource chip is a Multi-Function > Device with a GPIO expander and power supply alarm. An alarm with no interrupt? Are the GPIOs interrupt capable? > This patch adds documentation for the Altera A10-SR DT bindings. > > Signed-off-by: Thor Thayer <tthayer@opensource.altera.com> > --- > .../devicetree/bindings/mfd/altera-a10sr.txt | 35 ++++++++++++++++++++ > 1 file changed, 35 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/altera-a10sr.txt > -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 04/25/2016 07:55 AM, Rob Herring wrote: > On Fri, Apr 22, 2016 at 10:33:29AM -0500, tthayer@opensource.altera.com wrote: >> From: Thor Thayer <tthayer@opensource.altera.com> >> >> The Altera Arria10 Devkit System Resource chip is a Multi-Function >> Device with a GPIO expander and power supply alarm. > > An alarm with no interrupt? Are the GPIOs interrupt capable? The alarms have no interrupt but the GPIOs are interrupt capable (not implemented with interrupt capability in this patch series). > >> This patch adds documentation for the Altera A10-SR DT bindings. >> >> Signed-off-by: Thor Thayer <tthayer@opensource.altera.com> >> --- >> .../devicetree/bindings/mfd/altera-a10sr.txt | 35 ++++++++++++++++++++ >> 1 file changed, 35 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mfd/altera-a10sr.txt >> -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Apr 25, 2016 at 10:01:52AM -0500, Thor Thayer wrote: > > > On 04/25/2016 07:55 AM, Rob Herring wrote: > >On Fri, Apr 22, 2016 at 10:33:29AM -0500, tthayer@opensource.altera.com wrote: > >>From: Thor Thayer <tthayer@opensource.altera.com> > >> > >>The Altera Arria10 Devkit System Resource chip is a Multi-Function > >>Device with a GPIO expander and power supply alarm. > > > >An alarm with no interrupt? Are the GPIOs interrupt capable? > > The alarms have no interrupt but the GPIOs are interrupt capable (not > implemented with interrupt capability in this patch series). Please make the binding as complete as possible even if you don't support features in the driver yet. Driver features may evolve, but a specific h/w doesn't. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 04/26/2016 07:33 AM, Rob Herring wrote: > On Mon, Apr 25, 2016 at 10:01:52AM -0500, Thor Thayer wrote: >> >> >> On 04/25/2016 07:55 AM, Rob Herring wrote: >>> On Fri, Apr 22, 2016 at 10:33:29AM -0500, tthayer@opensource.altera.com wrote: >>>> From: Thor Thayer <tthayer@opensource.altera.com> >>>> >>>> The Altera Arria10 Devkit System Resource chip is a Multi-Function >>>> Device with a GPIO expander and power supply alarm. >>> >>> An alarm with no interrupt? Are the GPIOs interrupt capable? >> >> The alarms have no interrupt but the GPIOs are interrupt capable (not >> implemented with interrupt capability in this patch series). > > Please make the binding as complete as possible even if you don't > support features in the driver yet. Driver features may evolve, but a > specific h/w doesn't. > > Rob > Understood. Thanks for the explanation. -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" 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/altera-a10sr.txt b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt new file mode 100644 index 0000000..8e17d05 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt @@ -0,0 +1,35 @@ +* Altera Arria10 Development Kit System Resource Chip + +Required parent device properties: +- compatible : "altr,a10sr" +- spi-max-frequency : Maximum SPI frequency. +- reg : The SPI Chip Select address for the Arria10 + System Resource chip + +The A10SR consists of this varied group of sub-devices: + +Device Description +------ ---------- +a10sr_gpio GPIO Controller + +Arria10 GPIO +Required Properties: +- compatible : Should be "altr,a10sr-gpio" +- gpio-controller : Marks the device node as a GPIO Controller. +- #gpio-cells : Should be two. The first cell is the pin number and + the second cell is used to specify flags. + See ../gpio/gpio.txt for more information. + +Example: + + resource-manager@0 { + compatible = "altr,a10sr"; + reg = <0>; + spi-max-frequency = <100000>; + + a10sr_gpio: gpio-controller { + compatible = "altr,a10sr-gpio"; + gpio-controller; + #gpio-cells = <2>; + }; + };