mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@oss.nxp.com>
To: "Uwe Kleine-König (The Capable Hub)" <u.kleine-koenig@baylibre.com>
Cc: Linus Walleij <linusw@kernel.org>,
	Bartosz Golaszewski <brgl@kernel.org>,
	Andy Shevchenko <andy@kernel.org>,
	linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
	Yang Shen <shenyang39@huawei.com>,
	Lixu Zhang <lixu.zhang@intel.com>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Ray Jui <rjui@broadcom.com>,
	Broadcom internal kernel review list
	<bcm-kernel-feedback-list@broadcom.com>,
	Florian Fainelli <florian.fainelli@broadcom.com>,
	Scott Branden <sbranden@broadcom.com>,
	Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>,
	Frank Li <Frank.Li@nxp.com>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>, Imre Kaloz <kaloz@kernel.org>,
	Conor Dooley <conor.dooley@microchip.com>,
	Valentina Fernandez <valentina.fernandezalanis@microchip.com>,
	Daire McNamara <daire.mcnamara@microchip.com>,
	Daniel Palmer <daniel@thingy.jp>,
	Romain Perier <romain.perier@gmail.com>,
	Robert Jarzmik <robert.jarzmik@free.fr>,
	imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
	linux-riscv@lists.infradead.org,
	Hoan Tran <hoan@os.amperecomputing.com>,
	Alan Borzeszkowski <alan.borzeszkowski@linux.intel.com>,
	Mika Westerberg <westeri@kernel.org>,
	Israel Cepeda <israel.a.cepeda.lopez@intel.com>,
	Hans de Goede <hansg@kernel.org>,
	Matti Vaittinen <mazziesaccount@gmail.com>,
	"Enrico Weigelt, metux IT consult" <info@metux.net>,
	Viresh Kumar <vireshk@kernel.org>,
	virtualization@lists.linux.dev, Yinbo Zhu <zhuyinbo@loongson.cn>,
	Thierry Reding <thierry.reding@kernel.org>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	linux-tegra@vger.kernel.org,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>,
	Srinivas Neeli <srinivas.neeli@amd.com>,
	Michal Simek <michal.simek@amd.com>,
	linux-aspeed@lists.ozlabs.org, asahi@lists.linux.dev,
	linux-mediatek@lists.infradead.org, openbmc@lists.ozlabs.org,
	linux-omap@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	linux-unisoc@lists.infradead.org,
	linux-rockchip@lists.infradead.org,
	linux-stm32@st-md-mailman.stormreply.com,
	Michael Buesch <m@bues.ch>,
	William Breathitt Gray <wbg@kernel.org>,
	Robert Richter <rric@kernel.org>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	linux-acpi@vger.kernel.org
Subject: Re: [PATCH v3 00/22] gpio: Improvements around device-id arrays
Date: Mon, 26 Oct 2026 11:38:49 -0500	[thread overview]
Message-ID: <at-CGVq5jVsMQief@SMW015318> (raw)
In-Reply-To: <cover.1793030820.git.u.kleine-koenig@baylibre.com>

