lkml.org 
[lkml]   [2022]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v4 3/4] input/i8042: Add debug output for quirks
From
Hi,

On 3/22/22 21:50, kernel test robot wrote:
> Hi Werner,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on dtor-input/next]
> [also build test ERROR on v5.17 next-20220322]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url: https://github.com/0day-ci/linux/commits/Werner-Sembach/input-i8042-Move-__initconst-to-fix-code-styling-warning/20220323-012205
> base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
> config: ia64-defconfig (https://download.01.org/0day-ci/archive/20220323/202203230435.z5FH2fWa-lkp@intel.com/config)
> compiler: ia64-linux-gcc (GCC) 11.2.0
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://github.com/0day-ci/linux/commit/d92be72641d5753be3aa94fe5961e2dbe2b5bdc9
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review Werner-Sembach/input-i8042-Move-__initconst-to-fix-code-styling-warning/20220323-012205
> git checkout d92be72641d5753be3aa94fe5961e2dbe2b5bdc9
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=ia64 SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>

i8042_dritek is only defined when building on x86, I would solve this
by moving the new pr_debug to inside i8042_check_quirks();

The i8042_nopnp not being defined order is slightly tricker, you
can more or less cleanly fix this by doing this at the end of
i8042_check_quirks() and then put the new pr_debug after this:

#ifdef CONFIG_PNP
if (quirks & SERIO_QUIRK_NOPNP)
i8042_nopnp = true;
#else
#define i8042_nopnp 0
#endif

Regards,

Hans



>
> All errors (new ones prefixed by >>):
>
> In file included from include/asm-generic/bug.h:22,
> from arch/ia64/include/asm/bug.h:17,
> from include/linux/bug.h:5,
> from include/linux/thread_info.h:13,
> from include/asm-generic/preempt.h:5,
> from ./arch/ia64/include/generated/asm/preempt.h:1,
> from include/linux/preempt.h:78,
> from include/linux/rcupdate.h:27,
> from include/linux/rculist.h:11,
> from include/linux/pid.h:5,
> from include/linux/sched.h:14,
> from include/linux/delay.h:23,
> from drivers/input/serio/i8042.c:12:
> drivers/input/serio/i8042-x86ia64io.h: In function 'i8042_platform_init':
>>> drivers/input/serio/i8042-x86ia64io.h:1511:17: error: 'i8042_dritek' undeclared (first use in this function); did you mean 'i8042_direct'?
> 1511 | i8042_dritek ? " dritek" : "",
> | ^~~~~~~~~~~~
> include/linux/printk.h:418:33: note: in definition of macro 'printk_index_wrap'
> 418 | _p_func(_fmt, ##__VA_ARGS__); \
> | ^~~~~~~~~~~
> include/linux/printk.h:132:17: note: in expansion of macro 'printk'
> 132 | printk(fmt, ##__VA_ARGS__); \
> | ^~~~~~
> include/linux/printk.h:576:9: note: in expansion of macro 'no_printk'
> 576 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
> | ^~~~~~~~~
> drivers/input/serio/i8042-x86ia64io.h:1497:9: note: in expansion of macro 'pr_debug'
> 1497 | pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
> | ^~~~~~~~
> drivers/input/serio/i8042-x86ia64io.h:1511:17: note: each undeclared identifier is reported only once for each function it appears in
> 1511 | i8042_dritek ? " dritek" : "",
> | ^~~~~~~~~~~~
> include/linux/printk.h:418:33: note: in definition of macro 'printk_index_wrap'
> 418 | _p_func(_fmt, ##__VA_ARGS__); \
> | ^~~~~~~~~~~
> include/linux/printk.h:132:17: note: in expansion of macro 'printk'
> 132 | printk(fmt, ##__VA_ARGS__); \
> | ^~~~~~
> include/linux/printk.h:576:9: note: in expansion of macro 'no_printk'
> 576 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
> | ^~~~~~~~~
> drivers/input/serio/i8042-x86ia64io.h:1497:9: note: in expansion of macro 'pr_debug'
> 1497 | pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
> | ^~~~~~~~
>
>
> vim +1511 drivers/input/serio/i8042-x86ia64io.h
>
> 1494
> 1495 i8042_check_quirks();
> 1496
> 1497 pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
> 1498 i8042_nokbd ? " nokbd" : "",
> 1499 i8042_noaux ? " noaux" : "",
> 1500 i8042_nomux ? " nomux" : "",
> 1501 i8042_unlock ? " unlock" : "",
> 1502 i8042_probe_defer ? "probe_defer" : "",
> 1503 i8042_reset == I8042_RESET_DEFAULT ?
> 1504 "" : i8042_reset == I8042_RESET_ALWAYS ?
> 1505 " reset_always" : " reset_never",
> 1506 i8042_direct ? " direct" : "",
> 1507 i8042_dumbkbd ? " dumbkbd" : "",
> 1508 i8042_noloop ? " noloop" : "",
> 1509 i8042_notimeout ? " notimeout" : "",
> 1510 i8042_kbdreset ? " kbdreset" : "",
>> 1511 i8042_dritek ? " dritek" : "",
> 1512 i8042_nopnp ? " nopnp" : "");
> 1513
> 1514 retval = i8042_pnp_init();
> 1515 if (retval)
> 1516 return retval;
> 1517
>

\
 
 \ /
  Last update: 2022-03-28 14:09    [W:0.085 / U:0.284 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site