From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Francesco Dolcini <francesco@dolcini.it>
Cc: Sherry Sun <sherry.sun@nxp.com>,
robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
shawnguo@kernel.org, s.hauer@pengutronix.de, festevam@gmail.com,
kwilczynski@kernel.org, mani@kernel.org, bhelgaas@google.com,
hongxing.zhu@nxp.com, frank.li@nxp.com, l.stach@pengutronix.de,
lpieralisi@kernel.org, kernel@pengutronix.de,
devicetree@vger.kernel.org, imx@lists.linux.dev,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] fsl,imx6q-pcie: Remove the deprecated "reset-gpio-active-high" property
Date: Thu, 15 Jan 2026 10:08:47 +0200 [thread overview]
Message-ID: <aWigj011i0pGgTRi@smile.fi.intel.com> (raw)
In-Reply-To: <20260115073418.GA4366@francesco-nb>
On Thu, Jan 15, 2026 at 08:34:18AM +0100, Francesco Dolcini wrote:
> On Thu, Jan 15, 2026 at 03:18:13PM +0800, Sherry Sun wrote:
...
> And , the property is still handled, see commit b8b80348c57b
> ("gpiolib: of: add polarity quirk for Freescale PCIe controller"), the
> DT compatibility was not broken as you seem to imply in this series.
This is a quirk and not a 1st class support. There must be no such property
in new DTs, it's only present for the backward compatibility.
For the old ones, indeed the property is needed. The problem of reuse of Linux
DTs somewhere else is orthogonal to the Linux kernel. If this is a real problem
the DTs should have been maintained outside of Linux kernel and, if needed, be
synchronised from time to time.
TL;DR: we must discourage people to use deprecated and (historically) wrong
properties.
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2026-01-15 8:08 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-15 7:18 [PATCH 0/3] fsl,imx6q-pcie: Remove the deprecated "reset-gpio-active-high" property Sherry Sun
2026-01-15 7:18 ` [PATCH 1/3] ARM: dts: imx6q-apalis: " Sherry Sun
2026-01-15 7:18 ` [PATCH 2/3] ARM: dts: imx6sx-softing-vining-2000: " Sherry Sun
2026-01-15 7:18 ` [PATCH 3/3] dt-bindings: PCI: fsl,imx6q-pcie: " Sherry Sun
2026-01-15 9:07 ` Krzysztof Kozlowski
2026-01-15 9:23 ` Sherry Sun
2026-01-15 7:34 ` [PATCH 0/3] " Francesco Dolcini
2026-01-15 8:08 ` Andy Shevchenko [this message]
2026-01-15 8:11 ` Francesco Dolcini
2026-01-15 9:02 ` Sherry Sun
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=aWigj011i0pGgTRi@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=francesco@dolcini.it \
--cc=frank.li@nxp.com \
--cc=hongxing.zhu@nxp.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=kwilczynski@kernel.org \
--cc=l.stach@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=mani@kernel.org \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=sherry.sun@nxp.com \
/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