On Mon, Oct 26, 2026 at 05:28:11PM +0100, Uwe Kleine-König (The Capable Hub) wrote:
> Hello,
>
> as with v1 and v2 the original motivation for this series are the
> patches that convert the arrays to use named initializers, see
> https://lore.kernel.org/all/cover.1780048925.git.u.kleine-koenig@baylibre.com/
> for the idea behind it. Then by not closing the eyes for small issues
> spotted while working on that, this series grew a bit.
>
> Changes since v2 (available at
> https://lore.kernel.org/all/cover.1784133987.git.u.kleine-koenig@baylibre.com/)
>
>  - Rebase to current next/master (new changes to
>    drivers/gpio/gpio-mlxbf.c, drivers/gpio/gpio-mlxbf2.c,
>    drivers/gpio/gpio-74x164.c)
>  - Fix a few commit logs to reference the right *_device_id names
>  - Drop all tags for Andy
>  - Drop a questionable MODULE_DEVICE_TABLE(acpi, ...) for drivers/gpio/gpio-usbio.c
>
> Checkpatch only reports two undocumented device-tree compatibles:
>
> 	WARNING: DT compatible string "apm,xgene-gpio-v2" appears un-documented -- check ./Documentation/devicetree/bindings/
> 	WARNING: DT compatible string "marvell,pxa1928-gpio" appears un-documented -- check ./Documentation/devicetree/bindings/
>
> I felt free to ignore these as this is a pre-existing issue and
> out-of-context for this series.
>
> Andy didn't like how this series is split into patches before, I do. I
> think it's not possible to please everyone's opinion regarding minimal
> changes per patch vs grouping similar changes in patches and about what
> is considered churn. I'm happy about every patch in this series that
> goes in, so feel free to apply patches that you like and that don't get
> resistance even if other's do. If patches don't apply, just drop them,
> I'll catch up on them eventually.
>
> Best regards
> Uwe
>
> Uwe Kleine-König (The Capable Hub) (22):
>   gpio: Remove __maybe_unused annotations from acpi_device_id arrays
>   gpio: fxl6408: Remove __maybe_unused annotations from of_device_id
>     array
>   gpio: Drop unused assignment of acpi_device_id driver data
>   gpio: max7301: Drop unused assignment of spi_device_id driver data
>   gpio: mmio: Drop unused assignment of platform_device_id driver data
>   gpio: ljca: Drop unused assignment of auxiliary_device_id driver data
>   gpio: xgene: Add missing ACPI module annotation
>   gpio: sodaville: Add missing PCI module annotations
>   gpio: Add missing OF module annotations
>   gpio: pxa: Add missing platform module annotations
>   gpio: pl061: Use empty initializer for amba_id terminator
>   gpio: Use named initializers for acpi_device_id array
>   gpio: pcf857x: Use named initializers for of_device_id array
>   gpio: bd73800: Use named initializers for platform_device_id array
>   gpio: Use named initializers for spi_device_id array
>   gpio: virtio: Use a named initializer for virtio_device_id array
>   gpio: Unify style of acpi_device_id arrays
>   gpio: Unify style of of_device_id arrays
>   gpio: max77620: Unify style of platform_device_id array
>   gpio: Unify style of spi_device_id arrays
>   gpio: Unify style of pci_device_id arrays
>   gpio: Unify style of various *_device_id arrays
>
>  drivers/gpio/gpio-74x164.c          |  4 ++--
>  drivers/gpio/gpio-adnp.c            |  4 ++--
>  drivers/gpio/gpio-aggregator.c      |  2 +-
>  drivers/gpio/gpio-altera-a10sr.c    |  2 +-
>  drivers/gpio/gpio-altera.c          |  4 ++--
>  drivers/gpio/gpio-amd8111.c         |  2 +-
>  drivers/gpio/gpio-amdpt.c           |  8 ++++----
>  drivers/gpio/gpio-aspeed-sgpio.c    | 10 ++++-----
>  drivers/gpio/gpio-aspeed.c          | 10 ++++-----
>  drivers/gpio/gpio-ath79.c           |  2 +-
>  drivers/gpio/gpio-bcm-kona.c        |  1 +
>  drivers/gpio/gpio-bd73800.c         |  4 ++--
>  drivers/gpio/gpio-blzp1600.c        |  4 ++--
>  drivers/gpio/gpio-brcmstb.c         |  2 +-
>  drivers/gpio/gpio-bt8xx.c           |  2 +-
>  drivers/gpio/gpio-cadence.c         |  6 +++---
>  drivers/gpio/gpio-creg-snps.c       |  5 +++--
>  drivers/gpio/gpio-davinci.c         |  8 ++++----
>  drivers/gpio/gpio-dwapb.c           | 14 ++++++-------
>  drivers/gpio/gpio-em.c              |  4 ++--
>  drivers/gpio/gpio-ep93xx.c          |  1 +
>  drivers/gpio/gpio-ftgpio010.c       |  3 ++-
>  drivers/gpio/gpio-fxl6408.c         |  2 +-
>  drivers/gpio/gpio-graniterapids.c   |  4 ++--
>  drivers/gpio/gpio-grgpio.c          |  2 +-
>  drivers/gpio/gpio-gw-pld.c          |  4 ++--
>  drivers/gpio/gpio-hisi.c            |  6 +++---
>  drivers/gpio/gpio-hlwd.c            |  4 ++--
>  drivers/gpio/gpio-imx-scu.c         |  1 +
>  drivers/gpio/gpio-ixp4xx.c          |  4 ++--
>  drivers/gpio/gpio-ljca.c            |  4 ++--
>  drivers/gpio/gpio-loongson-64bit.c  |  4 ++--
>  drivers/gpio/gpio-lp3943.c          |  2 +-
>  drivers/gpio/gpio-lpc32xx.c         |  4 ++--
>  drivers/gpio/gpio-macsmc.c          |  4 ++--
>  drivers/gpio/gpio-max3191x.c        | 12 +++++------
>  drivers/gpio/gpio-max7301.c         |  2 +-
>  drivers/gpio/gpio-max7360.c         |  8 ++++----
>  drivers/gpio/gpio-max77620.c        |  4 ++--
>  drivers/gpio/gpio-max77759.c        |  2 +-
>  drivers/gpio/gpio-mb86s7x.c         |  2 +-
>  drivers/gpio/gpio-ml-ioh.c          |  2 +-
>  drivers/gpio/gpio-mlxbf.c           |  6 +++---
>  drivers/gpio/gpio-mlxbf2.c          |  6 +++---
>  drivers/gpio/gpio-mlxbf3.c          |  4 ++--
>  drivers/gpio/gpio-mm-lantiq.c       |  2 +-
>  drivers/gpio/gpio-mmio.c            |  3 +--
>  drivers/gpio/gpio-mockup.c          |  4 ++--
>  drivers/gpio/gpio-moxtet.c          |  4 ++--
>  drivers/gpio/gpio-mpc5200.c         |  9 ++++----
>  drivers/gpio/gpio-mpc8xxx.c         | 25 +++++++++++-----------
>  drivers/gpio/gpio-mpfs.c            |  1 +
>  drivers/gpio/gpio-mpsse.c           |  3 +--
>  drivers/gpio/gpio-msc313.c          |  1 +
>  drivers/gpio/gpio-mt7621.c          |  2 +-
>  drivers/gpio/gpio-mvebu.c           | 13 ++++++------
>  drivers/gpio/gpio-mxc.c             |  2 +-
>  drivers/gpio/gpio-mxs.c             |  4 ++--

For all imx part:

Acked-by: Frank Li <Frank.Li@nxp.com>

      parent reply	other threads:[~2026-10-26 16:39 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-26 16:28 Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 01/22] gpio: Remove __maybe_unused annotations from acpi_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 02/22] gpio: fxl6408: Remove __maybe_unused annotations from of_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 03/22] gpio: Drop unused assignment of acpi_device_id driver data Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 04/22] gpio: max7301: Drop unused assignment of spi_device_id " Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 05/22] gpio: mmio: Drop unused assignment of platform_device_id " Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 06/22] gpio: ljca: Drop unused assignment of auxiliary_device_id " Uwe Kleine-König (The Capable Hub)
