From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 787B1CA0FFE for ; Tue, 2 Sep 2025 15:45:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=P7IQUwrmaIJJyiAIRA3OBRlT5m+RLmtCALTMDM8tgUE=; b=xxuFwZaukhU/Gb 408Xu12RQwInuqJJ2zKFGgBHUdq3Ni6iByXzdmzYVSKY5mAzupHKba+75EdfUUaAgbmuXqRGWWHmc WiNRnddIVYZu7KsJGNM3zJ4JCBloEuTo4f6R0DA37UwuGbSm+nX3/Ark1VUbAQ/PM0AksK9f9rcms J2RLxYFu3m2EsOmt1QUbSINJK+R5p7yYBmGGO8+3wpXMUA4A/pnJYwpIre22ZMXyJ53If2Pz2HTHt 8d4L5jqDVSpuCNtHPflLKfY8zwfK7v1dVRWRMxmE2dpvhg3DlgG5j8xfj8D0sBX9/VUwVFAlSVbsv Y5HjNcZ6C+YJ3pq4yEzw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1utTBs-00000000km1-0CVa; Tue, 02 Sep 2025 15:45:00 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1utPoU-0000000H7RY-2jwB; Tue, 02 Sep 2025 12:08:38 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 175796020C; Tue, 2 Sep 2025 12:08:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A7627C4CEF7; Tue, 2 Sep 2025 12:08:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1756814917; bh=757pey5fnEg/qeq9CPxzMrLDiz9/8/oYvMF8D6dGc30=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=sfyuwHypWYSuAc0P6oMVNGBTCH4OCNymSjUysZLo4FjzWYj2V0Z7LyHxATqIXKDrx DcdeIoRWSEZGu8+amagySS7gT9WngVj6XzfCqEVZiXwvO2u/pdbCWNdYIfmroY4ELj ZBXBUti33S6pJXFsQuGsxLb2Jne+hrNYAhCntdn+LEZkzgJULTNc3I4RX4cmzLog36 EcfeinLB8Bk7BzJCJKYCHtnVDDaaHZNurdq5CfSmS9fXYPbQe8x5CP/Hx5Lv8JIdHN 5FMlG4wO3UMSzdI217+0223NMHbCO9OoE8F/vVW1GPZATc0AJHmaBHeepnxLSpLJG7 BpBWt1wp5oqQA== From: Sasha Levin To: patches@lists.linux.dev, stable@vger.kernel.org Cc: Alexey Gladkov , Xianwei Zhao , Linus Walleij , Neil Armstrong , Kevin Hilman , linux-amlogic@lists.infradead.org, linux-gpio@vger.kernel.org, kernel test robot , Sasha Levin , linux-arm-kernel@lists.infradead.org Subject: [PATCH AUTOSEL 6.16] pinctrl: meson: Fix typo in device table macro Date: Tue, 2 Sep 2025 08:08:13 -0400 Message-ID: <20250902120833.1342615-2-sashal@kernel.org> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20250902120833.1342615-1-sashal@kernel.org> References: <20250902120833.1342615-1-sashal@kernel.org> MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.16.4 X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org From: Alexey Gladkov [ 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 Cc: Linus Walleij Cc: Neil Armstrong Cc: Kevin Hilman Cc: linux-amlogic@lists.infradead.org Cc: linux-gpio@vger.kernel.org Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202507220009.8HKbNP16-lkp@intel.com/ Signed-off-by: Alexey Gladkov Reviewed-by: Neil Armstrong Link: https://lore.kernel.org/e548b7761302defec15aa2098172eabb1ce1ad4a.1755170493.git.legion@kernel.org Signed-off-by: Linus Walleij Signed-off-by: Sasha Levin --- 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 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic