Re: [PATCH] dt-bindings: timer: renesas, cmt: Document r8a774a1 CMT support

2018-11-22 Thread Daniel Lezcano
On 19/11/2018 17:14, Biju Das wrote: > Document SoC specific bindings for RZ/G2M (r8a774a1) SoC. > Applied, thanks! -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook |

Re: [PATCH] dt-bindings: timer: renesas, cmt: Document r8a774a1 CMT support

2018-11-22 Thread Daniel Lezcano
On 19/11/2018 17:14, Biju Das wrote: > Document SoC specific bindings for RZ/G2M (r8a774a1) SoC. > > Signed-off-by: Biju Das Please send to lkml@ also. > --- > Documentation/devicetree/bindings/timer/renesas,cmt.txt | 11 --- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff

Re: [PATCH] arm64: dts: renesas: r8a7796: Add CMT device nodes

2018-11-22 Thread Daniel Lezcano
On 22/11/2018 10:46, Biju Das wrote: > Hello Daniel, > > Thanks for the feedback. > >> -Original Message----- >> From: Daniel Lezcano >> Sent: 19 November 2018 17:15 >> To: Biju Das ; Rob Herring >> ; Mark Rutland >> Cc: Simon Horman ; Ma

Re: [PATCH] arm64: dts: renesas: r8a7796: Add CMT device nodes

2018-11-19 Thread Daniel Lezcano
On 19/11/2018 11:35, Biju Das wrote: > Hi Daniel, > > Thanks for the feedback. > >> -Original Message----- >> From: Daniel Lezcano >> Sent: 19 November 2018 10:26 >> To: Biju Das ; Rob Herring >> ; Mark Rutland >> Cc: Simon Horman ; Ma

Re: [PATCH] arm64: dts: renesas: r8a7796: Add CMT device nodes

2018-11-19 Thread Daniel Lezcano
On 26/10/2018 10:25, Biju Das wrote: > This patch adds CMT{0|1|2|3} device nodes for r8a7796 SoC. > > Signed-off-by: Biju Das > --- > This patch is tested against renesas-dev > > I have executed on inconsistency-check, nanosleep and clocksource_switch > selftests on this arm64 SoC. The

Re: [PATCH] arm64: dts: renesas: r8a7796: Add CMT device nodes

2018-11-19 Thread Daniel Lezcano
On 26/10/2018 10:25, Biju Das wrote: > This patch adds CMT{0|1|2|3} device nodes for r8a7796 SoC. > > Signed-off-by: Biju Das > --- > This patch is tested against renesas-dev > > I have executed on inconsistency-check, nanosleep and clocksource_switch > selftests on this arm64 SoC. The

Re: [PATCH] dt-bindings: timer: renesas, cmt: Document r8a77470 CMT support

2018-11-17 Thread Daniel Lezcano
On 26/10/2018 10:36, Biju Das wrote: > Document SoC specific compatible strings for r8a77470. No driver change > is needed as the fallback strings will activate the right code. > > Signed-off-by: Biju Das > --- Applied also, thanks! -- Linaro.org │ Open source

Re: [PATCH] dt-bindings: timer: renesas, cmt: Document r8a7796 CMT support

2018-11-17 Thread Daniel Lezcano
On 26/10/2018 10:01, Biju Das wrote: > Document SoC specific bindings for R-Car M3-W (r8a7796) SoC. > > Signed-off-by: Biju Das > --- > This patch is tested against linu-next > --- Applied, thanks! -- Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [PATCH] thermal: rcar_thermal: fix duplicate IRQ request

2018-10-04 Thread Daniel Lezcano
ylyov > > --- Reviewed-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] dt-bindings: thermal: rcar: Add device tree support for r8a7744

2018-09-25 Thread Daniel Lezcano
al". > > Signed-off-by: Biju Das > Reviewed-by: Chris Paterson Reviewed-by: Daniel Lezcano > --- > Documentation/devicetree/bindings/thermal/rcar-thermal.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.t

Re: [PATCH v2 0/3] Add R-Car gen3 SoC support to the CMT driver

2018-09-14 Thread Daniel Lezcano
On 12/09/2018 22:10, Sergei Shtylyov wrote: > Hello! > > Here's the set of 3 patches against the 'tip.git' repo's 'timers/core' branch > plus the CMT driver fixups for the 32/64-bit machines posted recently. We're > adding support for the CMT types0/1 found in the R-Car gen3 SoCs. > > [1/3]

Re: [PATCH] clocksource: sh_cmt: fix clocksource width for 32-bit machines

2018-09-14 Thread Daniel Lezcano
On 10/09/2018 22:22, Sergei Shtylyov wrote: > The driver seems to abuse *unsigned long* not only for the (32-bit) > register values but also for the 'sh_cmt_channel::total_cycles' which > needs to always be 64-bit -- as a result, the clocksource's mask is > needlessly clamped down to 32-bits on

Re: [PATCH v3] clocksource: sh_cmt: fixup for 64-bit machines

2018-09-14 Thread Daniel Lezcano
On 08/09/2018 22:54, Sergei Shtylyov wrote: > When trying to use CMT for clockevents on R-Car gen3 SoCs, I noticed > that 'max_delta_ns' for the broadcast timer (CMT) was shown as 1000 in > /proc/timer_list. It turned out that when calculating it, the driver did > 1 << 32 (causing what I think was

Re: [PATCH v3] clocksource: sh_cmt: fixup for 64-bit machines

2018-09-13 Thread Daniel Lezcano
On 08/09/2018 22:54, Sergei Shtylyov wrote: > When trying to use CMT for clockevents on R-Car gen3 SoCs, I noticed > that 'max_delta_ns' for the broadcast timer (CMT) was shown as 1000 in > /proc/timer_list. It turned out that when calculating it, the driver did > 1 << 32 (causing what I think was

Re: [PATCH] clocksource: sh_cmt: fix clocksource width for 32-bit machines

2018-09-13 Thread Daniel Lezcano
On 10/09/2018 22:22, Sergei Shtylyov wrote: > The driver seems to abuse *unsigned long* not only for the (32-bit) > register values but also for the 'sh_cmt_channel::total_cycles' which > needs to always be 64-bit -- as a result, the clocksource's mask is > needlessly clamped down to 32-bits on

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-09-13 Thread Daniel Lezcano
On 11/09/2018 20:42, Chris Brandt wrote: > On Tuesday, September 11, 2018 1, Rob Herring wrote: >> Well before we get to initcalls, the kernel calls the arch specific >> time_init() which (on ARM) calls of_clk_init (for all the reasons >> above) and then timer_probe(). When timer_probe returns, it

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-08-30 Thread Daniel Lezcano
[Added Arnd Bergmann, Bartosz Golaszewski and Mark Brown] On 30/08/2018 10:48, Geert Uytterhoeven wrote: > Hi Daniel, [ ... ] >> Yeah, I got this point. But it is the meaning of your sentence: "... >> which causes issues with complex dependencies.". >> >> It is ambiguous *what* causes the

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-08-30 Thread Daniel Lezcano
On 30/08/2018 10:27, Geert Uytterhoeven wrote: > Hi Daniel, > > On Thu, Aug 30, 2018 at 10:09 AM Daniel Lezcano > wrote: >> On 30/08/2018 09:54, Geert Uytterhoeven wrote: >>> On Wed, Aug 29, 2018 at 6:26 PM Daniel Lezcano >>> wrote: >>

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-08-30 Thread Daniel Lezcano
On 30/08/2018 09:54, Geert Uytterhoeven wrote: > Hi Daniel, > > On Wed, Aug 29, 2018 at 6:26 PM Daniel Lezcano > wrote: >> On 29/08/2018 17:44, Chris Brandt wrote: >>> On Wednesday, August 29, 2018 1, Daniel Lezcano wrote: >>>> Can the boot constraints [1]

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-08-29 Thread Daniel Lezcano
On 29/08/2018 17:44, Chris Brandt wrote: > On Wednesday, August 29, 2018 1, Daniel Lezcano wrote: >> Can the boot constraints [1] solve this issue instead of the changes you >> are proposing ? >> >> [1] https://lwn.net/Articles/747250/ > > Thanks for the

Re: [PATCH 1/2] clocksource/drivers/ostm: Delay driver registration

2018-08-29 Thread Daniel Lezcano
On 29/08/2018 15:29, Chris Brandt wrote: > The newer RZ/A clock control driver no longer registers all its clocks > using DT, therefore the clocks required by this driver are no longer > present at the beginning of boot. > > Because of this, TIMER_OF_DECLARE can no longer be used because this >

Re: [PATCH] thermal: rcar_gen3_thermal: convert to SPDX identifiers

2018-07-30 Thread Daniel Lezcano
On 30/07/2018 09:57, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > Signed-off-by: Kuninori Morimoto Reviewed-by: Daniel Lezcano Adding Cc: Philippe Ombredanne > --- > drivers/thermal/rcar_gen3_thermal.c | 11 +-- > 1 file changed, 1 inser

Re: [PATCH] thermal: rcar_thermal: convert to SPDX identifiers

2018-07-30 Thread Daniel Lezcano
ot;GPL" [GNU Public License v2 or later] > "GPL v2"[GNU Public License v2] > > Signed-off-by: Kuninori Morimoto Reviewed-by: Daniel Lezcano Adding Cc: Philippe Ombredanne > --- > drivers/thermal/rcar_thermal.c | 16 ++-- >

Re: [PATCH v2 1/3] dt-bindings: timer: renesas, cmt: Document r8a774[35] CMT support

2018-01-25 Thread Daniel Lezcano
On 25/01/2018 09:21, Geert Uytterhoeven wrote: > Hi Daniel, > > On Thu, Jan 25, 2018 at 9:09 AM, Daniel Lezcano > <daniel.lezc...@linaro.org> wrote: >> On 24/01/2018 17:49, Fabrizio Castro wrote: >>> I am sorry to bother you, just wondering if you think there ar

Re: [PATCH v2 1/3] dt-bindings: timer: renesas, cmt: Document r8a774[35] CMT support

2018-01-25 Thread Daniel Lezcano
On 24/01/2018 17:49, Fabrizio Castro wrote: > Hello Daniel, > > I am sorry to bother you, just wondering if you think there are any chances > for this patch to applied on top of v4.16? You can take it through the renesas tree. Acked-by: Daniel Lezcano <daniel.lezc...@linaro.or

Re: [PATCH v3 0/7] clocksource: sh_cmt: Update driver for DT binding rework

2017-09-26 Thread Daniel Lezcano
On 26/09/2017 10:26, Laurent Pinchart wrote: > Reviewed-by: Laurent Pinchart Thanks. I applied the whole series. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [PATCH v3 0/7] clocksource: sh_cmt: Update driver for DT binding rework

2017-09-25 Thread Daniel Lezcano
Hi all, last call for comments before I merge this series. Thanks. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter |

Re: [PATCH v4 00/06] clocksource: sh_cmt: DT binding rework V4

2017-08-10 Thread Daniel Lezcano
On 10/08/2017 11:01, Geert Uytterhoeven wrote: > On Tue, Jul 11, 2017 at 1:56 PM, Simon Horman wrote: >> On Thu, Nov 24, 2016 at 11:58:43AM +0100, Simon Horman wrote: >>> On Mon, Mar 14, 2016 at 11:23:42PM +0900, Magnus Damm wrote: clocksource: sh_cmt: DT binding rework

Re: [PATCH v4 00/06] clocksource: sh_cmt: DT binding rework V4

2017-07-31 Thread Daniel Lezcano
On 11/07/2017 13:56, Simon Horman wrote: > On Thu, Nov 24, 2016 at 11:58:43AM +0100, Simon Horman wrote: >> Hi Magnus, >> >> On Mon, Mar 14, 2016 at 11:23:42PM +0900, Magnus Damm wrote: >>> clocksource: sh_cmt: DT binding rework V4 >>> >>> [PATCH v4 01/06] devicetree: bindings: Remove sh7372 CMT

[PATCH 06/10] clocksource/drivers/ostm: Add renesas-ostm timer driver

2017-02-08 Thread Daniel Lezcano
evices as clock events. Signed-off-by: Chris Brandt <chris.bra...@renesas.com> Signed-off-by: Daniel Lezcano <daniel.lezc...@linaro.org> --- arch/arm/mach-shmobile/Kconfig | 1 + drivers/clocksource/Kconfig| 7 + drivers/clocksource/Makefile | 1 + drivers/clocksou

Re: [PATCH v7 0/2] clocksource: Add renesas-ostm timer driver

2017-01-30 Thread Daniel Lezcano
On Fri, Jan 27, 2017 at 03:02:13PM -0500, Chris Brandt wrote: > This patch set adds a new clocksource driver that uses the OS Timer > (OSTM) that exists in the R7S72100 (RZ/A1) SoC. > > The operation of the driver was tested with a simple user application > that does multiple calls to nanosleep()

Re: [PATCH v3 2/2] clocksource: Add renesas-ostm timer driver

2017-01-25 Thread Daniel Lezcano
On Tue, Jan 24, 2017 at 08:19:50PM +, Chris Brandt wrote: > Hi Daniel, > > On Tuesday, January 24, 2017, Daniel Lezcano wrote: > > > > > +early_platform_init("earlytimer", _timer); > > > > > +subsys_initcall(ostm_init); module_exit(ostm_e

Re: [PATCH v3 2/2] clocksource: Add renesas-ostm timer driver

2017-01-24 Thread Daniel Lezcano
On Tue, Jan 24, 2017 at 04:45:47AM +, Chris Brandt wrote: Hi Chris, [ ... ] > > > + bool "Renesas OSTM timer driver" if COMPILE_TEST > > > + depends on GENERIC_CLOCKEVENTS > > > + select CLKSRC_MMIO > > > + default SYS_SUPPORTS_RENESAS_OSTM > > > > - default SYS_SUPPORTS_RENESAS_OSTM > >

Re: [PATCH v3 2/2] clocksource: Add renesas-ostm timer driver

2017-01-23 Thread Daniel Lezcano
On Mon, Jan 23, 2017 at 08:54:23AM -0500, Chris Brandt wrote: > This patch adds a OSTM driver for the Renesas architecture. As it is a new driver, please give technical details for the log. Replace ioread/write by readl/writel in the code. > Signed-off-by: Chris Brandt

Re: [PATCH 0/9] clocksource/drivers/clksrc-of: Improve error handling

2016-06-08 Thread Daniel Lezcano
On 06/07/2016 11:54 AM, Geert Uytterhoeven wrote: [ ... ] in clockevents/clockevents/next breaks the boot on e.g. r8a7791/koelsch (arm32) and r8a7795/salvator-x (arm64). Using "earlycon keep_bootcon" on koelsch (this doesn't help on arm64) reveals it's stuck at: clocksource_probe: no

Re: [PATCH 0/9] clocksource/drivers/clksrc-of: Improve error handling

2016-06-07 Thread Daniel Lezcano
On 06/07/2016 11:54 AM, Geert Uytterhoeven wrote: Hi Daniel, Hi Geert, [ ... ] Using "earlycon keep_bootcon" on koelsch (this doesn't help on arm64) reveals it's stuck at: clocksource_probe: no matching clocksources found sched_clock: 32 bits at 100 Hz, resolution 1000ns,