[PATCH AUTOSEL 6.16] pinctrl: meson: Fix typo in device table macro
Sasha Levin
sashal at kernel.org
Tue Sep 2 05:08:13 PDT 2025
From: Alexey Gladkov <legion at kernel.org>
[ Upstream commit bd7c2312128e31d056d30d34d60503de056e15f0 ]
The typo when using the MODULE_DEVICE_TABLE macro was not noticeable
because the macro was defined only if the module was built as a separate
module.
Cc: Xianwei Zhao <xianwei.zhao at amlogic.com>
Cc: Linus Walleij <linus.walleij at linaro.org>
Cc: Neil Armstrong <neil.armstrong at linaro.org>
Cc: Kevin Hilman <khilman at baylibre.com>
Cc: linux-amlogic at lists.infradead.org
Cc: linux-gpio at vger.kernel.org
Reported-by: kernel test robot <lkp at intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202507220009.8HKbNP16-lkp@intel.com/
Signed-off-by: Alexey Gladkov <legion at kernel.org>
Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>
Link: https://lore.kernel.org/e548b7761302defec15aa2098172eabb1ce1ad4a.1755170493.git.legion@kernel.org
Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
Signed-off-by: Sasha Levin <sashal at kernel.org>
---
LLM Generated explanations, may be completely bogus:
**YES**
This commit should be backported to stable kernel trees for the
following reasons:
1. **Clear bug fix**: The commit fixes a typo where
`MODULE_DEVICE_TABLE(of, aml_pctl_dt_match)` incorrectly referenced
`aml_pctl_dt_match` instead of the actual device table name
`aml_pctl_of_match` (line 1022-1025 shows the table is named
`aml_pctl_of_match`).
2. **Prevents module loading failure**: This typo would cause the module
to fail to load properly when built as a loadable module, as the
MODULE_DEVICE_TABLE macro creates an alias for module autoloading.
The incorrect reference means the device matching wouldn't work
correctly for module loading.
3. **Small, contained change**: The fix is a simple one-line change that
only corrects the typo - changing `aml_pctl_dt_match` to
`aml_pctl_of_match`. There are no architectural changes or feature
additions.
4. **Low regression risk**: This is a straightforward typo fix with
minimal risk. The change only affects the MODULE_DEVICE_TABLE macro
invocation and doesn't modify any functional code paths.
5. **Affects user functionality**: When the pinctrl driver is built as a
module (CONFIG_PINCTRL_AMLOGIC_A4=m), this bug would prevent proper
module autoloading and device matching, affecting users of Amlogic
A4/S6/S7 SoCs.
6. **Recently introduced code**: The driver was added very recently
(February 2025 based on commit 6e9be3abb78c2), and this typo was
present from the initial submission. Fixing it in stable trees
ensures users get a working driver.
The commit message indicates it was caught by the kernel test robot, and
the fix has already been reviewed by the subsystem maintainer (Neil
Armstrong). This is exactly the type of bug fix that stable kernels
should include - it fixes a real problem, is minimal in scope, and has
very low risk of introducing regressions.
drivers/pinctrl/meson/pinctrl-amlogic-a4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/meson/pinctrl-amlogic-a4.c b/drivers/pinctrl/meson/pinctrl-amlogic-a4.c
index 385cc619df13c..95525e66e5c00 100644
--- a/drivers/pinctrl/meson/pinctrl-amlogic-a4.c
+++ b/drivers/pinctrl/meson/pinctrl-amlogic-a4.c
@@ -1023,7 +1023,7 @@ static const struct of_device_id aml_pctl_of_match[] = {
{ .compatible = "amlogic,pinctrl-a4", },
{ /* sentinel */ }
};
-MODULE_DEVICE_TABLE(of, aml_pctl_dt_match);
+MODULE_DEVICE_TABLE(of, aml_pctl_of_match);
static struct platform_driver aml_pctl_driver = {
.driver = {
--
2.50.1
More information about the linux-amlogic
mailing list