[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150219081432.GA21983@waimea>
Date: Thu, 19 Feb 2015 16:14:32 +0800
From: kbuild test robot <fengguang.wu@...el.com>
To: Bryan O'Donoghue <pure.logic@...us-software.ie>
Cc: kbuild-all@...org, Ingo Molnar <mingo@...nel.org>,
Thomas Gleixner <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org,
Andy Shevchenko <andy.schevchenko@...il.com>,
"Ong, Boon Leong" <boon.leong.ong@...el.com>,
Darren Hart <dvhart@...ux.intel.com>,
linux-kernel@...r.kernel.org
Subject: [PATCH] x86/intel/quark: fix ptr_ret.cocci warnings
arch/x86/platform/intel-quark/imr.c:280:1-3: WARNING: PTR_ERR_OR_ZERO can be used
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
Generated by: scripts/coccinelle/api/ptr_ret.cocci
CC: Bryan O'Donoghue <pure.logic@...us-software.ie>
Signed-off-by: Fengguang Wu <fengguang.wu@...el.com>
---
imr.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
--- a/arch/x86/platform/intel-quark/imr.c
+++ b/arch/x86/platform/intel-quark/imr.c
@@ -277,10 +277,7 @@ static int imr_debugfs_register(struct i
{
idev->file = debugfs_create_file("imr_state", S_IFREG | S_IRUGO, NULL,
idev, &imr_state_ops);
- if (IS_ERR(idev->file))
- return PTR_ERR(idev->file);
-
- return 0;
+ return PTR_ERR_OR_ZERO(idev->file);
}
/**
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists