All of lore.kernel.org
 help / color / mirror / Atom feed
* [Kernel-janitors] Re: Replacement for pci_for_each_dev()?
@ 2004-08-23 20:25 max attems
  2004-08-23 22:12 ` Greg KH
  2004-08-23 22:58 ` max attems
  0 siblings, 2 replies; 3+ messages in thread
From: max attems @ 2004-08-23 20:25 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 757 bytes --]

hello greg,

On Mon, 23 Aug 2004, Greg KH wrote:

> On Mon, Aug 23, 2004 at 10:04:23AM -0500, Timur Tabi wrote:
> > Greg KH wrote:
> > 
> > >>pci_for_each_dev()              Superseded by pci_find_device()
> > >
> > >No!  Use pci_get_device(), pci_find_device() is about to be deleted.
> > 
> > Looking at pci.txt, it appears that if I use pci_get_device(), I will 
> > also need to use pci_dev_put().
> 
> That is correct.

~/src/linux-2.6.8$ egrep -r 'pci_find_device' drivers/ | wc -l
195

i saw this in your weblog, 
could we add this item to the kernel janitor todo list?
have you any further info or pointer to help any takers?
like a patch you already submitted?

adding kj on cc, thanks


--
maks
kernel janitor  	http://janitor.kernelnewbies.org/


[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Kernel-janitors] Re: Replacement for pci_for_each_dev()?
  2004-08-23 20:25 [Kernel-janitors] Re: Replacement for pci_for_each_dev()? max attems
@ 2004-08-23 22:12 ` Greg KH
  2004-08-23 22:58 ` max attems
  1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2004-08-23 22:12 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 463 bytes --]

On Mon, Aug 23, 2004 at 10:25:07PM +0200, max attems wrote:
> 
> ~/src/linux-2.6.8$ egrep -r 'pci_find_device' drivers/ | wc -l
> 195
> 
> i saw this in your weblog, 
> could we add this item to the kernel janitor todo list?

Sure.

> have you any further info or pointer to help any takers?
> like a patch you already submitted?

I did a bunch of cleanups like it a long time ago (during 2.5), but I
don't have a pointer at this moment, sorry.

tanks,

greg k-h

[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Kernel-janitors] Re: Replacement for pci_for_each_dev()?
  2004-08-23 20:25 [Kernel-janitors] Re: Replacement for pci_for_each_dev()? max attems
  2004-08-23 22:12 ` Greg KH
@ 2004-08-23 22:58 ` max attems
  1 sibling, 0 replies; 3+ messages in thread
From: max attems @ 2004-08-23 22:58 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 686 bytes --]

On Mon, 23 Aug 2004, Greg KH wrote:

> On Mon, Aug 23, 2004 at 10:25:07PM +0200, max attems wrote:
> > 
> > ~/src/linux-2.6.8$ egrep -r 'pci_find_device' drivers/ | wc -l
> > 195
> > 
> > i saw this in your weblog, 
> > could we add this item to the kernel janitor todo list?
> 
> Sure.
> 
> > have you any further info or pointer to help any takers?
> > like a patch you already submitted?
> 
> I did a bunch of cleanups like it a long time ago (during 2.5), but I
> don't have a pointer at this moment, sorry.

added to the kj TODO -> http://janitor.kernelnewbies.org/TODO
will add pointer as soon one shows up. :)
thanks
 
--
maks
kernel janitor  	http://janitor.kernelnewbies.org/


[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2004-08-23 22:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-08-23 20:25 [Kernel-janitors] Re: Replacement for pci_for_each_dev()? max attems
2004-08-23 22:12 ` Greg KH
2004-08-23 22:58 ` max attems

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.