Received-SPF: pass (google.com: domain of linux-kernel+bounces-13177-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <ab993f3d-142c-4094-9320-772e1e3aad41@gmx.net>
Date: Fri, 29 Dec 2023 18:17:57 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v4 2/2] arm64: dts: imx93-var-som: Add Variscite
 VAR-SOM-MX93
To: Mathieu Othacehe <othacehe@gnu.org>
Cc: Rob Herring <robh+dt@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Conor Dooley <conor+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
 Sascha Hauer <s.hauer@pengutronix.de>,
 Pengutronix Kernel Team <kernel@pengutronix.de>,
 Fabio Estevam <festevam@gmail.com>, NXP Linux Team <linux-imx@nxp.com>,
 Li Yang <leoyang.li@nxp.com>, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
References: <20231227170919.8771-1-othacehe@gnu.org>
 <20231227170919.8771-3-othacehe@gnu.org>
 <d378ad67-2a75-4a14-a131-7eb91de9ad3d@gmx.net> <87plypqi2d.fsf@gnu.org>
 <4e3a0328-0ae7-40fa-b37d-16b661e28a1e@gmx.net> <87zfxtosyy.fsf@gnu.org>
Content-Language: en-US
From: Stefan Wahren <wahrenst@gmx.net>
In-Reply-To: <87zfxtosyy.fsf@gnu.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:NQIdbz97cZU=;EQ0dCoaZ0GyLvbLCDJMb/Iqn2EH
 RhsFD0DCopL8pOJJs/YWQdmFDkSgwgy6Vdy9qSMQY3zEyI/i2WZ16pzJD1mlh0JIvLAFt8QgY
 vGTX2cZFQvYe9DZp7S1rz1gOhdbBQlsIGQtVBi2h/SOMNnQOZxla1AodzIsY59Z95AASTrJnN
 9Pks2bLAQ69A84yY8EOzuYKMXSllyB2m3YqZoMzahk8lQyoC7EJcr+W3UroeDCnQPrxLuD0JR
 qUrDTKduG8UWZbPLgPfd0dtxGFIs0aAx5ppAW6PyVUKDBEmKfIyMcagkZBOB4Xjdk0DEirPeW
 /XN7z3OL57JPYOk5/PWVgY4v4gMzDAq0UBB4yY2xhKPhqkwRlQ/o3AZTYDQk0FX+qVacu/xx5
 OdoFgR/x/tpARg+YA8/ZHTreahf+5uj2K7NnrYz4AhWVKl23//L/m1gF7GjIgJhrTvNcFLCvK
 6yU7RIIkFH0ymaXuEh8ar5Gel8Hinx5zOTqSDigKTqoReHvYn9aTskBVWhItKvf33wt6AOjZL
 tB0ttUt8nz4NxvQnU3kq1QkuiZLwGkhOeA69Xz4vnEsnG2F7U/+iHTUsjthCh+2vhBE9SgL41
 2FKnh1iXgTp4MmsYqrhAuTfUGu7H3YDXhwaXb2iwYcxaKKQhplSpA2M2pQ0oQH+xSmZUCyLZe
 39mSB/xJN4kH7xYfutRk3UiGjqzY3q6ZqLRBA1eVmK0mOA+s+XOa4YS0bMagQkDMzasavVYvP
 wNkRToVFhMAuti7CLxP+SF76KY+s5bI4TnqHKLBzTyGa/2Y0ioIb5+CFbJ5y+LodofSeeXW3N
 1daTE7dtUyzxANoC6FYKLNfD0lgB2hM4KgL6uACndSzzaGd1QfLMP8ef6MqdXiCc235FTKAYM
 aAdmKILYNurb+W1sO8y3sGdunhaM5xSjqt/kNJMOe4nSDwhdJ3SlBz2Pp66DuGXdQCmrUDhAf
 NhJ5AGXiuJOL8Lb6UoM63qUz/oWG4pqAa9gbvNF4XPAO4Vg/

Hi,

Am 29.12.23 um 17:48 schrieb Mathieu Othacehe:
> Hey,
>
>> I also saw in the other Symphony DTS a LED for heartbeat. Is it usable,=
 too?
> Yes, I will add it to in v5.
>
>> According to the datasheet the SOM expose 7 UARTs, but only the debug
>> UART is enable. What is the reason for this?
> So those 7 UARTs are:
>
> UART1: console
> UART2: used by the microphone
> UART3: used by the resistive touch
> UART4: used by the ethernet/resistive touch
> UART5: on J18.4,6,8,10 (used by BT)
> UART6: on J18.7 and J18.9 (available)
> UART7: on J18.3 and J18.5 (used by the M33 firmware)
>
> I will enable UART6, and add a mention about UART7, something like:
>
> /* J18.7, J18.9 */
> &lpuart6 {
> 	pinctrl-names =3D "default";
> 	pinctrl-0 =3D <&pinctrl_uart6>;
> 	status =3D "okay";
> };
>
> /* J18.3, J18.5 - used by M33 firmware */
> &lpuart7 {
> 	pinctrl-names =3D "default";
> 	pinctrl-0 =3D <&pinctrl_uart7>;
> 	status =3D "disabled";
> };
in case this is used by the M33 firmware, this applies to all i.MX93 so
we don't need this. Just enable UART6 and keep UART7.
>> Btw the commit log mentions a Wifi (via SDIO?) and audio interface. Thi=
s
>> is currently not supported by Linux?
> I am not sure about that, and I would prefer to study that a bit later
> on if that's OK :) I can remove it from the commit log as this is
> misleading as not already supported / tested.
This sounds good. No need to have it all at first.
>
> Thanks,
>
> Mathieu
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel