Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-10-14 Thread Marc Zyngier
On 2019-10-14 10:31, Benjamin GAIGNARD wrote: On 9/27/19 2:59 PM, Marc Zyngier wrote: On 2019-09-27 13:44, Benjamin GAIGNARD wrote: On 9/27/19 2:41 PM, Marc Zyngier wrote: On 2019-09-27 13:36, Benjamin GAIGNARD wrote: On 9/27/19 1:22 PM, Marc Zyngier wrote: On 2019-09-27 09:48, Benjamin Gaig

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-10-14 Thread Benjamin GAIGNARD
On 9/27/19 2:59 PM, Marc Zyngier wrote: > On 2019-09-27 13:44, Benjamin GAIGNARD wrote: >> On 9/27/19 2:41 PM, Marc Zyngier wrote: >>> On 2019-09-27 13:36, Benjamin GAIGNARD wrote: On 9/27/19 1:22 PM, Marc Zyngier wrote: > On 2019-09-27 09:48, Benjamin Gaignard wrote: >> Adding always

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Sudeep Holla
On Fri, Sep 27, 2019 at 12:44:55PM +, Benjamin GAIGNARD wrote: [...] > > Even in low-power modes this timer is always powered and clocked so it > is working fine. > Is that tested ? I see only cpu_{on,off} available on this platform with PSCI v0.1. Did you add cpu_suspend, idle-states and the

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Marc Zyngier
On 2019-09-27 13:44, Benjamin GAIGNARD wrote: On 9/27/19 2:41 PM, Marc Zyngier wrote: On 2019-09-27 13:36, Benjamin GAIGNARD wrote: On 9/27/19 1:22 PM, Marc Zyngier wrote: On 2019-09-27 09:48, Benjamin Gaignard wrote: Adding always-on makes arm arch_timer claim to be an high resolution timer

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Benjamin GAIGNARD
On 9/27/19 2:41 PM, Marc Zyngier wrote: > On 2019-09-27 13:36, Benjamin GAIGNARD wrote: >> On 9/27/19 1:22 PM, Marc Zyngier wrote: >>> On 2019-09-27 09:48, Benjamin Gaignard wrote: Adding always-on makes arm arch_timer claim to be an high resolution timer. That is possible because p

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Marc Zyngier
On 2019-09-27 13:36, Benjamin GAIGNARD wrote: On 9/27/19 1:22 PM, Marc Zyngier wrote: On 2019-09-27 09:48, Benjamin Gaignard wrote: Adding always-on makes arm arch_timer claim to be an high resolution timer. That is possible because power mode won't stop clocking the timer. The "always-on" i

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Benjamin GAIGNARD
On 9/27/19 1:22 PM, Marc Zyngier wrote: > On 2019-09-27 09:48, Benjamin Gaignard wrote: >> Adding always-on makes arm arch_timer claim to be an high resolution >> timer. >> That is possible because power mode won't stop clocking the timer. > > The "always-on" is not about the clock. It is about t

Re: [PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Marc Zyngier
On 2019-09-27 09:48, Benjamin Gaignard wrote: Adding always-on makes arm arch_timer claim to be an high resolution timer. That is possible because power mode won't stop clocking the timer. The "always-on" is not about the clock. It is about the comparator. The clock itself is *guaranteed* to a

[PATCH] ARM: dts: stm32: Enable high resolution timer

2019-09-27 Thread Benjamin Gaignard
Adding always-on makes arm arch_timer claim to be an high resolution timer. That is possible because power mode won't stop clocking the timer. Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stm32mp157c.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/stm32mp157c