All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Matthias Kaehlcke <mka@chromium.org>
Cc: Lukasz Luba <lukasz.luba@arm.com>,
	linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	dri-devel@lists.freedesktop.org, linux-omap@vger.kernel.org,
	linux-mediatek@lists.infradead.org,
	linux-arm-msm@vger.kernel.org, linux-imx@nxp.com,
	Morten.Rasmussen@arm.com, Dietmar.Eggemann@arm.com,
	javi.merino@arm.com, cw00.choi@samsung.com,
	b.zolnierkie@samsung.com, rjw@rjwysocki.net,
	sudeep.holla@arm.com, viresh.kumar@linaro.org, nm@ti.com,
	sboyd@kernel.org, rui.zhang@intel.com,
	amit.kucheria@verdurent.com, mingo@redhat.com,
	peterz@infradead.org, juri.lelli@redhat.com,
	vincent.guittot@linaro.org, rostedt@goodmis.org,
	qperret@google.com, bsegall@google.com, mgorman@suse.de,
	shawnguo@kernel.org, s.hauer@pengutronix.de, festevam@gmail.com,
	kernel@pengutronix.de, khilman@kernel.org, agross@kernel.org,
	bjorn.andersson@linaro.org, robh@kernel.org,
	matthias.bgg@gmail.com, steven.price@arm.com,
	tomeu.vizoso@collabora.com, alyssa.rosenzweig@collabora.com,
	airlied@linux.ie, daniel@ffwll.ch, liviu.dudau@arm.com,
	lorenzo.pieralisi@arm.com, patrick.bellasi@matbug.net,
	orjan.eide@arm.com, rdunlap@infradead.org
Subject: Re: [PATCH v5 3/5] thermal: devfreq_cooling: Use PM QoS to set frequency limits
Date: Fri, 3 Apr 2020 19:19:33 +0200	[thread overview]
Message-ID: <82997cd2-1e4d-fb9d-8c64-6771b72456e9@linaro.org> (raw)
In-Reply-To: <20200403171818.GO199755@google.com>

On 03/04/2020 19:18, Matthias Kaehlcke wrote:
> Hi Daniel,
> 
> On Fri, Apr 03, 2020 at 06:43:20PM +0200, Daniel Lezcano wrote:
>> On 18/03/2020 12:45, Lukasz Luba wrote:
>>> From: Matthias Kaehlcke <mka@chromium.org>
>>>
>>> Now that devfreq supports limiting the frequency range of a device
>>> through PM QoS make use of it instead of disabling OPPs that should
>>> not be used.
>>>
>>> The switch from disabling OPPs to PM QoS introduces a subtle behavioral
>>> change in case of conflicting requests (min > max): PM QoS gives
>>> precedence to the MIN_FREQUENCY request, while higher OPPs disabled
>>> with dev_pm_opp_disable() would override MIN_FREQUENCY.
>>>
>>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>>> Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
>>> Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
>>
>> This patch is standalone, right? If yes, I will apply it.
> 
> Yes, it is standalone, please apply

Applied on 'testing', thanks


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Matthias Kaehlcke <mka@chromium.org>
Cc: nm@ti.com, juri.lelli@redhat.com, peterz@infradead.org,
	viresh.kumar@linaro.org, liviu.dudau@arm.com,
	dri-devel@lists.freedesktop.org, bjorn.andersson@linaro.org,
	bsegall@google.com, alyssa.rosenzweig@collabora.com,
	festevam@gmail.com, Morten.Rasmussen@arm.com, robh@kernel.org,
	amit.kucheria@verdurent.com, lorenzo.pieralisi@arm.com,
	vincent.guittot@linaro.org, khilman@kernel.org,
	agross@kernel.org, b.zolnierkie@samsung.com,
	steven.price@arm.com, cw00.choi@samsung.com, mingo@redhat.com,
	linux-imx@nxp.com, rui.zhang@intel.com, mgorman@suse.de,
	orjan.eide@arm.com, daniel@ffwll.ch, linux-pm@vger.kernel.org,
	linux-arm-msm@vger.kernel.org, s.hauer@pengutronix.de,
	rostedt@goodmis.org, linux-mediatek@lists.infradead.org,
	matthias.bgg@gmail.com, linux-omap@vger.kernel.org,
	Dietmar.Eggemann@arm.com, linux-arm-kernel@lists.infradead.org,
	airlied@linux.ie, javi.merino@arm.com,
	tomeu.vizoso@collabora.com, qperret@google.com, sboyd@kernel.org,
	rdunlap@infradead.org, rjw@rjwysocki.net,
	linux-kernel@vger.kernel.org, kernel@pengutronix.de,
	sudeep.holla@arm.com, patrick.bellasi@matbug.net,
	shawnguo@kernel.org, Lukasz Luba <lukasz.luba@arm.com>
Subject: Re: [PATCH v5 3/5] thermal: devfreq_cooling: Use PM QoS to set frequency limits
Date: Fri, 3 Apr 2020 19:19:33 +0200	[thread overview]
Message-ID: <82997cd2-1e4d-fb9d-8c64-6771b72456e9@linaro.org> (raw)
In-Reply-To: <20200403171818.GO199755@google.com>

On 03/04/2020 19:18, Matthias Kaehlcke wrote:
> Hi Daniel,
> 
> On Fri, Apr 03, 2020 at 06:43:20PM +0200, Daniel Lezcano wrote:
>> On 18/03/2020 12:45, Lukasz Luba wrote:
>>> From: Matthias Kaehlcke <mka@chromium.org>
>>>
>>> Now that devfreq supports limiting the frequency range of a device
>>> through PM QoS make use of it instead of disabling OPPs that should
>>> not be used.
>>>
>>> The switch from disabling OPPs to PM QoS introduces a subtle behavioral
>>> change in case of conflicting requests (min > max): PM QoS gives
>>> precedence to the MIN_FREQUENCY request, while higher OPPs disabled
>>> with dev_pm_opp_disable() would override MIN_FREQUENCY.
>>>
>>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>>> Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
>>> Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
>>
>> This patch is standalone, right? If yes, I will apply it.
> 
> Yes, it is standalone, please apply

Applied on 'testing', thanks


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Matthias Kaehlcke <mka@chromium.org>
Cc: nm@ti.com, juri.lelli@redhat.com, peterz@infradead.org,
	viresh.kumar@linaro.org, liviu.dudau@arm.com,
	dri-devel@lists.freedesktop.org, bjorn.andersson@linaro.org,
	bsegall@google.com, alyssa.rosenzweig@collabora.com,
	festevam@gmail.com, Morten.Rasmussen@arm.com, robh@kernel.org,
	amit.kucheria@verdurent.com, lorenzo.pieralisi@arm.com,
	khilman@kernel.org, agross@kernel.org, b.zolnierkie@samsung.com,
	steven.price@arm.com, cw00.choi@samsung.com, mingo@redhat.com,
	linux-imx@nxp.com, rui.zhang@intel.com, mgorman@suse.de,
	orjan.eide@arm.com, daniel@ffwll.ch, linux-pm@vger.kernel.org,
	linux-arm-msm@vger.kernel.org, s.hauer@pengutronix.de,
	rostedt@goodmis.org, linux-mediatek@lists.infradead.org,
	matthias.bgg@gmail.com, linux-omap@vger.kernel.org,
	Dietmar.Eggemann@arm.com, linux-arm-kernel@lists.infradead.org,
	airlied@linux.ie, javi.merino@arm.com,
	tomeu.vizoso@collabora.com, qperret@google.com, sboyd@kernel.org,
	rdunlap@infradead.org, rjw@rjwysocki.net,
	linux-kernel@vger.kernel.org, kernel@pengutronix.de,
	sudeep.holla@arm.com, patrick.bellasi@matbug.net,
	shawnguo@kernel.org, Lukasz Luba <lukasz.luba@arm.com>
Subject: Re: [PATCH v5 3/5] thermal: devfreq_cooling: Use PM QoS to set frequency limits
Date: Fri, 3 Apr 2020 19:19:33 +0200	[thread overview]
Message-ID: <82997cd2-1e4d-fb9d-8c64-6771b72456e9@linaro.org> (raw)
In-Reply-To: <20200403171818.GO199755@google.com>

On 03/04/2020 19:18, Matthias Kaehlcke wrote:
> Hi Daniel,
> 
> On Fri, Apr 03, 2020 at 06:43:20PM +0200, Daniel Lezcano wrote:
>> On 18/03/2020 12:45, Lukasz Luba wrote:
>>> From: Matthias Kaehlcke <mka@chromium.org>
>>>
>>> Now that devfreq supports limiting the frequency range of a device
>>> through PM QoS make use of it instead of disabling OPPs that should
>>> not be used.
>>>
>>> The switch from disabling OPPs to PM QoS introduces a subtle behavioral
>>> change in case of conflicting requests (min > max): PM QoS gives
>>> precedence to the MIN_FREQUENCY request, while higher OPPs disabled
>>> with dev_pm_opp_disable() would override MIN_FREQUENCY.
>>>
>>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>>> Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
>>> Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
>>
>> This patch is standalone, right? If yes, I will apply it.
> 
> Yes, it is standalone, please apply

