[PATCH V2 1/3] cpuidle: play_idle: Make play_idle more flexible

2019-10-03 Thread Daniel Lezcano
state. The current code keeps the default behavior which is go to the deepest idle state. Signed-off-by: Daniel Lezcano Acked-by: Mathieu Poirier --- drivers/cpuidle/cpuidle.c | 21 +++-- include/linux/cpuidle.h | 13 ++--- kernel/sched/idle.c | 10 +- 3

[PATCH V2 2/3] cpuidle: play_idle: Specify play_idle with an idle state

2019-10-03 Thread Daniel Lezcano
Currently, the play_idle function does not allow to tell which idle state we want to go. Improve this by passing the idle state as parameter to the function. There is no functional changes, the cpuidle state is the deepest one. Signed-off-by: Daniel Lezcano Acked-by: Mathieu Poirier

Re: [PATCH v5 3/3] clocksource/drivers: Suspend/resume Hyper-V clocksource for hibernation

2019-09-26 Thread Daniel Lezcano
On 26/09/2019 01:35, Dexuan Cui wrote: >> From: Daniel Lezcano >> Sent: Wednesday, September 25, 2019 4:21 PM >> To: Dexuan Cui ; a...@arndb.de; b...@alien8.de; >> Haiyang Zhang ; h...@zytor.com; KY Srinivasan >> ; linux-hyp...@vger.kernel.org; >> linux-kern

Re: [PATCH v5 3/3] clocksource/drivers: Suspend/resume Hyper-V clocksource for hibernation

2019-09-25 Thread Daniel Lezcano
On 06/09/2019 00:47, Dexuan Cui wrote: > This is needed for hibernation, e.g. when we resume the old kernel, we need > to disable the "current" kernel's TSC page and then resume the old kernel's. > > Signed-off-by: Dexuan Cui > Reviewed-by: Michael Kelley I can take this patch if needed. >

Re: [PATCH 0/7] add support for clocksource/clockevent DT selection

2019-09-25 Thread Daniel Lezcano
Hi Claudiu, On 10/09/2019 15:47, Claudiu Beznea wrote: > Hi, > > This series adds support to permit the selection of clocksource/clockevent > via DT. Thanks for the proposal and taking care of making some progress on this. I just wanted to let you know I've been traveling but the series is

Re: [PATCH] clocksource: mediatek: fix error handling

2019-09-25 Thread Daniel Lezcano
On 19/09/2019 21:13, Fabien Parent wrote: > When timer_of_init fails, it cleans up after itself by undoing > everything it did during the initialization function. > > mtk_syst_init and mtk_gpt_init both call timer_of_cleanup if > timer_of_init fails. timer_of_cleanup try to release the resource

Re: [PATCH 3/3] powercap/drivers/idle_inject: Specify the idle state to inject

2019-09-24 Thread Daniel Lezcano
Hi Mathieu, On 18/09/2019 21:35, Mathieu Poirier wrote: > On Mon, Sep 09, 2019 at 04:50:15PM +0200, Daniel Lezcano wrote: >> Currently the idle injection framework only allows to inject the >> deepest idle state available on the system. >> >> Give the opportunity to

Re: [PATCH] thermal: qoriq: add thermal monitor unit version 2 support

2019-09-23 Thread Daniel Lezcano
On 23/09/2019 11:24, Andy Tang wrote: > Hi Rui, Edubezval, > > Would you please review this patch? Eduardo, can you give an update about the thermal maintenance discussion we had at LPC2019? Thanks -- Daniel >> -Original Message- >> From: Andy Tang >> Sent: 2019年8月29日 16:38 >> To:

Re: [PATCH 11/32] clocksource: samsung_pwm_timer: Use pr_warn instead of pr_warning

2019-09-20 Thread Daniel Lezcano
On 20/09/2019 08:25, Kefeng Wang wrote: > As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of > pr_warning"), removing pr_warning so all logging messages use a > consistent _warn style. Let's do it. > > Cc: Daniel Lezcano > Signed-off-by: Kefeng Wan

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-18 Thread Daniel Lezcano
On 18/09/2019 11:24, Viresh Kumar wrote: > On 13-09-19, 00:33, Daniel Lezcano wrote: >> >> Hi Adam, >> >> On 12/09/2019 23:19, Adam Ford wrote: >>> On Thu, Sep 12, 2019 at 4:12 PM Daniel Lezcano >>> wrote: >>>> >>>> On 12/

Re: [PATCH 5/5] cpufreq: qcom-hw: Move driver initialisation earlier

2019-09-17 Thread Daniel Lezcano
On 12/09/2019 00:32, Amit Kucheria wrote: > Allow qcom-hw driver to initialise right after the cpufreq and thermal > subsystems are initialised in core_initcall so we get earlier access to > thermal mitigation. > > Signed-off-by: Amit Kucheria Acked-by: Daniel Lezcano > ---

Re: [PATCH 5/5] cpufreq: qcom-hw: Move driver initialisation earlier

2019-09-17 Thread Daniel Lezcano
Hi Sudeep, On 17/09/2019 11:34, Sudeep Holla wrote: > On Thu, Sep 12, 2019 at 04:02:34AM +0530, Amit Kucheria wrote: >> Allow qcom-hw driver to initialise right after the cpufreq and thermal >> subsystems are initialised in core_initcall so we get earlier access to >> thermal mitigation. >> >>

Re: [PATCH 1/5] thermal: Initialize thermal subsystem earlier

