From: Krzysztof Kozlowski <krzk@kernel.org>
To: Lukasz Majewski <lukma@denx.de>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Shawn Guo <shawnguo@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>,
Fabio Estevam <festevam@gmail.com>,
devicetree@vger.kernel.org, imx@lists.linux.dev,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm: dts: mxs: Remove not used "fsl,imx2[38]-icoll' compatibles
Date: Wed, 28 Aug 2024 16:39:27 +0200 [thread overview]
Message-ID: <f4b7e56f-50d1-486b-9866-ee1f82262b53@kernel.org> (raw)
In-Reply-To: <20240828093518.2628817-1-lukma@denx.de>
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
next prev parent reply other threads:[~2024-08-28 14:39 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-28 9:35 [PATCH] arm: dts: mxs: Remove not used "fsl,imx2[38]-icoll' compatibles Lukasz Majewski
2024-08-28 14:39 ` Krzysztof Kozlowski [this message]
2024-08-28 15:33 ` Lukasz Majewski
2024-08-28 16:15 ` Stefan Wahren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f4b7e56f-50d1-486b-9866-ee1f82262b53@kernel.org \
--to=krzk@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lukma@denx.de \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox