[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <084bc911-a51c-4257-b5bb-25bed445f3b9@kernel.org>
Date: Sun, 1 Sep 2024 12:26:55 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Tarang Raval <tarang.raval@...iconsignals.io>,
"shawnguo@...nel.org" <shawnguo@...nel.org>,
"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "robh@...nel.org"
<robh@...nel.org>, "festevam@...il.com" <festevam@...il.com>
Cc: Conor Dooley <conor+dt@...nel.org>, Sascha Hauer
<s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
Gregor Herburger <gregor.herburger@...tq-group.com>,
Francesco Dolcini <francesco.dolcini@...adex.com>,
Hugo Villeneuve <hvilleneuve@...onoff.com>,
Hiago De Franco <hiago.franco@...adex.com>,
Joao Paulo Goncalves <joao.goncalves@...adex.com>,
Mathieu Othacehe <m.othacehe@...il.com>,
Alexander Stein <alexander.stein@...tq-group.com>,
Parthiban Nallathambi <parthiban@...umiz.com>, Yannic Moog
<y.moog@...tec.de>, Josua Mayer <josua@...id-run.com>,
Li Yang <leoyang.li@....com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"imx@...ts.linux.dev" <imx@...ts.linux.dev>,
"linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 2/2] dt-bindings: arm: fsl: Add Variscite Symphony board
and VAR-SOM-MX8MP SoM
On 31/08/2024 15:51, Tarang Raval wrote:
> Hi ,
>
>>> - toradex,verdin-imx8mp-wifi # Verdin iMX8M Plus Wi-Fi / BT Modules
>>> + - variscite,var-som-mx8mp # i.MX8MP Variscite VAR-SOM-MX8M Plus module
>>
>> That's not correct. You cannot use this SoM alone.
>
> I took the reference from var-som-imx8mm, where they use "variscite,var-som-mx8mm" in a similar manner. However, if you still believe it's incorrect, I will change it.
I posted that patch, but I don't think it was correct.
>
> One more thing: should I change "VAR-SOM-MX8M Plus" to "VAR-SOM-MX8MP" in the DTSI model name and in the YAML file as well, or keep it as it is?
No clue, these comments are basically repeating compatible name so not
much helpful anyway.
Best regards,
Krzysztof
Powered by blists - more mailing lists