Commit fd1ba2965244 ("drivers: clk: Fix using assigned-clocks in the
node of the clock it sets up"), first included in v2020.01-rc2, added
a clock-uclass post_probe() hook that applies clock controllers own
assigned clocks.

Remove the redundant clk_set_defaults() call from the RK3308 clock
driver probe() ops.

The clock-uclass post_probe() hook does not treat errors as a hard
failure, so this introduce an acceptable change in behavior.

Fixes: fe9efbca7b99 ("rockchip: clk: Add clk driver for rk3308")
Signed-off-by: Jonas Karlman <[email protected]>
---
 drivers/clk/rockchip/clk_rk3308.c | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/drivers/clk/rockchip/clk_rk3308.c 
b/drivers/clk/rockchip/clk_rk3308.c
index 97043b8693c0..605566f71981 100644
--- a/drivers/clk/rockchip/clk_rk3308.c
+++ b/drivers/clk/rockchip/clk_rk3308.c
@@ -1162,16 +1162,9 @@ static void rk3308_clk_init(struct udevice *dev)
 
 static int rk3308_clk_probe(struct udevice *dev)
 {
-       int ret;
-
        rk3308_clk_init(dev);
 
-       /* Process 'assigned-{clocks/clock-parents/clock-rates}' properties */
-       ret = clk_set_defaults(dev, CLK_DEFAULTS_POST);
-       if (ret)
-               debug("%s clk_set_defaults failed %d\n", __func__, ret);
-
-       return ret;
+       return 0;
 }
 
 static int rk3308_clk_of_to_plat(struct udevice *dev)
-- 
2.55.0

Reply via email to