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

[kbuild test robot]

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

.config.gz
Description: Binary data