[openib-general] Re: Initial ipath review brain dump

[Robert Walsh]

> I came up with the patch below, which lets drivers do something like
> the following:
> 
> 	for (pos = pci_find_capability(pdev, <ID>);
> 	     pos;
> 	     pos = pci_find_next_capability(pdev, pos, <ID>)) {
> 		/* ... */
> 	}
> 
> I think this works well for infinipath_core.c.  What do you think?  If
> it looks OK to you, I'll send it on to Greg K-H for (I hope) inclusion
> in 2.6.15.

Hi Roland,

This looks reasonable enough, but we're a little short on cycles over
here at the moment to fully test it right away.

> [Assuming Linus releases 2.6.14 within the next few days, then the
> 2.6.15 window for core changes will close by the end of next week, so
> it's a good idea to get any generic stuff merged ASAP]

Right.

Regards,
 Robert.

-- 
Robert Walsh                                 Email: rjwalsh at pathscale.com
PathScale, Inc.                              Phone: +1 650 934 8117
2071 Stierlin Court, Suite 200                 Fax: +1 650 428 1969
Mountain View, CA 94043
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 481 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openfabrics.org/pipermail/general/attachments/20051017/42297ef7/attachment.sig>