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

Am 28.03.22 um 14:08 schrieb Hans de Goede:
> 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();

Thanks I missed that. But moving the debug function in i8042_check_quirks would make the output x86 exclusive, while the
quirks in general are not (on non x86 they must be set via command line but still have effects on the code if i see that
correctly)

>
> 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

As an alternative i would suggest putting defines in the pr_debug call. Otherwise it might not be immediately that the
#define i8042_nopnp 0 is there specifically for this one statement only.

    pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
        i8042_nokbd ? " nokbd" : "",
        i8042_noaux ? " noaux" : "",
        i8042_nomux ? " nomux" : "",
        i8042_unlock ? " unlock" : "",
        i8042_probe_defer ? "probe_defer" : "",
        i8042_reset == I8042_RESET_DEFAULT ?
            "" : i8042_reset == I8042_RESET_ALWAYS ?
                " reset_always" : " reset_never",
        i8042_direct ? " direct" : "",
        i8042_dumbkbd ? " dumbkbd" : "",
        i8042_noloop ? " noloop" : "",
        i8042_notimeout ? " notimeout" : "",
        i8042_kbdreset ? " kbdreset" : "",
#ifdef CONFIG_X86
        i8042_dritek ? " dritek" : "",
#else
        "",
#endif
#ifdef CONFIG_PNP
        i8042_nopnp ? " nopnp" : "");
#else
        "");
#endif

As a side effect: like this the code snipped also stays completely position independed.

>
> 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 18:29    [W:0.080 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site