Applied on 'testing', thanks


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Matthias Kaehlcke <mka@chromium.org>
Cc: nm@ti.com, juri.lelli@redhat.com, peterz@infradead.org,
	viresh.kumar@linaro.org, liviu.dudau@arm.com,
	dri-devel@lists.freedesktop.org, bjorn.andersson@linaro.org,
	bsegall@google.com, alyssa.rosenzweig@collabora.com,
	Morten.Rasmussen@arm.com, amit.kucheria@verdurent.com,
	lorenzo.pieralisi@arm.com, vincent.guittot@linaro.org,
	khilman@kernel.org, agross@kernel.org, b.zolnierkie@samsung.com,
	steven.price@arm.com, cw00.choi@samsung.com, mingo@redhat.com,
	linux-imx@nxp.com, rui.zhang@intel.com, mgorman@suse.de,
	orjan.eide@arm.com, linux-pm@vger.kernel.org,
	linux-arm-msm@vger.kernel.org, s.hauer@pengutronix.de,
	rostedt@goodmis.org, linux-mediatek@lists.infradead.org,
	matthias.bgg@gmail.com, linux-omap@vger.kernel.org,
	Dietmar.Eggemann@arm.com, linux-arm-kernel@lists.infradead.org,
	airlied@linux.ie, javi.merino@arm.com,
	tomeu.vizoso@collabora.com, qperret@google.com, sboyd@kernel.org,
	rdunlap@infradead.org, rjw@rjwysocki.net,
	linux-kernel@vger.kernel.org, kernel@pengutronix.de,
	sudeep.holla@arm.com, patrick.bellasi@matbug.net,
	shawnguo@kernel.org, Lukasz Luba <lukasz.luba@arm.com>
Subject: Re: [PATCH v5 3/5] thermal: devfreq_cooling: Use PM QoS to set frequency limits
Date: Fri, 3 Apr 2020 19:19:33 +0200	[thread overview]
Message-ID: <82997cd2-1e4d-fb9d-8c64-6771b72456e9@linaro.org> (raw)
In-Reply-To: <20200403171818.GO199755@google.com>

On 03/04/2020 19:18, Matthias Kaehlcke wrote:
> Hi Daniel,
> 
> On Fri, Apr 03, 2020 at 06:43:20PM +0200, Daniel Lezcano wrote:
>> On 18/03/2020 12:45, Lukasz Luba wrote:
>>> From: Matthias Kaehlcke <mka@chromium.org>
>>>
>>> Now that devfreq supports limiting the frequency range of a device
>>> through PM QoS make use of it instead of disabling OPPs that should
>>> not be used.
>>>
>>> The switch from disabling OPPs to PM QoS introduces a subtle behavioral
>>> change in case of conflicting requests (min > max): PM QoS gives
>>> precedence to the MIN_FREQUENCY request, while higher OPPs disabled
>>> with dev_pm_opp_disable() would override MIN_FREQUENCY.
>>>
>>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>>> Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
>>> Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
>>
>> This patch is standalone, right? If yes, I will apply it.
> 
> Yes, it is standalone, please apply