2019-09-16 Thread Daniel Lezcano
On 12/09/2019 00:32, Amit Kucheria wrote: > From: Lina Iyer > > Now that the thermal framework is built-in, in order to facilitate > thermal mitigation as early as possible in the boot cycle, move the > thermal framework initialization to core_initcall. > > However, netlink initialization

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-14 Thread Daniel Lezcano
Hi Nikolauss, On 13/ 09/2019 22:34, H. Nikolaus Schaller wrote: [ ... ] >> The governor continues to read the temperature and see the temperature >> decrease, it does nothing. > > Ah, I think our misunderstanding is that the govenor "enables" and > "disables" a set of OPPs. Rather it goes

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-13 Thread Daniel Lezcano
On 13/09/2019 20:46, Adam Ford wrote: > On Fri, Sep 13, 2019 at 12:18 PM Daniel Lezcano > wrote: >> >> On 13/09/2019 18:51, H. Nikolaus Schaller wrote: >> >> [ ... ] >> >>>> Good news (I think) >>>> >>>> With cooling-device

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-13 Thread Daniel Lezcano
On 13/09/2019 18:51, H. Nikolaus Schaller wrote: [ ... ] >> Good news (I think) >> >> With cooling-device = < 1 2> setup, I was able to ask the max >> frequency and it returned 600MHz. >> >> # cat /sys/devices/virtual/thermal/thermal_zone0/temp >> 58500 >> # cat

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-12 Thread Daniel Lezcano
Hi Adam, On 12/09/2019 23:19, Adam Ford wrote: > On Thu, Sep 12, 2019 at 4:12 PM Daniel Lezcano > wrote: >> >> On 12/09/2019 20:30, Adam Ford wrote: >>> The thermal sensor in the omap3 family isn't accurate, but it's >>> better than nothing. The va

Re: [RFC] ARM: dts: omap36xx: Enable thermal throttling

2019-09-12 Thread Daniel Lezcano
On 12/09/2019 20:30, Adam Ford wrote: > The thermal sensor in the omap3 family isn't accurate, but it's > better than nothing. The various OPP's enabled for the omap3630 > support up to OPP1G, however the datasheet for the DM3730 states > that OPP130 and OPP1G are not available above TJ of 90C. >

Re: [PATCH 1/2] arm64: dts: imx8mm: Remove incorrect fallback compatible for ocotp

2019-09-11 Thread Daniel Lezcano
On Wed, Sep 11, 2019 at 09:05:49AM +, Anson Huang wrote: > Hi, Daniel > > > On 11/09/2019 16:24, Anson Huang wrote: > > > Compared to i.MX7D, i.MX8MM has different ocotp layout, so it should > > > NOT use "fsl,imx7d-ocotp" as ocotp's fallback compatible, remove it. > > > > > > Signed-off-by:

Re: [PATCH 1/2] arm64: dts: imx8mm: Remove incorrect fallback compatible for ocotp

2019-09-11 Thread Daniel Lezcano
On 11/09/2019 16:24, Anson Huang wrote: > Compared to i.MX7D, i.MX8MM has different ocotp layout, so it should > NOT use "fsl,imx7d-ocotp" as ocotp's fallback compatible, remove it. > > Signed-off-by: Anson Huang > --- > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 2 +- > 1 file changed, 1

[PATCH 3/3] powercap/drivers/idle_inject: Specify the idle state to inject

2019-09-09 Thread Daniel Lezcano
, the cpuidle state is the deepest one. Signed-off-by: Daniel Lezcano --- drivers/powercap/idle_inject.c | 15 ++- include/linux/idle_inject.h| 3 +++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/drivers/powercap/idle_inject.c b/drivers/powercap/idle_inject.c index

[PATCH 2/3] cpuidle: play_idle: Specify play_idle with an idle state

2019-09-09 Thread Daniel Lezcano
Currently, the play_idle function does not allow to tell which idle state we want to go. Improve this by passing the idle state as parameter to the function. There is no functional changes, the cpuidle state is the deepest one. Signed-off-by: Daniel Lezcano --- drivers/powercap/idle_inject.c

[PATCH 1/3] cpuidle: play_idle: Make play_idle more flexible

2019-09-09 Thread Daniel Lezcano
state. The current code keeps the default behavior which is go to the deepest idle state. Signed-off-by: Daniel Lezcano --- drivers/cpuidle/cpuidle.c | 21 +++-- include/linux/cpuidle.h | 14 +++--- kernel/sched/idle.c | 10 +- 3 files changed, 23

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-05 Thread Daniel Lezcano
Hi, On 06/09/2019 03:48, Ming Lei wrote: [ ... ] >> You did not share yet the analysis of the problem (the kernel warnings >> give the symptoms) and gave the reasoning for the solution. It is hard >> to understand what you are looking for exactly and how to connect the dots. > > Let me

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-05 Thread Daniel Lezcano
On 06/09/2019 03:22, Long Li wrote: [ ... ] > > Tracing shows that the CPU was in either hardirq or softirq all the > time before warnings. During tests, the system was unresponsive at > times. > > Ming's patch fixed this problem. The system was responsive throughout > tests. > > As for

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-05 Thread Daniel Lezcano
Hi Ming, On 05/09/2019 11:06, Ming Lei wrote: > On Wed, Sep 04, 2019 at 07:31:48PM +0200, Daniel Lezcano wrote: >> Hi, >> >> On 04/09/2019 19:07, Bart Van Assche wrote: >>> On 9/3/19 12:50 AM, Daniel Lezcano wrote: >>>> On 03/09/2019 09:28, Ming Lei

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-04 Thread Daniel Lezcano
Hi, On 04/09/2019 19:07, Bart Van Assche wrote: > On 9/3/19 12:50 AM, Daniel Lezcano wrote: >> On 03/09/2019 09:28, Ming Lei wrote: >>> On Tue, Sep 03, 2019 at 08:40:35AM +0200, Daniel Lezcano wrote: >>>> It is a scheduler problem then ? >>> >>&

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-03 Thread Daniel Lezcano
On 03/09/2019 09:28, Ming Lei wrote: > On Tue, Sep 03, 2019 at 08:40:35AM +0200, Daniel Lezcano wrote: >> On 03/09/2019 08:31, Ming Lei wrote: >>> Hi Daniel, >>> >>> On Tue, Sep 03, 2019 at 07:59:39AM +0200, Daniel Lezcano wrote: >>>> >>>>

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-03 Thread Daniel Lezcano
On 03/09/2019 08:31, Ming Lei wrote: > Hi Daniel, > > On Tue, Sep 03, 2019 at 07:59:39AM +0200, Daniel Lezcano wrote: >> >> Hi Ming Lei, >> >> On 03/09/2019 05:30, Ming Lei wrote: >> >> [ ... ] >> >> >>>>> 2) irq/timing does

