From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org, Eddie James <eajames@linux.ibm.com>,
linux-leds@vger.kernel.org
Cc: lkp@intel.com, kbuild-all@lists.01.org,
linux-kernel@vger.kernel.org, pavel@ucw.cz, patrick@stwcx.xyz,
andy.shevchenko@gmail.com, openbmc@lists.ozlabs.org,
joel@jms.id.au, Eddie James <eajames@linux.ibm.com>
Subject: Re: [PATCH v3 4/4] leds: pca955x: Add HW blink support
Date: Tue, 12 Apr 2022 14:33:30 +0300 [thread overview]
Message-ID: <202204121953.zHZcX6EV-lkp@intel.com> (raw)
In-Reply-To: <20220411162033.39613-5-eajames@linux.ibm.com>
Hi Eddie,
url: https://github.com/intel-lab-lkp/linux/commits/Eddie-James/leds-pca955x-Add-HW-blink-support/20220412-002330
base: git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git for-next
config: i386-randconfig-m021-20220411 (https://download.01.org/0day-ci/archive/20220412/202204121953.zHZcX6EV-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.2.0-19) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/leds/leds-pca955x.c:455 pca955x_led_blink() error: uninitialized symbol 'ret'.
vim +/ret +455 drivers/leds/leds-pca955x.c
3b7b1899f6cc6d Eddie James 2022-04-11 390 static int pca955x_led_blink(struct led_classdev *led_cdev,
3b7b1899f6cc6d Eddie James 2022-04-11 391 unsigned long *delay_on, unsigned long *delay_off)
3b7b1899f6cc6d Eddie James 2022-04-11 392 {
3b7b1899f6cc6d Eddie James 2022-04-11 393 struct pca955x_led *pca955x_led = led_to_pca955x(led_cdev);
3b7b1899f6cc6d Eddie James 2022-04-11 394 struct pca955x *pca955x = pca955x_led->pca955x;
3b7b1899f6cc6d Eddie James 2022-04-11 395 unsigned long p = *delay_on + *delay_off;
3b7b1899f6cc6d Eddie James 2022-04-11 396 int ret;
3b7b1899f6cc6d Eddie James 2022-04-11 397
3b7b1899f6cc6d Eddie James 2022-04-11 398 mutex_lock(&pca955x->lock);
3b7b1899f6cc6d Eddie James 2022-04-11 399
3b7b1899f6cc6d Eddie James 2022-04-11 400 if (p) {
3b7b1899f6cc6d Eddie James 2022-04-11 401 if (*delay_on != *delay_off) {
3b7b1899f6cc6d Eddie James 2022-04-11 402 ret = -EINVAL;
3b7b1899f6cc6d Eddie James 2022-04-11 403 goto out;
3b7b1899f6cc6d Eddie James 2022-04-11 404 }
3b7b1899f6cc6d Eddie James 2022-04-11 405
3b7b1899f6cc6d Eddie James 2022-04-11 406 if (p < pca955x_psc_to_period(pca955x, 0) ||
3b7b1899f6cc6d Eddie James 2022-04-11 407 p > pca955x_psc_to_period(pca955x, 0xff)) {
3b7b1899f6cc6d Eddie James 2022-04-11 408 ret = -EINVAL;
3b7b1899f6cc6d Eddie James 2022-04-11 409 goto out;
3b7b1899f6cc6d Eddie James 2022-04-11 410 }
3b7b1899f6cc6d Eddie James 2022-04-11 411 } else {
3b7b1899f6cc6d Eddie James 2022-04-11 412 p = pca955x->active_blink ? pca955x->blink_period :
3b7b1899f6cc6d Eddie James 2022-04-11 413 PCA955X_BLINK_DEFAULT_MS;
3b7b1899f6cc6d Eddie James 2022-04-11 414 }
3b7b1899f6cc6d Eddie James 2022-04-11 415
3b7b1899f6cc6d Eddie James 2022-04-11 416 if (!pca955x->active_blink ||
3b7b1899f6cc6d Eddie James 2022-04-11 417 pca955x->active_blink == BIT(pca955x_led->led_num) ||
3b7b1899f6cc6d Eddie James 2022-04-11 418 pca955x->blink_period == p) {
3b7b1899f6cc6d Eddie James 2022-04-11 419 u8 psc = pca955x_period_to_psc(pca955x, p);
f46e9203d9a100 Nate Case 2008-07-16 420
3b7b1899f6cc6d Eddie James 2022-04-11 421 if (!test_and_set_bit(pca955x_led->led_num,
3b7b1899f6cc6d Eddie James 2022-04-11 422 &pca955x->active_blink)) {
3b7b1899f6cc6d Eddie James 2022-04-11 423 u8 ls;
3b7b1899f6cc6d Eddie James 2022-04-11 424 int reg = pca955x_led->led_num / 4;
3b7b1899f6cc6d Eddie James 2022-04-11 425 int bit = pca955x_led->led_num % 4;
3b7b1899f6cc6d Eddie James 2022-04-11 426
3b7b1899f6cc6d Eddie James 2022-04-11 427 ret = pca955x_read_ls(pca955x, reg, &ls);
3b7b1899f6cc6d Eddie James 2022-04-11 428 if (ret)
3b7b1899f6cc6d Eddie James 2022-04-11 429 goto out;
3b7b1899f6cc6d Eddie James 2022-04-11 430
3b7b1899f6cc6d Eddie James 2022-04-11 431 ls = pca955x_ledsel(ls, bit, PCA955X_LS_BLINK0);
9e58c2a7bb91f6 Eddie James 2022-04-11 432 ret = pca955x_write_ls(pca955x, reg, ls);
3b7b1899f6cc6d Eddie James 2022-04-11 433 if (ret)
3b7b1899f6cc6d Eddie James 2022-04-11 434 goto out;
3b7b1899f6cc6d Eddie James 2022-04-11 435 }
3b7b1899f6cc6d Eddie James 2022-04-11 436
3b7b1899f6cc6d Eddie James 2022-04-11 437 if (pca955x->blink_period != p) {
3b7b1899f6cc6d Eddie James 2022-04-11 438 pca955x->blink_period = p;
3b7b1899f6cc6d Eddie James 2022-04-11 439 ret = pca955x_write_psc(pca955x, 0, psc);
3b7b1899f6cc6d Eddie James 2022-04-11 440 if (ret)
3b7b1899f6cc6d Eddie James 2022-04-11 441 goto out;
3b7b1899f6cc6d Eddie James 2022-04-11 442 }
Can both the !test_and_set_bit() and pca955x->blink_period != p conditions
be false? If so then "ret" is uninitialized.
3b7b1899f6cc6d Eddie James 2022-04-11 443
3b7b1899f6cc6d Eddie James 2022-04-11 444 p = pca955x_psc_to_period(pca955x, psc);
3b7b1899f6cc6d Eddie James 2022-04-11 445 p /= 2;
3b7b1899f6cc6d Eddie James 2022-04-11 446 *delay_on = p;
3b7b1899f6cc6d Eddie James 2022-04-11 447 *delay_off = p;
3b7b1899f6cc6d Eddie James 2022-04-11 448 } else {
3b7b1899f6cc6d Eddie James 2022-04-11 449 ret = -EBUSY;
3b7b1899f6cc6d Eddie James 2022-04-11 450 }
e7e11d8ba807d4 Alexander Stein 2012-05-29 451
1591caf2d5eafd Cédric Le Goater 2017-08-30 452 out:
e7e11d8ba807d4 Alexander Stein 2012-05-29 453 mutex_unlock(&pca955x->lock);
f46e9203d9a100 Nate Case 2008-07-16 454
1591caf2d5eafd Cédric Le Goater 2017-08-30 @455 return ret;
f46e9203d9a100 Nate Case 2008-07-16 456 }
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-04-12 12:28 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-11 16:20 [PATCH v3 0/4] leds: pca955x: Add HW blink support Eddie James
2022-04-11 16:20 ` [PATCH v3 1/4] leds: pca955x: Refactor with helper functions and renaming Eddie James
2022-04-11 16:20 ` [PATCH v3 2/4] leds: pca955x: Use pointers to driver data rather than I2C client Eddie James
2022-04-11 16:20 ` [PATCH v3 3/4] leds: pca955x: Optimize probe led selection Eddie James
2022-05-04 17:24 ` Pavel Machek
2022-05-11 19:43 ` Eddie James
2022-04-11 16:20 ` [PATCH v3 4/4] leds: pca955x: Add HW blink support Eddie James
2022-04-12 11:33 ` Dan Carpenter [this message]
2022-05-04 17:24 ` Pavel Machek
2022-05-11 19:54 ` Eddie James
2022-04-11 17:06 ` [PATCH v3 0/4] " Andy Shevchenko
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=202204121953.zHZcX6EV-lkp@intel.com \
--to=dan.carpenter@oracle.com \
--cc=andy.shevchenko@gmail.com \
--cc=eajames@linux.ibm.com \
--cc=joel@jms.id.au \
--cc=kbuild-all@lists.01.org \
--cc=kbuild@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=lkp@intel.com \
--cc=openbmc@lists.ozlabs.org \
--cc=patrick@stwcx.xyz \
--cc=pavel@ucw.cz \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox