[kbuild-all] [zen-kernel-zen-kernel:5.8/zen-sauce 2/26] arch/m68k/include/asm/raw_io.h:83:7: error: variable '__w' set but not used

tree: https://github.com/zen-kernel/zen-kernel 5.8/zen-sauce head: 4ae00f4cbefe933616f33bf79935393ec49dbfa3 commit: 5664a4e6b6af231e79f53494742a7d7a08ba72d7 [2/26] ZEN: Add VHBA driver config: m68k-allmodconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 9.3.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 git checkout 5664a4e6b6af231e79f53494742a7d7a08ba72d7 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All errors (new ones prefixed by >>): In file included from arch/m68k/include/asm/io_mm.h:25, from arch/m68k/include/asm/io.h:8, from include/linux/io.h:13, from include/linux/irq.h:20, from include/asm-generic/hardirq.h:13, from ./arch/m68k/include/generated/asm/hardirq.h:1, from include/linux/hardirq.h:10, from include/linux/highmem.h:10, from drivers/scsi/vhba/vhba.c:25: arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ | ^~~ arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro 'rom_out_8' 430 | rom_out_8(port, *buf++); | ^~~~~~~~~ arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': arch/m68k/include/asm/raw_io.h:86:8: error: variable '__w' set but not used [-Werror=unused-but-set-variable] 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ | ^~~ arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro 'rom_out_be16' 448 | rom_out_be16(port, *buf++); | ^~~~~~~~~~~~ arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': arch/m68k/include/asm/raw_io.h:90:8: error: variable '__w' set but not used [-Werror=unused-but-set-variable] 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ | ^~~ arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro 'rom_out_le16' 466 | rom_out_le16(port, *buf++); | ^~~~~~~~~~~~ In file included from include/linux/init.h:5, from drivers/scsi/vhba/vhba.c:23: include/linux/scatterlist.h: In function 'sg_set_buf': arch/m68k/include/asm/page_mm.h:169:49: error: ordered comparison of pointer with null pointer [-Werror=extra] 169 | #define virt_addr_valid(kaddr) ((void *)(kaddr) >= (void *)PAGE_OFFSET && (void *)(kaddr) < high_memory) | ^~ include/linux/compiler.h:78:42: note: in definition of macro 'unlikely' 78 | # define unlikely(x) __builtin_expect(!!(x), 0) | ^ include/linux/scatterlist.h:143:2: note: in expansion of macro 'BUG_ON' 143 | BUG_ON(!virt_addr_valid(buf)); | ^~~~~~ include/linux/scatterlist.h:143:10: note: in expansion of macro 'virt_addr_valid' 143 | BUG_ON(!virt_addr_valid(buf)); | ^~~~~~~~~~~~~~~ In file included from arch/m68k/include/asm/bug.h:32, from include/linux/bug.h:5, from include/linux/thread_info.h:12, from include/asm-generic/preempt.h:5, from ./arch/m68k/include/generated/asm/preempt.h:1, from include/linux/preempt.h:78, from include/linux/spinlock.h:51, from include/linux/seqlock.h:36, from include/linux/time.h:6, from include/linux/stat.h:19, from include/linux/module.h:13, from drivers/scsi/vhba/vhba.c:24: include/linux/dma-mapping.h: In function 'dma_map_resource': arch/m68k/include/asm/page_mm.h:169:49: error: ordered comparison of pointer with null pointer [-Werror=extra] 169 | #define virt_addr_valid(kaddr) ((void *)(kaddr) >= (void *)PAGE_OFFSET && (void *)(kaddr) < high_memory) | ^~ include/asm-generic/bug.h:144:27: note: in definition of macro 'WARN_ON_ONCE' 144 | int __ret_warn_once = !!(condition); \ | ^~~~~~~~~ arch/m68k/include/asm/page_mm.h:170:25: note: in expansion of macro 'virt_addr_valid' 170 | #define pfn_valid(pfn) virt_addr_valid(pfn_to_virt(pfn)) | ^~~~~~~~~~~~~~~ include/linux/dma-mapping.h:352:19: note: in expansion of macro 'pfn_valid' 352 | if (WARN_ON_ONCE(pfn_valid(PHYS_PFN(phys_addr)))) | ^~~~~~~~~ In file included from drivers/scsi/vhba/vhba.c:42: drivers/scsi/vhba/vhba.c: At top level: drivers/scsi/vhba/vhba.c:483:15: error: no previous prototype for 'vhba_queuecommand' [-Werror=missing-prototypes] 483 | DEF_SCSI_QCMD(vhba_queuecommand) | ^~~~~~~~~~~~~~~~~ include/scsi/scsi_host.h:495:6: note: in definition of macro 'DEF_SCSI_QCMD' 495 | int func_name(struct Scsi_Host *shost, struct scsi_cmnd *cmd) \ | ^~~~~~~~~ cc1: all warnings being treated as errors # https://github.com/zen-kernel/zen-kernel/commit/5664a4e6b6af231e79f534947... git remote add zen-kernel-zen-kernel https://github.com/zen-kernel/zen-kernel git fetch --no-tags zen-kernel-zen-kernel 5.8/zen-sauce git checkout 5664a4e6b6af231e79f53494742a7d7a08ba72d7 vim +/__w +83 arch/m68k/include/asm/raw_io.h ^1da177e4c3f415 include/asm-m68k/raw_io.h Linus Torvalds 2005-04-16 49 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 50 /* 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 51 * Atari ROM port (cartridge port) ISA adapter, used for the EtherNEC NE2000 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 52 * network card driver. 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 53 * The ISA adapter connects address lines A9-A13 to ISA address lines A0-A4, 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 54 * and hardwires the rest of the ISA addresses for a base address of 0x300. 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 55 * 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 56 * Data lines D8-D15 are connected to ISA data lines D0-D7 for reading. 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 57 * For writes, address lines A1-A8 are latched to ISA data lines D0-D7 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 58 * (meaning the bit pattern on A1-A8 can be read back as byte). 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 59 * 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 60 * Read and write operations are distinguished by the base address used: 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 61 * reads are from the ROM A side range, writes are through the B side range 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 62 * addresses (A side base + 0x10000). 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 63 * 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 64 * Reads and writes are byte only. 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 65 * 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 66 * 16 bit reads and writes are necessary for the NetUSBee adapter's USB 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 67 * chipset - 16 bit words are read straight off the ROM port while 16 bit 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 68 * reads are split into two byte writes. The low byte is latched to the 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 69 * NetUSBee buffer by a read from the _read_ window (with the data pattern 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 70 * asserted as A1-A8 address pattern). The high byte is then written to the 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 71 * write range as usual, completing the write cycle. 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 72 */ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 73 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 74 #if defined(CONFIG_ATARI_ROM_ISA) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 75 #define rom_in_8(addr) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 76 ({ u16 __v = (*(__force volatile u16 *) (addr)); __v >>= 8; __v; }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 77 #define rom_in_be16(addr) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 78 ({ u16 __v = (*(__force volatile u16 *) (addr)); __v; }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 79 #define rom_in_le16(addr) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 80 ({ u16 __v = le16_to_cpu(*(__force volatile u16 *) (addr)); __v; }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 81 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 82 #define rom_out_8(addr, b) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 @83 ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 84 __w = ((*(__force volatile u8 *) ((_addr | 0x10000) + (__v<<1)))); }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 85 #define rom_out_be16(addr, w) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 86 ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 87 __w = ((*(__force volatile u16 *) ((_addr & 0xFFFF0000UL) + ((__v & 0xFF)<<1)))); \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 88 __w = ((*(__force volatile u16 *) ((_addr | 0x10000) + ((__v >> 8)<<1)))); }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 89 #define rom_out_le16(addr, w) \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 90 ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 91 __w = ((*(__force volatile u16 *) ((_addr & 0xFFFF0000UL) + ((__v >> 8)<<1)))); \ 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 92 __w = ((*(__force volatile u16 *) ((_addr | 0x10000) + ((__v & 0xFF)<<1)))); }) 84b16b7b0d5c818 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 93 :::::: The code at line 83 was first introduced by commit :::::: 84b16b7b0d5c818fadc731a69965dc76dce0c91e m68k/atari: ROM port ISA adapter support :::::: TO: Michael Schmitz <schmitz(a)opal.biophys.uni-duesseldorf.de&gt; :::::: CC: Geert Uytterhoeven <geert(a)linux-m68k.org&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org