linux-kernel - Re: [PATCH 1/1] PCI: hotplug: Remove "Returns" kerneldoc from void functions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240930170317.GA180055@bhelgaas>
Date: Mon, 30 Sep 2024 12:03:17 -0500
From: Bjorn Helgaas <helgaas@...nel.org>
To: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
Cc: Krzysztof Wilczyński <kw@...ux.com>,
	Lukas Wunner <lukas@...ner.de>, Bjorn Helgaas <bhelgaas@...gle.com>,
	linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] PCI: hotplug: Remove "Returns" kerneldoc from void
 functions

On Mon, Sep 30, 2024 at 03:44:36PM +0300, Ilpo Järvinen wrote:
> pci_hp_deregister() was converted to void by the commit 51bbf9bee34f
> ("PCI: hotplug: Demidlayer registration with the core") but its
> kerneldoc still describes the return value. pci_hp_del() and
> pci_hp_destroy() have been void since they were introduced in that same
> commit.
> 
> Remove the return value description from the kerneldoc of those
> functions.
> 
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>

Applied to pci/misc for v6.13, thanks Ilpo!

> ---
>  drivers/pci/hotplug/pci_hotplug_core.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/drivers/pci/hotplug/pci_hotplug_core.c b/drivers/pci/hotplug/pci_hotplug_core.c
> index 058d5937d8a9..69b18b07f60a 100644
> --- a/drivers/pci/hotplug/pci_hotplug_core.c
> +++ b/drivers/pci/hotplug/pci_hotplug_core.c
> @@ -498,8 +498,6 @@ EXPORT_SYMBOL_GPL(pci_hp_add);
>   *
>   * The @slot must have been registered with the pci hotplug subsystem
>   * previously with a call to pci_hp_register().
> - *
> - * Returns 0 if successful, anything else for an error.
>   */
>  void pci_hp_deregister(struct hotplug_slot *slot)
>  {
> @@ -513,8 +511,6 @@ EXPORT_SYMBOL_GPL(pci_hp_deregister);
>   * @slot: pointer to the &struct hotplug_slot to unpublish
>   *
>   * Remove a hotplug slot's sysfs interface.
> - *
> - * Returns 0 on success or a negative int on error.
>   */
>  void pci_hp_del(struct hotplug_slot *slot)
>  {
> @@ -545,8 +541,6 @@ EXPORT_SYMBOL_GPL(pci_hp_del);
>   * the driver may no longer invoke hotplug_slot_name() to get the slot's
>   * unique name.  The driver no longer needs to handle a ->reset_slot callback
>   * from this point on.
> - *
> - * Returns 0 on success or a negative int on error.
>   */
>  void pci_hp_destroy(struct hotplug_slot *slot)
>  {
> -- 
> 2.39.5
>