Commit de47653b authored by Dmitry Osipenko's avatar Dmitry Osipenko Committed by Krzysztof Kozlowski

memory: tegra20-emc: Use dev_pm_opp_set_clkname()

The dev_pm_opp_get_opp_table() shouldn't be used by drivers, use
dev_pm_opp_set_clkname() instead.
Suggested-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarDmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/20201111011456.7875-3-digetx@gmail.comSigned-off-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
parent 3a0b6b5a
...@@ -902,7 +902,7 @@ static int tegra_emc_interconnect_init(struct tegra_emc *emc) ...@@ -902,7 +902,7 @@ static int tegra_emc_interconnect_init(struct tegra_emc *emc)
static int tegra_emc_opp_table_init(struct tegra_emc *emc) static int tegra_emc_opp_table_init(struct tegra_emc *emc)
{ {
struct opp_table *opp_table; struct opp_table *reg_opp_table = NULL, *clk_opp_table;
const char *rname = "core"; const char *rname = "core";
int err; int err;
...@@ -917,19 +917,24 @@ static int tegra_emc_opp_table_init(struct tegra_emc *emc) ...@@ -917,19 +917,24 @@ static int tegra_emc_opp_table_init(struct tegra_emc *emc)
} }
/* voltage scaling is optional */ /* voltage scaling is optional */
if (device_property_present(emc->dev, "core-supply")) if (device_property_present(emc->dev, "core-supply")) {
opp_table = dev_pm_opp_set_regulators(emc->dev, &rname, 1); reg_opp_table = dev_pm_opp_set_regulators(emc->dev, &rname, 1);
else if (IS_ERR(reg_opp_table))
opp_table = dev_pm_opp_get_opp_table(emc->dev); return dev_err_probe(emc->dev, PTR_ERR(reg_opp_table),
"failed to set OPP regulator\n");
}
if (IS_ERR(opp_table)) clk_opp_table = dev_pm_opp_set_clkname(emc->dev, NULL);
return dev_err_probe(emc->dev, PTR_ERR(opp_table), err = PTR_ERR_OR_ZERO(clk_opp_table);
"failed to prepare OPP table\n"); if (err) {
dev_err(emc->dev, "failed to set OPP clk: %d\n", err);
goto put_reg_table;
}
err = dev_pm_opp_of_add_table(emc->dev); err = dev_pm_opp_of_add_table(emc->dev);
if (err) { if (err) {
dev_err(emc->dev, "failed to add OPP table: %d\n", err); dev_err(emc->dev, "failed to add OPP table: %d\n", err);
goto put_table; goto put_clk_table;
} }
dev_info(emc->dev, "current clock rate %lu MHz\n", dev_info(emc->dev, "current clock rate %lu MHz\n",
...@@ -946,8 +951,11 @@ static int tegra_emc_opp_table_init(struct tegra_emc *emc) ...@@ -946,8 +951,11 @@ static int tegra_emc_opp_table_init(struct tegra_emc *emc)
remove_table: remove_table:
dev_pm_opp_of_remove_table(emc->dev); dev_pm_opp_of_remove_table(emc->dev);
put_table: put_clk_table:
dev_pm_opp_put_regulators(opp_table); dev_pm_opp_put_clkname(clk_opp_table);
put_reg_table:
if (reg_opp_table)
dev_pm_opp_put_regulators(reg_opp_table);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment