[V3,0/4] opp: general cleanups

Message

Viresh Kumar Aug. 20, 2020, 7:56 a.m. UTC

Hi,

Here is another version of the cleanups I sent earlier.

Rajendra: Please see if these work fine now.

V3:
- Dropped v2 1/4 as it is already merged.
- New patch 4/4 added.
- Reordered the first two patches here (Stephen)
- disable regulator only if present

Viresh Kumar (4):
  opp: Rename regulator_enabled and use it as status of all resources
  opp: Track device's resources configuration status
  opp: Split out _opp_set_rate_zero()
  opp: Remove _dev_pm_opp_find_and_remove_table() wrapper

 drivers/opp/core.c | 103 +++++++++++++++++++++------------------------
 drivers/opp/cpu.c  |   2 +-
 drivers/opp/of.c   |   2 +-
 drivers/opp/opp.h  |   5 +--
 4 files changed, 52 insertions(+), 60 deletions(-)

Comments

Rajendra Nayak Aug. 23, 2020, 5:48 a.m. UTC | #1

On 8/20/2020 1:26 PM, Viresh Kumar wrote:
> Hi,
> 
> Here is another version of the cleanups I sent earlier.
> 
> Rajendra: Please see if these work fine now.

I gave these a quick spin, and they don';t result in the crash I
earlier observed

Tested-by: Rajendra Nayak <[e-mail]>

> 
> V3:
> - Dropped v2 1/4 as it is already merged.
> - New patch 4/4 added.
> - Reordered the first two patches here (Stephen)
> - disable regulator only if present
> 
> Viresh Kumar (4):
>    opp: Rename regulator_enabled and use it as status of all resources
>    opp: Track device's resources configuration status
>    opp: Split out _opp_set_rate_zero()
>    opp: Remove _dev_pm_opp_find_and_remove_table() wrapper
> 
>   drivers/opp/core.c | 103 +++++++++++++++++++++------------------------
>   drivers/opp/cpu.c  |   2 +-
>   drivers/opp/of.c   |   2 +-
>   drivers/opp/opp.h  |   5 +--
>   4 files changed, 52 insertions(+), 60 deletions(-)
>