linux-kernel - Re: [PATCH] arm: dts: mxs: Remove not used "fsl,imx2[38]-icoll' compatibles

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: <f4b7e56f-50d1-486b-9866-ee1f82262b53@kernel.org>
Date: Wed, 28 Aug 2024 16:39:27 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Lukasz Majewski <lukma@...x.de>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
 Sascha Hauer <s.hauer@...gutronix.de>
Cc: Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>, devicetree@...r.kernel.org,
 imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm: dts: mxs: Remove not used "fsl,imx2[38]-icoll'
 compatibles

On 28/08/2024 11:35, Lukasz Majewski wrote:
> The 'fsl,imx23-icoll' and 'fsl,imx28-icoll' are not used at any place
> in the Linux kernel - instead for imx2[38] the 'fsl,icoll' is used.
> 
> Hence, it is possible to remove them.

Preference is to have SoC-specific compatibles in the front, so instead
of dropping it, these should be documented as bindings.

imx23 and imx28 are quite old platforms, so I think no one really cared
about dtbs_check and their bindings. If the platform is being actively
used (as judging by your contributions) then some bigger cleanup could
be useful. Also, drop your email somewhere in the files or maintainer
entry, so we will know whom to ping when asking for platform removal.

But if there is no product on imx23/28, then I am afraid it might be
wasted effort - isn't it planned for deprecation/removal by Arnd?

Best regards,
Krzysztof