[3/5] arm64: dts: ti: k3-j7200: use ti,j7200-sci compatible

Message ID 20231129-j7200-tisci-s2r-v1-3-c1d5964ed574@bootlin.com
State New
Headers
Series Add suspend/resume support in ti_sci driver for j7200 |

Commit Message

Thomas Richard Nov. 29, 2023, 3:31 p.m. UTC
  On j7200, suspend to ram poweroff the SOC.
This compatible restores irqs at resume.

Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Conor Dooley Nov. 29, 2023, 3:35 p.m. UTC | #1
On Wed, Nov 29, 2023 at 04:31:19PM +0100, Thomas Richard wrote:
> On j7200, suspend to ram poweroff the SOC.
> This compatible restores irqs at resume.
> 
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---
>  arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 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 467970fab3a0..d8dc1421e75e 100644
> --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> @@ -7,7 +7,7 @@
>  
>  &cbass_mcu_wakeup {
>  	dmsc: system-controller@44083000 {
> -		compatible = "ti,k2g-sci";
> +		compatible = "ti,j7200-sci", "ti,k2g-sci";

This is not what your dt-binding change allows. Did you test this with
dtbs_check?
  

Patch

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 467970fab3a0..d8dc1421e75e 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -7,7 +7,7 @@ 
 
 &cbass_mcu_wakeup {
 	dmsc: system-controller@44083000 {
-		compatible = "ti,k2g-sci";
+		compatible = "ti,j7200-sci", "ti,k2g-sci";
 		ti,host-id = <12>;
 
 		mbox-names = "rx", "tx";