| 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
| ||
|
Message-ID: <202202261644.hcY0BU0k-lkp@intel.com>
Date: Sat, 26 Feb 2022 16:48:19 +0800
From: kernel test robot <lkp@...el.com>
To: Yusuf Khan <yusisamerican@...il.com>, linux-kernel@...r.kernel.org
Cc: kbuild-all@...ts.01.org, jasowang@...hat.com,
mikelley@...rosoft.com, mst@...hat.com, gregkh@...uxfoundation.org,
javier@...igon.com, arnd@...db.de, will@...nel.org,
axboe@...nel.dk, bjorn.andersson@...aro.org,
Yusuf Khan <yusisamerican@...il.com>
Subject: Re: [PATCH v2] drivers: ddcci: upstream DDCCI driver
Hi Yusuf,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linux/master]
[also build test ERROR on staging/staging-testing linus/master v5.17-rc5]
[cannot apply to next-20220225]
[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/Yusuf-Khan/drivers-ddcci-upstream-DDCCI-driver/20220226-120531
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 2c271fe77d52a0555161926c232cd5bc07178b39
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20220226/202202261644.hcY0BU0k-lkp@intel.com/config)
compiler: sh4-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/7eb8029b70584acc0e9ce82dfdcff13c5496f544
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Yusuf-Khan/drivers-ddcci-upstream-DDCCI-driver/20220226-120531
git checkout 7eb8029b70584acc0e9ce82dfdcff13c5496f544
# 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=sh SHELL=/bin/bash drivers/ddcci/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
All error/warnings (new ones prefixed by >>):
drivers/ddcci/ddcci.c: In function 'ddcci_modalias_clean':
>> drivers/ddcci/ddcci.c:76:17: error: expected '}' before 'else'
76 | else if (c < '0' || (c > '9' && c < 'A') || (c > 'Z' && c < 'a') || c > 'z')
| ^~~~
--
drivers/ddcci/ddcci-backlight.c: In function 'ddcci_monitor_next_vcp_item':
>> drivers/ddcci/ddcci-backlight.c:168:17: error: expected '}' before 'else'
168 | else if (depth > 0) {
| ^~~~
>> drivers/ddcci/ddcci-backlight.c:186:13: error: invalid storage class for function 'ddcci_monitor_find_vcp'
186 | static bool ddcci_monitor_find_vcp(unsigned char vcp, const char *s)
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:186:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
186 | static bool ddcci_monitor_find_vcp(unsigned char vcp, const char *s)
| ^~~~~~
>> drivers/ddcci/ddcci-backlight.c:212:12: error: invalid storage class for function 'ddcci_backlight_create_symlink'
212 | static int ddcci_backlight_create_symlink(struct ddcci_device *ddcci_dev)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:240:12: error: invalid storage class for function 'ddcci_backlight_remove_symlink'
240 | static int ddcci_backlight_remove_symlink(struct ddcci_device *ddcci_dev)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:274:12: error: invalid storage class for function 'ddcci_monitor_probe'
274 | static int ddcci_monitor_probe(struct ddcci_device *dev,
| ^~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:377:12: error: invalid storage class for function 'ddcci_monitor_remove'
377 | static int ddcci_monitor_remove(struct ddcci_device *dev)
| ^~~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:396:27: error: initializer element is not constant
396 | .probe = ddcci_monitor_probe,
| ^~~~~~~~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:396:27: note: (near initialization for 'ddcci_backlight_driver.probe')
drivers/ddcci/ddcci-backlight.c:397:27: error: initializer element is not constant
397 | .remove = ddcci_monitor_remove,
| ^~~~~~~~~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:397:27: note: (near initialization for 'ddcci_backlight_driver.remove')
In file included from include/linux/device.h:32,
from include/linux/backlight.h:12,
from drivers/ddcci/ddcci-backlight.c:16:
>> drivers/ddcci/ddcci-backlight.c:400:21: error: invalid storage class for function 'ddcci_backlight_driver_init'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/device/driver.h:260:19: note: in definition of macro 'module_driver'
260 | static int __init __driver##_init(void) \
| ^~~~~~~~
drivers/ddcci/ddcci-backlight.c:400:1: note: in expansion of macro 'module_ddcci_driver'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~
In file included from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/backlight.h:12,
from drivers/ddcci/ddcci-backlight.c:16:
>> include/linux/module.h:131:49: error: invalid storage class for function '__inittest'
131 | static inline initcall_t __maybe_unused __inittest(void) \
| ^~~~~~~~~~
include/linux/device/driver.h:264:1: note: in expansion of macro 'module_init'
264 | module_init(__driver##_init); \
| ^~~~~~~~~~~
include/linux/ddcci.h:138:9: note: in expansion of macro 'module_driver'
138 | module_driver(__ddcci_driver, ddcci_add_driver, \
| ^~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:400:1: note: in expansion of macro 'module_ddcci_driver'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:400:1: warning: 'alias' attribute ignored [-Wattributes]
In file included from include/linux/device.h:32,
from include/linux/backlight.h:12,
from drivers/ddcci/ddcci-backlight.c:16:
>> drivers/ddcci/ddcci-backlight.c:400:21: error: invalid storage class for function 'ddcci_backlight_driver_exit'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/device/driver.h:265:20: note: in definition of macro 'module_driver'
265 | static void __exit __driver##_exit(void) \
| ^~~~~~~~
drivers/ddcci/ddcci-backlight.c:400:1: note: in expansion of macro 'module_ddcci_driver'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~
>> include/linux/device/driver.h:265:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
265 | static void __exit __driver##_exit(void) \
| ^~~~~~
include/linux/ddcci.h:138:9: note: in expansion of macro 'module_driver'
138 | module_driver(__ddcci_driver, ddcci_add_driver, \
| ^~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:400:1: note: in expansion of macro 'module_ddcci_driver'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~
In file included from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/backlight.h:12,
from drivers/ddcci/ddcci-backlight.c:16:
>> include/linux/module.h:139:49: error: invalid storage class for function '__exittest'
139 | static inline exitcall_t __maybe_unused __exittest(void) \
| ^~~~~~~~~~
include/linux/device/driver.h:269:1: note: in expansion of macro 'module_exit'
269 | module_exit(__driver##_exit);
| ^~~~~~~~~~~
include/linux/ddcci.h:138:9: note: in expansion of macro 'module_driver'
138 | module_driver(__ddcci_driver, ddcci_add_driver, \
| ^~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:400:1: note: in expansion of macro 'module_ddcci_driver'
400 | module_ddcci_driver(ddcci_backlight_driver);
| ^~~~~~~~~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:400:1: warning: 'alias' attribute ignored [-Wattributes]
In file included from include/linux/module.h:22,
from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/backlight.h:12,
from drivers/ddcci/ddcci-backlight.c:16:
>> include/linux/moduleparam.h:409:45: error: invalid storage class for function '__check_convenience_symlink'
409 | static inline type __always_unused *__check_##name(void) { return(p); }
| ^~~~~~~~
include/linux/moduleparam.h:468:35: note: in expansion of macro '__param_check'
468 | #define param_check_bool(name, p) __param_check(name, p, bool)
| ^~~~~~~~~~~~~
include/linux/moduleparam.h:150:9: note: in expansion of macro 'param_check_bool'
150 | param_check_##type(name, &(value)); \
| ^~~~~~~~~~~~
include/linux/moduleparam.h:127:9: note: in expansion of macro 'module_param_named'
127 | module_param_named(name, name, type, perm)
| ^~~~~~~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:403:1: note: in expansion of macro 'module_param'
403 | module_param(convenience_symlink, bool, 0644);
| ^~~~~~~~~~~~
include/linux/moduleparam.h:409:9: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
409 | static inline type __always_unused *__check_##name(void) { return(p); }
| ^~~~~~
include/linux/moduleparam.h:468:35: note: in expansion of macro '__param_check'
468 | #define param_check_bool(name, p) __param_check(name, p, bool)
| ^~~~~~~~~~~~~
include/linux/moduleparam.h:150:9: note: in expansion of macro 'param_check_bool'
150 | param_check_##type(name, &(value)); \
| ^~~~~~~~~~~~
include/linux/moduleparam.h:127:9: note: in expansion of macro 'module_param_named'
127 | module_param_named(name, name, type, perm)
| ^~~~~~~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:403:1: note: in expansion of macro 'module_param'
403 | module_param(convenience_symlink, bool, 0644);
| ^~~~~~~~~~~~
include/linux/moduleparam.h:289:9: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
289 | static const char __param_str_##name[] = prefix #name; \
| ^~~~~~
include/linux/moduleparam.h:176:9: note: in expansion of macro '__module_param_call'
176 | __module_param_call(MODULE_PARAM_PREFIX, name, ops, arg, perm, -1, 0)
| ^~~~~~~~~~~~~~~~~~~
include/linux/moduleparam.h:151:9: note: in expansion of macro 'module_param_cb'
151 | module_param_cb(name, ¶m_ops_##type, &value, perm); \
| ^~~~~~~~~~~~~~~
include/linux/moduleparam.h:127:9: note: in expansion of macro 'module_param_named'
127 | module_param_named(name, name, type, perm)
| ^~~~~~~~~~~~~~~~~~
drivers/ddcci/ddcci-backlight.c:403:1: note: in expansion of macro 'module_param'
403 | module_param(convenience_symlink, bool, 0644);
| ^~~~~~~~~~~~
>> drivers/ddcci/ddcci-backlight.c:411:1: error: expected declaration or statement at end of input
411 | MODULE_ALIAS("ddcci:monitor-*-*-*-*");
| ^~~~~~~~~~~~
vim +76 drivers/ddcci/ddcci.c
65
66 /* Replace non-alphanumeric characters in a string (used for modalias) */
67 static void ddcci_modalias_clean(char *string, size_t n, char replacement)
68 {
69 int i;
70
71 for (i = 0; i < n; ++i) {
72 char c = string[i];
73
74 if (c == 0) {
75 return;
> 76 else if (c < '0' || (c > '9' && c < 'A') || (c > 'Z' && c < 'a') || c > 'z')
77 string[i] = replacement;
78 }
79 }
80
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Powered by blists - more mailing lists