Skip to contentSkip to conversation

arm64: dts: marvell: mark the cp110 crypto engine as dma coherent

PATCH New Antoine Tenart 2 participants 4 messages Jul 17, 2017 – Jul 18, 2017

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 arm64: dts: marvell: mark the cp110 crypto engine as dma coherent

The crypto engines found on the cp110 master and slave are dma coherent. This patch adds the relevant property to their dt nodes.

---
 arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 1 +
 arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi  | 1 +
 2 files changed, 2 insertions(+)
arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi +1 -0
index 726528ce54e9..4c68605675a8 100644
@@ -270,6 +270,7 @@
 				interrupt-names = "mem", "ring0", "ring1",
 				"ring2", "ring3", "eip";
 				clocks = <&cpm_clk 1 26>;
+				dma-coherent;
 			};
 		};
 
arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi +1 -0
index 95f8e5f607f6..0dbb59811d00 100644
@@ -261,6 +261,7 @@
 				interrupt-names = "mem", "ring0", "ring1",
 						  "ring2", "ring3", "eip";
 				clocks = <&cps_clk 1 26>;
+				dma-coherent;
 				/*
 				 * The cryptographic engine found on the cp110
 				 * master is enabled by default at the SoC
Signature · 1 line
2.13.3
#1 lore raw

PATCH 1/1 arm64: dts: marvell: mark the cp110 crypto engine as dma coherent

Hi Antoine,

Quoted 2 lines · Antoine Tenart

The crypto engines found on the cp110 master and slave are dma coherent. This patch adds the relevant property to their dt nodes.

Is it a fix?

Should it go to the current ans stable kernel?

Thanks,

Gregory

Quoted 34 lines
---
 arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 1 +
 arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi  | 1 +
 2 files changed, 2 insertions(+)
arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi +1 -0
index 726528ce54e9..4c68605675a8 100644
@@ -270,6 +270,7 @@
 				interrupt-names = "mem", "ring0", "ring1",
 				"ring2", "ring3", "eip";
 				clocks = <&cpm_clk 1 26>;
+				dma-coherent;
 			};
 		};
 
arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi +1 -0
index 95f8e5f607f6..0dbb59811d00 100644
@@ -261,6 +261,7 @@
 				interrupt-names = "mem", "ring0", "ring1",
 						  "ring2", "ring3", "eip";
 				clocks = <&cps_clk 1 26>;
+				dma-coherent;
 				/*
 				 * The cryptographic engine found on the cp110
 				 * master is enabled by default at the SoC
Signature · 1 line
2.13.3
Signature · 4 lines
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH 1/1 arm64: dts: marvell: mark the cp110 crypto engine as dma coherent

Hi Gregory,

Quoted 8 lines · Gregory CLEMENT
Quoted 2 lines · Antoine Tenart

The crypto engines found on the cp110 master and slave are dma coherent. This patch adds the relevant property to their dt nodes.

Is it a fix?

Should it go to the current ans stable kernel?

Yes I think that should go into stable. I forgot the stable tag, do you want me to send a v2 with it included?

Thanks!
Antoine

Signature · 10 lines
Antoine T?nart, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170718/7ad85945/attachment-0001.sig>
#3 lore raw

PATCH 1/1 arm64: dts: marvell: mark the cp110 crypto engine as dma coherent

Hi Antoine,

Quoted 8 lines · Antoine Tenart

Hi Gregory,

Quoted 5 lines · Gregory CLEMENT
Quoted 2 lines · Antoine Tenart

The crypto engines found on the cp110 master and slave are dma coherent. This patch adds the relevant property to their dt nodes.

Quoted 7 lines
Quoted 3 lines

Is it a fix?

Should it go to the current ans stable kernel?

Yes I think that should go into stable. I forgot the stable tag, do you want me to send a v2 with it included?

Yes please and also add the Fixes tag.

Thanks,

Gregory

Quoted 9 lines

Thanks!
Antoine

Signature · 3 lines
Antoine T?nart, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
Signature · 4 lines
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com