mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [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®