* [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning
@ 2023-07-21 10:12 Sricharan Ramabadhran
2023-07-21 20:27 ` Nick Desaulniers
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Sricharan Ramabadhran @ 2023-07-21 10:12 UTC (permalink / raw)
To: agross, andersson, konrad.dybcio, linus.walleij, quic_srichara,
quic_varada, linux-arm-msm, linux-gpio, linux-kernel, llvm,
oe-kbuild-all
When building with clang toolchain and arm64-randconfig-r015-20230712
kernel test robot reports the below warning.
drivers/pinctrl/qcom/pinctrl-ipq5018.c:244:27: warning: unused variable '_groups' [-Wunused-const-variable]
static const char * const _groups[] = {
^
1 warning generated.
static const char * const _groups[] = {
"gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
"gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
"gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
"gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
"gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
"gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
"gpio43", "gpio44", "gpio45", "gpio46",
};
Fixing it by removing the variable.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202307120814.vWPY6URk-lkp@intel.com/
Fixes: 725d1c891658 ("pinctrl: qcom: Add IPQ5018 pinctrl driver")
Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com>
---
drivers/pinctrl/qcom/pinctrl-ipq5018.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/drivers/pinctrl/qcom/pinctrl-ipq5018.c b/drivers/pinctrl/qcom/pinctrl-ipq5018.c
index ed58f75..e2951f8 100644
--- a/drivers/pinctrl/qcom/pinctrl-ipq5018.c
+++ b/drivers/pinctrl/qcom/pinctrl-ipq5018.c
@@ -241,16 +241,6 @@ static const char * const atest_char_groups[] = {
"gpio0", "gpio1", "gpio2", "gpio3", "gpio37",
};
-static const char * const _groups[] = {
- "gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
- "gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
- "gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
- "gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
- "gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
- "gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
- "gpio43", "gpio44", "gpio45", "gpio46",
-};
-
static const char * const wci_txd_groups[] = {
"gpio0", "gpio1", "gpio2", "gpio3",
"gpio42", "gpio43", "gpio44", "gpio45",
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning
2023-07-21 10:12 [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning Sricharan Ramabadhran
@ 2023-07-21 20:27 ` Nick Desaulniers
2023-07-24 19:09 ` Krzysztof Kozlowski
2023-08-07 9:02 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Nick Desaulniers @ 2023-07-21 20:27 UTC (permalink / raw)
To: Sricharan Ramabadhran
Cc: agross, andersson, konrad.dybcio, linus.walleij, quic_varada,
linux-arm-msm, linux-gpio, linux-kernel, llvm, oe-kbuild-all
On Fri, Jul 21, 2023 at 3:13 AM Sricharan Ramabadhran
<quic_srichara@quicinc.com> wrote:
>
> When building with clang toolchain and arm64-randconfig-r015-20230712
> kernel test robot reports the below warning.
>
> drivers/pinctrl/qcom/pinctrl-ipq5018.c:244:27: warning: unused variable '_groups' [-Wunused-const-variable]
> static const char * const _groups[] = {
> ^
> 1 warning generated.
>
> static const char * const _groups[] = {
> "gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
> "gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
> "gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
> "gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
> "gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
> "gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
> "gpio43", "gpio44", "gpio45", "gpio46",
> };
>
> Fixing it by removing the variable.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202307120814.vWPY6URk-lkp@intel.com/
> Fixes: 725d1c891658 ("pinctrl: qcom: Add IPQ5018 pinctrl driver")
> Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com>
Thanks for the patch!
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
> ---
> drivers/pinctrl/qcom/pinctrl-ipq5018.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
> diff --git a/drivers/pinctrl/qcom/pinctrl-ipq5018.c b/drivers/pinctrl/qcom/pinctrl-ipq5018.c
> index ed58f75..e2951f8 100644
> --- a/drivers/pinctrl/qcom/pinctrl-ipq5018.c
> +++ b/drivers/pinctrl/qcom/pinctrl-ipq5018.c
> @@ -241,16 +241,6 @@ static const char * const atest_char_groups[] = {
> "gpio0", "gpio1", "gpio2", "gpio3", "gpio37",
> };
>
> -static const char * const _groups[] = {
> - "gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
> - "gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
> - "gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
> - "gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
> - "gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
> - "gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
> - "gpio43", "gpio44", "gpio45", "gpio46",
> -};
> -
> static const char * const wci_txd_groups[] = {
> "gpio0", "gpio1", "gpio2", "gpio3",
> "gpio42", "gpio43", "gpio44", "gpio45",
> --
> 2.7.4
>
>
--
Thanks,
~Nick Desaulniers
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning
2023-07-21 10:12 [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning Sricharan Ramabadhran
2023-07-21 20:27 ` Nick Desaulniers
@ 2023-07-24 19:09 ` Krzysztof Kozlowski
2023-08-07 9:02 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-24 19:09 UTC (permalink / raw)
To: Sricharan Ramabadhran, agross, andersson, konrad.dybcio,
linus.walleij, quic_varada, linux-arm-msm, linux-gpio,
linux-kernel, llvm, oe-kbuild-all
On 21/07/2023 12:12, Sricharan Ramabadhran wrote:
> When building with clang toolchain and arm64-randconfig-r015-20230712
> kernel test robot reports the below warning.
>
> drivers/pinctrl/qcom/pinctrl-ipq5018.c:244:27: warning: unused variable '_groups' [-Wunused-const-variable]
> static const char * const _groups[] = {
> ^
> 1 warning generated.
>
> static const char * const _groups[] = {
> "gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
> "gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
> "gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
> "gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
> "gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
> "gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
> "gpio43", "gpio44", "gpio45", "gpio46",
> };
>
> Fixing it by removing the variable.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202307120814.vWPY6URk-lkp@intel.com/
> Fixes: 725d1c891658 ("pinctrl: qcom: Add IPQ5018 pinctrl driver")
> Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com>
This seems to be a duplicate of gpio_groups, so:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning
2023-07-21 10:12 [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning Sricharan Ramabadhran
2023-07-21 20:27 ` Nick Desaulniers
2023-07-24 19:09 ` Krzysztof Kozlowski
@ 2023-08-07 9:02 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2023-08-07 9:02 UTC (permalink / raw)
To: Sricharan Ramabadhran
Cc: agross, andersson, konrad.dybcio, quic_varada, linux-arm-msm,
linux-gpio, linux-kernel, llvm, oe-kbuild-all
On Fri, Jul 21, 2023 at 12:13 PM Sricharan Ramabadhran
<quic_srichara@quicinc.com> wrote:
> When building with clang toolchain and arm64-randconfig-r015-20230712
> kernel test robot reports the below warning.
>
> drivers/pinctrl/qcom/pinctrl-ipq5018.c:244:27: warning: unused variable '_groups' [-Wunused-const-variable]
> static const char * const _groups[] = {
> ^
> 1 warning generated.
>
> static const char * const _groups[] = {
> "gpio0", "gpio1", "gpio2", "gpio3", "gpio4", "gpio5", "gpio6", "gpio7",
> "gpio8", "gpio9", "gpio10", "gpio11", "gpio12", "gpio13", "gpio14",
> "gpio15", "gpio16", "gpio17", "gpio18", "gpio19", "gpio20", "gpio21",
> "gpio22", "gpio23", "gpio24", "gpio25", "gpio26", "gpio27", "gpio28",
> "gpio29", "gpio30", "gpio31", "gpio32", "gpio33", "gpio34", "gpio35",
> "gpio36", "gpio37", "gpio38", "gpio39", "gpio40", "gpio41", "gpio42",
> "gpio43", "gpio44", "gpio45", "gpio46",
> };
>
> Fixing it by removing the variable.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202307120814.vWPY6URk-lkp@intel.com/
> Fixes: 725d1c891658 ("pinctrl: qcom: Add IPQ5018 pinctrl driver")
> Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com>
Patch applied, thanks for fixing this!
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-08-07 9:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-21 10:12 [PATCH] pinctrl: qcom: Remove the unused _groups variable build warning Sricharan Ramabadhran
2023-07-21 20:27 ` Nick Desaulniers
2023-07-24 19:09 ` Krzysztof Kozlowski
2023-08-07 9:02 ` Linus Walleij
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.