diff mbox

[v2,3/3] arm64: dts: marvell: add a comment on the cp110 slave node status

Message ID 20170615121004.32257-3-antoine.tenart@free-electrons.com (mailing list archive)
State New, archived
Headers show

Commit Message

Antoine Tenart June 15, 2017, 12:10 p.m. UTC
The cryptographic engine found on the cp110 slave is disabled by default
because of some known limitations. Add a comment to explain why it is
disabled by default.

Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
---
 arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Gregory CLEMENT June 15, 2017, 4:27 p.m. UTC | #1
Hi Antoine,
 
 On jeu., juin 15 2017, Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> The cryptographic engine found on the cp110 slave is disabled by default
> because of some known limitations. Add a comment to explain why it is
> disabled by default.
>
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>


Applied on mvebu/dt64

Thanks,

Gregory
> ---
>  arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> index 7e98c155cc6a..385ac30d6012 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> @@ -234,6 +234,13 @@
>  						  "ring2", "ring3", "eip";
>  				clocks = <&cps_syscon0 1 26>;
>  				dma-mask = <0xff 0xffffffff>;
> +				/*
> +				 * The cryptographic engine found on the cp110
> +				 * master is enabled by default at the SoC
> +				 * level. Because it is not possible as of now
> +				 * to enable two cryptographic engines in
> +				 * parallel, disable this one by default.
> +				 */
>  				status = "disabled";
>  			};
>  		};
> -- 
> 2.9.4
>
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
index 7e98c155cc6a..385ac30d6012 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
@@ -234,6 +234,13 @@ 
 						  "ring2", "ring3", "eip";
 				clocks = <&cps_syscon0 1 26>;
 				dma-mask = <0xff 0xffffffff>;
+				/*
+				 * The cryptographic engine found on the cp110
+				 * master is enabled by default at the SoC
+				 * level. Because it is not possible as of now
+				 * to enable two cryptographic engines in
+				 * parallel, disable this one by default.
+				 */
 				status = "disabled";
 			};
 		};