On Thu, 2009-02-12 at 21:35 -0800, Luis R. Rodriguez wrote: > - if (unlikely(drv->wiphy_idx < 0)) { > + if (!wiphy_idx_valid(drv->wiphy_idx)) { > +/* Note 0 is valid, hence phy0 */ > +static inline > +bool wiphy_idx_valid(int wiphy_idx) > +{ > + return (likely(wiphy_idx >= 0)); > +} Does that really work properly with the likely in there? johannes
Attachment:
signature.asc
Description: This is a digitally signed message part