Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: Justin Yeh <justin.yeh@mediatek.com>,
	Sean Wang <sean.wang@kernel.org>,
	Linus Walleij <linusw@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>
Cc: Project_Global_Chrome_Upstream_Group@mediatek.com,
	linux-mediatek@lists.infradead.org, linux-gpio@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v4 00/32] pinctrl: mediatek: Enable module build support for all drivers
Date: Fri, 26 Jun 2026 12:04:14 +0200	[thread overview]
Message-ID: <ad1a6a17-aff4-4dbf-9cdb-62765064b670@collabora.com> (raw)
In-Reply-To: <20260626040112.2436185-1-justin.yeh@mediatek.com>

On 6/26/26 06:00, Justin Yeh wrote:
> Sorry for the quick v4 - v3 was sent with an incomplete cover letter
> (template placeholders) by mistake. This v4 also unifies MODULE_LICENSE
> to consistently use "GPL v2" across all patches.
> 
> This series enables all MediaTek pinctrl drivers to be built as loadable
> kernel modules. This is required for Android GKI (Generic Kernel Image) +
> vendor_dlkm deployments where vendor-specific drivers must be kept separate
> from the GKI vmlinux.
> 
> Each patch adds MODULE_LICENSE("GPL v2") and MODULE_DESCRIPTION() macros where
> missing, and changes the Kconfig option from bool to tristate. This allows
> these drivers to be properly packaged as vendor kernel modules while
> maintaining the existing built-in option.
> 

The entire series is

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

...but I'm not sure if one commit for each is ok, makes reverting easy but that
may be too much noise. Should Linus ask for joining them all, feel free to still
keep my R-b tag, as I'd be fine with that as well.

Cheers,
Angelo

