* [PATCH] pinctrl: qcom: sdm670-lpass-lpi: label variables as static
@ 2026-04-08 22:30 Richard Acayan
2026-04-09 7:20 ` Linus Walleij
0 siblings, 1 reply; 2+ messages in thread
From: Richard Acayan @ 2026-04-08 22:30 UTC (permalink / raw)
To: Bjorn Andersson, Linus Walleij, Konrad Dybcio, Dmitry Baryshkov,
linux-arm-msm, linux-gpio
Cc: kernel test robot, Richard Acayan
These variables are local to the driver and have no need to be exported
to the global namespace. Label them as static to fix compiler warnings.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202604080950.Mvm8aN0a-lkp@intel.com/
Fixes: 9826035a75da ("pinctrl: qcom: add sdm670 lpi tlmm")
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
drivers/pinctrl/qcom/pinctrl-sdm670-lpass-lpi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/qcom/pinctrl-sdm670-lpass-lpi.c b/drivers/pinctrl/qcom/pinctrl-sdm670-lpass-lpi.c
index 6270c6d09c22..858146c408d0 100644
--- a/drivers/pinctrl/qcom/pinctrl-sdm670-lpass-lpi.c
+++ b/drivers/pinctrl/qcom/pinctrl-sdm670-lpass-lpi.c
@@ -80,7 +80,7 @@ static const char * const pdm_sync_groups[] = { "gpio19" };
static const char * const pdm_tx_groups[] = { "gpio20" };
static const char * const slimbus_clk_groups[] = { "gpio18" };
-const struct lpi_pingroup sdm670_lpi_pinctrl_groups[] = {
+static const struct lpi_pingroup sdm670_lpi_pinctrl_groups[] = {
LPI_PINGROUP(0, LPI_NO_SLEW, _, _, _, _),
LPI_PINGROUP(1, LPI_NO_SLEW, _, _, _, _),
LPI_PINGROUP(2, LPI_NO_SLEW, _, _, _, _),
@@ -115,7 +115,7 @@ const struct lpi_pingroup sdm670_lpi_pinctrl_groups[] = {
LPI_PINGROUP(31, LPI_NO_SLEW, _, _, _, _),
};
-const struct lpi_function sdm670_lpi_pinctrl_functions[] = {
+static const struct lpi_function sdm670_lpi_pinctrl_functions[] = {
LPI_FUNCTION(comp_rx),
LPI_FUNCTION(dmic1_clk),
LPI_FUNCTION(dmic1_data),
--
2.53.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] pinctrl: qcom: sdm670-lpass-lpi: label variables as static
2026-04-08 22:30 [PATCH] pinctrl: qcom: sdm670-lpass-lpi: label variables as static Richard Acayan
@ 2026-04-09 7:20 ` Linus Walleij
0 siblings, 0 replies; 2+ messages in thread
From: Linus Walleij @ 2026-04-09 7:20 UTC (permalink / raw)
To: Richard Acayan
Cc: Bjorn Andersson, Konrad Dybcio, Dmitry Baryshkov, linux-arm-msm,
linux-gpio, kernel test robot
On Thu, Apr 9, 2026 at 12:29 AM Richard Acayan <mailingradian@gmail.com> wrote:
> These variables are local to the driver and have no need to be exported
> to the global namespace. Label them as static to fix compiler warnings.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202604080950.Mvm8aN0a-lkp@intel.com/
> Fixes: 9826035a75da ("pinctrl: qcom: add sdm670 lpi tlmm")
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
Oh I just made the same patch, nevermind I'll apply this one instead.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-04-09 7:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-08 22:30 [PATCH] pinctrl: qcom: sdm670-lpass-lpi: label variables as static Richard Acayan
2026-04-09 7:20 ` Linus Walleij
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox