Re: [PATCH v5 4/5] cpufreq: qcom: Update the bandwidth levels on frequency change

2020-06-01 Thread Sibi Sankar
On 2020-06-01 16:31, Viresh Kumar wrote: On 29-05-20, 17:00, Sibi Sankar wrote: > > +static int qcom_cpufreq_update_opp(struct device *cpu_dev, > > +unsigned long freq_khz, > > +unsigned long volt) > > +{ > > + unsigned long fre

Re: [PATCH v5 4/5] cpufreq: qcom: Update the bandwidth levels on frequency change

2020-06-01 Thread Viresh Kumar
On 29-05-20, 17:00, Sibi Sankar wrote: > > > +static int qcom_cpufreq_update_opp(struct device *cpu_dev, > > > +unsigned long freq_khz, > > > +unsigned long volt) > > > +{ > > > + unsigned long freq_hz = freq_khz * 1000; > > > + > > > + if (de

Re: [PATCH v5 4/5] cpufreq: qcom: Update the bandwidth levels on frequency change

2020-05-29 Thread Sibi Sankar
Hey Viresh, Thanks for taking time to review the series :) On 2020-05-29 15:30, Viresh Kumar wrote: On 28-05-20, 01:51, Sibi Sankar wrote: Add support to parse optional OPP table attached to the cpu node when the OPP bandwidth values are populated. This allows for scaling of DDR/L3 bandwidth le

Re: [PATCH v5 4/5] cpufreq: qcom: Update the bandwidth levels on frequency change

2020-05-29 Thread Viresh Kumar
On 28-05-20, 01:51, Sibi Sankar wrote: > Add support to parse optional OPP table attached to the cpu node when > the OPP bandwidth values are populated. This allows for scaling of > DDR/L3 bandwidth levels with frequency change. > > Signed-off-by: Sibi Sankar > --- > > V5: > * Use dev_pm_opp_ad

[PATCH v5 4/5] cpufreq: qcom: Update the bandwidth levels on frequency change

2020-05-27 Thread Sibi Sankar
Add support to parse optional OPP table attached to the cpu node when the OPP bandwidth values are populated. This allows for scaling of DDR/L3 bandwidth levels with frequency change. Signed-off-by: Sibi Sankar --- V5: * Use dev_pm_opp_adjust_voltage instead [Viresh] * Misc cleanup v4: * Spl