Re: [PATCH 1/4] softirq: implement IRQ flood detection mechanism

2019-09-02 Thread Daniel Lezcano
Hi Ming Lei, On 03/09/2019 05:30, Ming Lei wrote: [ ... ] >>> 2) irq/timing doesn't cover softirq >> >> That's solvable, right? > > Yeah, we can extend irq/timing, but ugly for irq/timing, since irq/timing > focuses on hardirq predication, and softirq isn't involved in that > purpose. > >>

Re: [PATCH 5/6] dt-bindings: timer: renesas: tmu: Document r8a774a1 bindings

2019-09-02 Thread Daniel Lezcano
;>> Is anybody willing to take this patch? >> >> <2c> I think Geert can take this > > If the timer people won't take it for v5.4, I can queue it in renesas-devel > for v5.5, in my branch for DT binding updates for subsystems that are > less DT-centric. Please d

Re: [PATCH 0/5] dt-bindings: arm: idle-states: Miscellaneous improvements

2019-08-30 Thread Daniel Lezcano
- Add punctuation marks to improve readability, > - Move exit-latency-us explanation to exit-latency-us section. > > Signed-off-by: Geert Uytterhoeven Acked-by: Daniel Lezcano > --- > Split to ease review. > Feel free to squash into a single commit. > > Geert Uytterh

Re: Default governor regardless of cpuidle driver

2019-08-29 Thread Daniel Lezcano
On 29/08/2019 23:12, Joao Martins wrote: [ ... ] >>> Say you wanted to have a kvm specific config, you would still see the same >>> problem if you happen to compile intel_idle together with haltpoll >>> driver+governor. >> >> Can a guest work with an intel_idle driver? >> > Yes. > > If you use

Re: Default governor regardless of cpuidle driver

2019-08-29 Thread Daniel Lezcano
On 29/08/2019 21:11, Joao Martins wrote: > On 8/29/19 7:28 PM, Daniel Lezcano wrote: >> On 29/08/2019 20:07, Joao Martins wrote: >>> On 8/29/19 6:42 PM, Daniel Lezcano wrote: >>>> On 29/08/2019 19:16, Joao Martins wrote: >>>>> On 8/29/19 4:10 PM, Jo

Re: Is: Default governor regardless of cpuidle driver Was: [PATCH v2] cpuidle-haltpoll: vcpu hotplug support

2019-08-29 Thread Daniel Lezcano
On 29/08/2019 20:07, Joao Martins wrote: > On 8/29/19 6:42 PM, Daniel Lezcano wrote: >> On 29/08/2019 19:16, Joao Martins wrote: >>> On 8/29/19 4:10 PM, Joao Martins wrote: >>>> When cpus != maxcpus cpuidle-haltpoll will fail to register all vcpus >>&

Re: Is: Default governor regardless of cpuidle driver Was: [PATCH v2] cpuidle-haltpoll: vcpu hotplug support

2019-08-29 Thread Daniel Lezcano
On 29/08/2019 19:16, Joao Martins wrote: > On 8/29/19 4:10 PM, Joao Martins wrote: >> When cpus != maxcpus cpuidle-haltpoll will fail to register all vcpus >> past the online ones and thus fail to register the idle driver. >> This is because cpuidle_add_sysfs() will return with -ENODEV as a >>

[GIT PULL] timers drivers v5.4

2019-08-26 Thread Daniel Lezcano
Hi Thomas, The following changes since commit 3e2d94535adb2df15f3907e4b4c7cd8a5a4c2b5a: clocksource/drivers/hyperv: Enable TSC page clocksource on 32bit (2019-08-23 16:59:54 +0200) are available in the Git repository at: https://git.linaro.org/people/daniel.lezcano/linux.git

Re: [GIT PULL] timers drivers v5.5

2019-08-26 Thread Daniel Lezcano
On 26/08/2019 22:59, Thomas Gleixner wrote: > On Mon, 26 Aug 2019, Daniel Lezcano wrote: > >> The following changes since commit 08a3c192c93f4359a94bf47971e55b0324b72b8b: >> >> posix-timers: Prepare for PREEMPT_RT (2019-08-01 20:51:25 +0200) >> >>

[PATCH 12/20] clocksource/drivers/timer-of: Do not warn on deferred probe

2019-08-26 Thread Daniel Lezcano
From: Jon Hunter Deferred probe is an expected return value for clk_get() on many platforms. The driver deals with it properly, so there's no need to output a warning that may potentially confuse users. Signed-off-by: Jon Hunter Signed-off-by: Daniel Lezcano --- drivers/clocksource/timer

[PATCH 10/20] clocksource/drivers/renesas-ostm: Use DIV_ROUND_CLOSEST() helper

2019-08-26 Thread Daniel Lezcano
From: Geert Uytterhoeven Use the DIV_ROUND_CLOSEST() helper instead of open-coding the same operation. Signed-off-by: Geert Uytterhoeven Reviewed-by: Simon Horman Signed-off-by: Daniel Lezcano --- drivers/clocksource/renesas-ostm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 09/20] arm64: dts: imx8mq: Add system counter node

2019-08-26 Thread Daniel Lezcano
From: Anson Huang Add i.MX8MQ system counter node to enable timer-imx-sysctr broadcast timer driver. Signed-off-by: Anson Huang Signed-off-by: Daniel Lezcano --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm64/boot/dts

[PATCH 04/20] clocksource: sun4i: Add missing compatibles

2019-08-26 Thread Daniel Lezcano
From: Maxime Ripard Newer Allwinner SoCs have different number of interrupts, let's add different compatibles for all of them to deal with this properly. Signed-off-by: Maxime Ripard Acked-by: Daniel Lezcano Signed-off-by: Daniel Lezcano --- drivers/clocksource/timer-sun4i.c | 4 1

[PATCH 11/20] clocksource/drivers/npcm: Fix GENMASK and timer operation

2019-08-26 Thread Daniel Lezcano
for the sake of clarity. Fixes: 1c00289ecd12 ("clocksource/drivers/npcm: Add NPCM7xx timer driver") Signed-off-by: Avi Fishman Signed-off-by: Daniel Lezcano --- drivers/clocksource/timer-npcm7xx.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/clockso

[PATCH 14/20] dt-bindings: timer: renesas, cmt: Add CMT0234 to sh73a0 and r8a7740

2019-08-26 Thread Daniel Lezcano
ob Herring Reviewed-by: Simon Horman Reviewed-by: Geert Uytterhoeven Signed-off-by: Daniel Lezcano --- Documentation/devicetree/bindings/timer/renesas,cmt.txt | 8 1 file changed, 8 insertions(+) diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation/devic

[PATCH 19/20] clocksource/drivers/sh_cmt: r8a7740 and sh73a0 SoC-specific match

2019-08-26 Thread Daniel Lezcano
esas,-" Signed-off-by: Magnus Damm Reviewed-by: Simon Horman Reviewed-by: Geert Uytterhoeven Signed-off-by: Daniel Lezcano --- drivers/clocksource/sh_cmt.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/clocksource/sh_cmt.c b/drivers/clocksource/sh_cmt.c index f6424b6

[PATCH 20/20] clocksource/drivers/sh_cmt: Document "cmt-48" as deprecated

2019-08-26 Thread Daniel Lezcano
DT compat string. In case of the only in-tree users r8a7740 and sh73a0 the compat strings "renesas,r8a7740-cmt1" and "renesas,sh73a0-cmt1" may be used instead. Signed-off-by: Magnus Damm Reviewed-by: Simon Horman Reviewed-by: Geert Uytterhoeven Signed-off-by: Daniel Lezcano ---

[PATCH 18/20] dt-bindings: timer: renesas, cmt: Update R-Car Gen3 CMT1 usage

2019-08-26 Thread Daniel Lezcano
en Reviewed-by: Rob Herring Reviewed-by: Simon Horman Signed-off-by: Daniel Lezcano --- .../devicetree/bindings/timer/renesas,cmt.txt | 20 +-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt b/Documentation

[PATCH 16/20] dt-bindings: timer: renesas, cmt: Add CMT0 and CMT1 to r8a7792

2019-08-26 Thread Daniel Lezcano
From: Magnus Damm This patch adds DT binding documentation for the CMT devices on the R-Car Gen2 V2H (r8a7792) SoC. Signed-off-by: Magnus Damm Reviewed-by: Geert Uytterhoeven Reviewed-by: Rob Herring Reviewed-by: Simon Horman Signed-off-by: Daniel Lezcano --- Documentation/devicetree

[PATCH 17/20] dt-bindings: timer: renesas, cmt: Add CMT0 and CMT1 to r8a77995

2019-08-26 Thread Daniel Lezcano
From: Magnus Damm This patch adds DT binding documentation for the CMT devices on the R-Car Gen3 D3 (r8a77995) SoC. Signed-off-by: Magnus Damm Reviewed-by: Geert Uytterhoeven Reviewed-by: Rob Herring Reviewed-by: Simon Horman Signed-off-by: Daniel Lezcano --- Documentation/devicetree

[PATCH 13/20] clocksource/drivers: Do not warn on probe defer

2019-08-26 Thread Daniel Lezcano
From: Jon Hunter Deferred probe is an expected return value on many platforms and so there's no need to output a warning that may potentially confuse users. Signed-off-by: Jon Hunter Signed-off-by: Daniel Lezcano --- drivers/clocksource/timer-probe.c | 4 +++- 1 file changed, 3 insertions

[PATCH 15/20] dt-bindings: timer: renesas, cmt: Update CMT1 on sh73a0 and r8a7740

2019-08-26 Thread Daniel Lezcano
is removed from the DT binding documentation, however software support for this deprecated binding will still remain in the CMT driver for some time. Signed-off-by: Magnus Damm Reviewed-by: Rob Herring Reviewed-by: Simon Horman Reviewed-by: Geert Uytterhoeven Signed-off-by: Dani

[PATCH 06/20] clocksource/drivers/tcb_clksrc: Register delay timer

2019-08-26 Thread Daniel Lezcano
From: Alexandre Belloni Implement and register delay timer to allow get_cycles() to work properly. Signed-off-by: Alexandre Belloni Signed-off-by: Daniel Lezcano --- drivers/clocksource/Kconfig | 2 +- drivers/clocksource/timer-atmel-tcb.c | 18 ++ 2 files changed

[PATCH 05/20] dt-bindings: timer: Convert Allwinner A13 HSTimer to a schema

2019-08-26 Thread Daniel Lezcano
-by: Rob Herring Signed-off-by: Daniel Lezcano --- .../timer/allwinner,sun5i-a13-hstimer.txt | 26 -- .../timer/allwinner,sun5i-a13-hstimer.yaml| 79 +++ 2 files changed, 79 insertions(+), 26 deletions(-) delete mode 100644 Documentation/devicetree/bindings/timer

[PATCH 08/20] arm64: dts: imx8mm: Add system counter node

2019-08-26 Thread Daniel Lezcano
From: Anson Huang Add i.MX8MM system counter node to enable timer-imx-sysctr broadcast timer driver. Signed-off-by: Anson Huang Signed-off-by: Daniel Lezcano --- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm64/boot/dts

[PATCH 03/20] dt-bindings: timer: Add missing compatibles

2019-08-26 Thread Daniel Lezcano
From: Maxime Ripard Newer Allwinner SoCs have different number of interrupts, let's add different compatibles for all of them to deal with this properly. Signed-off-by: Maxime Ripard Reviewed-by: Rob Herring Signed-off-by: Daniel Lezcano --- .../timer/allwinner,sun4i-a10-timer.yaml

[PATCH 02/20] dt-bindings: timer: Convert Allwinner A10 Timer to a schema

2019-08-26 Thread Daniel Lezcano
for that controller over to a YAML schemas. Signed-off-by: Maxime Ripard Reviewed-by: Rob Herring Signed-off-by: Daniel Lezcano --- .../timer/allwinner,sun4i-a10-timer.yaml | 76 +++ .../bindings/timer/allwinner,sun4i-timer.txt | 19 - 2 files changed, 76 insertions(+), 19

[PATCH 07/20] clocksource/drivers/imx-sysctr: Add internal clock divider handle

2019-08-26 Thread Daniel Lezcano
Signed-off-by: Daniel Lezcano --- drivers/clocksource/timer-imx-sysctr.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/clocksource/timer-imx-sysctr.c b/drivers/clocksource/timer-imx-sysctr.c index fd7d68066efb..b7c80a368a1b 100644 --- a/drivers/clocksource/timer-imx-sysctr.c

[PATCH 01/20] clocksource: Remove dev_err() usage after platform_get_irq()

2019-08-26 Thread Daniel Lezcano
an Cc: Daniel Lezcano Cc: Thomas Gleixner Signed-off-by: Stephen Boyd Reviewed-by: Geert Uytterhoeven Signed-off-by: Daniel Lezcano --- drivers/clocksource/em_sti.c | 4 +--- drivers/clocksource/sh_cmt.c | 5 + drivers/clocksource/sh_tmu.c | 5 + 3 files changed, 3 insertions(+), 11 del

[GIT PULL] timers drivers v5.5

2019-08-26 Thread Daniel Lezcano
The following changes since commit 08a3c192c93f4359a94bf47971e55b0324b72b8b: posix-timers: Prepare for PREEMPT_RT (2019-08-01 20:51:25 +0200) are available in the Git repository at: https://git.linaro.org/people/daniel.lezcano/linux.git tags/timers-v5.5 for you to fetch changes up to

Re: [PATCH 1/2] clocksource/drivers/timer-of: Do not warn on deferred probe

2019-08-22 Thread Daniel Lezcano
On 21/08/2019 17:02, Jon Hunter wrote: > Deferred probe is an expected return value for clk_get() on many > platforms. The driver deals with it properly, so there's no need > to output a warning that may potentially confuse users. > > Signed-off-by: Jon Hunter > --- Applied, thanks! >

Re: [PATCH v2 0/7] renesas, cmt: DT Binding Documentation and Minor Driver Updates V2

2019-08-22 Thread Daniel Lezcano
On 20/08/2019 14:34, Magnus Damm wrote: > renesas, cmt: DT Binding Documentation and Minor Driver Updates V2 > > [PATCH v2 1/7] dt-bindings: timer: renesas, cmt: Add CMT0234 to sh73a0 and > r8a7740 > [PATCH v2 2/7] dt-bindings: timer: renesas, cmt: Update CMT1 on sh73a0 and > r8a7740 > [PATCH

Re: [PATCH] [v5] clocksource/drivers/npcm: fix GENMASK and timer operation

2019-08-21 Thread Daniel Lezcano
On 29/07/2019 19:03, Avi Fishman wrote: > NPCM7XX_Tx_OPER GENMASK bits where wrong, > Since NPCM7XX_REG_TICR0 register reset value of those bits was 0, > it did not cause an issue. > in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was > read again after masking it with

Re: [PATCH 03/15] drivers: thermal: tsens: Add __func__ identifier to debug statements

2019-08-19 Thread Daniel Lezcano
On 26/07/2019 00:18, Amit Kucheria wrote: > Printing the function name when enabling debugging makes logs easier to > read. > > Signed-off-by: Amit Kucheria > --- Reviewed-by: Daniel Lezcano -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Re: [PATCH 02/15] drivers: thermal: tsens: Simplify code flow in tsens_probe

2019-08-19 Thread Daniel Lezcano
On 26/07/2019 00:18, Amit Kucheria wrote: > Move platform_set_drvdata up to avoid an extra 'if (ret)' check after > the call to tsens_register. > > Signed-off-by: Amit Kucheria Reviewed-by: Daniel Lezcano [ ... ] -- <http://www.linaro.org/> Linaro.org │ Open source sof

Re: [PATCH 01/15] drivers: thermal: tsens: Get rid of id field in tsens_sensor

2019-08-19 Thread Daniel Lezcano
op through the list of sensors, and then > return the actual hw_id that is registered on that index. > > Signed-off-by: Amit Kucheria Reviewed-by: Daniel Lezcano -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Lin

Re: [PATCH v2 1/5] RISC-V: Remove per cpu clocksource

2019-08-16 Thread Daniel Lezcano
On 31/07/2019 03:24, Atish Patra wrote: > There is only one clocksource in RISC-V. The boot cpu initializes > that clocksource. No need to keep a percpu data structure. That is not what is stated in the initial patch [1]. Can you clarify that ? Thanks -- Daniel [1]

Re: [PATCH V2 1/2] clocksource/Hyper-v: Allocate Hyper-V tsc page statically

2019-08-16 Thread Daniel Lezcano
method of getting its physical address. > > Signed-off-by: Tianyu Lan Acked-by: Daniel Lezcano > --- > Change since v1: >- Update commit log >- Remove and operation of tsc page's va with PAGE_MASK > > arch/x86/entry/vdso/vma.c

Re: [PATCH] clocksource: Add driver for the Ingenic JZ47xx OST

2019-08-16 Thread Daniel Lezcano
On 09/08/2019 14:38, Paul Cercueil wrote: > From: Maarten ter Huurne > > OST is the OS Timer, a 64-bit timer/counter with buffered reading. > > SoCs before the JZ4770 had (if any) a 32-bit OST; the JZ4770 and > JZ4780 have a 64-bit OST. > > This driver will register both a clocksource and a

Re: [PATCH v3 1/3] clocksource/drivers/renesas-ostm: Use DIV_ROUND_CLOSEST() helper

2019-08-16 Thread Daniel Lezcano
On 16/08/2019 14:58, Simon Horman wrote: > On Wed, Aug 07, 2019 at 10:46:33AM +0200, Geert Uytterhoeven wrote: >> Use the DIV_ROUND_CLOSEST() helper instead of open-coding the same >> operation. >> >> Signed-off-by: Geert Uytterhoeven > > Reviewed-by: Simon Horman Applied this one, thanks I'm

Re: [PATCH V6 4/4] arm64: dts: imx8mm: Enable cpu-idle driver

2019-08-16 Thread Daniel Lezcano
On 16/08/2019 02:38, Anson Huang wrote: > Enable i.MX8MM cpu-idle using generic ARM cpu-idle driver, 2 states > are supported, details as below: > > root@imx8mmevk:~# cat /sys/devices/system/cpu/cpu0/cpuidle/state0/name > WFI > root@imx8mmevk:~# cat

Re: [PATCH V5 5/5] arm64: dts: imx8mm: Enable cpu-idle driver

2019-08-15 Thread Daniel Lezcano
Hi Anson, sorry for the late review, I've been pretty busy. If Shawn is ok, I can pick the patches 1-4 in my tree and then this one after you fix the comments below. On 10/07/2019 08:30, anson.hu...@nxp.com wrote: [ ... ] > + idle-states { > + entry-method =

Re: [PATCH] thermal: armada: Fix -Wshift-negative-value

2019-08-14 Thread Daniel Lezcano
On 15/08/2019 00:12, Nick Desaulniers wrote: > On Tue, Aug 13, 2019 at 10:28 AM 'Nathan Huckleberry' via Clang Built > Linux wrote: >> >> Following up to see if this patch is going to be accepted. > > Miquel is listed as the maintainer of this file in MAINTAINERS. > Miquel, can you please pick

Re: [PATCH 06/15] arm64: dts: msm8916: thermal: Fixup HW ids for cpu sensors

2019-08-13 Thread Daniel Lezcano
a This change matches the documentation available in the Qcom website. Reviewed-by: Daniel Lezcano > --- > arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi > b/arch/arm64/b

Re: [PATCH 11/22] clocksource: ti-dmtimer: avoid using mach/hardware.h

2019-08-09 Thread Daniel Lezcano
On 09/08/2019 09:33, Arnd Bergmann wrote: > On Fri, Aug 9, 2019 at 9:01 AM Daniel Lezcano > wrote: >> >> On 08/08/2019 23:22, Arnd Bergmann wrote: >>> As a preparation for future omap1 multiplatform support, stop >>> using mach/hardware.h and instead incl

Re: [PATCH 11/22] clocksource: ti-dmtimer: avoid using mach/hardware.h

2019-08-09 Thread Daniel Lezcano
On 08/08/2019 23:22, Arnd Bergmann wrote: > As a preparation for future omap1 multiplatform support, stop > using mach/hardware.h and instead include the omap1-io.h > for low-level register access to MOD_CONF_CTRL_1. > > Signed-off-by: Arnd Bergmann > --- Do you want me to apply it through me

Re: [PATCH v6 04/57] clocksource: Remove dev_err() usage after platform_get_irq()

2019-08-09 Thread Daniel Lezcano
| > ... > -dev_err(...); > ) > ... > } > // > > While we're here, remove braces on if statements that only have one > statement (manually). > > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: Thomas Gleixner > Signed-off-by: Stephen Boyd > --- >

Re: [PATCH v3 2/3] clocksource/drivers/renesas-ostm: Fix probe error path

2019-08-07 Thread Daniel Lezcano
On 07/08/2019 10:46, Geert Uytterhoeven wrote: > Fix various issues in the error path of ostm_init(): > 1. Drop error message printing on of_iomap() failure, as the memory > allocation core already takes of that, > 2. Handle irq_of_parse_and_map() failures correctly: it returns >

Re: [PATCH 6/6] PSCI: cpuidle: Refactor CPU suspend power_state parameter handling

2019-08-07 Thread Daniel Lezcano
gt; Cc: Will Deacon > Cc: Ulf Hansson > Cc: Sudeep Holla > Cc: Daniel Lezcano > Cc: Catalin Marinas > Cc: Mark Rutland > Cc: "Rafael J. Wysocki" > --- AFAICT, Acked-by: Daniel Lezcano -- <http://www.linaro.org/> Linaro.org │ Open source software for

Re: [PATCH 4/6] ARM: psci: cpuidle: Introduce PSCI CPUidle driver

2019-08-07 Thread Daniel Lezcano
selection would clash with the > generic ARM CPUidle driver that provides a PSCI based idle driver > through the arm/arm64 arches back-ends CPU operations. > > Signed-off-by: Lorenzo Pieralisi > Cc: Ulf Hansson > Cc: Sudeep Holla > Cc: Daniel Lezcano > Cc: Mark Rutland > Cc

Re: [PATCH 3/6] drivers: firmware: psci: Decouple checker from generic ARM CPUidle

2019-08-07 Thread Daniel Lezcano
fing the code > in the process. > > Convert the PSCI checker suspend entry function to use > the struct cpuidle_state.enter() function callback. > > Signed-off-by: Lorenzo Pieralisi > Cc: Sudeep Holla > Cc: Mark Rutland Acked-by: Daniel Lezcano > --- > drivers/

Re: [PATCH 2/6] ARM: cpuidle: Remove overzealous error logging

2019-08-07 Thread Daniel Lezcano
be reported or not in the kernel log and do > not log it if the platform does not support CPUidle operations. > > Signed-off-by: Lorenzo Pieralisi > Cc: Ulf Hansson > Cc: Sudeep Holla > Cc: Daniel Lezcano > Cc: "Rafael J. Wysocki" Acked-by: Daniel Lezcano -- <http://www.

Re: [PATCH 1/6] ARM: cpuidle: Remove useless header include

2019-08-07 Thread Daniel Lezcano
On 22/07/2019 17:37, Lorenzo Pieralisi wrote: > The generic ARM CPUidle driver includes by mistake. > > Remove the topology header include. > > Signed-off-by: Lorenzo Pieralisi > Cc: Ulf Hansson > Cc: Sudeep Holla > Cc: Daniel Lezcano > Cc: "Rafael J. Wys

Re: [PATCH v4 1/4] RISC-V: Remove per cpu clocksource

2019-08-07 Thread Daniel Lezcano
Thanks, queued for v5.3-rc4. Please, in the future wait for my: Acked-by: Daniel Lezcano -- <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

Re: [PATCH V5 1/5] clocksource: imx-sysctr: Add internal clock divider handle

2019-08-06 Thread Daniel Lezcano
On 06/08/2019 03:55, Anson Huang wrote: > Gentle ping... Coming back from vacation. It is in the pipe ... :) >> From: Anson Huang >> >> The system counter block guide states that the base clock is internally >> divided >> by 3 before use, that means the clock input of system counter defined in

[PATCH 3/3] powercap/drivers/idle_inject: Specify the idle state to inject

2019-08-05 Thread Daniel Lezcano
Currently the idle injection framework only allows to inject the deepest idle state available on the system. Give the opportunity to specify which idle state we want to inject by adding a new function helper to set the state and use it when calling play_idle(). Signed-off-by: Daniel Lezcano

[PATCH 2/3] cpuidle: play_idle: Specify play_idle with an idle state

2019-08-05 Thread Daniel Lezcano
Currently, the play_idle function does not allow to tell which idle state we want to go. Improve this by passing the idle state as parameter to the function. Signed-off-by: Daniel Lezcano --- drivers/powercap/idle_inject.c | 3 ++- drivers/thermal/intel/intel_powerclamp.c | 4

[PATCH 1/3] cpuidle: play_idle: Make play_idle more flexible

2019-08-05 Thread Daniel Lezcano
to prepare the function to receive an idle state parameter, let's replace the 'use_deepest_state' boolean field with 'use_state' and use this value to enter the specific idle state. The current code keeps the default behavior which is go to the deepest idle state. Signed-off-by: Daniel Lezcano

Re: [PATCH v3 6/7] thermal/drivers/cpu_cooling: Introduce the cpu idle cooling driver

2019-08-05 Thread Daniel Lezcano
On 05/08/2019 09:42, Martin Kepplinger wrote: > On 05.08.19 09:39, Daniel Lezcano wrote: >> On 05/08/2019 08:53, Martin Kepplinger wrote: >> >> [ ... ] >> >>>>> +static s64 cpuidle_cooling_runtime(struct cpuidle_cooling_device >>>>> *idle

Re: [PATCH v3 6/7] thermal/drivers/cpu_cooling: Introduce the cpu idle cooling driver

2019-08-05 Thread Daniel Lezcano
On 05/08/2019 08:53, Martin Kepplinger wrote: [ ... ] >>> +static s64 cpuidle_cooling_runtime(struct cpuidle_cooling_device >>> *idle_cdev) >>> +{ >>> + s64 next_wakeup; >>> + unsigned long state = idle_cdev->state; >>> + >>> + /* >>> +* The function should not be called when there is

Re: [PATCH v3 6/7] thermal/drivers/cpu_cooling: Introduce the cpu idle cooling driver

2019-08-05 Thread Daniel Lezcano
On 05/08/2019 07:11, Martin Kepplinger wrote: > --- [ ... ] >> +static s64 cpuidle_cooling_runtime(struct cpuidle_cooling_device *idle_cdev) >> +{ >> +s64 next_wakeup; >> +unsigned long state = idle_cdev->state; >> + >> +/* >> + * The function should not be called when there is

Re: [PATCH v6 0/3] Make IPA use PM_EM

2019-08-01 Thread Daniel Lezcano
Hi Rui, is it possible to merge these patches? They are acked-by since May. Thanks -- Daniel On 01/08/2019 14:46, Quentin Perret wrote: > Changes in v6 > * > - Added Daniel's and Viresh's Acked-by to all patches > > Changes in v5: > ** > - Changed patch 02 to

Re: [PATCH] cpuidle/drivers/mobile: Add new governor for mobile/embedded systems

2019-07-08 Thread Daniel Lezcano
Hi Rafael, On 04/07/2019 12:14, Rafael J. Wysocki wrote: > On Thursday, June 20, 2019 1:58:08 PM CEST Daniel Lezcano wrote: >> The objective is the same for all the governors: save energy, but at >> the end the governors menu, ladder and teo aim to improve the >> performanc

Re: [PATCH] cpuidle/drivers/mobile: Add new governor for mobile/embedded systems

2019-07-03 Thread Daniel Lezcano
Hi Doug, On 03/07/2019 16:23, Doug Smythies wrote: > Hi Daniel, > > I tried your "mobile" governor, albeit not on a mobile device. > > On 2019.06.20 04:58 Daniel Lezcano wrote: > > ... > >> The mobile governor is a new governor targeting embe

Re: [PATCH RESEND V4 1/5] clocksource: timer-of: Support getting clock frequency from DT

2019-07-02 Thread Daniel Lezcano
On 02/07/2019 11:03, Anson Huang wrote: > Hi, Daniel > >> Hi Anson, >> >> why did you resend the series? > > Previous patch series has build warning and I did NOT notice, sorry for that, > > drivers/clocksource/timer-of.c: In function ‘timer_of_init’: > drivers/clocksource/timer-of.c:185:30:

Re: [PATCH RESEND V4 1/5] clocksource: timer-of: Support getting clock frequency from DT

2019-07-02 Thread Daniel Lezcano
Hi Anson, why did you resend the series? On 02/07/2019 09:55, anson.hu...@nxp.com wrote: > From: Anson Huang > > More and more platforms use platform driver model for clock driver, > so the clock driver is NOT ready during timer initialization phase, > it will cause timer initialization

Re: [PATCH V3 1/5] clocksource: timer-of: Support getting clock frequency from DT

2019-07-01 Thread Daniel Lezcano
Hi Anson, thanks for taking care of adding the clock-frequency handling in the timer-of. On 28/06/2019 05:30, anson.hu...@nxp.com wrote: > From: Anson Huang > > More and more platforms use platform driver model for clock driver, > so the clock driver is NOT ready during timer initialization

Re: [PATCH V4 2/3] thermal/drivers/cpu_cooling: Unregister with the policy

2019-06-28 Thread Daniel Lezcano
On 28/06/2019 12:03, Rafael J. Wysocki wrote: > On Fri, Jun 28, 2019 at 11:58 AM Daniel Lezcano > wrote: >> >> >> On 28/06/2019 11:12, Rafael J. Wysocki wrote: >>> On Thu, Jun 27, 2019 at 11:02 PM Daniel Lezcano >>> wrote: >>>> >>

Re: [PATCH V4 2/3] thermal/drivers/cpu_cooling: Unregister with the policy

2019-06-28 Thread Daniel Lezcano
On 28/06/2019 11:12, Rafael J. Wysocki wrote: > On Thu, Jun 27, 2019 at 11:02 PM Daniel Lezcano > wrote: >> >> Currently the function cpufreq_cooling_register() returns a cooling >> device pointer which is used back as a pointer to call the function >> cpuf

[PATCH V4 3/3] thermal/drivers/cpu_cooling: cpufreq_cooling_register returns an int

2019-06-27 Thread Daniel Lezcano
incorrect things with the returned pointers (like using it to unregister the cooling device). Replace the returned value an integer instead of a pointer to a thermal cooling device structure. Suggested-by: Rafael J. Wysocki Signed-off-by: Daniel Lezcano --- drivers/thermal/cpu_cooling.c

[PATCH V4 2/3] thermal/drivers/cpu_cooling: Unregister with the policy

2019-06-27 Thread Daniel Lezcano
is to unregister the cpu cooling device. As there is no more need of this pointer, remove it. Signed-off-by: Daniel Lezcano Acked-by: Viresh Kumar --- drivers/cpufreq/arm_big_little.c | 9 ++-- drivers/cpufreq/cpufreq.c | 8 ++-- drivers/thermal/cpu_cooling.c

<    3   4   5   6   7   8   9   10   11   12   >