On Tue, Feb 16, 2016 at 09:41:17AM +0100, Steffen Trumtrar wrote:
>
> Hi!
>
> Shawn Guo writes:
>
> > On Thu, Feb 11, 2016 at 03:04:43PM +0100, Steffen Trumtrar wrote:
> >> Add documentation for the Freescale Security Controller (SCC)
> >> found on i.MX25 SoCs.
> >>
> >> Signed-off-by: Steffen Trumtrar <s.trumtrar@xxxxxxxxxxxxxx>
> >> ---
> >> .../devicetree/bindings/crypto/fsl-imx-scc.txt | 17 +++++++++++++++++
> >> 1 file changed, 17 insertions(+)
> >> create mode 100644 Documentation/devicetree/bindings/crypto/fsl-imx-scc.txt
> >>
> >> diff --git a/Documentation/devicetree/bindings/crypto/fsl-imx-scc.txt b/Documentation/devicetree/bindings/crypto/fsl-imx-scc.txt
> >> new file mode 100644
> >> index 000000000000..9a15ab8da94f
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/crypto/fsl-imx-scc.txt
> >> @@ -0,0 +1,17 @@
> >> +Freescale Security Controller (SCC)
> >> +
> >> +Required properties:
> >> +- compatible : Should be "fsl,imx25-scc".
> >> +- reg : Should contain register location and length.
> >> +- interrupts : Should contain interrupt numbers for SCM IRQ and SMN IRQ.
> >> +
> >> +Example:
> >> +
> >> + scc: crypto@53fac000 {
> >> + compatible = "fsl,imx25-scc";
> >> + reg = <0x53fac000 0x4000>;
> >> + clocks = <&clks 111>;
> >> + clock-names = "ipg";
> >
> > Shouldn't these two be mentioned in properties list above too?
> >
>
> What is the current policy on clocks in bindings? There are some with
> and some without.
>
> Of course I can add them to the required properties if necessary.
I think any required properties should be listed in "Required
properties:" section.
Shawn
--
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
