Re: [PATCH 1/2] staging: fbtft: Moves ";" from macro definition to macro usage.

From: kbuild test robot
Date: Fri Aug 10 2018 - 07:28:47 EST


Hi Leonardo,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.18-rc8 next-20180809]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Leonardo-Br-s/staging-fbtft-Moves-from-macro-definition-to-macro-usage/20180810-154004
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> drivers/staging/fbtft/fbtft-bus.c:66:1: sparse: incorrect type in assignment (different base types) @@ expected restricted __be16 [usertype] <noident> @@ got unsignedrestricted __be16 [usertype] <noident> @@
drivers/staging/fbtft/fbtft-bus.c:66:1: expected restricted __be16 [usertype] <noident>
drivers/staging/fbtft/fbtft-bus.c:66:1: got unsigned short [unsigned] [usertype] <noident>
>> drivers/staging/fbtft/fbtft-bus.c:66:1: sparse: incorrect type in assignment (different base types) @@ expected restricted __be16 [usertype] <noident> @@ got unsignedrestricted __be16 [usertype] <noident> @@
drivers/staging/fbtft/fbtft-bus.c:66:1: expected restricted __be16 [usertype] <noident>
drivers/staging/fbtft/fbtft-bus.c:66:1: got unsigned short [unsigned] [usertype] <noident>
>> drivers/staging/fbtft/fbtft-bus.c:66:1: sparse: incorrect type in assignment (different base types) @@ expected restricted __be16 [usertype] <noident> @@ got unsignedrestricted __be16 [usertype] <noident> @@
drivers/staging/fbtft/fbtft-bus.c:66:1: expected restricted __be16 [usertype] <noident>
drivers/staging/fbtft/fbtft-bus.c:66:1: got unsigned short [unsigned] [usertype] <noident>
drivers/staging/fbtft/fbtft-bus.c:156:27: sparse: expression using sizeof(void)
drivers/staging/fbtft/fbtft-bus.c:156:27: sparse: expression using sizeof(void)
drivers/staging/fbtft/fbtft-bus.c:200:27: sparse: expression using sizeof(void)
drivers/staging/fbtft/fbtft-bus.c:200:27: sparse: expression using sizeof(void)

vim +66 drivers/staging/fbtft/fbtft-bus.c

64
65 define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8, );
> 66 define_fbtft_write_reg(fbtft_write_reg16_bus8, __be16, u16, );
67 define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16, );
68

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation