linux-kernel - Re: [PATCH 2/3] mtd: spi-nor: core code for the Altera Quadspi Flash Controller v2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <201706271711.zwTTiKny%fengguang.wu@intel.com>
Date:   Tue, 27 Jun 2017 17:30:47 +0800
From:   kbuild test robot <lkp@...el.com>
To:     matthew.gerlach@...ux.intel.com
Cc:     kbuild-all@...org, vndao@...era.com, dwmw2@...radead.org,
        computersforpeace@...il.com, boris.brezillon@...e-electrons.com,
        marek.vasut@...il.com, richard@....at, cyrille.pitchen@...ev4u.fr,
        robh+dt@...nel.org, mark.rutland@....com,
        linux-mtd@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, gregkh@...uxfoundation.org,
        davem@...emloft.net, mchehab@...nel.org,
        Matthew Gerlach <matthew.gerlach@...ux.intel.com>
Subject: Re: [PATCH 2/3] mtd: spi-nor: core code for the Altera Quadspi Flash
 Controller v2

Hi Matthew,

[auto build test WARNING on spi-nor/next]
[also build test WARNING on v4.12-rc7 next-20170627]
[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/matthew-gerlach-linux-intel-com/Altera-Quadspi-Controller-Version-2/20170627-120604
base:   git://github.com/spi-nor/linux next
config: m32r-allmodconfig (attached as .config)
compiler: m32r-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/01org/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=m32r 

All warnings (new ones prefixed by >>):

   In file included from include/linux/printk.h:329:0,
                    from include/linux/kernel.h:13,
                    from include/linux/list.h:8,
                    from include/linux/module.h:9,
                    from drivers/mtd/spi-nor/altera-quadspi.c:20:
   drivers/mtd/spi-nor/altera-quadspi.c: In function 'altera_quadspi_erase':
>> drivers/mtd/spi-nor/altera-quadspi.c:289:18: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'long unsigned int' [-Wformat=]
     dev_dbg(q->dev, "%s SR=0x%x FSR=0x%x\n", __func__,
                     ^
   include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
>> drivers/mtd/spi-nor/altera-quadspi.c:289:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(q->dev, "%s SR=0x%x FSR=0x%x\n", __func__,
     ^~~~~~~
   drivers/mtd/spi-nor/altera-quadspi.c:289:18: warning: format '%x' expects argument of type 'unsigned int', but argument 6 has type 'long unsigned int' [-Wformat=]
     dev_dbg(q->dev, "%s SR=0x%x FSR=0x%x\n", __func__,
                     ^
   include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
>> drivers/mtd/spi-nor/altera-quadspi.c:289:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(q->dev, "%s SR=0x%x FSR=0x%x\n", __func__,
     ^~~~~~~
   drivers/mtd/spi-nor/altera-quadspi.c: In function 'altera_quadspi_create':
   drivers/mtd/spi-nor/altera-quadspi.c:625:15: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'long unsigned int' [-Wformat=]
     dev_dbg(dev, "%s SR=0x%x FSR=0x%x\n", __func__,
                  ^
   include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
   drivers/mtd/spi-nor/altera-quadspi.c:625:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(dev, "%s SR=0x%x FSR=0x%x\n", __func__,
     ^~~~~~~
   drivers/mtd/spi-nor/altera-quadspi.c:625:15: warning: format '%x' expects argument of type 'unsigned int', but argument 6 has type 'long unsigned int' [-Wformat=]
     dev_dbg(dev, "%s SR=0x%x FSR=0x%x\n", __func__,
                  ^
   include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
   drivers/mtd/spi-nor/altera-quadspi.c:625:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(dev, "%s SR=0x%x FSR=0x%x\n", __func__,
     ^~~~~~~

vim +289 drivers/mtd/spi-nor/altera-quadspi.c

   273	
   274		altera_quadspi_chip_select(q, flash->bank);
   275	
   276		sector_value = altera_quadspi_addr_to_sector(mtd, offset);
   277	
   278		dev_dbg(q->dev, "%s sector %d\n", __func__, sector_value);
   279	
   280		if (sector_value < 0)
   281			return -EINVAL;
   282	
   283		val = (sector_value << 8) & QUADSPI_MEM_OP_SECTOR_VALUE_MASK;
   284	
   285		val |= QUADSPI_MEM_OP_SECTOR_ERASE_CMD;
   286	
   287		alt_qspi_writel(val, q->csr_base, QUADSPI_MEM_OP_REG);
   288	
 > 289		dev_dbg(q->dev, "%s SR=0x%x FSR=0x%x\n", __func__,
   290			alt_qspi_readl(q->csr_base, QUADSPI_SR_REG),
   291			alt_qspi_readl(q->csr_base, QUADSPI_FLAG_STATUS_REG));
   292	
   293		return altera_quadspi_write_erase_check(nor, ERASE_CHECK);
   294	}
   295	
   296	#define WINDOW_ALIGN 4
   297	#define WINDOW_MASK (WINDOW_ALIGN - 1)

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

Download attachment ".config.gz" of type "application/gzip" (42462 bytes)