W dniu 17 listopada 2010 21:29 uÅytkownik Michael BÃsch <mb@xxxxxxxxx> napisaÅ: > On Wed, 2010-11-17 at 21:23 +0100, RafaÅ MiÅecki wrote: >> >> Â/* Returns TRUE, if the radio is enabled in hardware. */ >> >> -bool b43_is_hw_radio_enabled(struct b43_wldev *dev) >> >> +inline bool b43_is_hw_radio_enabled(struct b43_wldev *dev) >> > >> > inline doesn't make sense here. >> >> Err, tip for compiler for optimization? To avoid some JUMPs in generated ASM? > > Inline doesn't really work that way. In this case it might generate > an inline version for callers inside of rfkill.c and an > always-out-of-line version for other callers. > If you really want it inline (Which I think isn't really necessary > as this isn't a fastpath), you'll need to make it static inline > and put it into rfkill.h Huh, I got no idea inline works differently for local calls and calls from other files. That's tricky. Thanks, I'll send V2. -- RafaÅ -- 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