Re: [KJ] replace deprecated pci_find_device

[trem <tremyfr@yahoo.fr>]

Hi

I see a function named "cleanup_module" with some code to free pci board.
I think that 'pci_dev_put' could be added there. Be carreful, it's just
a proposition.

By the way, last time I've tried to do such change (pci_find_device ->
pci_get_device),
gkh answers me to use new PCI driver model (pci_register_driver).  I
don't kown if
it should be done here too.


trem





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