Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932149AbbG1IZh (ORCPT ); Tue, 28 Jul 2015 04:25:37 -0400 Received: from smtpfb2-g21.free.fr ([212.27.42.10]:49103 "EHLO smtpfb2-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754956AbbG1IZe convert rfc822-to-8bit (ORCPT ); Tue, 28 Jul 2015 04:25:34 -0400 Date: Tue, 28 Jul 2015 10:25:26 +0200 From: Jean-Francois Moine To: Nicholas Mc Guire Cc: Russell King , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH RFC] drm/i2c: tda998x: dead-code or unhandled error condition ? Message-ID: <20150728102527.528f7d51@armhf> In-Reply-To: <1433941128-19798-1-git-send-email-hofrat@osadl.org> References: <1433941128-19798-1-git-send-email-hofrat@osadl.org> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.28; arm-unknown-linux-gnueabihf) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1098 Lines: 26 On Wed, 10 Jun 2015 14:58:48 +0200 Nicholas Mc Guire wrote: > event API conformance testing with coccinelle spatches are being > used to locate API usage inconsistencies this triggert with: > ./drivers/gpu/drm/i2c/tda998x_drv.c:1062 > incorrect check for negative return > > The return of wait_event_timeout is always >= 0, thus the negative > check was effectively being ignoring - as the timeout will be checked > a few lines below the error condition being checked here seems to be > wrong or this is simply dead code which would be my guess as the > timeout condition check covers the wait-queue related failure condition. > > Signed-off-by: Nicholas Mc Guire Acked-by: Jean-Francois Moine -- Ken ar c'hentaƱ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/