| 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: <20260116104810.u2pd3qsieuumcuyn@manpower>
Date: Fri, 16 Jan 2026 04:48:10 -0600
From: Nishanth Menon <nm@...com>
To: Anshul Dalal <anshuld@...com>
CC: Tero Kristo <kristo@...nel.org>, Santosh Shilimkar <ssantosh@...nel.org>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Andrew Davis <afd@...com>,
<linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, Vignesh Raghavendra <vigneshr@...com>,
"Dhruva Gole" <d-gole@...com>, Manorit Chawdhry <m-chawdhry@...com>
Subject: Re: [PATCH v9 2/2] arm64: dts: ti: update compatible to ti,am654-sci
On 15:45-20260116, Anshul Dalal wrote:
> K3 SoC's use a different architecture to ti,k2g-sci so update the
> compatible to reflect the same while also adding the missing boot_*
> mailboxes.
>
> Signed-off-by: Anshul Dalal <anshuld@...com>
> ---
> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 9 ++++++---
> arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 9 ++++++---
> arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi | 8 +++++---
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 8 +++++---
> arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 6 ++++--
> arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 8 +++++---
> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 9 +++++----
> arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 9 ++++++---
> arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi | 9 ++++++---
Split this into two patches at least:
a) tda4 arch devices -> with notify
b) am62 class devices without notify
All should have boot_notify, no?
Will also be good to point to reference docs where these came from.
> 9 files changed, 48 insertions(+), 27 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> index c5ee263d34a6..c790fc643c4b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> @@ -181,11 +181,14 @@ main_pktdma: dma-controller@...c0000 {
>
> dmsc: system-controller@...43000 {
> bootph-all;
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx", "boot_notify";
> mboxes = <&secure_proxy_main 12>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_main 0>,
> + <&secure_proxy_main 1>,
> + <&secure_proxy_sa3 0>;
> reg-names = "debug_messages";
> reg = <0x00 0x44043000 0x00 0xfe0>;
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
> index 9e5b75a4e88e..af459ed3d4ff 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
> @@ -214,13 +214,16 @@ main_bcdma_csi: dma-controller@...30000 {
> };
>
> dmsc: system-controller@...43000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> reg = <0x00 0x44043000 0x00 0xfe0>;
> reg-names = "debug_messages";
> ti,host-id = <12>;
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx", "boot_notify";
> mboxes = <&secure_proxy_main 12>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_main 0>,
> + <&secure_proxy_main 1>,
> + <&secure_proxy_sa3 0>;
>
> k3_pds: power-controller {
> compatible = "ti,sci-pm-domain";
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
> index 3cf7c2b3ce2d..5037efd73d8b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
> @@ -212,11 +212,13 @@ main_bcdma_csi: dma-controller@...30000 {
> };
>
> dmsc: system-controller@...43000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx";
> mboxes = <&secure_proxy_main 12>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_main 0>,
> + <&secure_proxy_main 1>;
> reg-names = "debug_messages";
> reg = <0x00 0x44043000 0x00 0xfe0>;
> bootph-all;
> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> index 1b1d3970888b..ab105fc1f718 100644
> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> @@ -209,11 +209,13 @@ main_pktdma: dma-controller@...c0000 {
>
> dmsc: system-controller@...43000 {
> bootph-all;
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx";
> mboxes = <&secure_proxy_main 12>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_main 0>,
> + <&secure_proxy_main 1>;
> reg-names = "debug_messages";
> reg = <0x00 0x44043000 0x00 0xfe0>;
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
> index d62a0be767c8..98608793fa22 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
> @@ -10,10 +10,12 @@ dmsc: system-controller@...83000 {
> compatible = "ti,am654-sci";
> ti,host-id = <12>;
>
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx";
>
> mboxes = <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_mcu 5>,
> + <&secure_proxy_mcu 4>;
>
> reg-names = "debug_messages";
> reg = <0x44083000 0x1000>;
> 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 dc5c02a025f8..a185aeb0ae2e 100644
> --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> @@ -7,13 +7,15 @@
>
> &cbass_mcu_wakeup {
> dmsc: system-controller@...83000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
>
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx";
>
> mboxes = <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_mcu 5>,
> + <&secure_proxy_mcu 4>;
>
> reg-names = "debug_messages";
> reg = <0x00 0x44083000 0x00 0x1000>;
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> index d5e5e89be5e9..21cc01ab9dba 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> @@ -7,14 +7,15 @@
>
> &cbass_mcu_wakeup {
> dmsc: system-controller@...83000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
>
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx";
>
> mboxes = <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> -
> + <&secure_proxy_main 13>,
> + <&secure_proxy_mcu 5>,
> + <&secure_proxy_mcu 4>;
> reg-names = "debug_messages";
> reg = <0x00 0x44083000 0x0 0x1000>;
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
> index fd01437726ab..81d352d7912d 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
> @@ -7,13 +7,16 @@
>
> &cbass_mcu_wakeup {
> sms: system-controller@...83000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
>
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx", "boot_notify";
>
> mboxes = <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_mcu 5>,
> + <&secure_proxy_mcu 4>,
> + <&secure_proxy_sa3 5>;
>
> reg-names = "debug_messages";
> reg = <0x00 0x44083000 0x00 0x1000>;
> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi
> index cc22bfb5f599..3f0cef544ca8 100644
> --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi
> @@ -7,13 +7,16 @@
>
> &cbass_mcu_wakeup {
> sms: system-controller@...83000 {
> - compatible = "ti,k2g-sci";
> + compatible = "ti,am654-sci";
> ti,host-id = <12>;
>
> - mbox-names = "rx", "tx";
> + mbox-names = "rx", "tx", "boot_rx", "boot_tx", "boot_notify";
>
> mboxes = <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + <&secure_proxy_main 13>,
> + <&secure_proxy_mcu 5>,
> + <&secure_proxy_mcu 4>,
> + <&secure_proxy_sa3 5>;
>
> reg-names = "debug_messages";
> reg = <0x00 0x44083000 0x00 0x1000>;
>
> --
> 2.52.0
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
Powered by blists - more mailing lists