clk: meson: only one loop index is necessary in probe

We don't need several loop index variables in the probe function
This is far from being critical but since we are doing a vast
rework of meson clock controllers, now is the time to lower the
entropy a bit

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>

diff --git a/drivers/clk/meson/axg.c b/drivers/clk/meson/axg.c
index e16d53b6be30a9f624dd957b75875fe36f9be397..3bb77b4f1e8d5b0056ea0be555203f1d3f559c9d 100644 (file)
--- a/drivers/clk/meson/axg.c
+++ b/drivers/clk/meson/axg.c
@@ -802,7 +802,7 @@ static int axg_clkc_probe(struct platform_device *pdev)
        const struct clkc_data *clkc_data;
        struct resource *res;
        void __iomem *clk_base;
-       int ret, clkid, i;
+       int ret, i;
 
        clkc_data = of_device_get_match_data(dev);
        if (!clkc_data)
@@ -841,13 +841,13 @@ static int axg_clkc_probe(struct platform_device *pdev)
                clkc_data->clk_dividers[i]->reg = clk_base +
                        (u64)clkc_data->clk_dividers[i]->reg;
 
-       for (clkid = 0; clkid < clkc_data->hw_onecell_data->num; clkid++) {
+       for (i = 0; i < clkc_data->hw_onecell_data->num; i++) {
                /* array might be sparse */
-               if (!clkc_data->hw_onecell_data->hws[clkid])
+               if (!clkc_data->hw_onecell_data->hws[i])
                        continue;
 
                ret = devm_clk_hw_register(dev,
-                                       clkc_data->hw_onecell_data->hws[clkid]);
+                                          clkc_data->hw_onecell_data->hws[i]);
                if (ret) {
                        dev_err(dev, "Clock registration failed\n");
                        return ret;

diff --git a/drivers/clk/meson/gxbb.c b/drivers/clk/meson/gxbb.c
index 17f44ac751b3361c7a77cf23aa2e7578a2dd8ff9..e6adab49c0ba8f0e6105e5c951c2a6382bdc20fb 100644 (file)
--- a/drivers/clk/meson/gxbb.c
+++ b/drivers/clk/meson/gxbb.c
@@ -1947,7 +1947,7 @@ static int gxbb_clkc_probe(struct platform_device *pdev)
 {
        const struct clkc_data *clkc_data;
        void __iomem *clk_base;
-       int ret, clkid, i;
+       int ret, i;
        struct device *dev = &pdev->dev;
 
        clkc_data = of_device_get_match_data(dev);
@@ -1988,16 +1988,15 @@ static int gxbb_clkc_probe(struct platform_device *pdev)
        for (i = 0; i < clkc_data->clk_audio_dividers_count; i++)
                clkc_data->clk_audio_dividers[i]->base = clk_base;
 
-       /*
-        * register all clks
-        */
-       for (clkid = 0; clkid < clkc_data->hw_onecell_data->num; clkid++) {
+
+       /* Register all clks */
+       for (i = 0; i < clkc_data->hw_onecell_data->num; i++) {
                /* array might be sparse */
-               if (!clkc_data->hw_onecell_data->hws[clkid])
+               if (!clkc_data->hw_onecell_data->hws[i])
                        continue;
 
                ret = devm_clk_hw_register(dev,
-                                       clkc_data->hw_onecell_data->hws[clkid]);
+                                          clkc_data->hw_onecell_data->hws[i]);
                if (ret)
                        goto iounmap;
        }

diff --git a/drivers/clk/meson/meson8b.c b/drivers/clk/meson/meson8b.c
index abac079ff77f76e7e8e5e56b6acb769997d02c45..ffadad27375e593c3e7cc774cfc7e0e99b8cac58 100644 (file)
--- a/drivers/clk/meson/meson8b.c
+++ b/drivers/clk/meson/meson8b.c
@@ -806,7 +806,7 @@ static const struct reset_control_ops meson8b_clk_reset_ops = {
 
 static int meson8b_clkc_probe(struct platform_device *pdev)
 {
-       int ret, clkid, i;
+       int ret, i;
        struct clk_hw *parent_hw;
        struct clk *parent_clk;
        struct device *dev = &pdev->dev;
@@ -844,13 +844,13 @@ static int meson8b_clkc_probe(struct platform_device *pdev)
         * register all clks
         * CLKID_UNUSED = 0, so skip it and start with CLKID_XTAL = 1
         */
-       for (clkid = CLKID_XTAL; clkid < CLK_NR_CLKS; clkid++) {
+       for (i = CLKID_XTAL; i < CLK_NR_CLKS; i++) {
                /* array might be sparse */
-               if (!meson8b_hw_onecell_data.hws[clkid])
+               if (!meson8b_hw_onecell_data.hws[i])
                        continue;
 
                /* FIXME convert to devm_clk_register */
-               ret = devm_clk_hw_register(dev, meson8b_hw_onecell_data.hws[clkid]);
+               ret = devm_clk_hw_register(dev, meson8b_hw_onecell_data.hws[i]);
                if (ret)
                        return ret;
        }