linux-kernel - Re: [PATCH] arm64: dts: ti: k3-j722s-evm: Enable 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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <d8b802dc-e65c-4b76-bfe0-a5b5c9dad472@ti.com>
Date: Wed, 12 Jun 2024 12:10:40 +0530
From: Bhavya Kapoor <b-kapoor@...com>
To: <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>
CC: <linux-arm-kernel@...ts.infradead.org>, <conor+dt@...nel.org>,
        <krzk+dt@...nel.org>, <robh@...nel.org>, <kristo@...nel.org>,
        <vigneshr@...com>, <nm@...com>
Subject: Re: [PATCH] arm64: dts: ti: k3-j722s-evm: Enable main_uart5

Hi all, please ignore this patch as main_uart5 needs to be kept 
reserved. I will send another patch for the same

regards

On 28/05/24 4:38 pm, Bhavya Kapoor wrote:
>
> On 28/05/24 3:09 pm, Bhavya Kapoor wrote:
>> main_uart5 node defined in the SoC dtsi file is incomplete
>> and will not be functional unless it is extended with pinmux
>> information at board integration level.
>>
>> Thus, add pinmux for main_uart5 in the board dts file and
>> enable it to make it functional.
>>
>> Signed-off-by: Bhavya Kapoor <b-kapoor@...com>
>> ---
> Test Logs: 
> https://gist.github.com/a0498981/dc8b0cb670a5726eeb6c10f6de1f468e
>>   arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 16 ++++++++++++++++
>>   1 file changed, 16 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts 
>> b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> index bf3c246d13d1..e3892a6f2fab 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts
>> @@ -18,6 +18,7 @@ / {
>>       aliases {
>>           serial0 = &wkup_uart0;
>>           serial2 = &main_uart0;
>> +        serial3 = &main_uart5;
>>           mmc0 = &sdhci0;
>>           mmc1 = &sdhci1;
>>       };
>> @@ -142,6 +143,14 @@ 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 */
>> +        >;
>> +        bootph-all;
>> +    };
>> +
>>       vdd_sd_dv_pins_default: vdd-sd-dv-default-pins {
>>           pinctrl-single,pins = <
>>               J722S_IOPAD(0x0120, PIN_INPUT, 7) /* (F27) 
>> MMC2_CMD.GPIO0_70 */
>> @@ -240,6 +249,13 @@ &main_uart0 {
>>       bootph-all;
>>   };
>>   +&main_uart5 {
>> +    pinctrl-names = "default";
>> +    pinctrl-0 = <&main_uart5_pins_default>;
>> +    status = "okay";
>> +    bootph-all;
>> +};
>> +
>>   &mcu_pmx0 {
>>         wkup_uart0_pins_default: wkup-uart0-default-pins {