On Wed, Jul 24, 2013 at 6:45 PM, Sakari Ailus <sakari.ailus@xxxxxx> wrote: [] >> + max_m = clamp_t(u32, max_m, sensor->limits[SMIAPP_LIMIT_SCALER_M_MIN], >> + sensor->limits[SMIAPP_LIMIT_SCALER_M_MAX]); > > Do you need clamp_t()? Wouldn't plain clamp() do? The *_t variants are preferred due to they are faster (no type checking). > I can change it if you're ok with that. I don't know why you may choose clamp instead of clamp_t here. Are you going to change variable types? -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
