From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_MUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 409F2C48BD6 for ; Tue, 25 Jun 2019 22:53:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 12F1F20665 for ; Tue, 25 Jun 2019 22:53:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726382AbfFYWxQ (ORCPT ); Tue, 25 Jun 2019 18:53:16 -0400 Received: from mga03.intel.com ([134.134.136.65]:27018 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726357AbfFYWxP (ORCPT ); Tue, 25 Jun 2019 18:53:15 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 25 Jun 2019 15:53:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,417,1557212400"; d="scan'208";a="188447174" Received: from lkp-server01.sh.intel.com (HELO lkp-server01) ([10.239.97.150]) by fmsmga002.fm.intel.com with ESMTP; 25 Jun 2019 15:53:08 -0700 Received: from kbuild by lkp-server01 with local (Exim 4.89) (envelope-from ) id 1hfuJH-000Cwq-Tu; Wed, 26 Jun 2019 06:53:07 +0800 Date: Wed, 26 Jun 2019 06:52:59 +0800 From: kbuild test robot To: Alastair D'Silva Cc: kbuild-all@01.org, alastair@d-silva.org, Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , David Airlie , Daniel Vetter , Dan Carpenter , Karsten Keil , Jassi Brar , Tom Lendacky , "David S. Miller" , Jose Abreu , Kalle Valo , Stanislaw Gruszka , Benson Leung , Enric Balletbo i Serra , "James E.J. Bottomley" , "Martin K. Petersen" , Greg Kroah-Hartman , Alexander Viro , Petr Mladek , Sergey Senozhatsky , Steven Rostedt , David Laight , Andrew Morton , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, ath10k@lists.infradead.org, linux-wireless@vger.kernel.org, linux-scsi@vger.kernel.org, linux-fbdev@vger.kernel.org, devel@driverdev.osuosl.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v4 4/7] lib/hexdump.c: Replace ascii bool in hex_dump_to_buffer with flags Message-ID: <201906260657.2cnctJGF%lkp@intel.com> References: <20190625031726.12173-5-alastair@au1.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190625031726.12173-5-alastair@au1.ibm.com> X-Patchwork-Hint: ignore User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org Hi Alastair, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v5.2-rc6 next-20190625] [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/Alastair-D-Silva/Hexdump-Enhancements/20190625-224046 reproduce: # apt-get install sparse # sparse version: v0.6.1-rc1-7-g2b96cd8-dirty make ARCH=x86_64 allmodconfig make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' If you fix the issue, kindly add following tag Reported-by: kbuild test robot sparse warnings: (new ones prefixed by >>) sound/soc/intel/skylake/skl-debug.c:191:34: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void [noderef] *to @@ got eref] *to @@ sound/soc/intel/skylake/skl-debug.c:191:34: sparse: expected void [noderef] *to sound/soc/intel/skylake/skl-debug.c:191:34: sparse: got unsigned char * sound/soc/intel/skylake/skl-debug.c:191:51: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void const *from @@ got void [noderef] void const *from @@ sound/soc/intel/skylake/skl-debug.c:191:51: sparse: expected void const *from sound/soc/intel/skylake/skl-debug.c:191:51: sparse: got void [noderef] *[assigned] fw_reg_addr >> sound/soc/intel/skylake/skl-debug.c:195:35: sparse: sparse: too many arguments for function hex_dump_to_buffer -- >> drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c:93:27: sparse: sparse: too many arguments for function hex_dump_to_buffer -- >> sound/soc/sof/xtensa/core.c:125:35: sparse: sparse: too many arguments for function hex_dump_to_buffer vim +195 sound/soc/intel/skylake/skl-debug.c d14700a0 Vinod Koul 2017-06-30 170 bdd0384a Vunny Sodhi 2017-06-30 171 static ssize_t fw_softreg_read(struct file *file, char __user *user_buf, bdd0384a Vunny Sodhi 2017-06-30 172 size_t count, loff_t *ppos) bdd0384a Vunny Sodhi 2017-06-30 173 { bdd0384a Vunny Sodhi 2017-06-30 174 struct skl_debug *d = file->private_data; bdd0384a Vunny Sodhi 2017-06-30 175 struct sst_dsp *sst = d->skl->skl_sst->dsp; bdd0384a Vunny Sodhi 2017-06-30 176 size_t w0_stat_sz = sst->addr.w0_stat_sz; bdd0384a Vunny Sodhi 2017-06-30 177 void __iomem *in_base = sst->mailbox.in_base; bdd0384a Vunny Sodhi 2017-06-30 178 void __iomem *fw_reg_addr; bdd0384a Vunny Sodhi 2017-06-30 179 unsigned int offset; bdd0384a Vunny Sodhi 2017-06-30 180 char *tmp; bdd0384a Vunny Sodhi 2017-06-30 181 ssize_t ret = 0; bdd0384a Vunny Sodhi 2017-06-30 182 bdd0384a Vunny Sodhi 2017-06-30 183 tmp = kzalloc(FW_REG_BUF, GFP_KERNEL); bdd0384a Vunny Sodhi 2017-06-30 184 if (!tmp) bdd0384a Vunny Sodhi 2017-06-30 185 return -ENOMEM; bdd0384a Vunny Sodhi 2017-06-30 186 bdd0384a Vunny Sodhi 2017-06-30 187 fw_reg_addr = in_base - w0_stat_sz; bdd0384a Vunny Sodhi 2017-06-30 188 memset(d->fw_read_buff, 0, FW_REG_BUF); bdd0384a Vunny Sodhi 2017-06-30 189 bdd0384a Vunny Sodhi 2017-06-30 190 if (w0_stat_sz > 0) bdd0384a Vunny Sodhi 2017-06-30 @191 __iowrite32_copy(d->fw_read_buff, fw_reg_addr, w0_stat_sz >> 2); bdd0384a Vunny Sodhi 2017-06-30 192 bdd0384a Vunny Sodhi 2017-06-30 193 for (offset = 0; offset < FW_REG_SIZE; offset += 16) { bdd0384a Vunny Sodhi 2017-06-30 194 ret += snprintf(tmp + ret, FW_REG_BUF - ret, "%#.4x: ", offset); bdd0384a Vunny Sodhi 2017-06-30 @195 hex_dump_to_buffer(d->fw_read_buff + offset, 16, 16, 4, bdd0384a Vunny Sodhi 2017-06-30 196 tmp + ret, FW_REG_BUF - ret, 0); bdd0384a Vunny Sodhi 2017-06-30 197 ret += strlen(tmp + ret); bdd0384a Vunny Sodhi 2017-06-30 198 bdd0384a Vunny Sodhi 2017-06-30 199 /* print newline for each offset */ bdd0384a Vunny Sodhi 2017-06-30 200 if (FW_REG_BUF - ret > 0) bdd0384a Vunny Sodhi 2017-06-30 201 tmp[ret++] = '\n'; bdd0384a Vunny Sodhi 2017-06-30 202 } bdd0384a Vunny Sodhi 2017-06-30 203 bdd0384a Vunny Sodhi 2017-06-30 204 ret = simple_read_from_buffer(user_buf, count, ppos, tmp, ret); bdd0384a Vunny Sodhi 2017-06-30 205 kfree(tmp); bdd0384a Vunny Sodhi 2017-06-30 206 bdd0384a Vunny Sodhi 2017-06-30 207 return ret; bdd0384a Vunny Sodhi 2017-06-30 208 } bdd0384a Vunny Sodhi 2017-06-30 209 :::::: The code at line 195 was first introduced by commit :::::: bdd0384a5ada8bb5745e5f29c10a5ba88827efad ASoC: Intel: Skylake: Add support to read firmware registers :::::: TO: Vunny Sodhi :::::: CC: Mark Brown --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation