[PATCH 3/4] clk: vt8500: Use of_init_clk_data()

[Stephen Boyd]

Reduce lines of code and simplify this driver by using the
generic clock binding parsing function.

Signed-off-by: Stephen Boyd <sb...@codeaurora.org>
Cc: Tony Prisk <li...@prisktech.co.nz>
---
 drivers/clk/clk-vt8500.c | 39 +++++++++++++++------------------------
 1 file changed, 15 insertions(+), 24 deletions(-)

diff --git a/drivers/clk/clk-vt8500.c b/drivers/clk/clk-vt8500.c
index fe25570..454efa7 100644
--- a/drivers/clk/clk-vt8500.c
+++ b/drivers/clk/clk-vt8500.c
@@ -200,8 +200,6 @@ static __init void vtwm_device_clk_init(struct device_node 
*node)
        u32 en_reg, div_reg;
        struct clk *clk;
        struct clk_device *dev_clk;
-       const char *clk_name = node->name;
-       const char *parent_name;
        struct clk_init_data init;
        int rc;
        int clk_init_flags = 0;
@@ -237,8 +235,6 @@ static __init void vtwm_device_clk_init(struct device_node 
*node)
                clk_init_flags |= CLK_INIT_DIVISOR;
        }
 
-       of_property_read_string(node, "clock-output-names", &clk_name);
-
        switch (clk_init_flags) {
        case CLK_INIT_GATED:
                init.ops = &vt8500_gated_clk_ops;
@@ -256,11 +252,11 @@ static __init void vtwm_device_clk_init(struct 
device_node *node)
                return;
        }
 
-       init.name = clk_name;
-       init.flags = 0;
-       parent_name = of_clk_get_parent_name(node, 0);
-       init.parent_names = &parent_name;
-       init.num_parents = 1;
+       rc = of_init_clk_data(node, &init);
+       if (WARN_ON(rc)) {
+               kfree(dev_clk);
+               return;
+       }
 
        dev_clk->hw.init = &init;
 
@@ -270,7 +266,7 @@ static __init void vtwm_device_clk_init(struct device_node 
*node)
                return;
        }
        rc = of_clk_add_provider(node, of_clk_src_simple_get, clk);
-       clk_register_clkdev(clk, clk_name, NULL);
+       clk_register_clkdev(clk, init.name, NULL);
 }
 
 
@@ -458,8 +454,6 @@ static __init void vtwm_pll_clk_init(struct device_node 
*node, int pll_type)
        u32 reg;
        struct clk *clk;
        struct clk_pll *pll_clk;
-       const char *clk_name = node->name;
-       const char *parent_name;
        struct clk_init_data init;
        int rc;
 
@@ -475,24 +469,21 @@ static __init void vtwm_pll_clk_init(struct device_node 
*node, int pll_type)
        pll_clk->lock = &_lock;
        pll_clk->type = pll_type;
 
-       of_property_read_string(node, "clock-output-names", &clk_name);
-
-       init.name = clk_name;
        init.ops = &vtwm_pll_ops;
-       init.flags = 0;
-       parent_name = of_clk_get_parent_name(node, 0);
-       init.parent_names = &parent_name;
-       init.num_parents = 1;
+       rc = of_init_clk_data(node, &init);
+       if (WARN_ON(rc))
+               goto err;
 
        pll_clk->hw.init = &init;
 
        clk = clk_register(NULL, &pll_clk->hw);
-       if (WARN_ON(IS_ERR(clk))) {
-               kfree(pll_clk);
-               return;
-       }
+       if (WARN_ON(IS_ERR(clk)))
+               goto err;
        rc = of_clk_add_provider(node, of_clk_src_simple_get, clk);
-       clk_register_clkdev(clk, clk_name, NULL);
+       clk_register_clkdev(clk, init.name, NULL);
+       return;
+err:
+       kfree(pll_clk);
 }
 
 
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/