> Changes in v4:
>    * Fix cover letter content (v3 accidentally sent with template placeholders)
>    * Unify MODULE_LICENSE to use "GPL v2" consistently across all drivers
>    * Update all commit messages to reflect "GPL v2" instead of "GPL"
> 
> Changes in v3:
>    * Add MODULE_DESCRIPTION() for all drivers (even those that already had MODULE_LICENSE)
>    * Update commit messages to reflect that we're adding MODULE_DESCRIPTION too
> 
> Changes in v2:
>    * Squash MODULE_LICENSE and tristate changes into single patch per driver
>    * Extend fix to all MediaTek pinctrl drivers (32 total), not just MT8189
>    * Add Android GKI + vendor_dlkm context to cover letter
>    * Add MODULE_DESCRIPTION() where it was missing
>    * Add Fixes: tags referencing the original commits that added each driver
> 
> Justin Yeh (32):
>    pinctrl: mediatek: mt8189: Enable module build support
>    pinctrl: mediatek: mt6878: Enable module build support
>    pinctrl: mediatek: mt6893: Enable module build support
>    pinctrl: mediatek: mt7622: Enable module build support
>    pinctrl: mediatek: mt7981: Enable module build support
>    pinctrl: mediatek: mt7986: Enable module build support
>    pinctrl: mediatek: mt7988: Enable module build support
>    pinctrl: mediatek: mt8167: Enable module build support
>    pinctrl: mediatek: mt8173: Enable module build support
>    pinctrl: mediatek: mt8183: Enable module build support
>    pinctrl: mediatek: mt8186: Enable module build support
>    pinctrl: mediatek: mt8188: Enable module build support
>    pinctrl: mediatek: mt8192: Enable module build support
>    pinctrl: mediatek: mt8195: Enable module build support
>    pinctrl: mediatek: mt8196: Enable module build support
>    pinctrl: mediatek: mt8365: Enable module build support
>    pinctrl: mediatek: mt8516: Enable module build support
>    pinctrl: mediatek: mt2701: Enable module build support
>    pinctrl: mediatek: mt7623: Enable module build support
>    pinctrl: mediatek: mt7629: Enable module build support
>    pinctrl: mediatek: mt8135: Enable module build support
>    pinctrl: mediatek: mt8127: Enable module build support
>    pinctrl: mediatek: mt7620: Enable module build support
>    pinctrl: mediatek: mt7621: Enable module build support
>    pinctrl: mediatek: mt76x8: Enable module build support
>    pinctrl: mediatek: rt2880: Enable module build support
>    pinctrl: mediatek: rt305x: Enable module build support
>    pinctrl: mediatek: rt3883: Enable module build support
>    pinctrl: mediatek: mt6397: Enable module build support
>    pinctrl: mediatek: mt2712: Enable module build support
>    pinctrl: mediatek: mt6795: Enable module build support
>    pinctrl: mediatek: mt6797: Enable module build support
> 
>   drivers/pinctrl/mediatek/Kconfig          | 64 +++++++++++------------
>   drivers/pinctrl/mediatek/pinctrl-mt2701.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt2712.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt6397.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt6795.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt6797.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt6878.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt6893.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt7620.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7621.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7622.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7623.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7629.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt76x8.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7981.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7986.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt7988.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8127.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8135.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8167.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8173.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8183.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8186.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8188.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt8189.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt8192.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt8195.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-mt8196.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt8365.c |  1 +
>   drivers/pinctrl/mediatek/pinctrl-mt8516.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-rt2880.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-rt305x.c |  3 ++
>   drivers/pinctrl/mediatek/pinctrl-rt3883.c |  3 ++
>   33 files changed, 114 insertions(+), 32 deletions(-)
> 



  parent reply	other threads:[~2026-06-26 10:04 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-26  4:00 [PATCH v4 00/32] pinctrl: mediatek: Enable module build support for all drivers Justin Yeh
2026-06-26  4:00 ` [PATCH v4 01/32] pinctrl: mediatek: mt8189: Enable module build support Justin Yeh
2026-06-26  4:00 ` [PATCH v4 02/32] pinctrl: mediatek: mt6878: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 03/32] pinctrl: mediatek: mt6893: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 04/32] pinctrl: mediatek: mt7622: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 05/32] pinctrl: mediatek: mt7981: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 06/32] pinctrl: mediatek: mt7986: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 07/32] pinctrl: mediatek: mt7988: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 08/32] pinctrl: mediatek: mt8167: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 09/32] pinctrl: mediatek: mt8173: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 10/32] pinctrl: mediatek: mt8183: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 11/32] pinctrl: mediatek: mt8186: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 12/32] pinctrl: mediatek: mt8188: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 13/32] pinctrl: mediatek: mt8192: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 14/32] pinctrl: mediatek: mt8195: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 15/32] pinctrl: mediatek: mt8196: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 16/32] pinctrl: mediatek: mt8365: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 17/32] pinctrl: mediatek: mt8516: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 18/32] pinctrl: mediatek: mt2701: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 19/32] pinctrl: mediatek: mt7623: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 20/32] pinctrl: mediatek: mt7629: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 21/32] pinctrl: mediatek: mt8135: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 22/32] pinctrl: mediatek: mt8127: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 23/32] pinctrl: mediatek: mt7620: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 24/32] pinctrl: mediatek: mt7621: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 25/32] pinctrl: mediatek: mt76x8: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 26/32] pinctrl: mediatek: rt2880: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 27/32] pinctrl: mediatek: rt305x: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 28/32] pinctrl: mediatek: rt3883: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 29/32] pinctrl: mediatek: mt6397: " Justin Yeh
2026-06-26  4:00 ` [PATCH v4 30/32] pinctrl: mediatek: mt2712: " Justin Yeh
2026-06-26  4:01 ` [PATCH v4 31/32] pinctrl: mediatek: mt6795: " Justin Yeh
2026-06-26  4:01 ` [PATCH v4 32/32] pinctrl: mediatek: mt6797: " Justin Yeh
2026-06-26 10:04 ` AngeloGioacchino Del Regno [this message]
2026-06-26 13:17 ` [PATCH v4 00/32] pinctrl: mediatek: Enable module build support for all drivers Linus Walleij

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=ad1a6a17-aff4-4dbf-9cdb-62765064b670@collabora.com \
    --to=angelogioacchino.delregno@collabora.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=justin.yeh@mediatek.com \
    --cc=linusw@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=sean.wang@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