summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-09-05 16:03:48 +0100
committerStephen Boyd <sboyd@codeaurora.org>2017-11-13 17:44:15 -0800
commit9416a5f8842a37dc356b7855867d20db6dfa9e3a (patch)
treeef0f903cf3f9aaecb6c470e3110a1edc6ba61506 /drivers
parenta72da43c92dc09488ac2944b103de037c435302a (diff)
downloadlinux-9416a5f8842a37dc356b7855867d20db6dfa9e3a.tar.bz2
clk: cdce925: remove redundant check for non-null parent_name
The check for a non-null parent_name is always true as there is an earlier check for a null parent_name that returns -ENODEV. Remove this redundant check and always set init.num_parents to 1. Detected by CoverityScan CID#1309477 ("Logically dead code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/clk/clk-cdce925.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/clk-cdce925.c b/drivers/clk/clk-cdce925.c
index c933be01c7db..0a7e7d5a7506 100644
--- a/drivers/clk/clk-cdce925.c
+++ b/drivers/clk/clk-cdce925.c
@@ -665,7 +665,7 @@ static int cdce925_probe(struct i2c_client *client,
init.ops = &cdce925_pll_ops;
init.flags = 0;
init.parent_names = &parent_name;
- init.num_parents = parent_name ? 1 : 0;
+ init.num_parents = 1;
/* Register PLL clocks */
for (i = 0; i < data->chip_info->num_plls; ++i) {