[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aWigj011i0pGgTRi@smile.fi.intel.com>
Date: Thu, 15 Jan 2026 10:08:47 +0200
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Francesco Dolcini <francesco@...cini.it>
Cc: Sherry Sun <sherry.sun@....com>, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, shawnguo@...nel.org, s.hauer@...gutronix.de,
festevam@...il.com, kwilczynski@...nel.org, mani@...nel.org,
bhelgaas@...gle.com, hongxing.zhu@....com, frank.li@....com,
l.stach@...gutronix.de, lpieralisi@...nel.org,
kernel@...gutronix.de, devicetree@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] fsl,imx6q-pcie: Remove the deprecated
"reset-gpio-active-high" property
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
Powered by blists - more mailing lists