Messages in this thread | | | From | "Liu, Chuansheng" <> | Subject | RE: [PATCH 1/7] I2c-nomadik: Fix the usage of wait_for_completion_timeout | Date | Tue, 13 Nov 2012 01:21:57 +0000 |
| |
> -----Original Message----- > From: Wolfram Sang [mailto:w.sang@pengutronix.de] > Sent: Tuesday, November 13, 2012 1:27 AM > To: Liu, Chuansheng > Cc: linus.walleij@linaro.org; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org; linux-i2c@vger.kernel.org > Subject: Re: [PATCH 1/7] I2c-nomadik: Fix the usage of > wait_for_completion_timeout > > On Wed, Nov 07, 2012 at 01:18:37AM +0800, Chuansheng Liu wrote: > > > > The return value of wait_for_completion_timeout() is always > > >= 0 with unsigned int type. > > > > So the condition "ret < 0" or "ret >= 0" is pointless. > > > > Signed-off-by: liu chuansheng <chuansheng.liu@intel.com> > > You forgot to add the i2c-list, so I nearly overlooked this patch. > Luckily I haven't, so applied to for-next, thanks! Did you use a script > to find all occurences? Would be nice to know. Also, please take care of > the subject line. "I2c" is misspelled. Thanks your pointing out. I just manually reviewed the code when found this point. Would like to try some auto-script but failed:)
| |