diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-29 03:24:23 -0400 |
---|---|---|
committer | Ralf Baechle <ralf@linux-mips.org> | 2014-08-01 18:06:47 -0400 |
commit | c6b7b9f290c0987194b966d5ea1383e10e1a01b1 (patch) | |
tree | 5476f7740b9fa64f7ce4c76d34d4619fa511ed0b | |
parent | 2d6a554dd5128195300f2d2cff36c590b1b88049 (diff) |
MIPS: jz4740: remove unnecessary null test before debugfs_remove
Fix checkpatch warning:
WARNING: debugfs_remove(NULL) is safe this check is probably not required
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: linux-mips@linux-mips.org
Cc: linux-kernel@vger.kernel.org
Patchwork: https://patchwork.linux-mips.org/patch/7225/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
-rw-r--r-- | arch/mips/jz4740/clock-debugfs.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/mips/jz4740/clock-debugfs.c b/arch/mips/jz4740/clock-debugfs.c index a8acdeff267e..325422d0d453 100644 --- a/arch/mips/jz4740/clock-debugfs.c +++ b/arch/mips/jz4740/clock-debugfs.c | |||
@@ -87,8 +87,7 @@ void jz4740_clock_debugfs_add_clk(struct clk *clk) | |||
87 | /* TODO: Locking */ | 87 | /* TODO: Locking */ |
88 | void jz4740_clock_debugfs_update_parent(struct clk *clk) | 88 | void jz4740_clock_debugfs_update_parent(struct clk *clk) |
89 | { | 89 | { |
90 | if (clk->debugfs_parent_entry) | 90 | debugfs_remove(clk->debugfs_parent_entry); |
91 | debugfs_remove(clk->debugfs_parent_entry); | ||
92 | 91 | ||
93 | if (clk->parent) { | 92 | if (clk->parent) { |
94 | char parent_path[100]; | 93 | char parent_path[100]; |