Re: [PATCH 2/2] pinctrl: tegra: Add pinmux support for Tegra194

[Krzysztof Kozlowski <krzk@kernel.org>]

On 05/11/2021 18:49, kernel test robot wrote:
> Hi Prathamesh,
> 
> Thank you for the patch! Perhaps something to improve:
> 
> [auto build test WARNING on tegra/for-next]
> [also build test WARNING on v5.15]
> [cannot apply to linusw-pinctrl/devel next-20211105]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]

(...)

                        \
>                               ^~
>>> drivers/pinctrl/tegra/pinctrl-tegra194.c:1113:27: warning: unused variable 'sdmmc1_hv_trim_pins' [-Wunused-const-variable]
>    static const unsigned int sdmmc1_hv_trim_pins[] = {
>                              ^
>>> drivers/pinctrl/tegra/pinctrl-tegra194.c:1119:27: warning: unused variable 'sdmmc3_hv_trim_pins' [-Wunused-const-variable]
>    static const unsigned int sdmmc3_hv_trim_pins[] = {
>                              ^
>>> drivers/pinctrl/tegra/pinctrl-tegra194.c:1128:27: warning: unused variable 'sys_reset_n_pins' [-Wunused-const-variable]
>    static const unsigned int sys_reset_n_pins[] = {
>                              ^

Hi Prathamesh,

I stumbled on issue with your commit and now I found that this was
already reported 4 times to you:
https://lore.kernel.org/all/?q=sys_reset_n_pins

Never fixed by the author.

Why these reports about errors in your commit where ignored? It's your
commit so leaving it as is in kernel is not responsible...

Best regards,
Krzysztof