2022-10-31 15:56:23

by Jayesh Choudhary

[permalink] [raw]
Subject: [PATCH 0/3] Drop dma-coherent property from SA2UL

crypto driver itself is not dma-coherent. It is the system-dma
that moves data and so 'dma-coherent' property should be dropped.

This series drop it from the crypto nodes for the J7 family of
TI SoCs.

DT binding fixes have already been merged[0].

[0]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a536208da6f7d877f1adbad4ff13f63f31f59d91

Jayesh Choudhary (3):
arm64: dts: ti: k3-am65-main: drop dma-coherent in crypto node
arm64: dts: ti: k3-j721e-main: drop dma-coherent in crypto node
arm64: dts: ti: k3-j7200-mcu-wakeup: drop dma-coherent in crypto node

arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 1 -
3 files changed, 3 deletions(-)

--
2.25.1



2022-10-31 15:58:12

by Jayesh Choudhary

[permalink] [raw]
Subject: [PATCH 3/3] arm64: dts: ti: k3-j7200-mcu-wakeup: drop dma-coherent in crypto node

crypto driver itself is not dma-coherent. So drop it.

Fixes: d683a73980a6 ("arm64: dts: ti: k3-j7200-mcu-wakeup: Add SA2UL node")
Signed-off-by: Jayesh Choudhary <[email protected]>
---
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 2006933f9b49..e85106f55bee 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -391,7 +391,6 @@ mcu_crypto: crypto@40900000 {
dmas = <&mcu_udmap 0xf501>, <&mcu_udmap 0x7502>,
<&mcu_udmap 0x7503>;
dma-names = "tx", "rx1", "rx2";
- dma-coherent;

rng: rng@40910000 {
compatible = "inside-secure,safexcel-eip76";
--
2.25.1


2022-11-15 20:26:44

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH 0/3] Drop dma-coherent property from SA2UL

Hi Jayesh Choudhary,

On Mon, 31 Oct 2022 20:55:17 +0530, Jayesh Choudhary wrote:
> crypto driver itself is not dma-coherent. It is the system-dma
> that moves data and so 'dma-coherent' property should be dropped.
>
> This series drop it from the crypto nodes for the J7 family of
> TI SoCs.
>
> DT binding fixes have already been merged[0].
>
> [...]

(note: I picked up Manorit's reviewed-by as well)

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/3] arm64: dts: ti: k3-am65-main: drop dma-coherent in crypto node
commit: b86833ab3653dbb0dc453eec4eef8615e63de4e2
[2/3] arm64: dts: ti: k3-j721e-main: drop dma-coherent in crypto node
commit: 26c5012403f3f1fd3bf8f7d3389ee539ae5cc162
[3/3] arm64: dts: ti: k3-j7200-mcu-wakeup: drop dma-coherent in crypto node
commit: f00f26711d7183f8675c5591ba8daaabe94be452

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D