linux-kernel - Re: [RESEND][V5 PATCH 2/2] ASoC: max98390: Added Amplifier Driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <202005181107.osye13l9%lkp@intel.com>
Date:   Mon, 18 May 2020 12:00:43 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Steve Lee <steves.lee@...imintegrated.com>, lgirdwood@...il.com,
        broonie@...nel.org, perex@...ex.cz, tiwai@...e.com,
        ckeepax@...nsource.cirrus.com, geert@...ux-m68k.org,
        rf@...nsource.wolfsonmicro.com, shumingf@...ltek.com,
        srinivas.kandagatla@...aro.org, krzk@...nel.org, dmurphy@...com,
        jack.yu@...ltek.com, nuno.sa@...log.com,
        linux-kernel@...r.kernel.org, alsa-devel@...a-project.org
Cc:     kbuild-all@...ts.01.org, ryan.lee.maxim@...il.com,
        ryans.lee@...imintegrated.com
Subject: Re: [RESEND][V5 PATCH 2/2] ASoC: max98390: Added Amplifier Driver

Hi Steve,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on asoc/for-next]
[also build test WARNING on v5.7-rc6 next-20200515]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Steve-Lee/dt-bindings-Added-device-tree-binding-for-max98390/20200518-085308
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: i386-randconfig-a004-20200518 (attached as .config)
compiler: gcc-4.9 (Ubuntu 4.9.3-13ubuntu2) 4.9.3
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@...el.com>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

In file included from include/linux/dynamic_debug.h:6:0,
from include/linux/printk.h:326,
from include/linux/kernel.h:15,
from include/linux/list.h:9,
from include/linux/kobject.h:19,
from include/linux/of.h:17,
from include/linux/irqdomain.h:35,
from include/linux/acpi.h:13,
from sound/soc/codecs/max98390.c:9:
sound/soc/codecs/max98390.c: In function 'max98390_dsm_init':
>> include/linux/jump_label.h:471:59: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'size_t' [-Wformat=]
else if (__builtin_types_compatible_p(typeof(*x), struct static_key_false))                                                               ^
include/linux/dynamic_debug.h:105:2: note: in expansion of macro 'static_branch_unlikely'
static_branch_unlikely(&descriptor.key.dd_key_false)
^
include/linux/dynamic_debug.h:124:6: note: in expansion of macro 'DYNAMIC_DEBUG_BRANCH'
if (DYNAMIC_DEBUG_BRANCH(id))            ^
include/linux/dynamic_debug.h:143:2: note: in expansion of macro '__dynamic_func_call'
__dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__)
^
include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call'
_dynamic_func_call(fmt,__dynamic_dev_dbg,        ^
include/linux/dev_printk.h:114:2: note: in expansion of macro 'dynamic_dev_dbg'
dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
^
sound/soc/codecs/max98390.c:780:2: note: in expansion of macro 'dev_dbg'
dev_dbg(component->dev,
^

vim +471 include/linux/jump_label.h

11276d5306b8e5 Peter Zijlstra 2015-07-24  465  
11276d5306b8e5 Peter Zijlstra 2015-07-24  466  #define static_branch_unlikely(x)						\
11276d5306b8e5 Peter Zijlstra 2015-07-24  467  ({										\
11276d5306b8e5 Peter Zijlstra 2015-07-24  468  	bool branch;								\
11276d5306b8e5 Peter Zijlstra 2015-07-24  469  	if (__builtin_types_compatible_p(typeof(*x), struct static_key_true))	\
11276d5306b8e5 Peter Zijlstra 2015-07-24  470  		branch = arch_static_branch_jump(&(x)->key, false);		\
11276d5306b8e5 Peter Zijlstra 2015-07-24 @471  	else if (__builtin_types_compatible_p(typeof(*x), struct static_key_false)) \
11276d5306b8e5 Peter Zijlstra 2015-07-24  472  		branch = arch_static_branch(&(x)->key, false);			\
11276d5306b8e5 Peter Zijlstra 2015-07-24  473  	else									\
11276d5306b8e5 Peter Zijlstra 2015-07-24  474  		branch = ____wrong_branch_error();				\
81dcf89f03204d Peter Zijlstra 2018-01-18  475  	unlikely(branch);							\
11276d5306b8e5 Peter Zijlstra 2015-07-24  476  })
11276d5306b8e5 Peter Zijlstra 2015-07-24  477  

:::::: The code at line 471 was first introduced by commit
:::::: 11276d5306b8e5b438a36bbff855fe792d7eaa61 locking/static_keys: Add a new static_key interface

:::::: TO: Peter Zijlstra <peterz@...radead.org>
:::::: CC: Ingo Molnar <mingo@...nel.org>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (44145 bytes)