| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 28 Jan 2022 13:24:19 +0900
From: Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>
To: Linus Walleij <linus.walleij@...aro.org>,
Masami Hiramatsu <mhiramat@...nel.org>
Cc: linux-gpio@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org,
Ryuta NAKANISHI <nakanishi.ryuta@...ionext.com>
Subject: Re: [PATCH 2/3] pinctrl: uniphier: Divide pinmux group to support 1ch
and 2ch I2S
On 2022/01/28 6:00, 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 v5.17-rc1 next-20220127]
> [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
> -Add-some-more-pinmux-settings/20220127-193715
> base:
> https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git
> devel
> config: openrisc-buildonly-randconfig-r003-20220124
> (https://download.01.org/0day-ci/archive/20220128/202201280448.njc6LUNd-lk
> p@...el.com/config)
> compiler: or1k-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/e0e366a006b79517336423f8a2c7266117
> 8d5e41
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review
> Kunihiko-Hayashi/pinctrl-uniphier-Add-some-more-pinmux-settings/20220127-1
> 93715
> git checkout e0e366a006b79517336423f8a2c72661178d5e41
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross
> O=build_dir ARCH=openrisc SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@...el.com>
>
> All errors (new ones prefixed by >>):
>
> In file included from drivers/pinctrl/uniphier/pinctrl-uniphier-pxs2.c:
> 12:
>>> drivers/pinctrl/uniphier/pinctrl-uniphier-pxs2.c:958:34: error:
> 'aout4_groups' undeclared here (not in a function); did you mean
> 'aout3_groups'?
> 958 | UNIPHIER_PINMUX_FUNCTION(aout4),
> | ^~~~~
> drivers/pinctrl/uniphier/pinctrl-uniphier.h:179:27: note: in definition
> of macro 'UNIPHIER_PINMUX_FUNCTION'
> 179 | .groups = func##_groups,
> \
> | ^~~~
> In file included from include/linux/container_of.h:5,
> from include/linux/kernel.h:21,
> from drivers/pinctrl/uniphier/pinctrl-uniphier-pxs2.c:
> 6:
>>> include/linux/build_bug.h:16:51: error: bit-field '<anonymous>' width
> not an integer constant
> 16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct {
> int:(-!!(e)); })))
> | ^
> include/linux/compiler.h:258:33: note: in expansion of macro
> 'BUILD_BUG_ON_ZERO'
> 258 | #define __must_be_array(a)
> BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
> | ^~~~~~~~~~~~~~~~~
> include/linux/kernel.h:55:59: note: in expansion of macro
> '__must_be_array'
> 55 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) +
> __must_be_array(arr))
> |
> ^~~~~~~~~~~~~~~
> drivers/pinctrl/uniphier/pinctrl-uniphier.h:180:31: note: in expansion
> of macro 'ARRAY_SIZE'
> 180 | .num_groups = ARRAY_SIZE(func##_groups),
> \
> | ^~~~~~~~~~
> drivers/pinctrl/uniphier/pinctrl-uniphier-pxs2.c:958:9: note: in
> expansion of macro 'UNIPHIER_PINMUX_FUNCTION'
> 958 | UNIPHIER_PINMUX_FUNCTION(aout4),
> | ^~~~~~~~~~~~~~~~~~~~~~~~
>
>
> vim +958 drivers/pinctrl/uniphier/pinctrl-uniphier-pxs2.c
>
> 949
> 950 static const struct uniphier_pinmux_function
> uniphier_pxs2_functions[] = {
> 951 UNIPHIER_PINMUX_FUNCTION(ain1),
> 952 UNIPHIER_PINMUX_FUNCTION(ain2),
> 953 UNIPHIER_PINMUX_FUNCTION(ain3),
> 954 UNIPHIER_PINMUX_FUNCTION(ainiec1),
> 955 UNIPHIER_PINMUX_FUNCTION(aout1),
> 956 UNIPHIER_PINMUX_FUNCTION(aout2),
> 957 UNIPHIER_PINMUX_FUNCTION(aout3),
> > 958 UNIPHIER_PINMUX_FUNCTION(aout4),
The aout4 function doesn't exist now. I'll remove them in next.
Thank you,
> 959 UNIPHIER_PINMUX_FUNCTION(aoutiec1),
> 960 UNIPHIER_PINMUX_FUNCTION(aoutiec2),
> 961 UNIPHIER_PINMUX_FUNCTION(emmc),
> 962 UNIPHIER_PINMUX_FUNCTION(ether_mii),
> 963 UNIPHIER_PINMUX_FUNCTION(ether_rgmii),
> 964 UNIPHIER_PINMUX_FUNCTION(ether_rmii),
> 965 UNIPHIER_PINMUX_FUNCTION(i2c0),
> 966 UNIPHIER_PINMUX_FUNCTION(i2c1),
> 967 UNIPHIER_PINMUX_FUNCTION(i2c2),
> 968 UNIPHIER_PINMUX_FUNCTION(i2c3),
> 969 UNIPHIER_PINMUX_FUNCTION(i2c5),
> 970 UNIPHIER_PINMUX_FUNCTION(i2c6),
> 971 UNIPHIER_PINMUX_FUNCTION(nand),
> 972 UNIPHIER_PINMUX_FUNCTION(sd),
> 973 UNIPHIER_PINMUX_FUNCTION(spi0),
> 974 UNIPHIER_PINMUX_FUNCTION(spi1),
> 975 UNIPHIER_PINMUX_FUNCTION(system_bus),
> 976 UNIPHIER_PINMUX_FUNCTION(uart0),
> 977 UNIPHIER_PINMUX_FUNCTION(uart1),
> 978 UNIPHIER_PINMUX_FUNCTION(uart2),
> 979 UNIPHIER_PINMUX_FUNCTION(uart3),
> 980 UNIPHIER_PINMUX_FUNCTION(usb0),
> 981 UNIPHIER_PINMUX_FUNCTION(usb1),
> 982 UNIPHIER_PINMUX_FUNCTION(usb2),
> 983 UNIPHIER_PINMUX_FUNCTION(usb3),
> 984 };
> 985
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>
--
---
Best Regards
Kunihiko Hayashi
Powered by blists - more mailing lists