From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161595AbcBQM3T (ORCPT ); Wed, 17 Feb 2016 07:29:19 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:37708 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161387AbcBQM3M (ORCPT ); Wed, 17 Feb 2016 07:29:12 -0500 X-IronPort-AV: E=Sophos;i="5.22,460,1449529200"; d="scan'208";a="165035396" Date: Wed, 17 Feb 2016 07:29:05 -0500 (EST) From: Julia Lawall X-X-Sender: jll@hadrien To: Raveendra Padasalagi cc: kbuild-all@01.org, Ray Jui , Scott Branden , Dmitry Torokhov , Jon Mason , Jonathan Richardson , linux-input@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org Subject: [PATCH] input: fix odd_ptr_err.cocci warnings Message-ID: User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org PTR_ERR should access the value just tested by IS_ERR Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall --- I haven't checked the context in detail. There can be false positives, when it is the call to IS_ERR that is wrong. tree: https://github.com/Broadcom/arm64-linux iproc-tsc-v2 head: ef32cf1baeb934bfcdf7c9eb8ad345aae266cde5 commit: c3afc7a1c330ce452c52e2183a23b2ab9f70bd4a [2/3] input: syscon support in bcm_iproc_tsc driver Please take the patch only if it's a positive warning. Thanks! bcm_iproc_tsc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/input/touchscreen/bcm_iproc_tsc.c +++ b/drivers/input/touchscreen/bcm_iproc_tsc.c @@ -492,7 +492,7 @@ static int iproc_ts_probe(struct platfor pdev->dev.of_node, "ts_syscon"); if (IS_ERR(priv->regmap)) { - error = PTR_ERR(priv->regs); + error = PTR_ERR(priv->regmap); dev_err(&pdev->dev, "unable to map I/O memory:%d\n", error); return error;