Hi Krzysztof, On Tue, 3 Mar 2026 16:19:54 +0100 Krzysztof Kozlowski <[email protected]> wrote:
> On 03/03/2026 15:55, Hugo Villeneuve wrote: > > Hi Krzysztof, > > > > On Tue, 3 Mar 2026 08:10:51 +0100 > > Krzysztof Kozlowski <[email protected]> wrote: > > > >> On Mon, Mar 02, 2026 at 02:03:40PM -0500, Hugo Villeneuve wrote: > >>> From: Hugo Villeneuve <[email protected]> > >>> > >>> There is no Variscite module named VAR-SOM-MX6UL. > >> > >> VAR-SOM-MX6? And there is. > >> https://dev.variscite.com/var-som-mx6/ > > > > Ok, > > I was confused by the "const: variscite,var-som-imx6ul" and thought the > > description was not matching. > > > > But there is still no module named "VAR-SOM-MX6UL", but "VAR-SOM-MX6", > > with different CPU variants like UL, ULL, etc. So I will modify the > > patch to reflect that. > > I don't understand what you want to reflect/modify. > > Description says: VAR-SOM-MX6 > My link says: VAR-SOM-MX6 > > What is incorrect? VAR-SOM-MX6 is ok, like your link shows, but not VAR-SOM-MX6UL. If someone looks at VAR-SOM-MX6UL, they may think that the "MX6" part refers to the VAR-SOM-MX6, or they may think that the "6UL" part refers to the VAR-SOM-6UL, and it can be extremely confusing. This was my understanding at first, and the reason why I submitted this patch. If you look at the associated board entry in the binding, it says "const: variscite,mx6ulconcerto" but the VAR-SOM-MX6 doesn't use the concerto board. It uses a board named "VAR-MX6CustomBoard": https://variscite.com/wp-content/uploads/2017/12/VAR-MX6CustomBoard-Datasheet.pdf The VAR-SOM-6UL uses the concerto board: https://variscite.com/wp-content/uploads/2019/07/Concerto-Board_Datasheet.pdf So the description in the binding "i.MX6UL Variscite VAR-SOM-MX6 Boards" is wrong, and needs to be replaced with VAR-SOM-6UL. By the way, support for the VAR-SOM-MX6 is provided by this binding entry: - description: i.MX6Q Variscite VAR-SOM-MX6 Boards items: - const: variscite,mx6customboard - const: variscite,var-som-imx6q - const: fsl,imx6q > > > > In fact, I will modify the description to reflect that it supports both > > VAR-SOM-MX6 and VAR-SOM-6UL modules. > > Are you sure that it does? With my new research, as explained above, this is not the case. I will modify the description to reflect that it supports VAR-SOM-6UL modules only. -- Hugo Villeneuve
