diff mbox series

arm64: dts: ti: k3-j722s-evm: Describe main_uart5

Message ID 20240822053538.10475-1-b-kapoor@ti.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: ti: k3-j722s-evm: Describe main_uart5 | expand

Commit Message

Bhavya Kapoor Aug. 22, 2024, 5:35 a.m. UTC
main_uart5 in J722S platform is used by the firmware. Thus,
describe it for completeness, adding the pinmux and mark
it as reserved.

Signed-off-by: Bhavya Kapoor <b-kapoor@ti.com>
---
 arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Nishanth Menon Aug. 24, 2024, 6:14 p.m. UTC | #1
On 11:05-20240822, Bhavya Kapoor wrote:
> main_uart5 in J722S platform is used by the firmware. Thus,
what platform? what firmware?

> describe it for completeness, adding the pinmux and mark
> it as reserved.
> 
> Signed-off-by: Bhavya Kapoor <b-kapoor@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> index 24e9f2ea509b..5addf1c0afc2 100644
> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
> @@ -20,6 +20,7 @@ / {
>  	aliases {
>  		serial0 = &wkup_uart0;
>  		serial2 = &main_uart0;
> +		serial3 = &main_uart5;
>  		mmc0 = &sdhci0;
>  		mmc1 = &sdhci1;
>  	};
> @@ -211,6 +212,13 @@ J722S_IOPAD(0x01cc, PIN_OUTPUT, 0)	/* (B22) UART0_TXD */
>  		bootph-all;
>  	};
>  
> +	main_uart5_pins_default: main-uart5-default-pins {
> +		pinctrl-single,pins = <
> +			J722S_IOPAD(0x0108, PIN_INPUT, 3)       /* (J27) UART5_RXD */
> +			J722S_IOPAD(0x010c, PIN_OUTPUT, 3)      /* (H27) UART5_TXD */
> +		>;
> +	};
> +
>  	vdd_sd_dv_pins_default: vdd-sd-dv-default-pins {
>  		pinctrl-single,pins = <
>  			J722S_IOPAD(0x0120, PIN_INPUT, 7) /* (F27) MMC2_CMD.GPIO0_70 */
> @@ -330,6 +338,12 @@ &main_uart0 {
>  	bootph-all;
>  };
>  
> +&main_uart5 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&main_uart5_pins_default>;

Document specifically why reserved.

> +	status = "reserved";
> +};
> +
>  &mcu_pmx0 {
>  
>  	mcu_mcan0_pins_default: mcu-mcan0-default-pins {
> -- 
> 2.34.1
>
Bhavya Kapoor Aug. 27, 2024, 8:48 a.m. UTC | #2
On 24/08/24 11:44 pm, Nishanth Menon wrote:
> On 11:05-20240822, Bhavya Kapoor wrote:
>> main_uart5 in J722S platform is used by the firmware. Thus,
> what platform? what firmware?

Hey Nishanth, it is used by system firmware for its logs

in J722S-EVM.

>
>> describe it for completeness, adding the pinmux and mark
>> it as reserved.
>>
>> Signed-off-by: Bhavya Kapoor <b-kapoor@ti.com>
>> ---
>>   arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> index 24e9f2ea509b..5addf1c0afc2 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> @@ -20,6 +20,7 @@ / {
>>   	aliases {
>>   		serial0 = &wkup_uart0;
>>   		serial2 = &main_uart0;
>> +		serial3 = &main_uart5;
>>   		mmc0 = &sdhci0;
>>   		mmc1 = &sdhci1;
>>   	};
>> @@ -211,6 +212,13 @@ J722S_IOPAD(0x01cc, PIN_OUTPUT, 0)	/* (B22) UART0_TXD */
>>   		bootph-all;
>>   	};
>>   
>> +	main_uart5_pins_default: main-uart5-default-pins {
>> +		pinctrl-single,pins = <
>> +			J722S_IOPAD(0x0108, PIN_INPUT, 3)       /* (J27) UART5_RXD */
>> +			J722S_IOPAD(0x010c, PIN_OUTPUT, 3)      /* (H27) UART5_TXD */
>> +		>;
>> +	};
>> +
>>   	vdd_sd_dv_pins_default: vdd-sd-dv-default-pins {
>>   		pinctrl-single,pins = <
>>   			J722S_IOPAD(0x0120, PIN_INPUT, 7) /* (F27) MMC2_CMD.GPIO0_70 */
>> @@ -330,6 +338,12 @@ &main_uart0 {
>>   	bootph-all;
>>   };
>>   
>> +&main_uart5 {
>> +	pinctrl-names = "default";
>> +	pinctrl-0 = <&main_uart5_pins_default>;
> Document specifically why reserved.
Will send out v2 for this.
>
>> +	status = "reserved";
>> +};
>> +
>>   &mcu_pmx0 {
>>   
>>   	mcu_mcan0_pins_default: mcu-mcan0-default-pins {
>> -- 
>> 2.34.1
>>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
index 24e9f2ea509b..5addf1c0afc2 100644
--- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
@@ -20,6 +20,7 @@  / {
 	aliases {
 		serial0 = &wkup_uart0;
 		serial2 = &main_uart0;
+		serial3 = &main_uart5;
 		mmc0 = &sdhci0;
 		mmc1 = &sdhci1;
 	};
@@ -211,6 +212,13 @@  J722S_IOPAD(0x01cc, PIN_OUTPUT, 0)	/* (B22) UART0_TXD */
 		bootph-all;
 	};
 
+	main_uart5_pins_default: main-uart5-default-pins {
+		pinctrl-single,pins = <
+			J722S_IOPAD(0x0108, PIN_INPUT, 3)       /* (J27) UART5_RXD */
+			J722S_IOPAD(0x010c, PIN_OUTPUT, 3)      /* (H27) UART5_TXD */
+		>;
+	};
+
 	vdd_sd_dv_pins_default: vdd-sd-dv-default-pins {
 		pinctrl-single,pins = <
 			J722S_IOPAD(0x0120, PIN_INPUT, 7) /* (F27) MMC2_CMD.GPIO0_70 */
@@ -330,6 +338,12 @@  &main_uart0 {
 	bootph-all;
 };
 
+&main_uart5 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&main_uart5_pins_default>;
+	status = "reserved";
+};
+
 &mcu_pmx0 {
 
 	mcu_mcan0_pins_default: mcu-mcan0-default-pins {