mirror of
https://github.com/torvalds/linux.git
synced 2026-09-22 12:44:03 +02:00
clk: st: clean-up simple provider misuse of the consumer API
Clock provider should not be using the consumer interface. In other words, a provider should not be dealing with struct clk. This change targets occurrences for which the provider uses the consumer interface and corresponding clk_hw interface exist. Reviewed-by: Brian Masney <bmasney@redhat.com> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> Reviewed-by: Thierry Reding <treding@nvidia.com> Signed-off-by: Brian Masney <bmasney@redhat.com>
This commit is contained in:
parent
ef6ca9c4df
commit
227d023952
|
|
@ -260,7 +260,7 @@ static int __clkgen_pll_enable(struct clk_hw *hw)
|
|||
if (pll->data->switch2pll_en)
|
||||
CLKGEN_WRITE(pll, switch2pll, 0);
|
||||
|
||||
pr_debug("%s:%s enabled\n", __clk_get_name(hw->clk), __func__);
|
||||
pr_debug("%s:%s enabled\n", clk_hw_get_name(hw), __func__);
|
||||
}
|
||||
|
||||
return ret;
|
||||
|
|
@ -295,7 +295,7 @@ static void __clkgen_pll_disable(struct clk_hw *hw)
|
|||
|
||||
CLKGEN_WRITE(pll, pdn_ctrl, 1);
|
||||
|
||||
pr_debug("%s:%s disabled\n", __clk_get_name(hw->clk), __func__);
|
||||
pr_debug("%s:%s disabled\n", clk_hw_get_name(hw), __func__);
|
||||
}
|
||||
|
||||
static void clkgen_pll_disable(struct clk_hw *hw)
|
||||
|
|
@ -405,14 +405,14 @@ static int stm_pll3200c32_determine_rate(struct clk_hw *hw,
|
|||
&req->rate);
|
||||
else {
|
||||
pr_debug("%s: %s rate %ld Invalid\n", __func__,
|
||||
__clk_get_name(hw->clk), req->rate);
|
||||
clk_hw_get_name(hw), req->rate);
|
||||
req->rate = 0;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
pr_debug("%s: %s new rate %ld [ndiv=%u] [idf=%u]\n",
|
||||
__func__, __clk_get_name(hw->clk),
|
||||
__func__, clk_hw_get_name(hw),
|
||||
req->rate, (unsigned int)params.ndiv,
|
||||
(unsigned int)params.idf);
|
||||
|
||||
|
|
@ -434,7 +434,7 @@ static int set_rate_stm_pll3200c32(struct clk_hw *hw, unsigned long rate,
|
|||
clk_pll3200c32_get_rate(parent_rate, ¶ms, &hwrate);
|
||||
|
||||
pr_debug("%s: %s new rate %ld [ndiv=0x%x] [idf=0x%x]\n",
|
||||
__func__, __clk_get_name(hw->clk),
|
||||
__func__, clk_hw_get_name(hw),
|
||||
hwrate, (unsigned int)params.ndiv,
|
||||
(unsigned int)params.idf);
|
||||
|
||||
|
|
@ -547,7 +547,7 @@ static unsigned long recalc_stm_pll4600c28(struct clk_hw *hw,
|
|||
|
||||
clk_pll4600c28_get_rate(parent_rate, ¶ms, &rate);
|
||||
|
||||
pr_debug("%s:%s rate %lu\n", __clk_get_name(hw->clk), __func__, rate);
|
||||
pr_debug("%s:%s rate %lu\n", clk_hw_get_name(hw), __func__, rate);
|
||||
|
||||
return rate;
|
||||
}
|
||||
|
|
@ -562,14 +562,14 @@ static int stm_pll4600c28_determine_rate(struct clk_hw *hw,
|
|||
&req->rate);
|
||||
} else {
|
||||
pr_debug("%s: %s rate %ld Invalid\n", __func__,
|
||||
__clk_get_name(hw->clk), req->rate);
|
||||
clk_hw_get_name(hw), req->rate);
|
||||
req->rate = 0;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
pr_debug("%s: %s new rate %ld [ndiv=%u] [idf=%u]\n",
|
||||
__func__, __clk_get_name(hw->clk),
|
||||
__func__, clk_hw_get_name(hw),
|
||||
req->rate, (unsigned int)params.ndiv,
|
||||
(unsigned int)params.idf);
|
||||
|
||||
|
|
@ -591,12 +591,12 @@ static int set_rate_stm_pll4600c28(struct clk_hw *hw, unsigned long rate,
|
|||
clk_pll4600c28_get_rate(parent_rate, ¶ms, &hwrate);
|
||||
} else {
|
||||
pr_debug("%s: %s rate %ld Invalid\n", __func__,
|
||||
__clk_get_name(hw->clk), rate);
|
||||
clk_hw_get_name(hw), rate);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
pr_debug("%s: %s new rate %ld [ndiv=0x%x] [idf=0x%x]\n",
|
||||
__func__, __clk_get_name(hw->clk),
|
||||
__func__, clk_hw_get_name(hw),
|
||||
hwrate, (unsigned int)params.ndiv,
|
||||
(unsigned int)params.idf);
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user