Applied on 'testing', thanks


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-04-03 17:19 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-18 11:45 [PATCH v5 0/5] Add support for devices in the Energy Model Lukasz Luba
2020-03-18 11:45 ` Lukasz Luba
2020-03-18 11:45 ` Lukasz Luba
2020-03-18 11:45 ` [PATCH v5 1/5] PM / EM: add devices to " Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-04-03 16:05   ` Daniel Lezcano
2020-04-03 16:05     ` Daniel Lezcano
2020-04-03 16:05     ` Daniel Lezcano
2020-04-03 16:05     ` Daniel Lezcano
2020-04-06 13:29     ` Lukasz Luba
2020-04-06 13:29       ` Lukasz Luba
2020-04-06 13:29       ` Lukasz Luba
2020-04-06 13:29       ` Lukasz Luba
2020-04-06 14:58       ` Daniel Lezcano
2020-04-06 14:58         ` Daniel Lezcano
2020-04-06 14:58         ` Daniel Lezcano
2020-04-06 14:58         ` Daniel Lezcano
2020-04-06 16:07         ` Lukasz Luba
2020-04-06 16:07           ` Lukasz Luba
2020-04-06 16:07           ` Lukasz Luba
2020-04-06 16:07           ` Lukasz Luba
2020-04-06 21:17           ` Daniel Lezcano
2020-04-06 21:17             ` Daniel Lezcano
2020-04-06 21:17             ` Daniel Lezcano
2020-04-06 21:17             ` Daniel Lezcano
2020-04-07  9:32             ` Lukasz Luba
2020-04-07  9:32               ` Lukasz Luba
2020-04-07  9:32               ` Lukasz Luba
2020-04-07  9:32               ` Lukasz Luba
2020-03-18 11:45 ` [PATCH v5 2/5] OPP: refactor dev_pm_opp_of_register_em() and update related drivers Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-04-01  7:19   ` Lukasz Luba
2020-04-01  7:19     ` Lukasz Luba
2020-04-01  7:19     ` Lukasz Luba
2020-04-01  7:19     ` Lukasz Luba
2020-04-03 16:21   ` Daniel Lezcano
2020-04-03 16:21     ` Daniel Lezcano
2020-04-03 16:21     ` Daniel Lezcano
2020-04-03 16:21     ` Daniel Lezcano
2020-04-06 14:05     ` Lukasz Luba
2020-04-06 14:05       ` Lukasz Luba
2020-04-06 14:05       ` Lukasz Luba
2020-04-06 14:05       ` Lukasz Luba
2020-03-18 11:45 ` [PATCH v5 3/5] thermal: devfreq_cooling: Use PM QoS to set frequency limits Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-04-03 16:43   ` Daniel Lezcano
2020-04-03 16:43     ` Daniel Lezcano
2020-04-03 16:43     ` Daniel Lezcano
2020-04-03 16:43     ` Daniel Lezcano
2020-04-03 17:18     ` Matthias Kaehlcke
2020-04-03 17:18       ` Matthias Kaehlcke
2020-04-03 17:18       ` Matthias Kaehlcke
2020-04-03 17:18       ` Matthias Kaehlcke
2020-04-03 17:19       ` Daniel Lezcano [this message]
2020-04-03 17:19         ` Daniel Lezcano
2020-04-03 17:19         ` Daniel Lezcano
2020-04-03 17:19         ` Daniel Lezcano
2020-03-18 11:45 ` [PATCH v5 4/5] thermal: devfreq_cooling: Refactor code and switch to use Energy Model Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-04-01 10:49   ` Lukasz Luba
2020-04-01 10:49     ` Lukasz Luba
2020-04-01 10:49     ` Lukasz Luba
2020-04-01 10:49     ` Lukasz Luba
2020-04-03 17:44   ` Daniel Lezcano
2020-04-03 17:44     ` Daniel Lezcano
2020-04-03 17:44     ` Daniel Lezcano
2020-04-03 17:44     ` Daniel Lezcano
2020-04-06 13:35     ` Lukasz Luba
2020-04-06 13:35       ` Lukasz Luba
2020-04-06 13:35       ` Lukasz Luba
2020-04-06 13:35       ` Lukasz Luba
2020-03-18 11:45 ` [PATCH v5 5/5] drm/panfrost: Register devfreq cooling and attempt to add " Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 11:45   ` Lukasz Luba
2020-03-18 13:11   ` Alyssa Rosenzweig
2020-03-18 13:11     ` Alyssa Rosenzweig
2020-03-18 13:11     ` Alyssa Rosenzweig
2020-03-18 13:11     ` Alyssa Rosenzweig
2020-03-23 13:50     ` Lukasz Luba
2020-03-23 13:50       ` Lukasz Luba
2020-03-23 13:50       ` Lukasz Luba
2020-03-23 13:50       ` Lukasz Luba

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=82997cd2-1e4d-fb9d-8c64-6771b72456e9@linaro.org \
    --to=daniel.lezcano@linaro.org \
    --cc=Dietmar.Eggemann@arm.com \
    --cc=Morten.Rasmussen@arm.com \
    --cc=agross@kernel.org \
    --cc=airlied@linux.ie \
    --cc=alyssa.rosenzweig@collabora.com \
    --cc=amit.kucheria@verdurent.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=bsegall@google.com \
    --cc=cw00.choi@samsung.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=javi.merino@arm.com \
    --cc=juri.lelli@redhat.com \
    --cc=kernel@pengutronix.de \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=liviu.dudau@arm.com \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=lukasz.luba@arm.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mgorman@suse.de \
    --cc=mingo@redhat.com \
    --cc=mka@chromium.org \
    --cc=nm@ti.com \
    --cc=orjan.eide@arm.com \
    --cc=patrick.bellasi@matbug.net \
    --cc=peterz@infradead.org \
    --cc=qperret@google.com \
    --cc=rdunlap@infradead.org \
    --cc=rjw@rjwysocki.net \
    --cc=robh@kernel.org \
    --cc=rostedt@goodmis.org \
    --cc=rui.zhang@intel.com \
    --cc=s.hauer@pengutronix.de \
    --cc=sboyd@kernel.org \
    --cc=shawnguo@kernel.org \
    --cc=steven.price@arm.com \
    --cc=sudeep.holla@arm.com \
    --cc=tomeu.vizoso@collabora.com \
    --cc=vincent.guittot@linaro.org \
    --cc=viresh.kumar@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.