Re: [pinctrl:devel 16/46] drivers/pinctrl/bcm/pinctrl-bcm2835.c:995:10: error: incompatible types when assigning to type 'volatile struct SHIFTER' from type 'unsigned int'

[Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>]

CC linux-m68k (shifter too generic a name?)

On Tue, Aug 6, 2019 at 5:00 AM kbuild test robot <lkp@xxxxxxxxx> wrote:
>
> tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git devel
> head:   d55b7fdd58ac12e76ef65979af4a13b9c15fc00d
> commit: e38a9a437fb93ddafab5030165e4c6a3a5021669 [16/46] pinctrl: bcm2835: Add support for BCM2711 pull-up functionality
> config: m68k-allmodconfig (attached as .config)
> compiler: m68k-linux-gcc (GCC) 7.4.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout e38a9a437fb93ddafab5030165e4c6a3a5021669
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.4.0 make.cross ARCH=m68k
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>
>
> All error/warnings (new ones prefixed by >>):
>
>    In file included from arch/m68k/include/asm/io_mm.h:32:0,
>                     from arch/m68k/include/asm/io.h:8,
>                     from include/linux/io.h:13,
>                     from include/linux/irq.h:20,
>                     from include/linux/gpio/driver.h:7,
>                     from drivers/pinctrl/bcm/pinctrl-bcm2835.c:17:
>    drivers/pinctrl/bcm/pinctrl-bcm2835.c: In function 'bcm2711_pull_config_set':
> >> arch/m68k/include/asm/atarihw.h:190:22: error: expected identifier or '(' before 'volatile'
>     # define shifter ((*(volatile struct SHIFTER *)SHF_BAS))
>                          ^
> >> drivers/pinctrl/bcm/pinctrl-bcm2835.c:990:6: note: in expansion of macro 'shifter'
>      u32 shifter;
>          ^~~~~~~
> >> arch/m68k/include/asm/atarihw.h:172:17: error: expected ')' before '(' token
>     #define SHF_BAS (0xffff8200)
>                     ^
> >> arch/m68k/include/asm/atarihw.h:190:48: note: in expansion of macro 'SHF_BAS'
>     # define shifter ((*(volatile struct SHIFTER *)SHF_BAS))
>                                                    ^~~~~~~
> >> drivers/pinctrl/bcm/pinctrl-bcm2835.c:990:6: note: in expansion of macro 'shifter'
>      u32 shifter;
>          ^~~~~~~
> >> drivers/pinctrl/bcm/pinctrl-bcm2835.c:995:10: error: incompatible types when assigning to type 'volatile struct SHIFTER' from type 'unsigned int'
>      shifter = PUD_2711_REG_SHIFT(pin);
>              ^
> >> drivers/pinctrl/bcm/pinctrl-bcm2835.c:998:27: error: invalid operands to binary << (have 'int' and 'volatile struct SHIFTER')
>      value &= ~(PUD_2711_MASK << shifter);
>                               ^~
> >> drivers/pinctrl/bcm/pinctrl-bcm2835.c:999:16: error: invalid operands to binary << (have 'unsigned int' and 'volatile struct SHIFTER')
>      value |= (arg << shifter);
>                    ^~
> --
>    In file included from arch/m68k/include/asm/io_mm.h:32:0,
>                     from arch/m68k/include/asm/io.h:8,
>                     from include/linux/io.h:13,
>                     from include/linux/irq.h:20,
>                     from include/linux/gpio/driver.h:7,
>                     from drivers/pinctrl//bcm/pinctrl-bcm2835.c:17:
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c: In function 'bcm2711_pull_config_set':
> >> arch/m68k/include/asm/atarihw.h:190:22: error: expected identifier or '(' before 'volatile'
>     # define shifter ((*(volatile struct SHIFTER *)SHF_BAS))
>                          ^
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c:990:6: note: in expansion of macro 'shifter'
>      u32 shifter;
>          ^~~~~~~
> >> arch/m68k/include/asm/atarihw.h:172:17: error: expected ')' before '(' token
>     #define SHF_BAS (0xffff8200)
>                     ^
> >> arch/m68k/include/asm/atarihw.h:190:48: note: in expansion of macro 'SHF_BAS'
>     # define shifter ((*(volatile struct SHIFTER *)SHF_BAS))
>                                                    ^~~~~~~
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c:990:6: note: in expansion of macro 'shifter'
>      u32 shifter;
>          ^~~~~~~
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c:995:10: error: incompatible types when assigning to type 'volatile struct SHIFTER' from type 'unsigned int'
>      shifter = PUD_2711_REG_SHIFT(pin);
>              ^
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c:998:27: error: invalid operands to binary << (have 'int' and 'volatile struct SHIFTER')
>      value &= ~(PUD_2711_MASK << shifter);
>                               ^~
>    drivers/pinctrl//bcm/pinctrl-bcm2835.c:999:16: error: invalid operands to binary << (have 'unsigned int' and 'volatile struct SHIFTER')
>      value |= (arg << shifter);
>                    ^~
>
> vim +995 drivers/pinctrl/bcm/pinctrl-bcm2835.c
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation