[0/5] qcom-cpr fixes for rjw bleeding-edge

Message

Niklas Cassel Dec. 23, 2019, 2:19 p.m. UTC

Hello Rafael,

Here comes some bug fixes for qcom-cpr that were detected
once the driver got some more build testing.

Patches 1-2 fix warnings detected by the intel test robot.

Patch 3 fixes an error I detected when doing an allnoconfig
and enabling simply the qcom-cpr driver.

Patches 4-5 are only detected when building with W=1.
However, I decided to fix these as well, in order to hopefully
avoid any further build test reports.

The series is based on your bleeding-edge branch.
Feel free to squash them with the existing commit if you
so desire.

Niklas Cassel (5):
  power: avs: qcom-cpr: fix invalid printk specifier in debug print
  power: avs: qcom-cpr: fix unsigned expression compared with zero
  power: avs: qcom-cpr: make sure that regmap is available
  power: avs: qcom-cpr: remove set but unused variable
  power: avs: qcom-cpr: make cpr_get_opp_hz_for_req() static

 drivers/power/avs/Kconfig    |  1 +
 drivers/power/avs/qcom-cpr.c | 18 ++++++++++--------
 2 files changed, 11 insertions(+), 8 deletions(-)

Comments

Rafael J. Wysocki Dec. 29, 2019, 10:45 p.m. UTC | #1

On Monday, December 23, 2019 3:19:29 PM CET Niklas Cassel wrote:
> Hello Rafael,
> 
> Here comes some bug fixes for qcom-cpr that were detected
> once the driver got some more build testing.
> 
> Patches 1-2 fix warnings detected by the intel test robot.
> 
> Patch 3 fixes an error I detected when doing an allnoconfig
> and enabling simply the qcom-cpr driver.
> 
> Patches 4-5 are only detected when building with W=1.
> However, I decided to fix these as well, in order to hopefully
> avoid any further build test reports.
> 
> The series is based on your bleeding-edge branch.
> Feel free to squash them with the existing commit if you
> so desire.
> 
> Niklas Cassel (5):
>   power: avs: qcom-cpr: fix invalid printk specifier in debug print
>   power: avs: qcom-cpr: fix unsigned expression compared with zero
>   power: avs: qcom-cpr: make sure that regmap is available
>   power: avs: qcom-cpr: remove set but unused variable
>   power: avs: qcom-cpr: make cpr_get_opp_hz_for_req() static
> 
>  drivers/power/avs/Kconfig    |  1 +
>  drivers/power/avs/qcom-cpr.c | 18 ++++++++++--------
>  2 files changed, 11 insertions(+), 8 deletions(-)
> 
> 

All patches in the series applied as 5.6 material, thanks!