Re: [PATCHv2 1/8] dt-bindings: mfd: Add Altera Arria10 System Resource Chip bindings

[Lee Jones <lee.jones@xxxxxxxxxx>]

On Mon, 10 Oct 2016, Thor Thayer wrote:

> Hi Lee,
> 
> On 06/10/2016 09:47 AM, Lee Jones wrote:
> > On Thu, 02 Jun 2016, tthayer@xxxxxxxxxxxxxxxxxxxxx wrote:
> > 
> > > From: Thor Thayer <tthayer@xxxxxxxxxxxxxxxxxxxxx>
> > > 
> > > The Altera Arria10 Devkit System Resource chip is a Multi-Function
> > > Device with a GPIO expander.
> > > 
> > > This patch adds documentation for the Altera A10-SR DT bindings.
> > > 
> > > Signed-off-by: Thor Thayer <tthayer@xxxxxxxxxxxxxxxxxxxxx>
> > > ---
> > > v2  Addition of IRQ controller bindings.
> > >     Removal of hw_mon sub-device.
> > > ---
> > >  .../devicetree/bindings/mfd/altera-a10sr.txt       |   46 ++++++++++++++++++++
> > >  1 file changed, 46 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/mfd/altera-a10sr.txt
> > 
> > Applied, thanks.
> 
> For some reason, I'm not seeing this in the 4.8 tree with the other mfd
> additions. Do I need to submit it to a different tree?

No, it looks like this one was dropped for some reason.

Reapplying now.

> > > diff --git a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt
> > > new file mode 100644
> > > index 0000000..ea151f2
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt
> > > @@ -0,0 +1,46 @@
> > > +* 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
> > > +- interrupt-parent	: The parent interrupt controller.
> > > +- interrupts		: The interrupt line the device is connected to.
> > > +- interrupt-controller	: Marks the device node as an interrupt controller.
> > > +- #interrupt-cells	: The number of cells to describe an IRQ, should be 2.
> > > +			    The first cell is the IRQ number.
> > > +			    The second cell is the flags, encoded as trigger
> > > +			    masks from ../interrupt-controller/interrupts.txt.
> > > +
> > > +The A10SR consists of these 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>;
> > > +		interrupt-parent = <&portb>;
> > > +		interrupts = <5 IRQ_TYPE_LEVEL_LOW>;
> > > +		interrupt-controller;
> > > +		#interrupt-cells = <2>;
> > > +
> > > +		a10sr_gpio: gpio-controller {
> > > +			compatible = "altr,a10sr-gpio";
> > > +			gpio-controller;
> > > +			#gpio-cells = <2>;
> > > +		};
> > > +	};
> > 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html