diff options
| author | Fengguang Wu <fengguang.wu@intel.com> | 2015-02-19 03:14:32 -0500 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2015-02-19 04:00:54 -0500 |
| commit | 32d39169d7f56849b8c6c8c51aca7b73194d05f1 (patch) | |
| tree | 12fea3ed04fe8c54ce531c2083e5a159fdfe6c67 | |
| parent | 8bbc2a135b63bee6b41fa90f415521a97995a49f (diff) | |
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
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Cc: Andy Shevchenko <andy.schevchenko@gmail.com>
Cc: Ong, Boon Leong <boon.leong.ong@intel.com>
Cc: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Cc: Darren Hart <dvhart@linux.intel.com>
Cc: kbuild-all@01.org
Link: http://lkml.kernel.org/r/20150219081432.GA21983@waimea
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | arch/x86/platform/intel-quark/imr.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/arch/x86/platform/intel-quark/imr.c b/arch/x86/platform/intel-quark/imr.c index 16e4df1c9290..60c01eb47dd6 100644 --- 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 imr_device *idev) | |||
| 277 | { | 277 | { |
| 278 | idev->file = debugfs_create_file("imr_state", S_IFREG | S_IRUGO, NULL, | 278 | idev->file = debugfs_create_file("imr_state", S_IFREG | S_IRUGO, NULL, |
| 279 | idev, &imr_state_ops); | 279 | idev, &imr_state_ops); |
| 280 | if (IS_ERR(idev->file)) | 280 | return PTR_ERR_OR_ZERO(idev->file); |
| 281 | return PTR_ERR(idev->file); | ||
| 282 | |||
| 283 | return 0; | ||
| 284 | } | 281 | } |
| 285 | 282 | ||
| 286 | /** | 283 | /** |
