clk: renesas: cpg-mssr: Use genpd of_node instead of local copy
authorGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 27 May 2019 08:55:26 +0000 (10:55 +0200)
committerGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 4 Jun 2019 08:56:49 +0000 (10:56 +0200)
Since commit 6a0ae73d95956f7e ("PM / Domain: Add support to parse
domain's OPP table"), of_genpd_add_provider_simple() fills in
the dev.of_node field in the generic_pm_domain structure.

Hence cpg_mssr_is_pm_clk() can use that instead of its own copy in the
driver-private cpg_mssr_clk_domain structure.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
drivers/clk/renesas/renesas-cpg-mssr.c

index 0201809..d105420 100644 (file)
@@ -447,7 +447,6 @@ fail:
 
 struct cpg_mssr_clk_domain {
        struct generic_pm_domain genpd;
-       struct device_node *np;
        unsigned int num_core_pm_clks;
        unsigned int core_pm_clks[0];
 };
@@ -459,7 +458,7 @@ static bool cpg_mssr_is_pm_clk(const struct of_phandle_args *clkspec,
 {
        unsigned int i;
 
-       if (clkspec->np != pd->np || clkspec->args_count != 2)
+       if (clkspec->np != pd->genpd.dev.of_node || clkspec->args_count != 2)
                return false;
 
        switch (clkspec->args[0]) {
@@ -549,7 +548,6 @@ static int __init cpg_mssr_add_clk_domain(struct device *dev,
        if (!pd)
                return -ENOMEM;
 
-       pd->np = np;
        pd->num_core_pm_clks = num_core_pm_clks;
        memcpy(pd->core_pm_clks, core_pm_clks, pm_size);