From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from casper.infradead.org ([85.118.1.10]:48899 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751613Ab3AKAwo (ORCPT ); Thu, 10 Jan 2013 19:52:44 -0500 Message-ID: <50EF62A1.5070306@infradead.org> Date: Thu, 10 Jan 2013 16:53:53 -0800 From: Randy Dunlap MIME-Version: 1.0 To: Don Dutile CC: linux-pci , Bjorn Helgaas , Linus Torvalds Subject: Re: [PATCH] pci: fix iov.c kernel-doc warnings References: <50EE1594.4070905@infradead.org> <50EF3E65.3030302@redhat.com> In-Reply-To: <50EF3E65.3030302@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-pci-owner@vger.kernel.org List-ID: On 01/10/13 14:19, Don Dutile wrote: > On 01/09/2013 08:12 PM, Randy Dunlap wrote: >> From: Randy Dunlap >> >> Fix kernel-doc warning in iov.c: >> >> Warning(drivers/pci/iov.c:752): No description found for parameter 'numvfs' >> >> Signed-off-by: Randy Dunlap >> --- >> drivers/pci/iov.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> --- lnx-38-rc2.orig/drivers/pci/iov.c >> +++ lnx-38-rc2/drivers/pci/iov.c >> @@ -739,7 +739,7 @@ EXPORT_SYMBOL_GPL(pci_num_vf); >> /** >> * pci_sriov_set_totalvfs -- reduce the TotalVFs available >> * @dev: the PCI PF device >> - * numvfs: number that should be used for TotalVFs supported >> + * @numvfs: number that should be used for TotalVFs supported >> * >> * Should be called from PF driver's probe routine with >> * device's mutex held. >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-pci" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > Thanks for cleanup; /my bad. > Should checkpatch report this issue? if it does, then > my (double-down) bad for missing that correction when I ran it against the patch. checkpatch doesn't check for missing kernel-doc notation. I run 'make htmldocs' on each kernel to look for kernel-doc errors and warnings. -- ~Randy