On Thu, Sep 15, 2011 at 10:52:34AM +0800, Axel Lin wrote:> @@ -722,7 +723,7 @@ static void wait_for_dc_servo(struct snd_soc_codec *codec, u16 mask)> > } else {> msleep(1);> - if (--i) {> + if (--i == 0) {You should avoid making random stylistic changes that aren't needed whendoing things like this.