On Fri, Aug 19, 2016 at 12:53:34PM +0200, Jonas Gorski wrote: > Add binding documentation for the pincontrol core found in BCM6328 SoCs. > > Signed-off-by: Jonas Gorski <jonas.gorski@xxxxxxxxx> > --- > .../bindings/pinctrl/brcm,bcm6328-pinctrl.txt | 61 ++++++++++++++++++++++ > 1 file changed, 61 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pinctrl/brcm,bcm6328-pinctrl.txt > > diff --git a/Documentation/devicetree/bindings/pinctrl/brcm,bcm6328-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/brcm,bcm6328-pinctrl.txt > new file mode 100644 > index 0000000..401224e > --- /dev/null > +++ b/Documentation/devicetree/bindings/pinctrl/brcm,bcm6328-pinctrl.txt > @@ -0,0 +1,61 @@ > +* Broadcom BCM6328 pin controller > + > +Required properties: > +- compatible: Must be "brcm,bcm6328-pinctrl". > +- reg: Register specifies of dirout, dat, mode, mux registers. > +- reg-names: Must be "dirout", "dat", "mode", "mux". > +- gpio-controller: Identifies this node as a GPIO controller. > +- #gpio-cells: Must be <2>. > + > +Example: > + > +pinctrl: pin-controller@10000080 { > + compatible = "brcm,bcm6328-pinctrl"; > + reg = <0x10000080 0x8>, > + <0x10000088 0x8>, > + <0x10000098 0x4>, > + <0x1000009c 0xc>; > + reg-names = "dirout", "dat", "mode", "mux"; What's in the holes? Just do one range or perhaps this should be part of some larger block. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html