On Fri, 2010-09-24 at 15:21 -0700, Ben Greear wrote: > Here's a patch for consideration. It's on top of my other patch, > but I can re-do it against a tree with that reverted if you prefer. You also want to remove the restriction on renaming to phy%d now. > + if (unlikely(!wiphy_idx_valid(rdev->wiphy_idx))) > + goto too_many_devs; > + > /* 64k wiphy devices is enough for anyone! */ > for (i = 0; i < 0xFFFF; i++) { why limit yourself to this? You're guaranteed to find a free index if you don't, so error handling could be easier? johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html