On Thu, 2006-11-09 at 09:01 +0100, Segher Boessenkool wrote:
> > +int pci_find_next_ht_capability(struct pci_dev *dev, int pos, int  
> > ht_cap)
> > +{
> > +   int rc;
> > +   u8 cap, mask;
> > +
> > +   if (ht_cap == HT_CAPTYPE_SLAVE || ht_cap == HT_CAPTYPE_HOST)
> > +           mask = HT_3BIT_CAP_MASK;
> > +   else
> > +           mask = HT_5BIT_CAP_MASK;
> > +
> 
> +     pos = pci_find_next_capability(dev, pos, PCI_CAP_ID_HT);
> 
> or the caller will loop forever if a second same type HT cap is found.

Er .. duh. Memo, don't send code just before leaving for the weekend :)

Putting that back in is going to break pci_find_ht_capability(), so I'll
have to rethink it.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to