summaryrefslogtreecommitdiff
path: root/drivers/usb
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2019-02-19 13:43:33 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2019-02-20 11:29:01 +0100
commiteb76b37aaf7e6bdd6638bf5b2ef29688645e03b8 (patch)
treea5cb5f0fb0660230cfdd7ebf2d17facf99eda733 /drivers/usb
parent98bba546e82d7785b9560c6b141b73ac8d3d820f (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>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/typec/mux.c2
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);
}
/**