Re: [PATCH v2 4/4] crypto: rockchip/crypto - add DT bindings documentation
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 >> --- >> >> 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 000..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@ff8a { >> +compatible = "rockchip,rk3288-crypto"; >> +reg = <0xff8a 0x4000>; >> +interrupts = ; >> +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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 4/4] crypto: rockchip/crypto - add DT bindings documentation
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 > --- > > 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 000..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@ff8a { > + compatible = "rockchip,rk3288-crypto"; > + reg = <0xff8a 0x4000>; > + interrupts = ; > + 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 4/4] crypto: rockchip/crypto - add DT bindings documentation
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 Acked-by: Rob Herring > --- > > 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 000..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@ff8a { > + compatible = "rockchip,rk3288-crypto"; > + reg = <0xff8a 0x4000>; > + interrupts = ; > + 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html