Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965063AbWJCD4x (ORCPT ); Mon, 2 Oct 2006 23:56:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965061AbWJCD4x (ORCPT ); Mon, 2 Oct 2006 23:56:53 -0400 Received: from xenotime.net ([66.160.160.81]:54673 "HELO xenotime.net") by vger.kernel.org with SMTP id S932339AbWJCD4v (ORCPT ); Mon, 2 Oct 2006 23:56:51 -0400 Date: Mon, 2 Oct 2006 20:58:17 -0700 From: Randy Dunlap To: Frederik Deweerdt Cc: Arjan van de Ven , Matthew Wilcox , linux-scsi@vger.kernel.org, "Linux-Kernel," , "J.A. Magall??n" , Alan Cox , Andrew Morton , Jeff Garzik Subject: Re: [RFC PATCH] pci_request_irq (was [-mm patch] aic7xxx: check irq validity) Message-Id: <20061002205817.ddebe747.rdunlap@xenotime.net> In-Reply-To: <20061002200048.GC3003@slug> References: <20060929143949.GL5017@parisc-linux.org> <1159550143.13029.36.camel@localhost.localdomain> <20060929235054.GB2020@slug> <1159573404.13029.96.camel@localhost.localdomain> <20060930140946.GA1195@slug> <451F049A.1010404@garzik.org> <20061001142807.GD16272@parisc-linux.org> <1159729523.2891.408.camel@laptopd505.fenrus.org> <20061001193616.GF16272@parisc-linux.org> <1159755141.2891.434.camel@laptopd505.fenrus.org> <20061002200048.GC3003@slug> Organization: YPO4 X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.8.10; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3099 Lines: 104 On Mon, 2 Oct 2006 20:00:48 +0000 Frederik Deweerdt wrote: > Hi all, > > I've tried to summarize the different proposals made by Jeff Garzik, > Matthew Wilcox and Arjan van de Ven in the "[-mm patch] aic7xxx: check > irq validity" thread. I've also added: > - some kerneldoc The kernel-doc needs some repair -- see below. > - renamed valid_irq to is_irq_valid() > - added pci_release_irq(). > > I'll send a follow-up patch showing the implied modifications for the > following - semi-randomly chosen :) - drivers: aic7xxx, aic79xx, tg3 > and drm. > > Regards, > Frederik > > diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c > index a544997..ae20a3a 100644 > --- a/drivers/pci/pci.c > +++ b/drivers/pci/pci.c > @@ -15,6 +15,7 @@ #include > #include > #include > #include > +#include > #include > #include /* isa_dma_bridge_buggy */ > #include "pci.h" > @@ -810,6 +811,49 @@ err_out: > } > > /** > + * pci_request_irq - Reserve an IRQ for a PCI device > + * @pdev: The PCI device whose irq is to be reserved > + * handler: The interrupt handler function, * @handler: ... > + * pci_get_drvdata(pdev) shall be passed as an argument to that function > + * @flags: The flags to be passed to request_irq() > + * @name: The name of the device to be associated with the irq > + * > + * Returns 0 on success, or a negative value on error. A warning > + * message is also printed on failure. > + */ > +int pci_request_irq(struct pci_dev *pdev, > + irqreturn_t (*handler)(int, void *, struct pt_regs *), > + unsigned long flags, const char *name) > +{ > + int rc; > + const char *actual_name = name; > + > + rc = is_irq_valid(pdev->irq); > + if (!rc) { > + dev_printk(KERN_ERR, &pdev->dev, "invalid irq #%d\n", pdev->irq); > + return -EINVAL; > + } > + > + if (!actual_name) > + actual_name = pci_name(pdev); > + > + return request_irq(pdev->irq, handler, flags | IRQF_SHARED, > + actual_name, pci_get_drvdata(pdev)); > +} > +EXPORT_SYMBOL(pci_request_irq); > + > +/** > + * pci_free_irq - releases the interrupt line reserved to the PCI > + * device pointed by @pdev The first line is function name and <> function description. It cannot extend more than one line (combined). If you want to use more text for function description, you can do so after the list of parameters. See example below. > + * @pdev: the PCI device whose interrupt is to be freed * * This froofroo_irq function only does this on odd phases of * the moon. > + */ > +void pci_free_irq(struct pci_dev *pdev) > +{ > + free_irq(pdev->irq, pci_get_drvdata(pdev)); > +} > +EXPORT_SYMBOL(pci_free_irq); > + > +/** > * pci_set_master - enables bus-mastering for device dev > * @dev: the PCI device to enable > * --- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/