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] [thread-next>] [day] [month] [year] [list]

Message-ID: <27a350e5-b3e0-427d-8897-89e1d34a4c00@ti.com>
Date: Tue, 28 May 2024 16:38:18 +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>, Bhavya Kapoor <b-kapoor@...com>
Subject: Re: [PATCH] arm64: dts: ti: k3-j722s-evm: Enable main_uart5


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 {