From mboxrd@z Thu Jan 1 00:00:00 1970 From: walter harms Date: Mon, 21 Aug 2006 16:47:13 +0000 Subject: Re: [KJ] replace deprecated pci_find_device Message-Id: <44E9E391.4080900@bfs.de> List-Id: References: <200608211744.51043.quinte17@gmx.de> In-Reply-To: <200608211744.51043.quinte17@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kernel-janitors@vger.kernel.org with begin of august Patrik Kullman posted a bunch of patches for replacing pci_find_device->pci_get_device make sure you do not duplicate that work. read the comments posted by others about that subject. re, walter Stephan Sperber wrote: > Hello everybody! > The included patch is my first try, so be critical :) > On the TODO list is mentioned that pci_find_device should be replaced with > pci_get_dev. It also mentions that after shutting down the device a > pci_dev_put(*dev) is needed to free the used memory again. And here is the > problem I have. I don't see a "shutdownfunction" in ip2main.c nor in the > other ip2 related files. Or did I miss something? > > Thank you for any input on this topic. > > Stephan > > diff --git a/drivers/char/ip2/ip2main.c b/drivers/char/ip2/ip2main.c > index 7907ae8..11b4f25 100644 > --- a/drivers/char/ip2/ip2main.c > +++ b/drivers/char/ip2/ip2main.c > @@ -589,7 +589,7 @@ ip2_loadmain(int *iop, int *irqp, unsign > #ifdef CONFIG_PCI > { > struct pci_dev *pci_dev_i = NULL; > - pci_dev_i = pci_find_device(PCI_VENDOR_ID_COMPUTONE, > + pci_dev_i = pci_get_device(PCI_VENDOR_ID_COMPUTONE, > PCI_DEVICE_ID_COMPUTONE_IP2EX, pci_dev_i); > if (pci_dev_i != NULL) { > unsigned int addr; > _______________________________________________ > Kernel-janitors mailing list > Kernel-janitors@lists.osdl.org > https://lists.osdl.org/mailman/listinfo/kernel-janitors > > > _______________________________________________ Kernel-janitors mailing list Kernel-janitors@lists.osdl.org https://lists.osdl.org/mailman/listinfo/kernel-janitors