Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-27 Thread Viresh Kumar
On 27-01-21, 22:01, Akhil P Oommen wrote: > On 1/22/2021 10:15 AM, Viresh Kumar wrote: > > On 22-01-21, 00:41, Dmitry Osipenko wrote: > > > 21.01.2021 14:17, Viresh Kumar пишет: > > > > @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, > > > > unsigned long target_freq) > > > >

Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-27 Thread Akhil P Oommen
On 1/22/2021 10:15 AM, Viresh Kumar wrote: On 22-01-21, 00:41, Dmitry Osipenko wrote: 21.01.2021 14:17, Viresh Kumar пишет: @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq) if (!ret) { ret = _set_opp_bw(opp_table, opp, dev, f

Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-24 Thread Viresh Kumar
On 22-01-21, 17:31, Dmitry Osipenko wrote: > This may not be true for all kinds of hardware, a display controller is > one example. If display's pixclock is raised before the memory bandwidth > of the display's memory client, then display controller may get a memory > underflow since it won't be ab

Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-22 Thread Dmitry Osipenko
22.01.2021 07:45, Viresh Kumar пишет: > On 22-01-21, 00:41, Dmitry Osipenko wrote: >> 21.01.2021 14:17, Viresh Kumar пишет: >>> @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, >>> unsigned long target_freq) >>> >>> if (!ret) { >>> ret = _set_opp_bw(opp_table,

Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-21 Thread Viresh Kumar
On 22-01-21, 00:41, Dmitry Osipenko wrote: > 21.01.2021 14:17, Viresh Kumar пишет: > > @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, > > unsigned long target_freq) > > > > if (!ret) { > > ret = _set_opp_bw(opp_table, opp, dev, false); > > - if (!re

Re: [PATCH 03/13] opp: Keep track of currently programmed OPP

2021-01-21 Thread Dmitry Osipenko
21.01.2021 14:17, Viresh Kumar пишет: > @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, unsigned > long target_freq) > > if (!ret) { > ret = _set_opp_bw(opp_table, opp, dev, false); > - if (!ret) > + if (!ret) { >