summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2020-03-27 20:26:23 +0000
committerMichael Ellerman <mpe@ellerman.id.au>2020-04-03 00:09:59 +1100
commit96efbab92cccbe3434501e5a77cbaa01c5bc2767 (patch)
tree4b318b29aae9c2b0598e0f3624b88633ca5a7110
parent7ee417497a29028502cf952f419ab2635f563d51 (diff)
downloadlinux-stable-96efbab92cccbe3434501e5a77cbaa01c5bc2767.tar.gz
linux-stable-96efbab92cccbe3434501e5a77cbaa01c5bc2767.tar.bz2
linux-stable-96efbab92cccbe3434501e5a77cbaa01c5bc2767.zip
powerpc/ps3: Remove an unneeded NULL check
Static checkers don't like the inconsistent NULL checking on "ops". This function is only called once and "ops" isn't NULL so the check can be removed. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Geoff Levand <geoff@infradead.org> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/ddc3513dc54d15456692c80df49287fe3babe40a.1585340156.git.geoff@infradead.org
-rw-r--r--drivers/ps3/sys-manager-core.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ps3/sys-manager-core.c b/drivers/ps3/sys-manager-core.c
index 24709c572c0c..e061b7d0632b 100644
--- a/drivers/ps3/sys-manager-core.c
+++ b/drivers/ps3/sys-manager-core.c
@@ -31,7 +31,7 @@ void ps3_sys_manager_register_ops(const struct ps3_sys_manager_ops *ops)
{
BUG_ON(!ops);
BUG_ON(!ops->dev);
- ps3_sys_manager_ops = ops ? *ops : ps3_sys_manager_ops;
+ ps3_sys_manager_ops = *ops;
}
EXPORT_SYMBOL_GPL(ps3_sys_manager_register_ops);