On Fri, May 02, 2008 at 05:19:40PM +0100, Ricardo Martins wrote: > There are 14 drivers in drivers/isdn/hisax/ still using calls to > pci_find_device() instead of the pci_get_device(). From what I gather, a > simple sed 's|pci_find_device|pci_get_device|' is enough to fix this, > since both functions use the same input and give the same output. > Is this correct? It's never correct. If it were that easy (a) there wouldn't be a rename in the first place and (b) it would have been done with a shell-script already. pci_get_device increments a refcount on the pci_dev. pci_find_device does not. Your patch must include calls to pci_dev_put() *when appropriate*. Note, for example, that pci_get_device decrements the refcount on the device passed in. > Also, since there are 14 drivers with only those 1-line changes, should each > driver have its own patch or could I send a single patch covering all of > them? It's not going to be a 1-line patch, so one per driver. -- Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html