diff options
| author | Colin Ian King <colin.king@canonical.com> | 2019-02-19 13:43:33 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-02-20 11:29:01 +0100 |
| commit | eb76b37aaf7e6bdd6638bf5b2ef29688645e03b8 (patch) | |
| tree | a5cb5f0fb0660230cfdd7ebf2d17facf99eda733 | |
| parent | 98bba546e82d7785b9560c6b141b73ac8d3d820f (diff) | |
usb: typec: mux: remove redundant check on variable match
All the code paths that lead to the return statement are where
match is always true, hence the check to see if it is true is
redundant and can be removed.
Detected by CoverityScan, CID#14769672 ("Logically dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/usb/typec/mux.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/typec/mux.c b/drivers/usb/typec/mux.c index 54d74978df9c..2ce54f3fc79c 100644 --- a/drivers/usb/typec/mux.c +++ b/drivers/usb/typec/mux.c @@ -184,7 +184,7 @@ find_mux: if (dev_fwnode(mux->dev) == con->fwnode) return mux; - return match ? ERR_PTR(-EPROBE_DEFER) : NULL; + return ERR_PTR(-EPROBE_DEFER); } /** |