All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: oe-kbuild-all@lists.linux.dev, Mark Brown <broonie@kernel.org>
Subject: [broonie-ci:fileyhQmWY 15/16] drivers/usb/core/driver.c:1816:16: error: void value not ignored as it ought to be
Date: Fri, 26 Dec 2025 05:08:25 +0800	[thread overview]
Message-ID: <202512260516.2eP96Prm-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git fileyhQmWY
head:   63a83a6e40a3f55fbf3887370f4a6b64fc656924
commit: 1d3dcf1b026121c179aec3afdec4fb22509dfc8d [15/16] PM: runtime: Change pm_runtime_put() return type to void
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251226/202512260516.2eP96Prm-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251226/202512260516.2eP96Prm-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/202512260516.2eP96Prm-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/usb/core/driver.c: In function 'usb_autopm_put_interface_async':
>> drivers/usb/core/driver.c:1816:16: error: void value not ignored as it ought to be
    1816 |         status = pm_runtime_put(&intf->dev);
         |                ^
--
   kernel/irq/chip.c: In function 'irq_chip_pm_put':
>> kernel/irq/chip.c:1546:24: error: void value not ignored as it ought to be
    1546 |                 retval = pm_runtime_put(dev);
         |                        ^


vim +1816 drivers/usb/core/driver.c

645daaab0b6adc Alan Stern 2006-08-30  1794  
9ac39f28b5237a Alan Stern 2008-11-12  1795  /**
9ac39f28b5237a Alan Stern 2008-11-12  1796   * usb_autopm_put_interface_async - decrement a USB interface's PM-usage counter
9ac39f28b5237a Alan Stern 2008-11-12  1797   * @intf: the usb_interface whose counter should be decremented
9ac39f28b5237a Alan Stern 2008-11-12  1798   *
9bbdf1e0afe771 Alan Stern 2010-01-08  1799   * This routine does much the same thing as usb_autopm_put_interface():
9bbdf1e0afe771 Alan Stern 2010-01-08  1800   * It decrements @intf's usage counter and schedules a delayed
9bbdf1e0afe771 Alan Stern 2010-01-08  1801   * autosuspend request if the counter is <= 0.  The difference is that it
9bbdf1e0afe771 Alan Stern 2010-01-08  1802   * does not perform any synchronization; callers should hold a private
9bbdf1e0afe771 Alan Stern 2010-01-08  1803   * lock and handle all synchronization issues themselves.
9ac39f28b5237a Alan Stern 2008-11-12  1804   *
9ac39f28b5237a Alan Stern 2008-11-12  1805   * Typically a driver would call this routine during an URB's completion
9ac39f28b5237a Alan Stern 2008-11-12  1806   * handler, if no more URBs were pending.
9ac39f28b5237a Alan Stern 2008-11-12  1807   *
9ac39f28b5237a Alan Stern 2008-11-12  1808   * This routine can run in atomic context.
9ac39f28b5237a Alan Stern 2008-11-12  1809   */
9ac39f28b5237a Alan Stern 2008-11-12  1810  void usb_autopm_put_interface_async(struct usb_interface *intf)
9ac39f28b5237a Alan Stern 2008-11-12  1811  {
9ac39f28b5237a Alan Stern 2008-11-12  1812  	struct usb_device	*udev = interface_to_usbdev(intf);
fcc4a01eb86612 Alan Stern 2010-11-15  1813  	int			status;
9ac39f28b5237a Alan Stern 2008-11-12  1814  
6ddf27cdbc218a Ming Lei   2010-11-15  1815  	usb_mark_last_busy(udev);
fcc4a01eb86612 Alan Stern 2010-11-15 @1816  	status = pm_runtime_put(&intf->dev);
9bbdf1e0afe771 Alan Stern 2010-01-08  1817  	dev_vdbg(&intf->dev, "%s: cnt %d -> %d\n",
9bbdf1e0afe771 Alan Stern 2010-01-08  1818  			__func__, atomic_read(&intf->dev.power.usage_count),
9bbdf1e0afe771 Alan Stern 2010-01-08  1819  			status);
9ac39f28b5237a Alan Stern 2008-11-12  1820  }
9ac39f28b5237a Alan Stern 2008-11-12  1821  EXPORT_SYMBOL_GPL(usb_autopm_put_interface_async);
9ac39f28b5237a Alan Stern 2008-11-12  1822  

:::::: The code at line 1816 was first introduced by commit
:::::: fcc4a01eb8661226e80632327673f67bf6a5840b USB: use the runtime-PM autosuspend implementation

:::::: TO: Alan Stern <stern@rowland.harvard.edu>
:::::: CC: Greg Kroah-Hartman <gregkh@suse.de>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

                 reply	other threads:[~2025-12-25 21:09 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=202512260516.2eP96Prm-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=broonie@kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=rjw@rjwysocki.net \
    /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.