Re: [PATCH] cpufreq: dt: check the error returned by dev_pm_opp_of_cpumask_add_table

2021-03-24 Thread quanyang.wang
Hi Viresh, On 3/25/21 1:24 PM, Viresh Kumar wrote: On 25-03-21, 13:15, quanyang.wang wrote: Thank you for pointing it out.  Do you mean that even if dev_pm_opp_of_cpumask_add_table returns an error, dev_pm_opp_get_opp_count may still return count > 0 because someone may call dev_pm_opp_add

Re: [PATCH] cpufreq: dt: check the error returned by dev_pm_opp_of_cpumask_add_table

2021-03-24 Thread Viresh Kumar
On 25-03-21, 13:15, quanyang.wang wrote: > Thank you for pointing it out.  Do you mean that even if > dev_pm_opp_of_cpumask_add_table returns > > an error, dev_pm_opp_get_opp_count may still return count > 0 because > someone may call dev_pm_opp_add > > to add OPP to cpu succcessfully at

Re: [PATCH] cpufreq: dt: check the error returned by dev_pm_opp_of_cpumask_add_table

2021-03-24 Thread quanyang.wang
Hi Viresh, On 3/25/21 12:45 PM, Viresh Kumar wrote: On 25-03-21, 12:31, quanyang.w...@windriver.com wrote: From: Quanyang Wang The function dev_pm_opp_of_cpumask_add_table may return zero or an error. When it returns an error, this means that no OPP table is added for the cpumask because

Re: [PATCH] cpufreq: dt: check the error returned by dev_pm_opp_of_cpumask_add_table

2021-03-24 Thread Viresh Kumar
On 25-03-21, 12:31, quanyang.w...@windriver.com wrote: > From: Quanyang Wang > > The function dev_pm_opp_of_cpumask_add_table may return zero or an > error. When it returns an error, this means that no OPP table is > added for the cpumask because _dev_pm_opp_cpumask_remove_table is > called to

[PATCH] cpufreq: dt: check the error returned by dev_pm_opp_of_cpumask_add_table

2021-03-24 Thread quanyang . wang
From: Quanyang Wang The function dev_pm_opp_of_cpumask_add_table may return zero or an error. When it returns an error, this means that no OPP table is added for the cpumask because _dev_pm_opp_cpumask_remove_table is called to free all OPPs associated with the cpu devices in the error label