From mboxrd@z Thu Jan 1 00:00:00 1970 From: kbuild test robot Subject: Re: [PATCH v7 2/3] fpga manager: Add cyclone-ps-spi driver for Altera FPGAs Date: Fri, 20 Jan 2017 20:07:23 +0800 Message-ID: <201701202037.FWBSni3B%fengguang.wu@intel.com> References: <1b85ccf11096b8ef6c3a9eee9e5897c27b89a825.1484869881.git.stillcompiling@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1b85ccf11096b8ef6c3a9eee9e5897c27b89a825.1484869881.git.stillcompiling@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org Cc: Mark Rutland , Moritz Fischer , Joshua Clayton , devicetree@vger.kernel.org, Alan Tull , linux-fpga@vger.kernel.org, Russell King , linux-kernel@vger.kernel.org, Rob Herring , kbuild-all@01.org, Sascha Hauer , Fabio Estevam , Anatolij Gustschin , Shawn Guo , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hi Joshua, [auto build test WARNING on linus/master] [also build test WARNING on v4.10-rc4 next-20170120] [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/Joshua-Clayton/Altera-Cyclone-Passive-Serial-SPI-FPGA-Manager/20170120-172349 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) include/linux/compiler.h:253:8: sparse: attribute 'no_sanitize_address': unknown attribute >> drivers/fpga/cyclone-ps-spi.c:86:20: sparse: incompatible types in comparison expression (different signedness) drivers/fpga/cyclone-ps-spi.c:101:33: sparse: incompatible types in comparison expression (different type sizes) drivers/fpga/cyclone-ps-spi.c: In function 'rev_buf': drivers/fpga/cyclone-ps-spi.c:86:13: warning: comparison of distinct pointer types lacks a cast while (buf < fw_end) { ^ In file included from include/linux/delay.h:10:0, from drivers/fpga/cyclone-ps-spi.c:17: drivers/fpga/cyclone-ps-spi.c: In function 'cyclonespi_write': include/linux/kernel.h:753:16: warning: comparison of distinct pointer types lacks a cast (void) (&min1 == &min2); \ ^ include/linux/kernel.h:756:2: note: in expansion of macro '__min' __min(typeof(x), typeof(y), \ ^~~~~ drivers/fpga/cyclone-ps-spi.c:101:19: note: in expansion of macro 'min' size_t stride = min(fw_data_end - fw_data, SZ_4K); ^~~ drivers/fpga/cyclone-ps-spi.c:103:11: warning: passing argument 1 of 'rev_buf' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers] rev_buf(fw_data, stride); ^~~~~~~ drivers/fpga/cyclone-ps-spi.c:81:13: note: expected 'char *' but argument is of type 'const char *' static void rev_buf(char *buf, size_t len) ^~~~~~~ vim +86 drivers/fpga/cyclone-ps-spi.c 70 gpiod_set_value(conf->config, 0); 71 for (i = 0; i < (FPGA_MAX_DELAY / FPGA_MIN_DELAY); i++) { 72 usleep_range(FPGA_MIN_DELAY, FPGA_MIN_DELAY + 20); 73 if (!gpiod_get_value(conf->status)) 74 return 0; 75 } 76 77 dev_err(&mgr->dev, "Status pin not ready.\n"); 78 return -EIO; 79 } 80 81 static void rev_buf(char *buf, size_t len) 82 { 83 const u8 *fw_end = (buf + len); 84 85 /* set buffer to lsb first */ > 86 while (buf < fw_end) { 87 *buf = bitrev8(*buf); 88 buf++; 89 } 90 } 91 92 static int cyclonespi_write(struct fpga_manager *mgr, const char *buf, 93 size_t count) 94 { --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation