Re: [PATCH v11 3/3] fpga: Add support for Lattice iCE40 FPGAs

From: kbuild test robot <lkp@intel.com>
To: Joel Holdsworth <joel@airwebreathe.org.uk>
Cc: kbuild-all@01.org, atull@opensource.altera.com,
	moritz.fischer@ettus.com, robh@kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-spi@vger.kernel.org, marex@denx.de,
	linux-fpga@vger.kernel.org, vladimir_zapolskiy@mentor.com,
	geert@linux-m68k.org, Joel Holdsworth <joel@airwebreathe.org.uk>
Subject: Re: [PATCH v11 3/3] fpga: Add support for Lattice iCE40 FPGAs
Date: Mon, 9 Jan 2017 14:34:34 +0800	[thread overview]
Message-ID: <201701091403.6NaSB52X%fengguang.wu@intel.com> (raw)
In-Reply-To: <1483939426-4941-3-git-send-email-joel@airwebreathe.org.uk>

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

Hi Joel,

[auto build test WARNING on next-20170106]
[also build test WARNING on v4.10-rc3]
[cannot apply to linus/master linux/master robh/for-next v4.9-rc8 v4.9-rc7 v4.9-rc6]
[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/Joel-Holdsworth/of-Add-vendor-prefix-for-Lattice-Semiconductor/20170109-134955
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All warnings (new ones prefixed by >>):

   drivers/fpga/ice40-spi.c: In function 'ice40_fpga_probe':
>> drivers/fpga/ice40-spi.c:166:45: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'int' [-Wformat=]
      dev_err(dev, "Failed to get CDONE GPIO: %ld\n", ret);
                                                ^
   drivers/fpga/ice40-spi.c:173:48: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'int' [-Wformat=]
      dev_err(dev, "Failed to get CRESET_B GPIO: %ld\n", ret);
                                                   ^

vim +166 drivers/fpga/ice40-spi.c

   150	
   151		if (spi->max_speed_hz < ICE40_SPI_MIN_SPEED) {
   152			dev_err(dev, "SPI speed is too low, minimum speed is "
   153				__stringify(ICE40_SPI_MIN_SPEED) "\n");
   154			return -EINVAL;
   155		}
   156	
   157		if (spi->mode & SPI_CPHA) {
   158			dev_err(dev, "Bad SPI mode, CPHA not supported\n");
   159			return -EINVAL;
   160		}
   161	
   162		/* Set up the GPIOs */
   163		priv->cdone = devm_gpiod_get(dev, "cdone", GPIOD_IN);
   164		if (IS_ERR(priv->cdone)) {
   165			ret = PTR_ERR(priv->cdone);
 > 166			dev_err(dev, "Failed to get CDONE GPIO: %ld\n", ret);
   167			return ret;
   168		}
   169	
   170		priv->reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);
   171		if (IS_ERR(priv->reset)) {
   172			ret = PTR_ERR(priv->reset);
   173			dev_err(dev, "Failed to get CRESET_B GPIO: %ld\n", ret);
   174			return ret;

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

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