* Re: [PATCH v4] platform/x86: dell-uart-backlight: Bind through normal serdev matching
[not found] <20261024022930.2704030-3-an.wu@canonical.com>
@ 2026-10-24 17:42 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2026-10-24 17:42 UTC (permalink / raw)
To: ChunAn Wu, linux-serial, platform-driver-x86
Cc: llvm, oe-kbuild-all, linux-kernel, markus.probst, gregkh,
jirislaby, hansg, ilpo.jarvinen, mpearson-lenovo, derekjohn.clark
Hi ChunAn,
kernel test robot noticed the following build errors:
[auto build test ERROR on driver-core/driver-core-testing]
[also build test ERROR on driver-core/driver-core-next driver-core/driver-core-linus linus/master v7.3 next-20261023]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/ChunAn-Wu/platform-x86-dell-uart-backlight-Bind-through-normal-serdev-matching/20261024-102929
base: driver-core/driver-core-testing
patch link: https://lore.kernel.org/r/20261024022930.2704030-3-an.wu%40canonical.com
patch subject: [PATCH v4] platform/x86: dell-uart-backlight: Bind through normal serdev matching
config: x86_64-randconfig-1014-20261024 (https://download.01.org/0day-ci/archive/20261025/202610250118.t0UAW41j-lkp@intel.com/config)
compiler: clang version 22.1.8 (https://github.com/llvm/llvm-project ca7933e47d3a3451d81e72ac174dcb5aa28b59d1)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261025/202610250118.t0UAW41j-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202610250118.t0UAW41j-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/platform/x86/dell/dell-uart-backlight.c:328:60: error: array has incomplete element type 'const struct serdev_device_id'
328 | static const struct serdev_device_id dell_uart_bl_serdev_id[] = {
| ^
drivers/platform/x86/dell/dell-uart-backlight.c:328:21: note: forward declaration of 'struct serdev_device_id'
328 | static const struct serdev_device_id dell_uart_bl_serdev_id[] = {
| ^
>> drivers/platform/x86/dell/dell-uart-backlight.c:363:34: error: too many arguments to function call, expected 1, have 2
363 | ret = serdev_device_add(serdev, KBUILD_MODNAME);
| ~~~~~~~~~~~~~~~~~ ^~~~~~~~~~~~~~
<command line>:8:24: note: expanded from macro 'KBUILD_MODNAME'
8 | #define KBUILD_MODNAME "dell_uart_backlight"
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/serdev.h:161:5: note: 'serdev_device_add' declared here
161 | int serdev_device_add(struct serdev_device *);
| ^ ~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.
vim +328 drivers/platform/x86/dell/dell-uart-backlight.c
327
> 328 static const struct serdev_device_id dell_uart_bl_serdev_id[] = {
329 { KBUILD_MODNAME },
330 { }
331 };
332
333 static struct serdev_device_driver dell_uart_bl_serdev_driver = {
334 .probe = dell_uart_bl_serdev_probe,
335 .driver = {
336 .name = KBUILD_MODNAME,
337 },
338 .id_table = dell_uart_bl_serdev_id,
339 };
340
341 static int dell_uart_bl_pdev_probe(struct platform_device *pdev)
342 {
343 enum acpi_backlight_type bl_type;
344 struct serdev_device *serdev;
345 struct device *ctrl_dev;
346 int ret;
347
348 bl_type = acpi_video_get_backlight_type();
349 if (bl_type != acpi_backlight_dell_uart) {
350 dev_dbg(&pdev->dev, "Not loading (ACPI backlight type = %d)\n", bl_type);
351 return -ENODEV;
352 }
353
354 ctrl_dev = get_serdev_controller("DELL0501", NULL, 0, "serial0");
355 if (IS_ERR(ctrl_dev))
356 return PTR_ERR(ctrl_dev);
357
358 serdev = serdev_device_alloc(to_serdev_controller(ctrl_dev));
359 put_device(ctrl_dev);
360 if (!serdev)
361 return -ENOMEM;
362
> 363 ret = serdev_device_add(serdev, KBUILD_MODNAME);
364 if (ret) {
365 dev_err(&pdev->dev, "error %d adding serdev\n", ret);
366 serdev_device_put(serdev);
367 return ret;
368 }
369
370 ret = serdev_device_driver_register(&dell_uart_bl_serdev_driver);
371 if (ret)
372 goto err_remove_serdev;
373
374 /* So that dell_uart_bl_pdev_remove() can remove the serdev */
375 platform_set_drvdata(pdev, serdev);
376 return 0;
377
378 err_remove_serdev:
379 serdev_device_remove(serdev);
380 return ret;
381 }
382
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread