Re: [PATCH] qlogic_stub: Fixup NULL argument to host_reset()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hannes,

I love your patch! Yet something to improve:

[auto build test ERROR on mkp-scsi/for-next]
[also build test ERROR on v4.17-rc3 next-20180504]
[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/Hannes-Reinecke/qlogic_stub-Fixup-NULL-argument-to-host_reset/20180505-172602
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
config: parisc-allmodconfig (attached as .config)
compiler: hppa-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=parisc 

All errors (new ones prefixed by >>):

   drivers/scsi/pcmcia/qlogic_stub.c: In function 'qlogic_resume':
>> drivers/scsi/pcmcia/qlogic_stub.c:270:6: error: invalid type argument of '->' (have 'struct scsi_cmnd')
     scmd->device = &sdev;
         ^~
>> drivers/scsi/pcmcia/qlogic_stub.c:272:26: error: incompatible type for argument 1 of 'qlogicfas408_host_reset'
     qlogicfas408_host_reset(scmd);
                             ^~~~
   In file included from drivers/scsi/pcmcia/qlogic_stub.c:49:0:
   drivers/scsi/pcmcia/../qlogicfas408.h:112:12: note: expected 'struct scsi_cmnd *' but argument is of type 'struct scsi_cmnd'
    extern int qlogicfas408_host_reset(struct scsi_cmnd *cmd);
               ^~~~~~~~~~~~~~~~~~~~~~~

vim +270 drivers/scsi/pcmcia/qlogic_stub.c

   253	
   254	static int qlogic_resume(struct pcmcia_device *link)
   255	{
   256		scsi_info_t *info = link->priv;
   257		/* Dummy command for host_reset function */
   258		struct scsi_cmnd scmd;
   259		struct scsi_device sdev;
   260	
   261		pcmcia_enable_device(link);
   262		if ((info->manf_id == MANFID_MACNICA) ||
   263		    (info->manf_id == MANFID_PIONEER) ||
   264		    (info->manf_id == 0x0098)) {
   265			outb(0x80, link->resource[0]->start + 0xd);
   266			outb(0x24, link->resource[0]->start + 0x9);
   267			outb(0x04, link->resource[0]->start + 0xd);
   268		}
   269		sdev.host = info->host;
 > 270		scmd->device = &sdev;
   271	
 > 272		qlogicfas408_host_reset(scmd);
   273	
   274		return 0;
   275	}
   276	

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

Attachment: .config.gz
Description: application/gzip


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux