| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20251030160635.1388401-1-b-padhi@ti.com>
Date: Thu, 30 Oct 2025 21:36:35 +0530
From: Beleswar Padhi <b-padhi@...com>
To: <nm@...com>, <vigneshr@...com>, <kristo@...nel.org>, <robh@...nel.org>,
<krzk+dt@...nel.org>, <conor+dt@...nel.org>
CC: <afd@...com>, <u-kumar1@...com>, <hnagalla@...com>, <jm@...com>,
<hiago.franco@...adex.com>, <francesco.dolcini@...adex.com>,
<b-padhi@...com>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>
Subject: [PATCH] arm64: dts: ti: k3-{j7/am6}*-ti-ipc-firmware: Limit FIFOs to Linux use
Each mailbox cluster has 16 hardware FIFOs shared among 4 users (CPUs).
Each FIFO supports one-way communication between two users and is
configured by the firmware.
For TI IPC firmware, the FIFOs starting from 0 are assigned for
communication between the Cortex A-core (running Linux) and remote
processors (running RTOS). The remaining FIFOs are used for
RTOS-to-RTOS communication.
In some cases, pending messages may remain in the RTOS-to-RTOS FIFOs if
a remote processor is powered off or in a bad state. To avoid issues
such as suspend failures, restrict the 'ti,mbox-num-fifos' property in
the device tree to only include the FIFOs used for Linux-to-RTOS
communication. This ensures the Linux mailbox driver checks only its
own FIFOs and does not interfere with those used by other remote
processors.
Fixes: a49f991e740f ("arm64: dts: ti: k3-am62-verdin: Add missing cfg for TI IPC Firmware")
Closes: https://lore.kernel.org/all/sid7gtg5vay5qgicsl6smnzwg5mnneoa35cempt5ddwjvedaio@hzsgcx6oo74l/
Signed-off-by: Beleswar Padhi <b-padhi@...com>
---
Cc: Francesco Dolcini <francesco.dolcini@...adex.com>
Cc: Hiago De Franco <hiago.franco@...adex.com>
Please help in testing the patch on Toradex platforms.
Testing Done:
1. Tested Boot across all TI K3 EVM/SK boards.
2. Tested IPC on all TI K3 J7* EVM/SK boards (& AM62x SK).
3. Tested that the patch generates no new warnings/errors.
arch/arm64/boot/dts/ti/k3-am62-ti-ipc-firmware.dtsi | 1 +
arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi | 3 +++
arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi | 2 ++
arch/arm64/boot/dts/ti/k3-am64-ti-ipc-firmware.dtsi | 3 +++
arch/arm64/boot/dts/ti/k3-am65-ti-ipc-firmware.dtsi | 2 ++
arch/arm64/boot/dts/ti/k3-j7200-ti-ipc-firmware.dtsi | 2 ++
arch/arm64/boot/dts/ti/k3-j721e-ti-ipc-firmware.dtsi | 5 +++++
arch/arm64/boot/dts/ti/k3-j721s2-ti-ipc-firmware.dtsi | 4 ++++
arch/arm64/boot/dts/ti/k3-j722s-ti-ipc-firmware.dtsi | 4 ++++
.../dts/ti/k3-j784s4-j742s2-ti-ipc-firmware-common.dtsi | 6 ++++++
arch/arm64/boot/dts/ti/k3-j784s4-ti-ipc-firmware.dtsi | 1 +
11 files changed, 33 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am62-ti-ipc-firmware.dtsi
index ea69fab9b52b..913bd5ff49f7 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-ti-ipc-firmware.dtsi
@@ -21,6 +21,7 @@ mcu_m4fss_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
+ ti,mbox-num-fifos = <4>;
mbox_m4_0: mbox-m4-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
index 950f4f37d477..b377edb52bc9 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
@@ -33,6 +33,7 @@ mcu_r5fss0_core0_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_r5_0: mbox-r5-0 {
ti,mbox-rx = <0 0 0>;
@@ -42,6 +43,7 @@ mbox_r5_0: mbox-r5-0 {
&mailbox0_cluster1 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_c7x_0: mbox-c7x-0 {
ti,mbox-rx = <0 0 0>;
@@ -51,6 +53,7 @@ mbox_c7x_0: mbox-c7x-0 {
&mailbox0_cluster2 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_mcu_r5_0: mbox-mcu-r5-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
index d29a5dbe13ef..82512e7a44ea 100644
--- a/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
@@ -21,6 +21,7 @@ mcu_r5fss0_core0_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_r5_0: mbox-r5-0 {
ti,mbox-rx = <0 0 0>;
@@ -30,6 +31,7 @@ mbox_r5_0: mbox-r5-0 {
&mailbox0_cluster1 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_mcu_r5_0: mbox-mcu-r5-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-am64-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am64-ti-ipc-firmware.dtsi
index 6b10646ae64a..793e965c5a4b 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-ti-ipc-firmware.dtsi
@@ -63,6 +63,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster2 {
status = "okay";
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss0_core0: mbox-main-r5fss0-core0 {
ti,mbox-rx = <0 0 2>;
@@ -77,6 +78,7 @@ mbox_main_r5fss0_core1: mbox-main-r5fss0-core1 {
&mailbox0_cluster4 {
status = "okay";
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss1_core0: mbox-main-r5fss1-core0 {
ti,mbox-rx = <0 0 2>;
@@ -91,6 +93,7 @@ mbox_main_r5fss1_core1: mbox-main-r5fss1-core1 {
&mailbox0_cluster6 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_m4_0: mbox-m4-0 {
ti,mbox-rx = <0 0 2>;
diff --git a/arch/arm64/boot/dts/ti/k3-am65-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am65-ti-ipc-firmware.dtsi
index 61ab0357fc0d..579533df6bd4 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-ti-ipc-firmware.dtsi
@@ -28,6 +28,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
interrupts = <436>;
+ ti,mbox-num-fifos = <2>;
mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
ti,mbox-tx = <1 0 0>;
@@ -38,6 +39,7 @@ mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
&mailbox0_cluster1 {
status = "okay";
interrupts = <432>;
+ ti,mbox-num-fifos = <2>;
mbox_mcu_r5fss0_core1: mbox-mcu-r5fss0-core1 {
ti,mbox-tx = <1 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-ti-ipc-firmware.dtsi
index 9477f1efbbc6..4245a5319085 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-ti-ipc-firmware.dtsi
@@ -52,6 +52,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
interrupts = <436>;
+ ti,mbox-num-fifos = <4>;
mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -67,6 +68,7 @@ mbox_mcu_r5fss0_core1: mbox-mcu-r5fss0-core1 {
&mailbox0_cluster1 {
status = "okay";
interrupts = <432>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss0_core0: mbox-main-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-ti-ipc-firmware.dtsi
index 40c6cc99c405..4003f125dbe7 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-ti-ipc-firmware.dtsi
@@ -114,6 +114,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
interrupts = <436>;
+ ti,mbox-num-fifos = <4>;
mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -129,6 +130,7 @@ mbox_mcu_r5fss0_core1: mbox-mcu-r5fss0-core1 {
&mailbox0_cluster1 {
status = "okay";
interrupts = <432>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss0_core0: mbox-main-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -144,6 +146,7 @@ mbox_main_r5fss0_core1: mbox-main-r5fss0-core1 {
&mailbox0_cluster2 {
status = "okay";
interrupts = <428>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss1_core0: mbox-main-r5fss1-core0 {
ti,mbox-rx = <0 0 0>;
@@ -159,6 +162,7 @@ mbox_main_r5fss1_core1: mbox-main-r5fss1-core1 {
&mailbox0_cluster3 {
status = "okay";
interrupts = <424>;
+ ti,mbox-num-fifos = <4>;
mbox_c66_0: mbox-c66-0 {
ti,mbox-rx = <0 0 0>;
@@ -174,6 +178,7 @@ mbox_c66_1: mbox-c66-1 {
&mailbox0_cluster4 {
status = "okay";
interrupts = <420>;
+ ti,mbox-num-fifos = <2>;
mbox_c71_0: mbox-c71-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-ti-ipc-firmware.dtsi
index ebab0cc580bb..2ef5c95f6e93 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-ti-ipc-firmware.dtsi
@@ -100,6 +100,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
interrupts = <436>;
+ ti,mbox-num-fifos = <4>;
mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -115,6 +116,7 @@ mbox_mcu_r5fss0_core1: mbox-mcu-r5fss0-core1 {
&mailbox0_cluster1 {
status = "okay";
interrupts = <432>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss0_core0: mbox-main-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -130,6 +132,7 @@ mbox_main_r5fss0_core1: mbox-main-r5fss0-core1 {
&mailbox0_cluster2 {
status = "okay";
interrupts = <428>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss1_core0: mbox-main-r5fss1-core0 {
ti,mbox-rx = <0 0 0>;
@@ -145,6 +148,7 @@ mbox_main_r5fss1_core1: mbox-main-r5fss1-core1 {
&mailbox0_cluster4 {
status = "okay";
interrupts = <420>;
+ ti,mbox-num-fifos = <4>;
mbox_c71_0: mbox-c71-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j722s-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j722s-ti-ipc-firmware.dtsi
index cb7cd385a165..c25dec00cc2a 100644
--- a/arch/arm64/boot/dts/ti/k3-j722s-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j722s-ti-ipc-firmware.dtsi
@@ -63,6 +63,7 @@ rtos_ipc_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_wkup_r5_0: mbox-wkup-r5-0 {
ti,mbox-rx = <0 0 0>;
@@ -72,6 +73,7 @@ mbox_wkup_r5_0: mbox-wkup-r5-0 {
&mailbox0_cluster1 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_mcu_r5_0: mbox-mcu-r5-0 {
ti,mbox-rx = <0 0 0>;
@@ -81,6 +83,7 @@ mbox_mcu_r5_0: mbox-mcu-r5-0 {
&mailbox0_cluster2 {
status = "okay";
+ ti,mbox-num-fifos = <2>;
mbox_c7x_0: mbox-c7x-0 {
ti,mbox-rx = <0 0 0>;
@@ -90,6 +93,7 @@ mbox_c7x_0: mbox-c7x-0 {
&mailbox0_cluster3 {
status = "okay";
+ ti,mbox-num-fifos = <4>;
mbox_main_r5_0: mbox-main-r5-0 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-ti-ipc-firmware-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-ti-ipc-firmware-common.dtsi
index 455397227d4a..809ecf26ddd1 100644
--- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-ti-ipc-firmware-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-ti-ipc-firmware-common.dtsi
@@ -130,6 +130,7 @@ c71_2_memory_region: memory@...00000 {
&mailbox0_cluster0 {
status = "okay";
interrupts = <436>;
+ ti,mbox-num-fifos = <4>;
mbox_mcu_r5fss0_core0: mbox-mcu-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -145,6 +146,7 @@ mbox_mcu_r5fss0_core1: mbox-mcu-r5fss0-core1 {
&mailbox0_cluster1 {
status = "okay";
interrupts = <432>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss0_core0: mbox-main-r5fss0-core0 {
ti,mbox-rx = <0 0 0>;
@@ -160,6 +162,7 @@ mbox_main_r5fss0_core1: mbox-main-r5fss0-core1 {
&mailbox0_cluster2 {
status = "okay";
interrupts = <428>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss1_core0: mbox-main-r5fss1-core0 {
ti,mbox-rx = <0 0 0>;
@@ -175,6 +178,7 @@ mbox_main_r5fss1_core1: mbox-main-r5fss1-core1 {
&mailbox0_cluster3 {
status = "okay";
interrupts = <424>;
+ ti,mbox-num-fifos = <4>;
mbox_main_r5fss2_core0: mbox-main-r5fss2-core0 {
ti,mbox-rx = <0 0 0>;
@@ -190,6 +194,7 @@ mbox_main_r5fss2_core1: mbox-main-r5fss2-core1 {
&mailbox0_cluster4 {
status = "okay";
interrupts = <420>;
+ ti,mbox-num-fifos = <4>;
mbox_c71_0: mbox-c71-0 {
ti,mbox-rx = <0 0 0>;
@@ -205,6 +210,7 @@ mbox_c71_1: mbox-c71-1 {
&mailbox0_cluster5 {
status = "okay";
interrupts = <416>;
+ ti,mbox-num-fifos = <2>;
mbox_c71_2: mbox-c71-2 {
ti,mbox-rx = <0 0 0>;
diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-ti-ipc-firmware.dtsi
index 81b508b9b05e..67af04b25fd0 100644
--- a/arch/arm64/boot/dts/ti/k3-j784s4-ti-ipc-firmware.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j784s4-ti-ipc-firmware.dtsi
@@ -20,6 +20,7 @@ c71_3_memory_region: memory@...00000 {
};
&mailbox0_cluster5 {
+ ti,mbox-num-fifos = <4>;
mbox_c71_3: mbox-c71-3 {
ti,mbox-rx = <2 0 0>;
--
2.34.1
Powered by blists - more mailing lists