linux-kernel - Re: [PATCH] rapidio: fix kernel-doc warning

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: <495C4916.3080700@oracle.com>
Date:	Wed, 31 Dec 2008 20:39:50 -0800
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	Jianjun Kong <jianjun@...ux.org>
CC:	Linux-Kernel-Mailing-List <linux-kernel@...r.kernel.org>,
	mporter@...nel.crashing.org
Subject: Re: [PATCH] rapidio: fix kernel-doc warning

Jianjun Kong wrote:
> rapidio: fix kernel-doc warning
> 
> Fix kernel-doc parameter warning and correct the descriptio:
> Warning(/media/sda1/Project/linux-2.6//drivers/rapidio/rio-driver.c:88): Excess function parameter 'id'
> description in 'rio_device_probe'
> The parameter 'id' may be deleted.
> 
> Signed-off-by: Jianjun Kong <jianjun@...ux.org>
> Cc: Matt Porter <mporter@...nel.crashing.org>

Hi and thanks, but this is already fixed in Andrew Morton's mm patch series.
You should probably check there for other such patches.


> ---
>  drivers/rapidio/rio-driver.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/rapidio/rio-driver.c b/drivers/rapidio/rio-driver.c
> index 956d3e7..cd6f14e 100644
> --- a/drivers/rapidio/rio-driver.c
> +++ b/drivers/rapidio/rio-driver.c
> @@ -78,8 +78,7 @@ void rio_dev_put(struct rio_dev *rdev)
>  }
>  
>  /**
> - *  rio_device_probe - Tell if a RIO device structure has a matching RIO device id structure
> - *  @id: the RIO device id structure to match against
> + *  rio_device_probe - Tell if a RIO device structure has a matching RIO device structure
>   *  @dev: the RIO device structure to match against
>   *
>   * return 0 and set rio_dev->driver when drv claims rio_dev, else error


-- 
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/