From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752410AbdATXj4 (ORCPT ); Fri, 20 Jan 2017 18:39:56 -0500 Received: from mail-pf0-f194.google.com ([209.85.192.194]:36106 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751125AbdATXjy (ORCPT ); Fri, 20 Jan 2017 18:39:54 -0500 Subject: Re: [PATCH v7 2/3] fpga manager: Add cyclone-ps-spi driver for Altera FPGAs To: kbuild test robot References: <201701202037.FWBSni3B%fengguang.wu@intel.com> Cc: kbuild-all@01.org, Alan Tull , Moritz Fischer , Russell King , Shawn Guo , Sascha Hauer , Fabio Estevam , Mark Rutland , Rob Herring , Anatolij Gustschin , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-fpga@vger.kernel.org From: Joshua Clayton Message-ID: <8dff470a-e1b9-1aef-101c-287abcfea380@gmail.com> Date: Fri, 20 Jan 2017 15:39:50 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <201701202037.FWBSni3B%fengguang.wu@intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Sigh... On 01/20/2017 04:07 AM, kbuild test robot wrote: > 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); Bah! u8 * should be char * > 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 Guess I had better add sparse to my pre send regimen. v8 posted shortly. ~Joshua