On 11/01/17 23:11, Linus Walleij wrote: > On Wed, Jan 11, 2017 at 6:49 PM, Colin King <colin.king@xxxxxxxxxxxxx> wrote: > >> From: Colin Ian King <colin.king@xxxxxxxxxxxxx> >> >> At the end of the delay loop timeout will always be zero >> and hence the check for !timeout will always be true. Remove >> the redundant check and the redundant return 0 at the end of >> the function. >> >> Fixes CoverityScan CID#1357168 ("Logically dead code") >> >> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Clever Coverity scan. > Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > Yours, > Linus Walleij > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html