Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp()

[PATCH 00/13] opp: Implement dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 01/13] opp: Rename _opp_set_rate_zero() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 03/13] opp: Keep track of currently programmed OPP · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Dmitry Osipenko <digetx@gmail.com> · 2021-01-21
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-22
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Dmitry Osipenko <digetx@gmail.com> · 2021-01-22
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-25
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Ionela Voinescu <hidden> · 2021-07-07
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Viresh Kumar <viresh.kumar@linaro.org> · 2021-07-08
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Ionela Voinescu <hidden> · 2021-07-09
Re: [PATCH 03/13] opp: Keep track of currently programmed OPP · Viresh Kumar <viresh.kumar@linaro.org> · 2021-07-12
[PATCH 08/13] opp: Update parameters of _set_opp_custom() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 10/13] cpufreq: qcom: Migrate to dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Dmitry Osipenko <digetx@gmail.com> · 2021-01-21
Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-22
Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Dmitry Osipenko <digetx@gmail.com> · 2021-01-22
Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-25
Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() · Dmitry Osipenko <digetx@gmail.com> · 2021-01-26
[PATCH 12/13] drm: msm: Migrate to dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 13/13] opp: Remove dev_pm_opp_set_bw() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 09/13] opp: Implement dev_pm_opp_set_opp() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 07/13] opp: Allow _generic_set_opp_clk_only() to work for non-freq devices · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
Re: [PATCH 07/13] opp: Allow _generic_set_opp_clk_only() to work for non-freq devices · Dmitry Osipenko <digetx@gmail.com> · 2021-01-21
Re: [PATCH 07/13] opp: Allow _generic_set_opp_clk_only() to work for non-freq devices · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-22
Re: [PATCH 07/13] opp: Allow _generic_set_opp_clk_only() to work for non-freq devices · Dmitry Osipenko <digetx@gmail.com> · 2021-01-25
Re: [PATCH 07/13] opp: Allow _generic_set_opp_clk_only() to work for non-freq devices · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-27
[PATCH 06/13] opp: Allow _generic_set_opp_regulator() to work for non-freq devices · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 05/13] opp: Allow _set_opp() to work for non-freq devices · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 04/13] opp: Split _set_opp() out of dev_pm_opp_set_rate() · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21
[PATCH 02/13] opp: No need to check clk for errors · Viresh Kumar <viresh.kumar@linaro.org> · 2021-01-21

From: Viresh Kumar <viresh.kumar@linaro.org>
Date: 2021-01-25 03:15:43
Also in: linux-pm, linux-tegra, lkml

On 22-01-21, 18:28, Dmitry Osipenko wrote:

Either way will work, but maybe keeping the dev_pm_opp_set_bw() is a bit
more straightforward variant for now since it will avoid the code's
changes and it's probably a bit more obvious variant for the OPP users.

The problem is it creates unnecessary paths which we need to support. For
example, in case of bandwidth itself we may want to update regulator/pm
domain/required OPPs and this should all be done for everyone. I really do not
want to keep separate paths for such stuff, it makes it hard to maintain..

-- 
viresh

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help