* [kj]patch7:replace pci_find_device in sound/oss/trident.c
@ 2007-03-06 10:01 Surya
2007-03-06 16:41 ` Alan Cox
0 siblings, 1 reply; 2+ messages in thread
From: Surya @ 2007-03-06 10:01 UTC (permalink / raw)
To: alan, mulix; +Cc: linux-kernel, kernel-janitors
Hi,
Cleanup of pci_find_device to pci_get_device in sound/oss/trident.c
Applies and compiles clean to latest Linus tree. Not tested!
thanks..
Signed-off-by: Surya Prabhakar <surya.prabhakar@wipro.com>
---
diff --git a/sound/oss/trident.c b/sound/oss/trident.c
index 72a8a0e..882ab27 100644
--- a/sound/oss/trident.c
+++ b/sound/oss/trident.c
@@ -4450,8 +4450,8 @@ trident_probe(struct pci_dev *pci_dev, c
/* Add H/W Volume Control By Matt Wu Jul. 06, 2001 */
card->hwvolctl = 0;
- pci_dev_m1533 = pci_find_device(PCI_VENDOR_ID_AL,
- PCI_DEVICE_ID_AL_M1533,
+ pci_dev_m1533 = pci_get_device(PCI_VENDOR_ID_AL,
+ PCI_DEVICE_ID_AL_M1533,
pci_dev_m1533);
rc = -ENODEV;
if (pci_dev_m1533 == NULL)
@@ -4481,6 +4481,7 @@ trident_probe(struct pci_dev *pci_dev, c
/* claim our irq */
rc = -ENODEV;
+ pci_dev_put(pci_dev_m1533);
if (request_irq(card->irq, &trident_interrupt, IRQF_SHARED,
card_names[pci_id->driver_data], card)) {
printk(KERN_ERR "trident: unable to allocate irq %d\n",
--
surya.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [kj]patch7:replace pci_find_device in sound/oss/trident.c
2007-03-06 10:01 [kj]patch7:replace pci_find_device in sound/oss/trident.c Surya
@ 2007-03-06 16:41 ` Alan Cox
0 siblings, 0 replies; 2+ messages in thread
From: Alan Cox @ 2007-03-06 16:41 UTC (permalink / raw)
To: Surya; +Cc: mulix, linux-kernel, kernel-janitors
On Tue, 06 Mar 2007 15:31:27 +0530
Surya <surya.prabhakar@wipro.com> wrote:
> Hi,
> Cleanup of pci_find_device to pci_get_device in sound/oss/trident.c
> Applies and compiles clean to latest Linus tree. Not tested!
Thanks.
I would move the pci_dev_put inside the if as well (the variable is only
used in the ALi section so could itself also be moved), but the fix
appears correct.
Alan
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-03-06 15:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-03-06 10:01 [kj]patch7:replace pci_find_device in sound/oss/trident.c Surya
2007-03-06 16:41 ` Alan Cox
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®