2026-10-26 17:17   ` Sakari Ailus
2026-10-26 16:28 ` [PATCH v3 07/22] gpio: xgene: Add missing ACPI module annotation Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 08/22] gpio: sodaville: Add missing PCI module annotations Uwe Kleine-König (The Capable Hub)
2026-10-26 19:18   ` Andy Shevchenko
2026-10-26 16:28 ` [PATCH v3 09/22] gpio: Add missing OF " Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 10/22] gpio: pxa: Add missing platform " Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 11/22] gpio: pl061: Use empty initializer for amba_id terminator Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 12/22] gpio: Use named initializers for acpi_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 17:04   ` Sakari Ailus
2026-10-26 16:28 ` [PATCH v3 13/22] gpio: pcf857x: Use named initializers for of_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 14/22] gpio: bd73800: Use named initializers for platform_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-27  6:05   ` Matti Vaittinen
2026-10-26 16:28 ` [PATCH v3 15/22] gpio: Use named initializers for spi_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 16/22] gpio: virtio: Use a named initializer for virtio_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 17/22] gpio: Unify style of acpi_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 18/22] gpio: Unify style of of_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 19/22] gpio: max77620: Unify style of platform_device_id array Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 20/22] gpio: Unify style of spi_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 21/22] gpio: Unify style of pci_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:28 ` [PATCH v3 22/22] gpio: Unify style of various *_device_id arrays Uwe Kleine-König (The Capable Hub)
2026-10-26 16:38 ` Frank Li [this message]

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=at-CGVq5jVsMQief@SMW015318 \
    --to=frank.li@oss.nxp.com \
    --cc=Eugeniy.Paltsev@synopsys.com \
    --cc=Frank.Li@nxp.com \
    --cc=alan.borzeszkowski@linux.intel.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=andy@kernel.org \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=asahi@lists.linux.dev \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=brgl@kernel.org \
    --cc=conor.dooley@microchip.com \
    --cc=daire.mcnamara@microchip.com \
    --cc=daniel@thingy.jp \
    --cc=festevam@gmail.com \
    --cc=florian.fainelli@broadcom.com \
    --cc=geert+renesas@glider.be \
    --cc=hansg@kernel.org \
    --cc=hoan@os.amperecomputing.com \
    --cc=imx@lists.linux.dev \
    --cc=info@metux.net \
    --cc=israel.a.cepeda.lopez@intel.com \
    --cc=jonathanh@nvidia.com \
    --cc=kaloz@kernel.org \
    --cc=kernel@pengutronix.de \
    --cc=linusw@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-aspeed@lists.ozlabs.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-stm32@st-md-mailman.stormreply.com \
    --cc=linux-tegra@vger.kernel.org \
    --cc=linux-unisoc@lists.infradead.org \
    --cc=lixu.zhang@intel.com \
    --cc=m@bues.ch \
    --cc=mazziesaccount@gmail.com \
    --cc=michal.simek@amd.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=rjui@broadcom.com \
    --cc=robert.jarzmik@free.fr \
    --cc=romain.perier@gmail.com \
    --cc=rric@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=sakari.ailus@linux.intel.com \
    --cc=sbranden@broadcom.com \
    --cc=shenyang39@huawei.com \
    --cc=srinivas.neeli@amd.com \
    --cc=thierry.reding@kernel.org \
    --cc=u.kleine-koenig@baylibre.com \
    --cc=valentina.fernandezalanis@microchip.com \
    --cc=vireshk@kernel.org \
    --cc=virtualization@lists.linux.dev \
    --cc=wbg@kernel.org \
    --cc=westeri@kernel.org \
    --cc=zhuyinbo@loongson.cn \
    /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

all inboxes | Powered by JetHome®