Re: [PATCH 1/1] qla2xxx: Add EDIF support

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2674 bytes --]

Hi Nilesh,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on scsi/for-next]
[also build test WARNING on mkp-scsi/for-next next-20210507]
[cannot apply to v5.12]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nilesh-Javali/qla2xxx-Add-EDIF-support/20210507-212636
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-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
        # https://github.com/0day-ci/linux/commit/123765e7c3f96d59ba2239401abb1a3ed3adf910
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nilesh-Javali/qla2xxx-Add-EDIF-support/20210507-212636
        git checkout 123765e7c3f96d59ba2239401abb1a3ed3adf910
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross W=1 ARCH=xtensa 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/scsi/qla2xxx/qla_edif.c: In function 'ql_print_bsg_sglist':
>> drivers/scsi/qla2xxx/qla_edif.c:2288:24: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
    2288 |       "%x: dma(adr=%#llx len=%#x) off=%#x len=%#x\n",
         |                    ~~~~^
         |                        |
         |                        long long unsigned int
         |                    %#x


vim +2288 drivers/scsi/qla2xxx/qla_edif.c

  2275	
  2276	void
  2277	ql_print_bsg_sglist(uint level, scsi_qla_host_t *vha, uint id, char *str,
  2278			struct bsg_buffer *p)
  2279	{
  2280		struct scatterlist *sg;
  2281		uint i;
  2282	
  2283		ql_dbg(level, vha, id,
  2284		    "%s->(sg_cnt=%#x payload_len=%#x):\n",
  2285		    str, p->sg_cnt, p->payload_len);
  2286		for_each_sg(p->sg_list, sg, p->sg_cnt, i) {
  2287			ql_dbg(level, vha, id,
> 2288			    "%x: dma(adr=%#llx len=%#x) off=%#x len=%#x\n",
  2289			    i, sg_dma_address(sg), sg_dma_len(sg), sg->offset,
  2290			    sg->length);
  2291		}
  2292	}
  2293	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 67327 bytes --]