summaryrefslogtreecommitdiffstats
path: root/arch/mips
diff options
context:
space:
mode:
authorAmitoj Kaur Chawla <amitoj1606@gmail.com>2016-07-29 13:58:46 +0530
committerRalf Baechle <ralf@linux-mips.org>2016-08-01 22:54:48 +0200
commit33799a6d1aeb892862d5f69ee87195becabf8d0c (patch)
treefa54fb209fae92fe9636b28474fa5a3fe2e7c253 /arch/mips
parent11f769039e1d10652bb45e83ecaad18a8681d5e5 (diff)
downloadop-kernel-dev-33799a6d1aeb892862d5f69ee87195becabf8d0c.zip
op-kernel-dev-33799a6d1aeb892862d5f69ee87195becabf8d0c.tar.gz
MIPS: Modify error handling
debugfs_create_file returns NULL on error so an IS_ERR test is incorrect here and a NULL check is required. The Coccinelle semantic patch used to make this change is as follows: @@ expression e; @@ e = debugfs_create_file(...); if( - IS_ERR(e) + !e ) { <+... return - PTR_ERR(e) + -ENOMEM ; ...+> } Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com> Cc: julia.lawall@lip6.fr Cc: linux-mips@linux-mips.org Cc: linux-kernel@vger.kernel.org Patchwork: https://patchwork.linux-mips.org/patch/13834/ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Diffstat (limited to 'arch/mips')
-rw-r--r--arch/mips/mm/sc-debugfs.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/mips/mm/sc-debugfs.c b/arch/mips/mm/sc-debugfs.c
index 5eefe32..01f1154 100644
--- a/arch/mips/mm/sc-debugfs.c
+++ b/arch/mips/mm/sc-debugfs.c
@@ -73,8 +73,8 @@ static int __init sc_debugfs_init(void)
file = debugfs_create_file("prefetch", S_IRUGO | S_IWUSR, dir,
NULL, &sc_prefetch_fops);
- if (IS_ERR(file))
- return PTR_ERR(file);
+ if (!file)
+ return -ENOMEM;
return 0;
}
OpenPOWER on IntegriCloud