On Saturday 16 December 2006 07:40, Michael Wu wrote: > On Thursday 14 December 2006 13:20, Michael Buesch wrote: > > -int ieee80211_update_hw(struct ieee80211_hw *hw) > > +int ieee80211_register_hwmode(struct ieee80211_hw *hw, > > + struct ieee80211_hw_mode *mode) > Looks like this function never returns nonzero now. Can it return void > instead? This would simplify the fixes for drivers a bit.
Well, theoretically, yes. Maybe it's only my personal way to do things, but I usually let such API register functions return an error code, because if we internally change it only slightly, it often turns out we need an error code. For example let's say we need to kmalloc something in that function for some reason. If you say, well, breaking the API again is ok, when this is required, send a patch. ;) -- Greetings Michael. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
