From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Julia Lawall <julia.lawall@inria.fr>
Subject: drivers/power/supply/pf1550-charger.c:594:8-33: WARNING: Threaded IRQ with no primary handler requested without IRQF_ONESHOT (unless it is nested IRQ)
Date: Tue, 31 Mar 2026 03:09:00 +0800 [thread overview]
Message-ID: <202603310329.V0dDFga3-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Samuel Kayode <samuel.kayode@savoirfairelinux.com>
CC: Lee Jones <lee@kernel.org>
CC: Frank Li <Frank.Li@nxp.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 7aaa8047eafd0bd628065b15757d9b48c5f9c07d
commit: 4b6b6433a97d5863b5340fc87f866d784fdf0783 power: supply: pf1550: add battery charger support
date: 5 months ago
:::::: branch date: 20 hours ago
:::::: commit date: 5 months ago
config: powerpc64-randconfig-r052-20260330 (https://download.01.org/0day-ci/archive/20260331/202603310329.V0dDFga3-lkp@intel.com/config)
compiler: clang version 23.0.0git (https://github.com/llvm/llvm-project 054e11d1a17e5ba88bb1a8ef32fad3346e80b186)
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>
| Reported-by: Julia Lawall <julia.lawall@inria.fr>
| Closes: https://lore.kernel.org/r/202603310329.V0dDFga3-lkp@intel.com/
cocci warnings: (new ones prefixed by >>)
>> drivers/power/supply/pf1550-charger.c:594:8-33: WARNING: Threaded IRQ with no primary handler requested without IRQF_ONESHOT (unless it is nested IRQ)
vim +594 drivers/power/supply/pf1550-charger.c
4b6b6433a97d58 Samuel Kayode 2025-10-01 548
4b6b6433a97d58 Samuel Kayode 2025-10-01 549 static int pf1550_charger_probe(struct platform_device *pdev)
4b6b6433a97d58 Samuel Kayode 2025-10-01 550 {
4b6b6433a97d58 Samuel Kayode 2025-10-01 551 const struct pf1550_ddata *pf1550 = dev_get_drvdata(pdev->dev.parent);
4b6b6433a97d58 Samuel Kayode 2025-10-01 552 struct power_supply_config psy_cfg = {};
4b6b6433a97d58 Samuel Kayode 2025-10-01 553 struct pf1550_charger *chg;
4b6b6433a97d58 Samuel Kayode 2025-10-01 554 int i, irq, ret;
4b6b6433a97d58 Samuel Kayode 2025-10-01 555
4b6b6433a97d58 Samuel Kayode 2025-10-01 556 chg = devm_kzalloc(&pdev->dev, sizeof(*chg), GFP_KERNEL);
4b6b6433a97d58 Samuel Kayode 2025-10-01 557 if (!chg)
4b6b6433a97d58 Samuel Kayode 2025-10-01 558 return -ENOMEM;
4b6b6433a97d58 Samuel Kayode 2025-10-01 559
4b6b6433a97d58 Samuel Kayode 2025-10-01 560 chg->dev = &pdev->dev;
4b6b6433a97d58 Samuel Kayode 2025-10-01 561 chg->pf1550 = pf1550;
4b6b6433a97d58 Samuel Kayode 2025-10-01 562
4b6b6433a97d58 Samuel Kayode 2025-10-01 563 if (!chg->pf1550->regmap)
4b6b6433a97d58 Samuel Kayode 2025-10-01 564 return dev_err_probe(&pdev->dev, -ENODEV,
4b6b6433a97d58 Samuel Kayode 2025-10-01 565 "failed to get regmap\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 566
4b6b6433a97d58 Samuel Kayode 2025-10-01 567 platform_set_drvdata(pdev, chg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 568
4b6b6433a97d58 Samuel Kayode 2025-10-01 569 ret = devm_delayed_work_autocancel(chg->dev, &chg->vbus_sense_work,
4b6b6433a97d58 Samuel Kayode 2025-10-01 570 pf1550_chg_vbus_work);
4b6b6433a97d58 Samuel Kayode 2025-10-01 571 if (ret)
4b6b6433a97d58 Samuel Kayode 2025-10-01 572 return dev_err_probe(chg->dev, ret,
4b6b6433a97d58 Samuel Kayode 2025-10-01 573 "failed to add vbus sense work\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 574
4b6b6433a97d58 Samuel Kayode 2025-10-01 575 ret = devm_delayed_work_autocancel(chg->dev, &chg->chg_sense_work,
4b6b6433a97d58 Samuel Kayode 2025-10-01 576 pf1550_chg_chg_work);
4b6b6433a97d58 Samuel Kayode 2025-10-01 577 if (ret)
4b6b6433a97d58 Samuel Kayode 2025-10-01 578 return dev_err_probe(chg->dev, ret,
4b6b6433a97d58 Samuel Kayode 2025-10-01 579 "failed to add charger sense work\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 580
4b6b6433a97d58 Samuel Kayode 2025-10-01 581 ret = devm_delayed_work_autocancel(chg->dev, &chg->bat_sense_work,
4b6b6433a97d58 Samuel Kayode 2025-10-01 582 pf1550_chg_bat_work);
4b6b6433a97d58 Samuel Kayode 2025-10-01 583 if (ret)
4b6b6433a97d58 Samuel Kayode 2025-10-01 584 return dev_err_probe(chg->dev, ret,
4b6b6433a97d58 Samuel Kayode 2025-10-01 585 "failed to add battery sense work\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 586
4b6b6433a97d58 Samuel Kayode 2025-10-01 587 for (i = 0; i < PF1550_CHARGER_IRQ_NR; i++) {
4b6b6433a97d58 Samuel Kayode 2025-10-01 588 irq = platform_get_irq(pdev, i);
4b6b6433a97d58 Samuel Kayode 2025-10-01 589 if (irq < 0)
4b6b6433a97d58 Samuel Kayode 2025-10-01 590 return irq;
4b6b6433a97d58 Samuel Kayode 2025-10-01 591
4b6b6433a97d58 Samuel Kayode 2025-10-01 592 chg->virqs[i] = irq;
4b6b6433a97d58 Samuel Kayode 2025-10-01 593
4b6b6433a97d58 Samuel Kayode 2025-10-01 @594 ret = devm_request_threaded_irq(&pdev->dev, irq, NULL,
4b6b6433a97d58 Samuel Kayode 2025-10-01 595 pf1550_charger_irq_handler,
4b6b6433a97d58 Samuel Kayode 2025-10-01 596 IRQF_NO_SUSPEND,
4b6b6433a97d58 Samuel Kayode 2025-10-01 597 "pf1550-charger", chg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 598 if (ret)
4b6b6433a97d58 Samuel Kayode 2025-10-01 599 return dev_err_probe(&pdev->dev, ret,
4b6b6433a97d58 Samuel Kayode 2025-10-01 600 "failed irq request\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 601 }
4b6b6433a97d58 Samuel Kayode 2025-10-01 602
4b6b6433a97d58 Samuel Kayode 2025-10-01 603 psy_cfg.drv_data = chg;
4b6b6433a97d58 Samuel Kayode 2025-10-01 604
4b6b6433a97d58 Samuel Kayode 2025-10-01 605 chg->charger = devm_power_supply_register(&pdev->dev,
4b6b6433a97d58 Samuel Kayode 2025-10-01 606 &pf1550_charger_desc,
4b6b6433a97d58 Samuel Kayode 2025-10-01 607 &psy_cfg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 608 if (IS_ERR(chg->charger))
4b6b6433a97d58 Samuel Kayode 2025-10-01 609 return dev_err_probe(&pdev->dev, PTR_ERR(chg->charger),
4b6b6433a97d58 Samuel Kayode 2025-10-01 610 "failed: power supply register\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 611
4b6b6433a97d58 Samuel Kayode 2025-10-01 612 chg->battery = devm_power_supply_register(&pdev->dev,
4b6b6433a97d58 Samuel Kayode 2025-10-01 613 &pf1550_battery_desc,
4b6b6433a97d58 Samuel Kayode 2025-10-01 614 &psy_cfg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 615 if (IS_ERR(chg->battery))
4b6b6433a97d58 Samuel Kayode 2025-10-01 616 return dev_err_probe(&pdev->dev, PTR_ERR(chg->battery),
4b6b6433a97d58 Samuel Kayode 2025-10-01 617 "failed: power supply register\n");
4b6b6433a97d58 Samuel Kayode 2025-10-01 618
4b6b6433a97d58 Samuel Kayode 2025-10-01 619 pf1550_dt_parse_dev_info(chg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 620
4b6b6433a97d58 Samuel Kayode 2025-10-01 621 return pf1550_reg_init(chg);
4b6b6433a97d58 Samuel Kayode 2025-10-01 622 }
4b6b6433a97d58 Samuel Kayode 2025-10-01 623
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2026-03-30 19:09 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-30 19:09 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2026-02-15 3:04 drivers/power/supply/pf1550-charger.c:594:8-33: WARNING: Threaded IRQ with no primary handler requested without IRQF_ONESHOT (unless it is nested IRQ) kernel test robot
2025-12-17 2:10 kernel test robot
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=202603310329.V0dDFga3-lkp@intel.com \
--to=lkp@intel.com \
--cc=julia.lawall@inria.fr \
--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.