On Sat, May 13, 2023 at 12:17:25AM +0300, Dmitry Baryshkov wrote:
> The gcc and lcc devices have been switched to the DT-defined cxo_board
> clock. Now we can drop the manually defined cxo clock.
>
Perhaps I'm reading this incorrectly, but don't you introduce that clock
in DT in the next commit? It would be nice to allow for at least some
overlap between existing dtb and the new scheme.
Regards,
Bjorn
> Reviewed-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> ---
> drivers/clk/qcom/gcc-mdm9615.c | 17 -----------------
> 1 file changed, 17 deletions(-)
>
> diff --git a/drivers/clk/qcom/gcc-mdm9615.c b/drivers/clk/qcom/gcc-mdm9615.c
> index 458c18b639db..64d4f508e43a 100644
> --- a/drivers/clk/qcom/gcc-mdm9615.c
> +++ b/drivers/clk/qcom/gcc-mdm9615.c
> @@ -26,17 +26,6 @@
> #include "clk-branch.h"
> #include "reset.h"
>
> -static struct clk_fixed_factor cxo = {
> - .mult = 1,
> - .div = 1,
> - .hw.init = &(struct clk_init_data){
> - .name = "cxo",
> - .parent_names = (const char *[]){ "cxo_board" },
> - .num_parents = 1,
> - .ops = &clk_fixed_factor_ops,
> - },
> -};
> -
> enum {
> DT_CXO,
> DT_PLL4,
> @@ -1623,10 +1612,6 @@ static struct clk_branch ebi2_aon_clk = {
> },
> };
>
> -static struct clk_hw *gcc_mdm9615_hws[] = {
> - &cxo.hw,
> -};
> -
> static struct clk_regmap *gcc_mdm9615_clks[] = {
> [PLL0] = &pll0.clkr,
> [PLL0_VOTE] = &pll0_vote,
> @@ -1736,8 +1721,6 @@ static const struct qcom_cc_desc gcc_mdm9615_desc = {
> .num_clks = ARRAY_SIZE(gcc_mdm9615_clks),
> .resets = gcc_mdm9615_resets,
> .num_resets = ARRAY_SIZE(gcc_mdm9615_resets),
> - .clk_hws = gcc_mdm9615_hws,
> - .num_clk_hws = ARRAY_SIZE(gcc_mdm9615_hws),
> };
>
> static const struct of_device_id gcc_mdm9615_match_table[] = {
> --
> 2.39.2
>
