* [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 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 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 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
* [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
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.