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 /drivers/ps3 | |
| parent | powerpc/ps3: Remove duplicate error message (diff) | |
| download | linux-dev-96efbab92cccbe3434501e5a77cbaa01c5bc2767.tar.xz linux-dev-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
Diffstat (limited to 'drivers/ps3')
| -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); |