| Message ID | 20221210191013.453040-1-dmitry.baryshkov@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | a622c1dc84d920f231df539b3013c58a15fa2a19 |
| Headers | show |
| Series | [v2] clk: qcom: clk-spmi-pmic-div: convert to use parent_data | expand |
On Sat, 10 Dec 2022 22:10:13 +0300, Dmitry Baryshkov wrote: > The clk-spmi-pmic-div driver gets the parent name from the first (and > the only) clock parent. So, use parent data, index 0 to setup the clock. > > Applied, thanks! [1/1] clk: qcom: clk-spmi-pmic-div: convert to use parent_data commit: a622c1dc84d920f231df539b3013c58a15fa2a19 Best regards,
diff --git a/drivers/clk/qcom/clk-spmi-pmic-div.c b/drivers/clk/qcom/clk-spmi-pmic-div.c index f2cf55cee2fd..f846be285f51 100644 --- a/drivers/clk/qcom/clk-spmi-pmic-div.c +++ b/drivers/clk/qcom/clk-spmi-pmic-div.c @@ -204,7 +204,7 @@ static int spmi_pmic_clkdiv_probe(struct platform_device *pdev) struct regmap *regmap; struct device *dev = &pdev->dev; struct device_node *of_node = dev->of_node; - const char *parent_name; + struct clk_parent_data parent_data = { .index = 0, }; int nclks, i, ret, cxo_hz; char name[20]; u32 start; @@ -246,14 +246,8 @@ static int spmi_pmic_clkdiv_probe(struct platform_device *pdev) cxo_hz = clk_get_rate(cxo); clk_put(cxo); - parent_name = of_clk_get_parent_name(of_node, 0); - if (!parent_name) { - dev_err(dev, "missing parent clock\n"); - return -ENODEV; - } - init.name = name; - init.parent_names = &parent_name; + init.parent_data = &parent_data; init.num_parents = 1; init.ops = &clk_spmi_pmic_div_ops;
The clk-spmi-pmic-div driver gets the parent name from the first (and the only) clock parent. So, use parent data, index 0 to setup the clock. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- Changes since v1: - Moved parent_data to be the onstack var. --- drivers/clk/qcom/clk-spmi-pmic-div.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-)