Re: [PATCH] opp: Don't always remove static OPPs in _of_add_opp_table_v1()

2020-10-28 Thread Geert Uytterhoeven
Hi Viresh, On Wed, 14 Oct 2020, Viresh Kumar wrote: The patch missed returning 0 early in case of success and hence the static OPPs got removed by mistake. Fix it. Fixes: 90d46d71cce2 ("opp: Handle multiple calls for same OPP table in _of_add_opp_table_v1()") Reported-by: Aisheng Dong

Re: [PATCH] opp: Don't always remove static OPPs in _of_add_opp_table_v1()

2020-10-19 Thread Viresh Kumar
On 15-10-20, 10:01, Viresh Kumar wrote: > On 15-10-20, 02:35, Aisheng Dong wrote: > > Hi Viresh > > > > Thanks for the quick fix. > > > > > From: Viresh Kumar > > > Sent: Wednesday, October 14, 2020 12:26 PM > > > > > > The patch missed returning 0 early in case of success and hence the > > >

Re: [PATCH] opp: Don't always remove static OPPs in _of_add_opp_table_v1()

2020-10-14 Thread Viresh Kumar
On 15-10-20, 02:35, Aisheng Dong wrote: > Hi Viresh > > Thanks for the quick fix. > > > From: Viresh Kumar > > Sent: Wednesday, October 14, 2020 12:26 PM > > > > The patch missed returning 0 early in case of success and hence the static > > OPPs > > got removed by mistake. Fix it. > > > >

RE: [PATCH] opp: Don't always remove static OPPs in _of_add_opp_table_v1()

2020-10-14 Thread Aisheng Dong
Hi Viresh Thanks for the quick fix. > From: Viresh Kumar > Sent: Wednesday, October 14, 2020 12:26 PM > > The patch missed returning 0 early in case of success and hence the static > OPPs > got removed by mistake. Fix it. > > Fixes: 90d46d71cce2 ("opp: Handle multiple calls for same OPP