clk: st: clkgen-mux: search reg within node or parent
authorAlain Volmat <avolmat@me.com>
Sat, 18 Dec 2021 21:11:57 +0000 (22:11 +0100)
committerStephen Boyd <sboyd@kernel.org>
Thu, 6 Jan 2022 01:21:28 +0000 (17:21 -0800)
In order to avoid having duplicated addresses within the DT,
only have one unit-address per clockgen and each driver within
the clockgen should look at the parent node (overall clockgen)
to figure out the reg property.  Such behavior is already in
place in other STi platform clock drivers such as clk-flexgen
and clkgen-pll.  Keep backward compatibility by first looking
at reg within the node before looking into the parent node.

Signed-off-by: Alain Volmat <avolmat@me.com>
Link: https://lore.kernel.org/r/20211218211157.188214-3-avolmat@me.com
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/st/clkgen-mux.c

index ce583ded968a0b810b72169aa48d821da3ad8ecf..ee39af7a0b7215411b607940a5c52a4c989cfaf9 100644 (file)
@@ -57,10 +57,17 @@ static void __init st_of_clkgen_mux_setup(struct device_node *np,
        const char **parents;
        int num_parents = 0;
 
+       /*
+        * First check for reg property within the node to keep backward
+        * compatibility, then if reg doesn't exist look at the parent node
+        */
        reg = of_iomap(np, 0);
        if (!reg) {
-               pr_err("%s: Failed to get base address\n", __func__);
-               return;
+               reg = of_iomap(of_get_parent(np), 0);
+               if (!reg) {
+                       pr_err("%s: Failed to get base address\n", __func__);
+                       return;
+               }
        }
 
        parents = clkgen_mux_get_parents(np, &num_parents);