On Thu, 2010-10-28 at 14:19 -0700, Stephen Boyd wrote: > + > +static void (*delay_fn)(unsigned long) = delay_loop; > + > +void set_delay_fn(void (*fn)(unsigned long)) > +{ > + delay_fn = fn; > +} This needs to be a static inline in the header file. > +/* > + * loops = usecs * HZ * loops_per_jiffy / 1000000 > + */ > +void __delay(unsigned long loops) > +{ > + delay_fn(loops); > +} > EXPORT_SYMBOL(__delay); Can we make this static inline also? I'm sure about the module issues.. Daniel -- Sent by a consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html