public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: ti: k3-j722s: use ti,j7200-padconf compatible
@ 2026-04-20 13:17 Richard Genoud (TI)
  2026-04-20 13:38 ` Thomas Richard
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Genoud (TI) @ 2026-04-20 13:17 UTC (permalink / raw)
  To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley
  Cc: Thomas Petazzoni, Gregory CLEMENT, Thomas Richard, Udit Kumar,
	Abhash Kumar, linux-arm-kernel, devicetree, linux-kernel,
	Richard Genoud (TI)

From: Abhash Kumar Jha <a-kumar2@ti.com>

The pinctrl contexts for j722s should be saved and restored during
suspend-to-ram, just like it is done for j7200 and j784s4 SoCs.

Use ti,j7200-padconf compatible to save and restore pinctrl contexts during
suspend-to-ram.

Signed-off-by: Abhash Kumar Jha <a-kumar2@ti.com>
Signed-off-by: Richard Genoud (TI) <richard.genoud@bootlin.com>
---
 arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
index e66330c71593..757eb6abcd72 100644
--- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
@@ -242,6 +242,7 @@ &phy_gmii_sel {
 };
 
 &main_pmx0 {
+	compatible = "ti,j7200-padconf", "pinctrl-single";
 
 	main_mcan0_pins_default: main-mcan0-default-pins {
 		pinctrl-single,pins = <
@@ -418,6 +419,7 @@ &main_uart5 {
 };
 
 &mcu_pmx0 {
+	compatible = "ti,j7200-padconf", "pinctrl-single";
 
 	mcu_i2c0_pins_default: mcu-i2c0-default-pins {
 		pinctrl-single,pins = <
-- 
2.47.3



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: ti: k3-j722s: use ti,j7200-padconf compatible
  2026-04-20 13:17 [PATCH] arm64: dts: ti: k3-j722s: use ti,j7200-padconf compatible Richard Genoud (TI)
@ 2026-04-20 13:38 ` Thomas Richard
  2026-04-20 13:51   ` Richard GENOUD
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Richard @ 2026-04-20 13:38 UTC (permalink / raw)
  To: Richard Genoud (TI), Nishanth Menon, Vignesh Raghavendra,
	Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: Thomas Petazzoni, Gregory CLEMENT, Udit Kumar, Abhash Kumar,
	linux-arm-kernel, devicetree, linux-kernel

Hello Richard,

On 4/20/26 3:17 PM, Richard Genoud (TI) wrote:
> From: Abhash Kumar Jha <a-kumar2@ti.com>
> 
> The pinctrl contexts for j722s should be saved and restored during
> suspend-to-ram, just like it is done for j7200 and j784s4 SoCs.
> 
> Use ti,j7200-padconf compatible to save and restore pinctrl contexts during
> suspend-to-ram.
> 
> Signed-off-by: Abhash Kumar Jha <a-kumar2@ti.com>
> Signed-off-by: Richard Genoud (TI) <richard.genoud@bootlin.com>
> ---
>  arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> index e66330c71593..757eb6abcd72 100644
> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> @@ -242,6 +242,7 @@ &phy_gmii_sel {
>  };
>  
>  &main_pmx0 {
> +	compatible = "ti,j7200-padconf", "pinctrl-single";
>  
>  	main_mcan0_pins_default: main-mcan0-default-pins {
>  		pinctrl-single,pins = <
> @@ -418,6 +419,7 @@ &main_uart5 {
>  };
>  
>  &mcu_pmx0 {
> +	compatible = "ti,j7200-padconf", "pinctrl-single";
>  
>  	mcu_i2c0_pins_default: mcu-i2c0-default-pins {
>  		pinctrl-single,pins = <

It should be done at SoC level, not just for the EVM board.
You should modify k3-j722s-main.dtsi and create k3-j722s-mcu.dtsi.

Best Regards,
Thomas


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: ti: k3-j722s: use ti,j7200-padconf compatible
  2026-04-20 13:38 ` Thomas Richard
@ 2026-04-20 13:51   ` Richard GENOUD
  0 siblings, 0 replies; 3+ messages in thread
From: Richard GENOUD @ 2026-04-20 13:51 UTC (permalink / raw)
  To: Thomas Richard, Nishanth Menon, Vignesh Raghavendra, Tero Kristo,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: Thomas Petazzoni, Gregory CLEMENT, Udit Kumar, Abhash Kumar,
	linux-arm-kernel, devicetree, linux-kernel

Hi Thomas,

Le 20/04/2026 à 15:38, Thomas Richard a écrit :
> Hello Richard,
> 
> On 4/20/26 3:17 PM, Richard Genoud (TI) wrote:
>> From: Abhash Kumar Jha <a-kumar2@ti.com>
>>
>> The pinctrl contexts for j722s should be saved and restored during
>> suspend-to-ram, just like it is done for j7200 and j784s4 SoCs.
>>
>> Use ti,j7200-padconf compatible to save and restore pinctrl contexts during
>> suspend-to-ram.
>>
>> Signed-off-by: Abhash Kumar Jha <a-kumar2@ti.com>
>> Signed-off-by: Richard Genoud (TI) <richard.genoud@bootlin.com>
>> ---
>>   arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> index e66330c71593..757eb6abcd72 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> @@ -242,6 +242,7 @@ &phy_gmii_sel {
>>   };
>>   
>>   &main_pmx0 {
>> +	compatible = "ti,j7200-padconf", "pinctrl-single";
>>   
>>   	main_mcan0_pins_default: main-mcan0-default-pins {
>>   		pinctrl-single,pins = <
>> @@ -418,6 +419,7 @@ &main_uart5 {
>>   };
>>   
>>   &mcu_pmx0 {
>> +	compatible = "ti,j7200-padconf", "pinctrl-single";
>>   
>>   	mcu_i2c0_pins_default: mcu-i2c0-default-pins {
>>   		pinctrl-single,pins = <
> 
> It should be done at SoC level, not just for the EVM board.
> You should modify k3-j722s-main.dtsi and create k3-j722s-mcu.dtsi.
Indeed.

Thanks!

> 
> Best Regards,
> Thomas




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-04-20 13:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-20 13:17 [PATCH] arm64: dts: ti: k3-j722s: use ti,j7200-padconf compatible Richard Genoud (TI)
2026-04-20 13:38 ` Thomas Richard
2026-04-20 13:51   ` Richard GENOUD

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox