On Fri, Feb 24, 2012 at 02:52:45PM +0800, Axel Lin wrote: > This patch includes below fixes: > 1. The mask variable is not used at all here, remove it. > 2. We already have the new_sel and old_sel, simply returns the delay by: > DIV_ROUND_UP(desc->step * (new_sel - old_sel), s5m8767->ramp_delay); Applied, thanks.