diff options
Diffstat (limited to 'p00')
| -rw-r--r-- | p00 | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/p00 b/p00 new file mode 100644 index 00000000000000..bb502ad76b9076 --- /dev/null +++ b/p00 @@ -0,0 +1,44 @@ +From e82f2d8eab210dc7eb0dba2bd8e3c64ad485442b Mon Sep 17 00:00:00 2001 +From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +Date: Wed, 30 May 2018 16:15:35 +0200 +Subject: [PATCH] microblaze: no need to check return value of debugfs_create functions + +When calling debugfs functions, there is no need to ever check the +return value. The function can work or not, but the code logic should +never do something different based on this. + +Cc: Michal Simek <monstr@monstr.eu> +Cc: Rob Herring <robh@kernel.org> +Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +--- + arch/microblaze/kernel/setup.c | 13 ++----------- + 1 file changed, 2 insertions(+), 11 deletions(-) + +--- a/arch/microblaze/kernel/setup.c ++++ b/arch/microblaze/kernel/setup.c +@@ -192,23 +192,14 @@ struct dentry *of_debugfs_root; + static int microblaze_debugfs_init(void) + { + of_debugfs_root = debugfs_create_dir("microblaze", NULL); +- +- return of_debugfs_root == NULL; ++ return 0; + } + arch_initcall(microblaze_debugfs_init); + + # ifdef CONFIG_MMU + static int __init debugfs_tlb(void) + { +- struct dentry *d; +- +- if (!of_debugfs_root) +- return -ENODEV; +- +- d = debugfs_create_u32("tlb_skip", S_IRUGO, of_debugfs_root, &tlb_skip); +- if (!d) +- return -ENOMEM; +- ++ debugfs_create_u32("tlb_skip", S_IRUGO, of_debugfs_root, &tlb_skip); + return 0; + } + device_initcall(debugfs_tlb); |