Re: [PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-27 Thread Wolfram Sang
On Tue, Mar 17, 2015 at 03:51:13AM -0400, Nicholas Mc Guire wrote: > wait_for_completion_timeout return 0 (timeout) or >=1 (completion) so the > check > for >= 0 is always true and can be dropped implying that r==-EREMOTEIO and > thus > the return of -EREMOTEIO can be done in the if

Re: [PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-27 Thread Wolfram Sang
On Tue, Mar 17, 2015 at 03:51:13AM -0400, Nicholas Mc Guire wrote: wait_for_completion_timeout return 0 (timeout) or =1 (completion) so the check for = 0 is always true and can be dropped implying that r==-EREMOTEIO and thus the return of -EREMOTEIO can be done in the if (dev-buf_len)

Re: [PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-17 Thread Alexander Sverdlin
Hi! On 17/03/15 08:51, ext Nicholas Mc Guire wrote: > wait_for_completion_timeout return 0 (timeout) or >=1 (completion) so the > check > for >= 0 is always true and can be dropped implying that r==-EREMOTEIO and > thus > the return of -EREMOTEIO can be done in the if (dev->buf_len) branch.

[PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-17 Thread Nicholas Mc Guire
wait_for_completion_timeout return 0 (timeout) or >=1 (completion) so the check for >= 0 is always true and can be dropped implying that r==-EREMOTEIO and thus the return of -EREMOTEIO can be done in the if (dev->buf_len) branch. As wait_for_completion_timeout returns unsigned long not int, and

[PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-17 Thread Nicholas Mc Guire
wait_for_completion_timeout return 0 (timeout) or =1 (completion) so the check for = 0 is always true and can be dropped implying that r==-EREMOTEIO and thus the return of -EREMOTEIO can be done in the if (dev-buf_len) branch. As wait_for_completion_timeout returns unsigned long not int, and

Re: [PATCH] i2c: davinci: fixup wait_for_completion_timeout handling

2015-03-17 Thread Alexander Sverdlin
Hi! On 17/03/15 08:51, ext Nicholas Mc Guire wrote: wait_for_completion_timeout return 0 (timeout) or =1 (completion) so the check for = 0 is always true and can be dropped implying that r==-EREMOTEIO and thus the return of -EREMOTEIO can be done in the if (dev-buf_len) branch. As