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

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

[-- Attachment #1: Type: text/plain, Size: 2874 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: arm-randconfig-r033-20210507 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project a3a8a1a15b524d91b5308db68e9d293b34cd88dd)
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
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # 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=clang make.cross W=1 ARCH=arm 

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:2289:10: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
                       i, sg_dma_address(sg), sg_dma_len(sg), sg->offset,
                          ^~~~~~~~~~~~~~~~~~
   include/linux/scatterlist.h:28:28: note: expanded from macro 'sg_dma_address'
   #define sg_dma_address(sg)      ((sg)->dma_address)
                                   ^~~~~~~~~~~~~~~~~~~
   1 warning generated.


vim +2289 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: 32904 bytes --]