Messages in this thread | | | Date | Thu, 17 Jun 2021 09:30:27 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH] opp: of: Allow lazy-linking of required-opps to non genpd |
| |
On 17-06-21, 13:09, Chanwoo Choi wrote: > On 6/17/21 12:33 PM, Viresh Kumar wrote: > > On 17-06-21, 10:13, Chanwoo Choi wrote: > >> The devfreq driver(exynos-bus.c) has used the dev_pm_opp_set_rate() > >> and used the passive governor without the required-opp property. > > > > Which is fine. > > > >> I have a plan to use the required-opp property > >> between devfreq drivers (exynos-bus.c) with dev_pm_opp_set_rate(). > >> > >> I'll support them on later if this approach doesn't break the any > >> rule of required-opp property. > > > > You will be required to make some changes in core for that I am > > afraid. It won't work automatically. > > Do you think that better to use clk_enable/regulator_enable directly > instead of dev_pm_opp_set_rate() for using required-opp property?
No. All I am saying is that the OPP core won't work for your use case today and may need some updates.
-- viresh
| |