On Wed, Jan 28, 2015 at 12:22:51PM -0800, Marcel Holtmann wrote: > I am still curious why you specified it like this. It would expect something like this: > > usleep_range(TIMEGAP_MIN, TIMEGAP_MAX); > > Btw. _US seems a bit to unclear if that is suppose to represent usec > or something chip internal called US. If you want to keep it in there, > then I prefer _USEC so I do not have to second guess every time I look > at the code. > > Otherwise this is fine. Just use two constants and not some magic > range calculation. > > Regards > > Marcel Good point, will resend, thanks. -- Adam Lee -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
