mbox series

[RESEND,v8,0/4] Make IPA use PM_EM

Message ID 20190911130314.29973-1-qperret@qperret.net (mailing list archive)
Headers show
Series Make IPA use PM_EM | expand

Message

Quentin Perret Sept. 11, 2019, 1:03 p.m. UTC
Re-sending this from an email address I can access. For a cover letter,
see:
https://lore.kernel.org/lkml/20190812084235.21440-1-quentin.perret@arm.com/

Changes in v8:
 - Fixed checkpatch errors (Rui)

Changes in v7
 - Added patch 02/04 to fix the build error reported by the kbuild bot

Changes in v6
 - Added Daniel's and Viresh's Acked-by to all patches

Changes in v5:
 - Changed patch 02 to guard IPA-specific code in cpu_cooling.c with
   appropriate ifdefery (Daniel)
 - Rebased on 5.2-rc2

Changes in v4:
 - Added Viresh's Acked-by to all 3 patches
 - Improved commit message of patch 3/3 to explain how it has no
   functional impact on existing users (Eduardo)

Changes in v3:
 - Changed warning message for unordered tables to something more
   explicit (Viresh)
 - Changed WARN() into a pr_err() for consistency

Changes in v2:
 - Fixed patch 01/03 to actually enable CONFIG_ENERGY_MODEL
 - Added "depends on ENERGY_MODEL" to IPA (Daniel)
 - Added check to bail out if the freq table is unsorted (Viresh)


Quentin Perret (4):
  arm64: defconfig: Enable CONFIG_ENERGY_MODEL
  PM / EM: Declare EM data types unconditionally
  thermal: cpu_cooling: Make the power-related code depend on IPA
  thermal: cpu_cooling: Migrate to using the EM framework

 arch/arm64/configs/defconfig  |   1 +
 drivers/thermal/Kconfig       |   1 +
 drivers/thermal/cpu_cooling.c | 427 ++++++++++++++--------------------
 include/linux/energy_model.h  |   3 +-
 4 files changed, 178 insertions(+), 254 deletions(-)

Comments

Daniel Lezcano Oct. 7, 2019, 5:35 a.m. UTC | #1
Hi Quentin,

the series does no longer apply, do you think it is possible to give it
a respin?



On 11/09/2019 15:03, Quentin Perret wrote:
> Re-sending this from an email address I can access. For a cover letter,
> see:
> https://lore.kernel.org/lkml/20190812084235.21440-1-quentin.perret@arm.com/
> 
> Changes in v8:
>  - Fixed checkpatch errors (Rui)
> 
> Changes in v7
>  - Added patch 02/04 to fix the build error reported by the kbuild bot
> 
> Changes in v6
>  - Added Daniel's and Viresh's Acked-by to all patches
> 
> Changes in v5:
>  - Changed patch 02 to guard IPA-specific code in cpu_cooling.c with
>    appropriate ifdefery (Daniel)
>  - Rebased on 5.2-rc2
> 
> Changes in v4:
>  - Added Viresh's Acked-by to all 3 patches
>  - Improved commit message of patch 3/3 to explain how it has no
>    functional impact on existing users (Eduardo)
> 
> Changes in v3:
>  - Changed warning message for unordered tables to something more
>    explicit (Viresh)
>  - Changed WARN() into a pr_err() for consistency
> 
> Changes in v2:
>  - Fixed patch 01/03 to actually enable CONFIG_ENERGY_MODEL
>  - Added "depends on ENERGY_MODEL" to IPA (Daniel)
>  - Added check to bail out if the freq table is unsorted (Viresh)
> 
> 
> Quentin Perret (4):
>   arm64: defconfig: Enable CONFIG_ENERGY_MODEL
>   PM / EM: Declare EM data types unconditionally
>   thermal: cpu_cooling: Make the power-related code depend on IPA
>   thermal: cpu_cooling: Migrate to using the EM framework
> 
>  arch/arm64/configs/defconfig  |   1 +
>  drivers/thermal/Kconfig       |   1 +
>  drivers/thermal/cpu_cooling.c | 427 ++++++++++++++--------------------
>  include/linux/energy_model.h  |   3 +-
>  4 files changed, 178 insertions(+), 254 deletions(-)
>
Quentin Perret Oct. 7, 2019, 1:37 p.m. UTC | #2
Hi Daniel,

On Mon, Oct 7, 2019 at 6:35 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
> the series does no longer apply, do you think it is possible to give it
> a respin?

Right, I'll try to fix the conflicts and post a v9 shortly.

Thanks,
Quentin
Daniel Lezcano Oct. 30, 2019, 10:54 a.m. UTC | #3
Hi Quentin,

On 07/10/2019 15:37, Quentin Perret wrote:
> Hi Daniel,
> 
> On Mon, Oct 7, 2019 at 6:35 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
>> the series does no longer apply, do you think it is possible to give it
>> a respin?
> 
> Right, I'll try to fix the conflicts and post a v9 shortly.

we are getting close to the merge window, did you have time to respin
the series?
Quentin Perret Oct. 30, 2019, 11:44 a.m. UTC | #4
On Wednesday 30 Oct 2019 at 11:54:30 (+0100), Daniel Lezcano wrote:
> Hi Quentin,
> 
> On 07/10/2019 15:37, Quentin Perret wrote:
> > Hi Daniel,
> > 
> > On Mon, Oct 7, 2019 at 6:35 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
> >> the series does no longer apply, do you think it is possible to give it
> >> a respin?
> > 
> > Right, I'll try to fix the conflicts and post a v9 shortly.
> 
> we are getting close to the merge window, did you have time to respin
> the series?

Yes, sorry for the delay, but I have a 5.4-rc4-based branch that seems
to work. I'll push that ASAP.

Thanks,
Quentin