[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150810091725.GB2273@Sanchayan-Arch.toradex.int>
Date: Mon, 10 Aug 2015 14:47:25 +0530
From: maitysanchayan@...il.com
To: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
Cc: linux-arm-kernel@...ts.infradead.org,
maxime.ripard@...e-electrons.com, stefan@...er.ch,
shawn.guo@...nel.org, kernel@...gutronix.de,
stefan.wahren@...e.com, linux-kernel@...r.kernel.org,
p.zabel@...gutronix.de
Subject: Re: [PATCH v7 4/4] nvmem: Add DT binding documentation for Vybrid
OCOTP driver
Hello,
On 15-08-10 10:18:01, Srinivas Kandagatla wrote:
>
>
> On 06/08/15 16:27, Sanchayan Maity wrote:
> > Add the devicetree bindings for the Freescale Vybrid On-Chip
> > OTP driver.
> >
> > Signed-off-by: Sanchayan Maity <maitysanchayan@...il.com>
> > ---
> > .../devicetree/bindings/nvmem/vf610-ocotp.txt | 20 ++++++++++++++++++++
> > 1 file changed, 20 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/nvmem/vf610-ocotp.txt
> >
> > diff --git a/Documentation/devicetree/bindings/nvmem/vf610-ocotp.txt b/Documentation/devicetree/bindings/nvmem/vf610-ocotp.txt
> > new file mode 100644
> > index 0000000..5556810
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/nvmem/vf610-ocotp.txt
> > @@ -0,0 +1,20 @@
> > +On-Chip OTP Memory for Freescale Vybrid
> > +
> > +Required Properties:
> > + compatible:
> > + - "fsl,vf610-ocotp" for VF5xx/VF6xx
> > + #address-cells : Should be 1
> > + #size-cells : Should be 1
> > + reg : Address and length of OTP controller registers
>
> Is there a reason to not add clocks property in to the bindings?
An error on my part. Will fix with the next revision.
- Sanchayan.
>
> > +
> > +Example for Vybrid VF5xx/VF6xx:
> > +
> > + ocotp: ocotp@...a5000 {
> > + compatible = "fsl,vf610-ocotp";
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + reg = <0x400a5000 0xD00>;
> > + clocks = <&clks VF610_CLK_OCOTP>;
> > + clock-names = "ocotp";
> > + };
> > +
> >
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists