Re: [PATCH 3/3] pinctrl: uniphier: Add UniPhier NX1 pinctrl driver

[Kunihiko Hayashi]

On 2021/10/05 22:06, kernel test robot wrote:
> Hi Kunihiko,
>
> I love your patch! Yet something to improve:
>
> [auto build test ERROR on linusw-pinctrl/devel]
> [also build test ERROR on robh/for-next v5.15-rc3 next-20210922]
> [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]
>
> url:
> https://github.com/0day-ci/linux/commits/Kunihiko-Hayashi/pinctrl-uniphier
> -Introduce-some-features-and-NX1-support/20211005-101346
> base:
> https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git
> devel
> config: powerpc-allyesconfig (attached as .config)
> compiler: powerpc64-linux-gcc (GCC) 11.2.0
> reproduce (this is a W=1 build):
>          wget
> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
> -O ~/bin/make.cross
>          chmod +x ~/bin/make.cross
>          #
> https://github.com/0day-ci/linux/commit/c68781cbddcc323070eee04896f6f85884
> 0a60dc
>          git remote add linux-review https://github.com/0day-ci/linux
>          git fetch --no-tags linux-review
> Kunihiko-Hayashi/pinctrl-uniphier-Introduce-some-features-and-NX1-support/
> 20211005-101346
>          git checkout c68781cbddcc323070eee04896f6f858840a60dc
>          # save the attached .config to linux build tree
>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross
> ARCH=powerpc
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
>
> All errors (new ones prefixed by >>):
>
> > > drivers/pinctrl/uniphier/pinctrl-uniphier-nx1.c:429:27: error:
> 'usb3_groups' defined but not used [-Werror=unused-const-variable=]
>       429 | static const char * const usb3_groups[] = {"usb3"};
>           |                           ^~~~~~~~~~~
> > > drivers/pinctrl/uniphier/pinctrl-uniphier-nx1.c:428:27: error:
> 'usb2_groups' defined but not used [-Werror=unused-const-variable=]
>       428 | static const char * const usb2_groups[] = {"usb2"};
>           |                           ^~~~~~~~~~~
>     cc1: all warnings being treated as errors

The usb2 and usb3 groups don't exist. I'll remove them in next.

Thank you,

---
Best Regards
Kunihiko Hayashi