From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: [lunn:leds-offload-support-reduced-auto-netdev 15/21] drivers/leds/led-class.c:362 led_classdev_register_ext() warn: curly braces intended?
Date: Tue, 2 May 2023 09:27:45 +0800 [thread overview]
Message-ID: <202305020950.QBbVMspm-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
TO: Andrew Lunn <andrew@lunn.ch>
tree: https://github.com/lunn/linux.git leds-offload-support-reduced-auto-netdev
head: c60ee6ed7e664a425ce612fc68bcbc0a79d75aa5
commit: 1b3bca97ae847a2c323567950f7b7334703992c2 [15/21] drivers: leds: led-class: Keep fwnode from init_data for triggers
:::::: branch date: 13 hours ago
:::::: commit date: 13 hours ago
config: x86_64-randconfig-m001 (https://download.01.org/0day-ci/archive/20230502/202305020950.QBbVMspm-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Link: https://lore.kernel.org/r/202305020950.QBbVMspm-lkp@intel.com/
smatch warnings:
drivers/leds/led-class.c:362 led_classdev_register_ext() warn: curly braces intended?
vim +362 drivers/leds/led-class.c
a96aa64cb5723d Ricardo Ribalda 2015-03-30 326
c72a1d608dd0eb Richard Purdie 2006-03-31 327 /**
b2b998c0f94499 Jacek Anaszewski 2019-06-09 328 * led_classdev_register_ext - register a new object of led_classdev class
b2b998c0f94499 Jacek Anaszewski 2019-06-09 329 * with init data.
442c609830e989 Rafał Miłecki 2017-03-06 330 *
442c609830e989 Rafał Miłecki 2017-03-06 331 * @parent: parent of LED device
c72a1d608dd0eb Richard Purdie 2006-03-31 332 * @led_cdev: the led_classdev structure for this device.
b2b998c0f94499 Jacek Anaszewski 2019-06-09 333 * @init_data: LED class device initialization data
c72a1d608dd0eb Richard Purdie 2006-03-31 334 */
b2b998c0f94499 Jacek Anaszewski 2019-06-09 335 int led_classdev_register_ext(struct device *parent,
b2b998c0f94499 Jacek Anaszewski 2019-06-09 336 struct led_classdev *led_cdev,
b2b998c0f94499 Jacek Anaszewski 2019-06-09 337 struct led_init_data *init_data)
c72a1d608dd0eb Richard Purdie 2006-03-31 338 {
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 339 char composed_name[LED_MAX_NAME_SIZE];
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 340 char final_name[LED_MAX_NAME_SIZE];
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 341 const char *proposed_name = composed_name;
a96aa64cb5723d Ricardo Ribalda 2015-03-30 342 int ret;
a96aa64cb5723d Ricardo Ribalda 2015-03-30 343
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 344 if (init_data) {
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 345 if (init_data->devname_mandatory && !init_data->devicename) {
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 346 dev_err(parent, "Mandatory device name is missing");
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 347 return -EINVAL;
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 348 }
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 349 ret = led_compose_name(parent, init_data, composed_name);
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 350 if (ret < 0)
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 351 return ret;
c49d6cab0d7fde Marek Behún 2020-09-19 352
419066324e19a4 Eddie James 2021-07-16 353 if (init_data->fwnode) {
c49d6cab0d7fde Marek Behún 2020-09-19 354 fwnode_property_read_string(init_data->fwnode,
c49d6cab0d7fde Marek Behún 2020-09-19 355 "linux,default-trigger",
c49d6cab0d7fde Marek Behún 2020-09-19 356 &led_cdev->default_trigger);
419066324e19a4 Eddie James 2021-07-16 357
419066324e19a4 Eddie James 2021-07-16 358 if (fwnode_property_present(init_data->fwnode,
419066324e19a4 Eddie James 2021-07-16 359 "retain-state-shutdown"))
419066324e19a4 Eddie James 2021-07-16 360 led_cdev->flags |= LED_RETAIN_AT_SHUTDOWN;
1b3bca97ae847a Andrew Lunn 2023-04-30 361 #ifdef CONFIG_LEDS_TRIGGERS
1b3bca97ae847a Andrew Lunn 2023-04-30 @362 led_cdev->fwnode = init_data->fwnode;
1b3bca97ae847a Andrew Lunn 2023-04-30 363 #endif
419066324e19a4 Eddie James 2021-07-16 364 }
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 365 } else {
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 366 proposed_name = led_cdev->name;
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 367 }
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 368
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 369 ret = led_classdev_next_name(proposed_name, final_name, sizeof(final_name));
a96aa64cb5723d Ricardo Ribalda 2015-03-30 370 if (ret < 0)
a96aa64cb5723d Ricardo Ribalda 2015-03-30 371 return ret;
a96aa64cb5723d Ricardo Ribalda 2015-03-30 372
6d71021ab3b0bd Luis Henriques 2018-05-23 373 mutex_init(&led_cdev->led_access);
6d71021ab3b0bd Luis Henriques 2018-05-23 374 mutex_lock(&led_cdev->led_access);
d0d480cce8f522 Johan Hovold 2014-06-25 375 led_cdev->dev = device_create_with_groups(leds_class, parent, 0,
bb4e9af0348dfe Jacek Anaszewski 2019-06-09 376 led_cdev, led_cdev->groups, "%s", final_name);
6d71021ab3b0bd Luis Henriques 2018-05-23 377 if (IS_ERR(led_cdev->dev)) {
6d71021ab3b0bd Luis Henriques 2018-05-23 378 mutex_unlock(&led_cdev->led_access);
f8a7c6fe14f556 Richard Purdie 2007-07-08 379 return PTR_ERR(led_cdev->dev);
6d71021ab3b0bd Luis Henriques 2018-05-23 380 }
495b8966f7ad92 Sander Vanheule 2021-11-13 381 if (init_data && init_data->fwnode)
495b8966f7ad92 Sander Vanheule 2021-11-13 382 device_set_node(led_cdev->dev, init_data->fwnode);
c72a1d608dd0eb Richard Purdie 2006-03-31 383
a96aa64cb5723d Ricardo Ribalda 2015-03-30 384 if (ret)
6f06c7f87cf54f Sakari Ailus 2015-03-26 385 dev_warn(parent, "Led %s renamed to %s due to name collision",
64ed6588c2ea61 Ricardo Ribalda 2020-04-01 386 proposed_name, dev_name(led_cdev->dev));
a96aa64cb5723d Ricardo Ribalda 2015-03-30 387
0cb8eb30d425d2 Hans de Goede 2017-01-29 388 if (led_cdev->flags & LED_BRIGHT_HW_CHANGED) {
0cb8eb30d425d2 Hans de Goede 2017-01-29 389 ret = led_add_brightness_hw_changed(led_cdev);
0cb8eb30d425d2 Hans de Goede 2017-01-29 390 if (ret) {
0cb8eb30d425d2 Hans de Goede 2017-01-29 391 device_unregister(led_cdev->dev);
1dbb9fb4082ce2 Andy Shevchenko 2019-08-16 392 led_cdev->dev = NULL;
6d71021ab3b0bd Luis Henriques 2018-05-23 393 mutex_unlock(&led_cdev->led_access);
0cb8eb30d425d2 Hans de Goede 2017-01-29 394 return ret;
0cb8eb30d425d2 Hans de Goede 2017-01-29 395 }
0cb8eb30d425d2 Hans de Goede 2017-01-29 396 }
0cb8eb30d425d2 Hans de Goede 2017-01-29 397
a9c6ce57ec2f13 Hans de Goede 2016-11-08 398 led_cdev->work_flags = 0;
270c3957dbc5be Richard Purdie 2008-10-20 399 #ifdef CONFIG_LEDS_TRIGGERS
270c3957dbc5be Richard Purdie 2008-10-20 400 init_rwsem(&led_cdev->trigger_lock);
0cb8eb30d425d2 Hans de Goede 2017-01-29 401 #endif
0cb8eb30d425d2 Hans de Goede 2017-01-29 402 #ifdef CONFIG_LEDS_BRIGHTNESS_HW_CHANGED
0cb8eb30d425d2 Hans de Goede 2017-01-29 403 led_cdev->brightness_hw_changed = -1;
270c3957dbc5be Richard Purdie 2008-10-20 404 #endif
c72a1d608dd0eb Richard Purdie 2006-03-31 405 /* add to the list of leds */
72f8da329e07ad Richard Purdie 2007-12-31 406 down_write(&leds_list_lock);
c72a1d608dd0eb Richard Purdie 2006-03-31 407 list_add_tail(&led_cdev->node, &leds_list);
72f8da329e07ad Richard Purdie 2007-12-31 408 up_write(&leds_list_lock);
c72a1d608dd0eb Richard Purdie 2006-03-31 409
1bd465e6b0e2b5 Guennadi Liakhovetski 2009-01-10 410 if (!led_cdev->max_brightness)
1bd465e6b0e2b5 Guennadi Liakhovetski 2009-01-10 411 led_cdev->max_brightness = LED_FULL;
1bd465e6b0e2b5 Guennadi Liakhovetski 2009-01-10 412
29d76dfa29fe22 Henrique de Moraes Holschuh 2008-03-18 413 led_update_brightness(led_cdev);
29d76dfa29fe22 Henrique de Moraes Holschuh 2008-03-18 414
757b06ae04b3b6 Jacek Anaszewski 2015-09-28 415 led_init_core(led_cdev);
5ada28bf76752e Johannes Berg 2010-11-11 416
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
reply other threads:[~2023-05-02 1:28 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202305020950.QBbVMspm-lkp@intel.com \
--to=lkp@intel.com \
--cc=error27@gmail.com \
--cc=oe-kbuild@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.