Re: [PATCH 07/11] qla2xxx: Avoid side effects when using endianizer macros.

[kbuild test robot <lkp@xxxxxxxxx>]

Hi Joe,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.5-rc1 next-20160125]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Himanshu-Madhani/qla2xxx-Remove-unneeded-link-offline-message/20160127-015650
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: x86_64-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/scsi/qla2xxx/qla_dbg.c: In function 'qla25xx_fw_dump':
>> drivers/scsi/qla2xxx/qla_dbg.c:1426:56: warning: right-hand operand of comma expression has no effect [-Wunused-value]
     for (cnt = 0; cnt < sizeof(fw->mailbox_reg) / 2; cnt++, mbx_reg)
                                                           ^

vim +1426 drivers/scsi/qla2xxx/qla_dbg.c

  1410	
  1411		WRT_REG_DWORD(&reg->iobase_select, 0xB0800000);
  1412		fw->shadow_reg[8] = htonl(RD_REG_DWORD(&reg->iobase_sdata));
  1413	
  1414		WRT_REG_DWORD(&reg->iobase_select, 0xB0900000);
  1415		fw->shadow_reg[9] = htonl(RD_REG_DWORD(&reg->iobase_sdata));
  1416	
  1417		WRT_REG_DWORD(&reg->iobase_select, 0xB0A00000);
  1418		fw->shadow_reg[10] = htonl(RD_REG_DWORD(&reg->iobase_sdata));
  1419	
  1420		/* RISC I/O register. */
  1421		WRT_REG_DWORD(&reg->iobase_addr, 0x0010);
  1422		fw->risc_io_reg = htonl(RD_REG_DWORD(&reg->iobase_window));
  1423	
  1424		/* Mailbox registers. */
  1425		mbx_reg = &reg->mailbox0;
> 1426		for (cnt = 0; cnt < sizeof(fw->mailbox_reg) / 2; cnt++, mbx_reg)
  1427			fw->mailbox_reg[cnt] = htons(RD_REG_WORD(mbx_reg));
  1428	
  1429		/* Transfer sequence registers. */
  1430		iter_reg = fw->xseq_gp_reg;
  1431		iter_reg = qla24xx_read_window(reg, 0xBF00, 16, iter_reg);
  1432		iter_reg = qla24xx_read_window(reg, 0xBF10, 16, iter_reg);
  1433		iter_reg = qla24xx_read_window(reg, 0xBF20, 16, iter_reg);
  1434		iter_reg = qla24xx_read_window(reg, 0xBF30, 16, iter_reg);

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

Attachment: .config.gz
Description: Binary data