From: kbuild test robot <lkp@intel.com>
To: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Cc: kbuild-all@01.org, broonie@kernel.org,
linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org,
patches@opensource.wolfsonmicro.com
Subject: Re: [PATCH] ASoC: wm_adsp: Don't overrun firmware file buffer when reading region data
Date: Tue, 20 Dec 2016 03:20:03 +0800 [thread overview]
Message-ID: <201612200355.immIPrXm%fengguang.wu@intel.com> (raw)
In-Reply-To: <1482160688-31365-1-git-send-email-rf@opensource.wolfsonmicro.com>
[-- Attachment #1: Type: text/plain, Size: 4118 bytes --]
Hi Richard,
[auto build test WARNING on asoc/for-next]
[also build test WARNING on v4.9 next-20161219]
[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/Richard-Fitzgerald/ASoC-wm_adsp-Don-t-overrun-firmware-file-buffer-when-reading-region-data/20161220-021733
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
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=sparc64
All warnings (new ones prefixed by >>):
sound/soc/codecs/wm_adsp.c: In function 'wm_adsp_load':
>> sound/soc/codecs/wm_adsp.c:40:21: warning: format '%d' expects argument of type 'int', but argument 8 has type 'size_t {aka const long unsigned int}' [-Wformat=]
dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
^
>> sound/soc/codecs/wm_adsp.c:1705:4: note: in expansion of macro 'adsp_err'
adsp_err(dsp,
^~~~~~~~
sound/soc/codecs/wm_adsp.c: In function 'wm_adsp_load_coeff':
>> sound/soc/codecs/wm_adsp.c:40:21: warning: format '%d' expects argument of type 'int', but argument 8 has type 'size_t {aka const long unsigned int}' [-Wformat=]
dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
^
sound/soc/codecs/wm_adsp.c:2250:5: note: in expansion of macro 'adsp_err'
adsp_err(dsp,
^~~~~~~~
vim +40 sound/soc/codecs/wm_adsp.c
cdcd7f72 Charles Keepax 2014-11-14 24 #include <linux/vmalloc.h>
6ab2b7b4 Dimitris Papastamos 2013-05-08 25 #include <linux/workqueue.h>
f9f55e31 Richard Fitzgerald 2015-06-11 26 #include <linux/debugfs.h>
2159ad93 Mark Brown 2012-10-11 27 #include <sound/core.h>
2159ad93 Mark Brown 2012-10-11 28 #include <sound/pcm.h>
2159ad93 Mark Brown 2012-10-11 29 #include <sound/pcm_params.h>
2159ad93 Mark Brown 2012-10-11 30 #include <sound/soc.h>
2159ad93 Mark Brown 2012-10-11 31 #include <sound/jack.h>
2159ad93 Mark Brown 2012-10-11 32 #include <sound/initval.h>
2159ad93 Mark Brown 2012-10-11 33 #include <sound/tlv.h>
2159ad93 Mark Brown 2012-10-11 34
2159ad93 Mark Brown 2012-10-11 35 #include "wm_adsp.h"
2159ad93 Mark Brown 2012-10-11 36
2159ad93 Mark Brown 2012-10-11 37 #define adsp_crit(_dsp, fmt, ...) \
2159ad93 Mark Brown 2012-10-11 38 dev_crit(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown 2012-10-11 39 #define adsp_err(_dsp, fmt, ...) \
2159ad93 Mark Brown 2012-10-11 @40 dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown 2012-10-11 41 #define adsp_warn(_dsp, fmt, ...) \
2159ad93 Mark Brown 2012-10-11 42 dev_warn(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown 2012-10-11 43 #define adsp_info(_dsp, fmt, ...) \
2159ad93 Mark Brown 2012-10-11 44 dev_info(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown 2012-10-11 45 #define adsp_dbg(_dsp, fmt, ...) \
2159ad93 Mark Brown 2012-10-11 46 dev_dbg(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown 2012-10-11 47
2159ad93 Mark Brown 2012-10-11 48 #define ADSP1_CONTROL_1 0x00
:::::: The code at line 40 was first introduced by commit
:::::: 2159ad936b7e7a8b26c99cf5b4476cfbb8c13e22 ASoC: adsp: Add ADSP base support
:::::: TO: Mark Brown <broonie@opensource.wolfsonmicro.com>
:::::: CC: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
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: 47887 bytes --]
prev parent reply other threads:[~2016-12-19 19:21 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-19 15:18 [PATCH] ASoC: wm_adsp: Don't overrun firmware file buffer when reading region data Richard Fitzgerald
2016-12-19 19:16 ` kbuild test robot
2016-12-19 19:20 ` kbuild test robot [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201612200355.immIPrXm%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=kbuild-all@01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=patches@opensource.wolfsonmicro.com \
--cc=rf@opensource.wolfsonmicro.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).