clk: Change clk_ops->determine_rate to return a clk_hw as the best parent

This is in preparation for clock providers to not have to deal with struct clk.

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Michael Turquette <mturquette@linaro.org>

14 files changed, 65 insertions, 49 deletions

diff --git a/Documentation/clk.txt b/Documentation/clk.txt
index 1fee72f4d331..4ff84623d5e1 100644
--- a/Documentation/clk.txt
+++ b/Documentation/clk.txt
@@ -74,7 +74,7 @@ the operations defined in clk.h:
                 long            (*determine_rate)(struct clk_hw *hw,
                                                 unsigned long rate,
                                                 unsigned long *best_parent_rate,
-                                                struct clk **best_parent_clk);
+                                                struct clk_hw **best_parent_clk);
                 int             (*set_parent)(struct clk_hw *hw, u8 index);
                 u8              (*get_parent)(struct clk_hw *hw);
                 int             (*set_rate)(struct clk_hw *hw,
diff --git a/arch/mips/alchemy/common/clock.c b/arch/mips/alchemy/common/clock.c
index d7557cde271a..f42dd0a4af20 100644
--- a/arch/mips/alchemy/common/clock.c
+++ b/arch/mips/alchemy/common/clock.c
@@ -375,7 +375,7 @@ static long alchemy_calc_div(unsigned long rate, unsigned long prate,
 
 static long alchemy_clk_fgcs_detr(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk,
+                                        struct clk_hw **best_parent_clk,
                                         int scale, int maxdiv)
 {
         struct clk *pc, *bpc, *free;
@@ -454,7 +454,7 @@ static long alchemy_clk_fgcs_detr(struct clk_hw *hw, unsigned long rate,
         }
 
         *best_parent_rate = bpr;
-        *best_parent_clk = bpc;
+        *best_parent_clk = __clk_get_hw(bpc);
         return br;
 }
 
@@ -548,7 +548,7 @@ static unsigned long alchemy_clk_fgv1_recalc(struct clk_hw *hw,
 
 static long alchemy_clk_fgv1_detr(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk)
+                                        struct clk_hw **best_parent_clk)
 {
         return alchemy_clk_fgcs_detr(hw, rate, best_parent_rate,
                                      best_parent_clk, 2, 512);
@@ -680,7 +680,7 @@ static unsigned long alchemy_clk_fgv2_recalc(struct clk_hw *hw,
 
 static long alchemy_clk_fgv2_detr(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk)
+                                        struct clk_hw **best_parent_clk)
 {
         struct alchemy_fgcs_clk *c = to_fgcs_clk(hw);
         int scale, maxdiv;
@@ -899,7 +899,7 @@ static int alchemy_clk_csrc_setr(struct clk_hw *hw, unsigned long rate,
 
 static long alchemy_clk_csrc_detr(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk)
+                                        struct clk_hw **best_parent_clk)
 {
         struct alchemy_fgcs_clk *c = to_fgcs_clk(hw);
         int scale = c->dt[2] == 3 ? 1 : 2; /* au1300 check */
diff --git a/drivers/clk/at91/clk-programmable.c b/drivers/clk/at91/clk-programmable.c
index 62e2509f9df1..bbdb1b985c91 100644
--- a/drivers/clk/at91/clk-programmable.c
+++ b/drivers/clk/at91/clk-programmable.c
@@ -57,7 +57,7 @@ static unsigned long clk_programmable_recalc_rate(struct clk_hw *hw,
 static long clk_programmable_determine_rate(struct clk_hw *hw,
                                             unsigned long rate,
                                             unsigned long *best_parent_rate,
-                                            struct clk **best_parent_clk)
+                                            struct clk_hw **best_parent_hw)
 {
         struct clk *parent = NULL;
         long best_rate = -EINVAL;
@@ -84,7 +84,7 @@ static long clk_programmable_determine_rate(struct clk_hw *hw,
                 if (best_rate < 0 || (rate - tmp_rate) < (rate - best_rate)) {
                         best_rate = tmp_rate;
                         *best_parent_rate = parent_rate;
-                        *best_parent_clk = parent;
+                        *best_parent_hw = __clk_get_hw(parent);
                 }
 
                 if (!best_rate)
diff --git a/drivers/clk/bcm/clk-kona.c b/drivers/clk/bcm/clk-kona.c
index 95af2e665dd3..1c06f6f3a8c5 100644
--- a/drivers/clk/bcm/clk-kona.c
+++ b/drivers/clk/bcm/clk-kona.c
@@ -1032,7 +1032,7 @@ static long kona_peri_clk_round_rate(struct clk_hw *hw, unsigned long rate,
 }
 
 static long kona_peri_clk_determine_rate(struct clk_hw *hw, unsigned long rate,
-                unsigned long *best_parent_rate, struct clk **best_parent)
+                unsigned long *best_parent_rate, struct clk_hw **best_parent)
 {
         struct kona_clk *bcm_clk = to_kona_clk(hw);
         struct clk *clk = hw->clk;
@@ -1075,7 +1075,7 @@ static long kona_peri_clk_determine_rate(struct clk_hw *hw, unsigned long rate,
                 if (delta < best_delta) {
                         best_delta = delta;
                         best_rate = other_rate;
-                        *best_parent = parent;
+                        *best_parent = __clk_get_hw(parent);
                         *best_parent_rate = parent_rate;
                 }
         }
diff --git a/drivers/clk/clk-composite.c b/drivers/clk/clk-composite.c
index b9355daf8065..4386697236a7 100644
--- a/drivers/clk/clk-composite.c
+++ b/drivers/clk/clk-composite.c
@@ -57,7 +57,7 @@ static unsigned long clk_composite_recalc_rate(struct clk_hw *hw,
 
 static long clk_composite_determine_rate(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_p)
+                                        struct clk_hw **best_parent_p)
 {
         struct clk_composite *composite = to_clk_composite(hw);
         const struct clk_ops *rate_ops = composite->rate_ops;
@@ -80,8 +80,9 @@ static long clk_composite_determine_rate(struct clk_hw *hw, unsigned long rate,
                 *best_parent_p = NULL;
 
                 if (__clk_get_flags(hw->clk) & CLK_SET_RATE_NO_REPARENT) {
-                        *best_parent_p = clk_get_parent(mux_hw->clk);
-                        *best_parent_rate = __clk_get_rate(*best_parent_p);
+                        parent = clk_get_parent(mux_hw->clk);
+                        *best_parent_p = __clk_get_hw(parent);
+                        *best_parent_rate = __clk_get_rate(parent);
 
                         return rate_ops->round_rate(rate_hw, rate,
                                                     best_parent_rate);
@@ -103,7 +104,7 @@ static long clk_composite_determine_rate(struct clk_hw *hw, unsigned long rate,
 
                         if (!rate_diff || !*best_parent_p
                                        || best_rate_diff > rate_diff) {
-                                *best_parent_p = parent;
+                                *best_parent_p = __clk_get_hw(parent);
                                 *best_parent_rate = parent_rate;
                                 best_rate_diff = rate_diff;
                                 best_rate = tmp_rate;
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index f549e8b1d5ed..44cdc47a6cc5 100644
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -702,7 +702,7 @@ struct clk *__clk_lookup(const char *name)
  */
 long __clk_mux_determine_rate(struct clk_hw *hw, unsigned long rate,
                               unsigned long *best_parent_rate,
-                              struct clk **best_parent_p)
+                              struct clk_hw **best_parent_p)
 {
         struct clk *clk = hw->clk, *parent, *best_parent = NULL;
         int i, num_parents;
@@ -738,7 +738,7 @@ long __clk_mux_determine_rate(struct clk_hw *hw, unsigned long rate,
 
 out:
         if (best_parent)
-                *best_parent_p = best_parent;
+                *best_parent_p = best_parent->hw;
         *best_parent_rate = best;
 
         return best;
@@ -946,6 +946,7 @@ unsigned long __clk_round_rate(struct clk *clk, unsigned long rate)
 {
         unsigned long parent_rate = 0;
         struct clk *parent;
+        struct clk_hw *parent_hw;
 
         if (!clk)
                 return 0;
@@ -954,10 +955,11 @@ unsigned long __clk_round_rate(struct clk *clk, unsigned long rate)
         if (parent)
                 parent_rate = parent->rate;
 
-        if (clk->ops->determine_rate)
+        if (clk->ops->determine_rate) {
+                parent_hw = parent ? parent->hw : NULL;
                 return clk->ops->determine_rate(clk->hw, rate, &parent_rate,
-                                                &parent);
-        else if (clk->ops->round_rate)
+                                                &parent_hw);
+        } else if (clk->ops->round_rate)
                 return clk->ops->round_rate(clk->hw, rate, &parent_rate);
         else if (clk->flags & CLK_SET_RATE_PARENT)
                 return __clk_round_rate(clk->parent, rate);
@@ -1345,6 +1347,7 @@ static struct clk *clk_calc_new_rates(struct clk *clk, unsigned long rate)
 {
         struct clk *top = clk;
         struct clk *old_parent, *parent;
+        struct clk_hw *parent_hw;
         unsigned long best_parent_rate = 0;
         unsigned long new_rate;
         int p_index = 0;
@@ -1360,9 +1363,11 @@ static struct clk *clk_calc_new_rates(struct clk *clk, unsigned long rate)
 
         /* find the closest rate and parent clk/rate */
         if (clk->ops->determine_rate) {
+                parent_hw = parent ? parent->hw : NULL;
                 new_rate = clk->ops->determine_rate(clk->hw, rate,
                                                     &best_parent_rate,
-                                                    &parent);
+                                                    &parent_hw);
+                parent = parent_hw->clk;
         } else if (clk->ops->round_rate) {
                 new_rate = clk->ops->round_rate(clk->hw, rate,
                                                 &best_parent_rate);
diff --git a/drivers/clk/hisilicon/clk-hi3620.c b/drivers/clk/hisilicon/clk-hi3620.c
index 640ea3327c3e..007144f81f50 100644
--- a/drivers/clk/hisilicon/clk-hi3620.c
+++ b/drivers/clk/hisilicon/clk-hi3620.c
@@ -296,7 +296,7 @@ static unsigned long mmc_clk_recalc_rate(struct clk_hw *hw,
 
 static long mmc_clk_determine_rate(struct clk_hw *hw, unsigned long rate,
                               unsigned long *best_parent_rate,
-                              struct clk **best_parent_p)
+                              struct clk_hw **best_parent_p)
 {
         struct clk_mmc *mclk = to_mmc(hw);
         unsigned long best = 0;
diff --git a/drivers/clk/mmp/clk-mix.c b/drivers/clk/mmp/clk-mix.c
index b79742c47d53..48fa53c7ce5e 100644
--- a/drivers/clk/mmp/clk-mix.c
+++ b/drivers/clk/mmp/clk-mix.c
@@ -203,7 +203,7 @@ error:
 
 static long mmp_clk_mix_determine_rate(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk)
+                                        struct clk_hw **best_parent_clk)
 {
         struct mmp_clk_mix *mix = to_clk_mix(hw);
         struct mmp_clk_mix_clk_table *item;
@@ -264,7 +264,7 @@ static long mmp_clk_mix_determine_rate(struct clk_hw *hw, unsigned long rate,
 
 found:
         *best_parent_rate = parent_rate_best;
-        *best_parent_clk = parent_best;
+        *best_parent_clk = __clk_get_hw(parent_best);
 
         return mix_rate_best;
 }
diff --git a/drivers/clk/qcom/clk-pll.c b/drivers/clk/qcom/clk-pll.c
index b823bc3b6250..60873a7f45d9 100644
--- a/drivers/clk/qcom/clk-pll.c
+++ b/drivers/clk/qcom/clk-pll.c
@@ -141,7 +141,7 @@ struct pll_freq_tbl *find_freq(const struct pll_freq_tbl *f, unsigned long rate)
 
 static long
 clk_pll_determine_rate(struct clk_hw *hw, unsigned long rate,
-                       unsigned long *p_rate, struct clk **p)
+                       unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_pll *pll = to_clk_pll(hw);
         const struct pll_freq_tbl *f;
diff --git a/drivers/clk/qcom/clk-rcg.c b/drivers/clk/qcom/clk-rcg.c
index b6e6959e89aa..0b93972c8807 100644
--- a/drivers/clk/qcom/clk-rcg.c
+++ b/drivers/clk/qcom/clk-rcg.c
@@ -368,16 +368,17 @@ clk_dyn_rcg_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)
 
 static long _freq_tbl_determine_rate(struct clk_hw *hw,
                 const struct freq_tbl *f, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p_hw)
 {
         unsigned long clk_flags;
+        struct clk *p;
 
         f = qcom_find_freq(f, rate);
         if (!f)
                 return -EINVAL;
 
         clk_flags = __clk_get_flags(hw->clk);
-        *p = clk_get_parent_by_index(hw->clk, f->src);
+        p = clk_get_parent_by_index(hw->clk, f->src);
         if (clk_flags & CLK_SET_RATE_PARENT) {
                 rate = rate * f->pre_div;
                 if (f->n) {
@@ -387,15 +388,16 @@ static long _freq_tbl_determine_rate(struct clk_hw *hw,
                         rate = tmp;
                 }
         } else {
-                rate =  __clk_get_rate(*p);
+                rate =  __clk_get_rate(p);
         }
+        *p_hw = __clk_get_hw(p);
         *p_rate = rate;
 
         return f->freq;
 }
 
 static long clk_rcg_determine_rate(struct clk_hw *hw, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_rcg *rcg = to_clk_rcg(hw);
 
@@ -403,7 +405,7 @@ static long clk_rcg_determine_rate(struct clk_hw *hw, unsigned long rate,
 }
 
 static long clk_dyn_rcg_determine_rate(struct clk_hw *hw, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_dyn_rcg *rcg = to_clk_dyn_rcg(hw);
 
@@ -411,13 +413,15 @@ static long clk_dyn_rcg_determine_rate(struct clk_hw *hw, unsigned long rate,
 }
 
 static long clk_rcg_bypass_determine_rate(struct clk_hw *hw, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p_hw)
 {
         struct clk_rcg *rcg = to_clk_rcg(hw);
         const struct freq_tbl *f = rcg->freq_tbl;
+        struct clk *p;
 
-        *p = clk_get_parent_by_index(hw->clk, f->src);
-        *p_rate = __clk_round_rate(*p, rate);
+        p = clk_get_parent_by_index(hw->clk, f->src);
+        *p_hw = __clk_get_hw(p);
+        *p_rate = __clk_round_rate(p, rate);
 
         return *p_rate;
 }
diff --git a/drivers/clk/qcom/clk-rcg2.c b/drivers/clk/qcom/clk-rcg2.c
index cfa9eb4fe9ca..08b8b3729f53 100644
--- a/drivers/clk/qcom/clk-rcg2.c
+++ b/drivers/clk/qcom/clk-rcg2.c
@@ -175,16 +175,17 @@ clk_rcg2_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)
 
 static long _freq_tbl_determine_rate(struct clk_hw *hw,
                 const struct freq_tbl *f, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p_hw)
 {
         unsigned long clk_flags;
+        struct clk *p;
 
         f = qcom_find_freq(f, rate);
         if (!f)
                 return -EINVAL;
 
         clk_flags = __clk_get_flags(hw->clk);
-        *p = clk_get_parent_by_index(hw->clk, f->src);
+        p = clk_get_parent_by_index(hw->clk, f->src);
         if (clk_flags & CLK_SET_RATE_PARENT) {
                 if (f->pre_div) {
                         rate /= 2;
@@ -198,15 +199,16 @@ static long _freq_tbl_determine_rate(struct clk_hw *hw,
                         rate = tmp;
                 }
         } else {
-                rate =  __clk_get_rate(*p);
+                rate =  __clk_get_rate(p);
         }
+        *p_hw = __clk_get_hw(p);
         *p_rate = rate;
 
         return f->freq;
 }
 
 static long clk_rcg2_determine_rate(struct clk_hw *hw, unsigned long rate,
-                unsigned long *p_rate, struct clk **p)
+                unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_rcg2 *rcg = to_clk_rcg2(hw);
 
@@ -359,7 +361,7 @@ static int clk_edp_pixel_set_rate_and_parent(struct clk_hw *hw,
 }
 
 static long clk_edp_pixel_determine_rate(struct clk_hw *hw, unsigned long rate,
-                                 unsigned long *p_rate, struct clk **p)
+                                 unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_rcg2 *rcg = to_clk_rcg2(hw);
         const struct freq_tbl *f = rcg->freq_tbl;
@@ -371,7 +373,7 @@ static long clk_edp_pixel_determine_rate(struct clk_hw *hw, unsigned long rate,
         u32 hid_div;
 
         /* Force the correct parent */
-        *p = clk_get_parent_by_index(hw->clk, f->src);
+        *p = __clk_get_hw(clk_get_parent_by_index(hw->clk, f->src));
 
         if (src_rate == 810000000)
                 frac = frac_table_810m;
@@ -410,18 +412,20 @@ const struct clk_ops clk_edp_pixel_ops = {
 EXPORT_SYMBOL_GPL(clk_edp_pixel_ops);
 
 static long clk_byte_determine_rate(struct clk_hw *hw, unsigned long rate,
-                         unsigned long *p_rate, struct clk **p)
+                         unsigned long *p_rate, struct clk_hw **p_hw)
 {
         struct clk_rcg2 *rcg = to_clk_rcg2(hw);
         const struct freq_tbl *f = rcg->freq_tbl;
         unsigned long parent_rate, div;
         u32 mask = BIT(rcg->hid_width) - 1;
+        struct clk *p;
 
         if (rate == 0)
                 return -EINVAL;
 
-        *p = clk_get_parent_by_index(hw->clk, f->src);
-        *p_rate = parent_rate = __clk_round_rate(*p, rate);
+        p = clk_get_parent_by_index(hw->clk, f->src);
+        *p_hw = __clk_get_hw(p);
+        *p_rate = parent_rate = __clk_round_rate(p, rate);
 
         div = DIV_ROUND_UP((2 * parent_rate), rate) - 1;
         div = min_t(u32, div, mask);
@@ -472,14 +476,16 @@ static const struct frac_entry frac_table_pixel[] = {
 };
 
 static long clk_pixel_determine_rate(struct clk_hw *hw, unsigned long rate,
-                                 unsigned long *p_rate, struct clk **p)
+                                 unsigned long *p_rate, struct clk_hw **p)
 {
         struct clk_rcg2 *rcg = to_clk_rcg2(hw);
         unsigned long request, src_rate;
         int delta = 100000;
         const struct freq_tbl *f = rcg->freq_tbl;
         const struct frac_entry *frac = frac_table_pixel;
-        struct clk *parent = *p = clk_get_parent_by_index(hw->clk, f->src);
+        struct clk *parent = clk_get_parent_by_index(hw->clk, f->src);
+
+        *p = __clk_get_hw(parent);
 
         for (; frac->num; frac++) {
                 request = (rate * frac->den) / frac->num;
diff --git a/drivers/clk/sunxi/clk-factors.c b/drivers/clk/sunxi/clk-factors.c
index 5521e866fa5e..62e08fb58554 100644
--- a/drivers/clk/sunxi/clk-factors.c
+++ b/drivers/clk/sunxi/clk-factors.c
@@ -81,7 +81,7 @@ static long clk_factors_round_rate(struct clk_hw *hw, unsigned long rate,
 
 static long clk_factors_determine_rate(struct clk_hw *hw, unsigned long rate,
                                        unsigned long *best_parent_rate,
-                                       struct clk **best_parent_p)
+                                       struct clk_hw **best_parent_p)
 {
         struct clk *clk = hw->clk, *parent, *best_parent = NULL;
         int i, num_parents;
@@ -108,7 +108,7 @@ static long clk_factors_determine_rate(struct clk_hw *hw, unsigned long rate,
         }
 
         if (best_parent)
-                *best_parent_p = best_parent;
+                *best_parent_p = __clk_get_hw(best_parent);
         *best_parent_rate = best;
 
         return best_child_rate;
diff --git a/drivers/clk/sunxi/clk-sun6i-ar100.c b/drivers/clk/sunxi/clk-sun6i-ar100.c
index acca53290be2..3d282fb8f85c 100644
--- a/drivers/clk/sunxi/clk-sun6i-ar100.c
+++ b/drivers/clk/sunxi/clk-sun6i-ar100.c
@@ -46,7 +46,7 @@ static unsigned long ar100_recalc_rate(struct clk_hw *hw,
 
 static long ar100_determine_rate(struct clk_hw *hw, unsigned long rate,
                                  unsigned long *best_parent_rate,
-                                 struct clk **best_parent_clk)
+                                 struct clk_hw **best_parent_clk)
 {
         int nparents = __clk_get_num_parents(hw->clk);
         long best_rate = -EINVAL;
@@ -100,7 +100,7 @@ static long ar100_determine_rate(struct clk_hw *hw, unsigned long rate,
 
                 tmp_rate = (parent_rate >> shift) / div;
                 if (!*best_parent_clk || tmp_rate > best_rate) {
-                        *best_parent_clk = parent;
+                        *best_parent_clk = __clk_get_hw(parent);
                         *best_parent_rate = parent_rate;
                         best_rate = tmp_rate;
                 }
diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index 5e06f23eed41..d936409520f8 100644
--- a/include/linux/clk-provider.h
+++ b/include/linux/clk-provider.h
@@ -176,7 +176,7 @@ struct clk_ops {
                                         unsigned long *parent_rate);
         long            (*determine_rate)(struct clk_hw *hw, unsigned long rate,
                                         unsigned long *best_parent_rate,
-                                        struct clk **best_parent_clk);
+                                        struct clk_hw **best_parent_hw);
         int             (*set_parent)(struct clk_hw *hw, u8 index);
         u8              (*get_parent)(struct clk_hw *hw);
         int             (*set_rate)(struct clk_hw *hw, unsigned long rate,
@@ -551,7 +551,7 @@ bool __clk_is_enabled(struct clk *clk);
 struct clk *__clk_lookup(const char *name);
 long __clk_mux_determine_rate(struct clk_hw *hw, unsigned long rate,
                               unsigned long *best_parent_rate,
-                              struct clk **best_parent_p);
+                              struct clk_hw **best_parent_p);
 
 /*
  * FIXME clock api without lock protection