* [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level
@ 2017-06-15 12:10 Antoine Tenart
2017-06-15 12:10 ` [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files Antoine Tenart
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Antoine Tenart @ 2017-06-15 12:10 UTC (permalink / raw)
To: linux-arm-kernel
Enable the cryptographic engine at the SoC level on the master cp110.
This engine is always present and do not depends on any pinmux
configuration.
Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons•com>
---
arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
index c4294403276b..54cc3fc7b6a4 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
@@ -245,7 +245,6 @@
"ring2", "ring3", "eip";
clocks = <&cpm_syscon0 1 26>;
dma-mask = <0xff 0xffffffff>;
- status = "disabled";
};
};
--
2.9.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files
2017-06-15 12:10 [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Antoine Tenart
@ 2017-06-15 12:10 ` Antoine Tenart
2017-06-15 16:27 ` Gregory CLEMENT
2017-06-15 12:10 ` [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status Antoine Tenart
2017-06-15 16:25 ` [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Gregory CLEMENT
2 siblings, 1 reply; 6+ messages in thread
From: Antoine Tenart @ 2017-06-15 12:10 UTC (permalink / raw)
To: linux-arm-kernel
The cryptographic engine on the master cp110 is now enabled by default
at the SoC level. Remove its dts nodes that were only enabling it.
Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons•com>
---
arch/arm64/boot/dts/marvell/armada-7040-db.dts | 4 ----
arch/arm64/boot/dts/marvell/armada-8040-db.dts | 4 ----
2 files changed, 8 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-7040-db.dts b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
index 771311051421..92c761c380d3 100644
--- a/arch/arm64/boot/dts/marvell/armada-7040-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
@@ -187,7 +187,3 @@
phy = <&phy1>;
phy-mode = "rgmii-id";
};
-
-&cpm_crypto {
- status = "okay";
-};
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-db.dts b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
index 6af3c39bbb99..1e8f7242ed6f 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
@@ -142,10 +142,6 @@
phy-mode = "rgmii-id";
};
-&cpm_crypto {
- status = "okay";
-};
-
/* CON5 on CP1 expansion */
&cps_pcie2 {
status = "okay";
--
2.9.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status
2017-06-15 12:10 [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Antoine Tenart
2017-06-15 12:10 ` [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files Antoine Tenart
@ 2017-06-15 12:10 ` Antoine Tenart
2017-06-15 16:27 ` Gregory CLEMENT
2017-06-15 16:25 ` [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Gregory CLEMENT
2 siblings, 1 reply; 6+ messages in thread
From: Antoine Tenart @ 2017-06-15 12:10 UTC (permalink / raw)
To: linux-arm-kernel
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(+)
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
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level
2017-06-15 12:10 [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Antoine Tenart
2017-06-15 12:10 ` [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files Antoine Tenart
2017-06-15 12:10 ` [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status Antoine Tenart
@ 2017-06-15 16:25 ` Gregory CLEMENT
2 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2017-06-15 16:25 UTC (permalink / raw)
To: linux-arm-kernel
Hi Antoine,
On jeu., juin 15 2017, Antoine Tenart <antoine.tenart@free-electrons•com> wrote:
> Enable the cryptographic engine at the SoC level on the master cp110.
> This engine is always present and do not depends on any pinmux
> configuration.
>
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons•com>
Applied on mvebu/dt64
Thanks,
Gregory
> ---
> arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
> index c4294403276b..54cc3fc7b6a4 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
> @@ -245,7 +245,6 @@
> "ring2", "ring3", "eip";
> clocks = <&cpm_syscon0 1 26>;
> dma-mask = <0xff 0xffffffff>;
> - status = "disabled";
> };
> };
>
> --
> 2.9.4
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files
2017-06-15 12:10 ` [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files Antoine Tenart
@ 2017-06-15 16:27 ` Gregory CLEMENT
0 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2017-06-15 16:27 UTC (permalink / raw)
To: linux-arm-kernel
Hi Antoine,
On jeu., juin 15 2017, Antoine Tenart <antoine.tenart@free-electrons•com> wrote:
> The cryptographic engine on the master cp110 is now enabled by default
> at the SoC level. Remove its dts nodes that were only enabling it.
>
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons•com>
Applied on mvebu/dt64
Thanks,
Gregory
> ---
> arch/arm64/boot/dts/marvell/armada-7040-db.dts | 4 ----
> arch/arm64/boot/dts/marvell/armada-8040-db.dts | 4 ----
> 2 files changed, 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-7040-db.dts b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> index 771311051421..92c761c380d3 100644
> --- a/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> @@ -187,7 +187,3 @@
> phy = <&phy1>;
> phy-mode = "rgmii-id";
> };
> -
> -&cpm_crypto {
> - status = "okay";
> -};
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-db.dts b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
> index 6af3c39bbb99..1e8f7242ed6f 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-db.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
> @@ -142,10 +142,6 @@
> phy-mode = "rgmii-id";
> };
>
> -&cpm_crypto {
> - status = "okay";
> -};
> -
> /* CON5 on CP1 expansion */
> &cps_pcie2 {
> status = "okay";
> --
> 2.9.4
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status
2017-06-15 12:10 ` [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status Antoine Tenart
@ 2017-06-15 16:27 ` Gregory CLEMENT
0 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2017-06-15 16:27 UTC (permalink / raw)
To: linux-arm-kernel
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
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-06-15 16:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-15 12:10 [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Antoine Tenart
2017-06-15 12:10 ` [PATCH v2 2/3] arm64: dts: marvell: remove cpm crypto nodes from dts files Antoine Tenart
2017-06-15 16:27 ` Gregory CLEMENT
2017-06-15 12:10 ` [PATCH v2 3/3] arm64: dts: marvell: add a comment on the cp110 slave node status Antoine Tenart
2017-06-15 16:27 ` Gregory CLEMENT
2017-06-15 16:25 ` [PATCH v2 1/3] arm64: dts: marvell: cp110: enable the crypto engine at the SoC level Gregory CLEMENT
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox