the stable rockchip clocksource.
Let's disable the arm_global_timer in order to have the rockchip
clocksource selected by default.
Signed-off-by: Alexander Kochetkov
Reviewed-by: Heiko Stuebner
---
arch/arm/boot/dts/rk3188.dtsi |1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot
'clockevent' attribute should be added to the
timer.
rk3288 (and probably anything newer) is irrelevant to this patch,
as it has the arch timer interface. This patch may be useful
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/mach-ro
'clockevent' attribute should be added to the
timer.
rk3288 (and probably anything newer) is irrelevant to this patch,
as it has the arch timer interface. This patch may be useful
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov
---
arch/arm/mach-rockchip/rockchip.c|2
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner <he...@sntech.de>
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Acked-by: Rob Herring <r...@kernel.o
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner
Signed-off-by: Alexander Kochetkov
Acked-by: Rob Herring
Reviewed-by: Heiko Stuebner
---
.../bindings/timer/r
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviewed-by: Heiko Stuebner <he...@sntech.de>
---
arch/arm/boot/dts/rk3188.dtsi | 18 ++
1 file changed, 18 insertions(+)
diff --gi
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov
Reviewed-by: Heiko Stuebner
---
arch/arm/boot/dts/rk3188.dtsi | 18 ++
1 file changed, 18 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Suggested-by: Heiko Stübner <he...@sntech.de>
Reviewed-by: Heiko Stuebner <he...@sntech.de>
---
arch
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov
Suggested-by: Heiko Stübner
Reviewed-by: Heiko Stuebner
---
arch/arm/boot/dts/rk322x.dtsi |2 +-
1 file changed, 1 insertion(+), 1 del
because clock
framework will try initialize them as clocksource.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3036.dtsi|1 +
arch/arm/boot/dts/rk322x.dtsi|1 +
arch/arm/boot/dts/rk3288.dtsi|1 +
arch/arm64/bo
because clock
framework will try initialize them as clocksource.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3036.dtsi|1 +
arch/arm/boot/dts/rk322x.dtsi|1 +
arch/arm/boot/dts/rk3288.dtsi|1 +
arch/arm64/boot/dts/rockchip/rk3368.dtsi
rockchip.c.
This series should be applied after the commit:
https://lkml.org/lkml/2017/3/22/426
As without the commit, you will get linker error ("clkevt-probe.c:63:
undefined reference to `__clkevt_of_table’")
Regards,
Alexander.
Alexander Kochetkov (6):
dt-bindings: clarify
rockchip.c.
This series should be applied after the commit:
https://lkml.org/lkml/2017/3/22/426
As without the commit, you will get linker error ("clkevt-probe.c:63:
undefined reference to `__clkevt_of_table’")
Regards,
Alexander.
Alexander Kochetkov (6):
dt-bindings: clarify
ch, we allow a mechanism to clearly define a clocksource or a
clockevent without aerobatics we can find around in some drivers:
timer-sp804.c, arc-timer.c, dw_apb_timer_of.c, mps2-timer.c,
renesas-ostm.c, time-efm32.c, time-lpc32xx.c.
Signed-off-by: Daniel Lezcano <daniel.lez
early define a clocksource or a
clockevent without aerobatics we can find around in some drivers:
timer-sp804.c, arc-timer.c, dw_apb_timer_of.c, mps2-timer.c,
renesas-ostm.c, time-efm32.c, time-lpc32xx.c.
Signed-off-by: Daniel Lezcano
Signed-off-by: Alexander Kochetkov
---
Docume
Fixes: 0c8893c9095d ("clockevents: Add a clkevt-of mechanism like clksrc-of")
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
include/asm-generic/vmlinux.lds.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/include/asm-generic/vmlinux.lds.h
b/include/asm-generic/
Fixes: 0c8893c9095d ("clockevents: Add a clkevt-of mechanism like clksrc-of")
Signed-off-by: Alexander Kochetkov
---
include/asm-generic/vmlinux.lds.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/include/asm-generic/vmlinux.lds.h
b/include/asm-generic/vmlinux.lds.h
index 2456397
The patch fix syntax errors introduced by commit 0c8893c9095d
("clockevents: Add a clkevt-of mechanism like clksrc-of").
Fixes: 0c8893c9095d ("clockevents: Add a clkevt-of mechanism like clksrc-of")
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
drivers/
The patch fix syntax errors introduced by commit 0c8893c9095d
("clockevents: Add a clkevt-of mechanism like clksrc-of").
Fixes: 0c8893c9095d ("clockevents: Add a clkevt-of mechanism like clksrc-of")
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/clkevt-probe.
: 'mmc_host mmc0: Bus speed (slot 0) = 9600Hz
(slot req 5000Hz, actual 4800HZ div = 1)'
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3188-radxarock.dts |1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
: 'mmc_host mmc0: Bus speed (slot 0) = 9600Hz
(slot req 5000Hz, actual 4800HZ div = 1)'
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3188-radxarock.dts |1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
b/arch/arm/boot/dts/rk3188
).
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3188.dtsi | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi
index 44da3d42..de786f8 100644
--- a/arch/arm/boot/dts/rk3188.dtsi
+++
).
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3188.dtsi | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi
index 44da3d42..de786f8 100644
--- a/arch/arm/boot/dts/rk3188.dtsi
+++ b/arch/arm/boot/dts/rk3188
/scm/linux/kernel/git/next/linux-next.git/commit/drivers/clocksource?id=376bc27150f180d9f5eddec6a14117780177589d
Regards,
Alexander.
> 31 янв. 2017 г., в 15:43, Alexander Kochetkov <al.koc...@gmail.com>
> написал(а):
>
> From: Alexander Kochetkov <akochet...@lintech.ru>
&
/scm/linux/kernel/git/next/linux-next.git/commit/drivers/clocksource?id=376bc27150f180d9f5eddec6a14117780177589d
Regards,
Alexander.
> 31 янв. 2017 г., в 15:43, Alexander Kochetkov
> написал(а):
>
> From: Alexander Kochetkov
>
> Hello, Daniel, Heiko.
>
> Here
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Suggested-by: Heiko Stübner <he...@sntech.de>
---
arch/arm/boot/dts/rk322x.dtsi |2 +-
1 file changed, 1 i
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov
Suggested-by: Heiko Stübner
---
arch/arm/boot/dts/rk322x.dtsi |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi
index 31f81b2
the stable rockchip clocksource.
Let's disable the arm_global_timer in order to have the rockchip
clocksource selected by default.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3188.dtsi |1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/bo
the stable rockchip clocksource.
Let's disable the arm_global_timer in order to have the rockchip
clocksource selected by default.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3188.dtsi |1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm
may be useful
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
drivers/clocksource/Kconfig |1 +
drivers/clocksource/rockchip_timer.c | 218 --
2 files changed, 158 insertions(+), 61 deletions(-)
diff
From: Alexander Kochetkov <akochet...@lintech.ru>
Hello, Daniel, Heiko.
Here is try 6 :) Thanks a lot for helping me to bring the code
into kernel!
This patch series contain:
- devicetree bindings clarification for rockchip timers
- dts files fixes for rk3228-evb, rk3229-evb and
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner <he...@sntech.de>
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Acked-by: Rob Herring <r...@kernel.or
may be useful
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/Kconfig |1 +
drivers/clocksource/rockchip_timer.c | 218 --
2 files changed, 158 insertions(+), 61 deletions(-)
diff --git a/drivers/clocksource
From: Alexander Kochetkov
Hello, Daniel, Heiko.
Here is try 6 :) Thanks a lot for helping me to bring the code
into kernel!
This patch series contain:
- devicetree bindings clarification for rockchip timers
- dts files fixes for rk3228-evb, rk3229-evb and rk3188
- implementation of clocksource
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner
Signed-off-by: Alexander Kochetkov
Acked-by: Rob Herring
---
.../bindings/timer/rockchip,rk-timer.txt | 12
> 30 янв. 2017 г., в 16:12, Daniel Lezcano
> написал(а):
>
> I don't get the point of these changes. The patch does not explain why they
> are
> needed.
I’d like to extract timer API from current implementation.
And to make code more readable I’d like to introduce
> 30 янв. 2017 г., в 16:12, Daniel Lezcano
> написал(а):
>
> I don't get the point of these changes. The patch does not explain why they
> are
> needed.
I’d like to extract timer API from current implementation.
And to make code more readable I’d like to introduce 'struct rk_timer’ what can
> 30 янв. 2017 г., в 15:04, Daniel Lezcano
> написал(а):
>
> There is no case when the rockchip timer is used for the clockevent.
The is already timer entry for rk3228 in the DT. And it act as clockevent. I
guess it work as backup,
but I cannot test it also. In
> 30 янв. 2017 г., в 15:04, Daniel Lezcano
> написал(а):
>
> There is no case when the rockchip timer is used for the clockevent.
The is already timer entry for rk3228 in the DT. And it act as clockevent. I
guess it work as backup,
but I cannot test it also. In order to not break DT
> 30 янв. 2017 г., в 14:04, Daniel Lezcano
> написал(а):
>
> t is up to the RTC to wake up the system from suspend and there is no
> idle state stopping the local timers on these SoCs. So, the rockchip timers
> won't be ever used on the rk3188, right ?
No rockchip
> 30 янв. 2017 г., в 14:04, Daniel Lezcano
> написал(а):
>
> t is up to the RTC to wake up the system from suspend and there is no
> idle state stopping the local timers on these SoCs. So, the rockchip timers
> won't be ever used on the rk3188, right ?
No rockchip timers where used on rk3188
> 25 янв. 2017 г., в 15:21, Daniel Lezcano
> написал(а):
>
> Hopefully, that can help to do some housework in the directory, perhaps
> split the drivers in to entities, for example:
> - clksrc-rockchip.c
> - clkevt-rockchip.c
>
> Also, it gives the
> 25 янв. 2017 г., в 15:21, Daniel Lezcano
> написал(а):
>
> Hopefully, that can help to do some housework in the directory, perhaps
> split the drivers in to entities, for example:
> - clksrc-rockchip.c
> - clkevt-rockchip.c
>
> Also, it gives the possibility to declare
apply the patches without the tag.
Let me know is all ok with series now? Or may be I have to do something else?
Regards,
Alexander.
> 23 янв. 2017 г., в 20:57, Daniel Lezcano <daniel.lezc...@linaro.org>
> написал(а):
>
> On Mon, Jan 23, 2017 at 08:24:47PM +0300, Alexander Ko
apply the patches without the tag.
Let me know is all ok with series now? Or may be I have to do something else?
Regards,
Alexander.
> 23 янв. 2017 г., в 20:57, Daniel Lezcano
> написал(а):
>
> On Mon, Jan 23, 2017 at 08:24:47PM +0300, Alexander Kochetkov wrote:
>> Heik
> 24 янв. 2017 г., в 18:02, Heiko Stübner написал(а):
>
> Please don't add Reviewed-by tags without explicit mention of them by
> reviewers. (Also it's spelled wrong).
>
> I haven't looked that deeply into the driver itself and the changes made to
> feel comfortable with a
> 24 янв. 2017 г., в 18:02, Heiko Stübner написал(а):
>
> Please don't add Reviewed-by tags without explicit mention of them by
> reviewers. (Also it's spelled wrong).
>
> I haven't looked that deeply into the driver itself and the changes made to
> feel comfortable with a "Reviewed-by".
43987/
https://patchwork.kernel.org/patch/9443977/
https://patchwork.kernel.org/patch/9443991/
Alexander Kochetkov (8):
dt-bindings: clarify compatible property for rockchip timers
ARM: dts: rockchip: update compatible property for rk322x timer
ARM: dts: rockchip: add timer entries to rk3188 SoC
ARM: dts: rockc
://patchwork.kernel.org/patch/9443991/
Alexander Kochetkov (8):
dt-bindings: clarify compatible property for rockchip timers
ARM: dts: rockchip: update compatible property for rk322x timer
ARM: dts: rockchip: add timer entries to rk3188 SoC
ARM: dts: rockchip: disable arm-global-timer for rk3188
at the same time. Because arm-global-timer
looks like a better candidate for the kernel: it has higher
frequency and rating.
Disabling arm-global-timer doesn't leave kernel without
clockevents as there is another clockevent provider (smp-twd).
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.
at the same time. Because arm-global-timer
looks like a better candidate for the kernel: it has higher
frequency and rating.
Disabling arm-global-timer doesn't leave kernel without
clockevents as there is another clockevent provider (smp-twd).
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Suggested-by: Heiko Stübner <he...@sntech.de>
---
arch/arm/boot/dts/rk322x.dtsi |2 +-
1 file changed, 1 i
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov
Suggested-by: Heiko Stübner
---
arch/arm/boot/dts/rk322x.dtsi |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/
commit.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviwed-by: Heiko Stübner <he...@sntech.de>
---
drivers/clocksource/rockchip_timer.c | 33 ++---
1 file changed, 22 insertions(+), 11 deleti
Pass rk_timer instead of clock_event_device to low lever timer routines.
So that code could be reused by clocksource implementation.
Drop rk_base() and rk_ctrl().
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviwed-by:
may be usefull
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviwed-by: Heiko Stübner <he...@sntech.de>
---
drivers/clocksource/rockchip_timer.c | 137 +-
1 file changed, 117 insertions(+), 20 deletions(-)
This allow to enable timer without enabling interrupts from it.
As that mode will be used in clocksource implementation.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviwed-by: Heiko Stübner <he...@sntech.de>
commit.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko Stübner
---
drivers/clocksource/rockchip_timer.c | 33 ++---
1 file changed, 22 insertions(+), 11 deletions(-)
diff --git a/drivers/clocksource
Pass rk_timer instead of clock_event_device to low lever timer routines.
So that code could be reused by clocksource implementation.
Drop rk_base() and rk_ctrl().
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko Stübner
---
drivers
may be usefull
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko Stübner
---
drivers/clocksource/rockchip_timer.c | 137 +-
1 file changed, 117 insertions(+), 20 deletions(-)
diff --git a/drivers/clocksource/rockchip_timer.c
b
This allow to enable timer without enabling interrupts from it.
As that mode will be used in clocksource implementation.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko Stübner
---
drivers/clocksource/rockchip_timer.c |8
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Reviwed-by: Heiko Stübner <he...@sntech.de>
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/a
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov
Reviwed-by: Heiko Stübner
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner <he...@sntech.de>
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
Acked-by: Rob Herring <r...@kernel.or
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner
Signed-off-by: Alexander Kochetkov
Acked-by: Rob Herring
---
.../bindings/timer/rockchip,rk-timer.txt | 12
, Heiko Stübner <he...@sntech.de> написал(а):
>
> Am Montag, 23. Januar 2017, 15:47:44 CET schrieb Daniel Lezcano:
>> On Wed, Dec 21, 2016 at 05:21:05PM +0300, Alexander Kochetkov wrote:
>>> Hello Heiko, Daniel!
>>>
>>> Are there any reasons why the p
3. Januar 2017, 15:47:44 CET schrieb Daniel Lezcano:
>> On Wed, Dec 21, 2016 at 05:21:05PM +0300, Alexander Kochetkov wrote:
>>> Hello Heiko, Daniel!
>>>
>>> Are there any reasons why the patches [1][2] are not applied yet into
>>> kernel? How can I help in
Daniel, thanks for reviewing patches!
> 23 янв. 2017 г., в 19:26, Daniel Lezcano <daniel.lezc...@linaro.org>
> написал(а):
>
> On Tue, Nov 29, 2016 at 07:14:49PM +0300, Alexander Kochetkov wrote:
>> The patch move ce field out of struct bc_timer into struct
>> rk
Daniel, thanks for reviewing patches!
> 23 янв. 2017 г., в 19:26, Daniel Lezcano
> написал(а):
>
> On Tue, Nov 29, 2016 at 07:14:49PM +0300, Alexander Kochetkov wrote:
>> The patch move ce field out of struct bc_timer into struct
>> rk_clock_event_device and rename s
Hello Daniel!
:
> +
> + {
> + compatible = "rockchip,rk3228-timer", "rockchip,rk3288-timer";
> +};
> 23 янв. 2017 г., в 18:40, Daniel Lezcano
> написал(а):
>
> I'm not sure this is correct, to which timer will refer ?
>
>timer {
>
Hello Daniel!
:
> +
> + {
> + compatible = "rockchip,rk3228-timer", "rockchip,rk3288-timer";
> +};
> 23 янв. 2017 г., в 18:40, Daniel Lezcano
> написал(а):
>
> I'm not sure this is correct, to which timer will refer ?
>
>timer {
>compatible = "arm,armv7-timer";
>
for rockchip SoC using rockchip timer
[2]
http://lists.infradead.org/pipermail/linux-rockchip/2016-December/013308.html
> 29 нояб. 2016 г., в 19:14, Alexander Kochetkov <al.koc...@gmail.com>
> написал(а):
>
> Hello,
>
> This patch series contain:
> - devicetree bindings
for rockchip SoC using rockchip timer
[2]
http://lists.infradead.org/pipermail/linux-rockchip/2016-December/013308.html
> 29 нояб. 2016 г., в 19:14, Alexander Kochetkov
> написал(а):
>
> Hello,
>
> This patch series contain:
> - devicetree bindings clarification for rockchip
> 1 дек. 2016 г., в 0:30, Rob Herring написал(а):
>
> 1st and 2nd are ambiguous. Plus this is an OS implementation detail that
> doesn't belong in the binding.
>
>> +If you want to bind specific timer as clockevent (i.e. one from alive
>> subsystem)
>> +and specific timer as
> 1 дек. 2016 г., в 0:30, Rob Herring написал(а):
>
> 1st and 2nd are ambiguous. Plus this is an OS implementation detail that
> doesn't belong in the binding.
>
>> +If you want to bind specific timer as clockevent (i.e. one from alive
>> subsystem)
>> +and specific timer as clocksource, you
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner <he...@sntech.de>
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
.../bindings/timer/rockch
Make all properties description in form '"rockchip,-timer",
"rockchip,rk3288-timer"' for all chips found in linux kernel.
Suggested-by: Heiko Stübner
Signed-off-by: Alexander Kochetkov
---
.../bindings/timer/rockchip,rk-timer.txt | 12 +---
1 file c
may be usefull
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
drivers/clocksource/rockchip_timer.c | 137 +-
1 file changed, 117 insertions(+), 20 deletions(-)
diff --git a/drivers/clocksource/rockchip_timer.c
b/d
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3228-evb.dts |4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts
may be usefull
for Cortex-A9/A5 based parts.
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/rockchip_timer.c | 137 +-
1 file changed, 117 insertions(+), 20 deletions(-)
diff --git a/drivers/clocksource/rockchip_timer.c
b/drivers/clocksource
Property set to '"rockchip,rk3228-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3228-evb.dts |4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/rk3228-evb.dts b/arch/arm/boo
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/
Property set to '"rockchip,rk3229-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
arch/arm/boot/dts/rk3229-evb.dts |4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts
Property set to '"rockchip,rk3229-timer", "rockchip,rk3288-timer"'
to match devicetree bindings.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3229-evb.dts |4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/rk3229-evb.dts b/arch/arm/boo
for the arm-global-timer clocksource and sched clock. It run
at stable frequency 24MHz.
Signed-off-by: Alexander Kochetkov
---
arch/arm/boot/dts/rk3188.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi
index 31f81b2
This allow to enable timer without enabling interrupts from it.
As that mode will be used in clocksource implementation.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
drivers/clocksource/rockchip_timer.c |8
rating
because ratings greater 300 used for high frequency clocksources.
clocksource and shed_clock is quite unstable, because their rate depends
on cpu frequency. So disable arm-global-timer and use clocksource and
sched_clock from rockchip_timer.
Signed-off-by: Alexander Kochetkov <al.
Pass rk_timer instead of clock_event_device to low lever timer routines.
So that code could be reused by clocksource implementation.
Drop rk_base() and rk_ctrl().
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
d
This allow to enable timer without enabling interrupts from it.
As that mode will be used in clocksource implementation.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/rockchip_timer.c |8
1 file changed, 4
rating
because ratings greater 300 used for high frequency clocksources.
clocksource and shed_clock is quite unstable, because their rate depends
on cpu frequency. So disable arm-global-timer and use clocksource and
sched_clock from rockchip_timer.
Signed-off-by: Alexander Kochetkov
---
arch/arm
Pass rk_timer instead of clock_event_device to low lever timer routines.
So that code could be reused by clocksource implementation.
Drop rk_base() and rk_ctrl().
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/rockchip_timer.c
/9443977/
https://patchwork.kernel.org/patch/9443991/
Alexander Kochetkov (9):
dt-bindings: clarify compatible property for rockchip timers
ARM: dts: rockchip: update compatible property for rk3228 timer
ARM: dts: rockchip: update compatible property for rk3229 timer
ARM: dts: rockchip: add timer
The patch move ce field out of struct bc_timer into struct
rk_clock_event_device and rename struct bc_timer to struct rk_timer.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov <al.koc...@gmail.com>
---
drivers/clocksource/rockchip_timer.c
/9443977/
https://patchwork.kernel.org/patch/9443991/
Alexander Kochetkov (9):
dt-bindings: clarify compatible property for rockchip timers
ARM: dts: rockchip: update compatible property for rk3228 timer
ARM: dts: rockchip: update compatible property for rk3229 timer
ARM: dts: rockchip: add timer
The patch move ce field out of struct bc_timer into struct
rk_clock_event_device and rename struct bc_timer to struct rk_timer.
This is refactoring step without functional changes.
Signed-off-by: Alexander Kochetkov
---
drivers/clocksource/rockchip_timer.c | 33
Hello Heiko!
Thank you for patch review!
> 29 нояб. 2016 г., в 18:01, Heiko Stübner написал(а):
>
> you introduced the issue yourself in patch 11/13. In general any patch should
> never leave the kernel in a worse state than it was before, so no patch
> should
> ever
Hello Heiko!
Thank you for patch review!
> 29 нояб. 2016 г., в 18:01, Heiko Stübner написал(а):
>
> you introduced the issue yourself in patch 11/13. In general any patch should
> never leave the kernel in a worse state than it was before, so no patch
> should
> ever introduce known issues
> 29 нояб. 2016 г., в 17:51, Marc Zyngier написал(а):
>
> That'd be my preferred course of action. I've located some documentation
> over there [1], and page 1126 seems to indicate a profusion of
> additional timers, some of which are in an always-on domain. Seems like
> a
101 - 200 of 455 matches
Mail list logo