| Message ID | 1446772644-2352-5-git-send-email-zain.wang@rock-chips.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Herbert Xu |
| Headers | show |
On Fri, Nov 06, 2015 at 09:17:24AM +0800, Zain Wang wrote: > Add DT bindings documentation for the rk3288 crypto drivers. > > Signed-off-by: Zain Wang <zain.wang@rock-chips.com> Acked-by: Rob Herring <robh@kernel.org> > --- > > Changde in v2: > - None > > Changed in v1: > - remove the _crypto suffix > - use "rockchip,rk3288-crypto" instead of "rockchip,rk3288" > - remove the description of status > > .../devicetree/bindings/crypto/rockchip-crypto.txt | 29 ++++++++++++++++++++++ > 1 file changed, 29 insertions(+) > create mode 100644 Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > > diff --git a/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > new file mode 100644 > index 0000000..d27e203 > --- /dev/null > +++ b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > @@ -0,0 +1,29 @@ > +Rockchip Electronics And Security Accelerator > + > +Required properties: > +- compatible: Should be "rockchip,rk3288-crypto" > +- reg: base physical address of the engine and length of memory mapped > + region > +- interrupts: interrupt number > +- clocks: reference to the clocks about crypto > +- clock-names: "aclk" used to clock data > + "hclk" used to clock data > + "srst" used to clock crypto accelerator > + "apb_pclk" used to clock dma > + > +Examples: > + > + crypto: cypto-controller@ff8a0000 { > + compatible = "rockchip,rk3288-crypto"; > + reg = <0xff8a0000 0x4000>; > + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru ACLK_CRYPTO>, > + <&cru HCLK_CRYPTO>, > + <&cru SCLK_CRYPTO>, > + <&cru ACLK_DMAC1>; > + clock-names = "aclk", > + "hclk", > + "sclk", > + "apb_pclk"; > + status = "okay"; > + }; > -- > 1.9.1 > > -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Am Freitag, 6. November 2015, 09:17:24 schrieb Zain Wang: > Add DT bindings documentation for the rk3288 crypto drivers. > > Signed-off-by: Zain Wang <zain.wang@rock-chips.com> > --- > > Changde in v2: > - None > > Changed in v1: > - remove the _crypto suffix > - use "rockchip,rk3288-crypto" instead of "rockchip,rk3288" > - remove the description of status > > .../devicetree/bindings/crypto/rockchip-crypto.txt | 29 ++++++++++++++++++++++ > 1 file changed, 29 insertions(+) > create mode 100644 Documentation/devicetree/bindings/crypto/rockchip- crypto.txt > > diff --git a/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > new file mode 100644 > index 0000000..d27e203 > --- /dev/null > +++ b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > @@ -0,0 +1,29 @@ > +Rockchip Electronics And Security Accelerator > + > +Required properties: > +- compatible: Should be "rockchip,rk3288-crypto" > +- reg: base physical address of the engine and length of memory mapped > + region > +- interrupts: interrupt number > +- clocks: reference to the clocks about crypto > +- clock-names: "aclk" used to clock data > + "hclk" used to clock data > + "srst" used to clock crypto accelerator mistmatch between description and example. I think you mean "sclk" instead of "srst", like we talked about in the first version. > + "apb_pclk" used to clock dma > + > +Examples: > + > + crypto: cypto-controller@ff8a0000 { > + compatible = "rockchip,rk3288-crypto"; > + reg = <0xff8a0000 0x4000>; > + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru ACLK_CRYPTO>, > + <&cru HCLK_CRYPTO>, > + <&cru SCLK_CRYPTO>, > + <&cru ACLK_DMAC1>; > + clock-names = "aclk", > + "hclk", > + "sclk", > + "apb_pclk"; > + status = "okay"; > + }; > -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi On 2015?11?06? 17:00, Heiko Stuebner wrote: > Am Freitag, 6. November 2015, 09:17:24 schrieb Zain Wang: >> Add DT bindings documentation for the rk3288 crypto drivers. >> >> Signed-off-by: Zain Wang <zain.wang@rock-chips.com> >> --- >> >> Changde in v2: >> - None >> >> Changed in v1: >> - remove the _crypto suffix >> - use "rockchip,rk3288-crypto" instead of "rockchip,rk3288" >> - remove the description of status >> >> .../devicetree/bindings/crypto/rockchip-crypto.txt | 29 > ++++++++++++++++++++++ >> 1 file changed, 29 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/crypto/rockchip- > crypto.txt >> diff --git a/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt > b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt >> new file mode 100644 >> index 0000000..d27e203 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt >> @@ -0,0 +1,29 @@ >> +Rockchip Electronics And Security Accelerator >> + >> +Required properties: >> +- compatible: Should be "rockchip,rk3288-crypto" >> +- reg: base physical address of the engine and length of memory mapped >> + region >> +- interrupts: interrupt number >> +- clocks: reference to the clocks about crypto >> +- clock-names: "aclk" used to clock data >> + "hclk" used to clock data >> + "srst" used to clock crypto accelerator > mistmatch between description and example. I think you mean "sclk" instead of > "srst", like we talked about in the first version. I am sorry that forgetting to change it here. You can see I have changed it in other part like example below. > >> + "apb_pclk" used to clock dma >> + >> +Examples: >> + >> + crypto: cypto-controller@ff8a0000 { >> + compatible = "rockchip,rk3288-crypto"; >> + reg = <0xff8a0000 0x4000>; >> + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cru ACLK_CRYPTO>, >> + <&cru HCLK_CRYPTO>, >> + <&cru SCLK_CRYPTO>, >> + <&cru ACLK_DMAC1>; >> + clock-names = "aclk", >> + "hclk", >> + "sclk", >> + "apb_pclk"; >> + status = "okay"; >> + }; >> > > > Thanks Zain -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt new file mode 100644 index 0000000..d27e203 --- /dev/null +++ b/Documentation/devicetree/bindings/crypto/rockchip-crypto.txt @@ -0,0 +1,29 @@ +Rockchip Electronics And Security Accelerator + +Required properties: +- compatible: Should be "rockchip,rk3288-crypto" +- reg: base physical address of the engine and length of memory mapped + region +- interrupts: interrupt number +- clocks: reference to the clocks about crypto +- clock-names: "aclk" used to clock data + "hclk" used to clock data + "srst" used to clock crypto accelerator + "apb_pclk" used to clock dma + +Examples: + + crypto: cypto-controller@ff8a0000 { + compatible = "rockchip,rk3288-crypto"; + reg = <0xff8a0000 0x4000>; + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&cru ACLK_CRYPTO>, + <&cru HCLK_CRYPTO>, + <&cru SCLK_CRYPTO>, + <&cru ACLK_DMAC1>; + clock-names = "aclk", + "hclk", + "sclk", + "apb_pclk"; + status = "okay"; + };
Add DT bindings documentation for the rk3288 crypto drivers. Signed-off-by: Zain Wang <zain.wang@rock-chips.com> --- Changde in v2: - None Changed in v1: - remove the _crypto suffix - use "rockchip,rk3288-crypto" instead of "rockchip,rk3288" - remove the description of status .../devicetree/bindings/crypto/rockchip-crypto.txt | 29 ++++++++++++++++++++++ 1 file changed, 29 insertions(+) create mode 100644 Documentation/devicetree/bindings/crypto/rockchip-crypto.txt