On Fri, Oct 07, 2016 at 06:18:30PM +0300, Pantelis Antoniou wrote: > From: Georgi Vlaev <gvlaev@xxxxxxxxxxx> > > Add DT bindings document for the SAM MFD device. > > Signed-off-by: Georgi Vlaev <gvlaev@xxxxxxxxxxx> > [Ported from Juniper kernel] > Signed-off-by: Pantelis Antoniou <pantelis.antoniou@xxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/mfd/jnx-sam.txt | 94 +++++++++++++++++++++++ > 1 file changed, 94 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/jnx-sam.txt > > diff --git a/Documentation/devicetree/bindings/mfd/jnx-sam.txt b/Documentation/devicetree/bindings/mfd/jnx-sam.txt > new file mode 100644 > index 0000000..b4af7ea > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/jnx-sam.txt > @@ -0,0 +1,94 @@ > +Device-Tree bindings for Juniper Networks SAM MFD > + > +Required properties: > + > +- compatible - Must be: "jnx,sam" Kind of generic. Only 1 FPGA version or some other way to tell the version? > + > +Optional properties: > + > +- pma-coefficients: A set of tupples containing the configuration of the PMA. What's a PMA? What type of configuration? How many entries? > +Device Description > +------ ----------- > +jnx,i2c-sam : I2C mux driver > +jnx,gpio-sam : GPIO block > +jnx,flash-sam : MTD Flash > +jnx,mdio-sam : MDIO interfaces > + > +All these optional nodes are described in their respective binding > +documents. > + > +Example node: > + > +pci-0000-10-00.0 { What are the numbers? > + compatible = "jnx,sam"; If this is a PCI device, then it should use PCI compatible string syntax. > + #address-cells = <1>; > + #size-cells = <0>; > + pma-coefficients = <4 0x0>; > + > + i2c-sam@10 { > + compatible = "jnx,i2c-sam"; > + mux-channels = <2>; > + master-offset = <0x10000>; > + }; > + > + gpiogpqam0: gpio-sam@10 { gpio@... Where does 10 come from? > + compatible = "jnx,gpio-sam"; > + gpio-controller; > + #gpio-cells = <2>; > + gpio-count = <297>; > + interrupt-controller; > + /* > + * 1st cell: gpio interrupt status bit > + * 2nd cell: 1st pin > + * 3rd cell: # of pins > + */ > + gpio-interrupts = > + <0 0 12>, /* phy_int_monitor_en [16] */ > + <1 235 24>, /* qsfpp_fpga_int_monitor [17] */ > + <2 259 24>, /* qsfpp_fpga_modprs_monitor [18] */ > + <3 295 1>, /* si5345_fpga_monitor [19] */ > + <4 294 1>; /* fpc_pic_int_monitor [20] */ > + }; > + > + flash-sam@10 { > + compatible = "jnx,flash-sam"; > + #address-cells = <1>; > + #size-cells = <1>; > + partition@0 { > + reg = <0x0 0x400000>; > + label = "pic0-golden"; > + read-only; > + }; > + partition@400000 { > + reg = <0x400000 0x400000>; > + label = "pic0-user"; > + }; > + }; > + > + mdio-sam@10 { > + compatible = "jnx,mdio-sam"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x40000>; > + > + /* mii_bus types */ > + mdio0: mdio-sam@0 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x0>; > + }; > + > + mdio1: mdio-sam@4000 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x4000>; > + }; > + > + mdio2: mdio-sam@8000 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x8000>; > + }; > + }; > +}; > -- > 1.9.1 > -- 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