lkml.org 
[lkml]   [2016]   [Dec]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ASoC: wm_adsp: Don't overrun firmware file buffer when reading region data
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
[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2016-12-19 20:21    [W:1.719 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site