diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2020-03-27 20:26:23 +0000 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2020-04-03 00:09:59 +1100 |
| commit | 96efbab92cccbe3434501e5a77cbaa01c5bc2767 (patch) | |
| tree | 4b318b29aae9c2b0598e0f3624b88633ca5a7110 | |
| parent | 7ee417497a29028502cf952f419ab2635f563d51 (diff) | |
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.c | 2 |
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); |