linux-kernel - Re: [PATCH] arm64: dts: ti: k3-j722s-evm: Describe main

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6b147080-cd58-4430-a17c-4de4e29f1348@ti.com>
Date: Tue, 27 Aug 2024 14:18:24 +0530
From: Bhavya Kapoor <b-kapoor@...com>
To: Nishanth Menon <nm@...com>
CC: <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <vigneshr@...com>, <kristo@...nel.org>, <robh@...nel.org>,
        <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] arm64: dts: ti: k3-j722s-evm: Describe main_uart5


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@...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
>>