Hi Romain, Kukjin, On Wednesday 09 of January 2013 23:43:35 Romain Naour wrote: > This series of patches converts the s3c and s5pc100 timer driver to the > clocksource/clockevent API. I made some test on a mini2440 board and I > had to reduce timers frequency to 1MHz in order to produce a timer's > overflow every 64ms. Initial timer's frequency (8,45MHz) provide only > 7ms between each overflow. It is not enough. As timers were previously > used to produce an IRQ at 200Hz, some board (Osiris, Anubis board) use > an external 12MHz signal to clock the timers (tclk1). So, I changed > their configuration to select internal pclk clock instead, but I can't > test it. > > Since clockevent/clocksource API becomes available, we can use High > Resolution Timer and Tickless mode. > > Most of the work is already done in s5p-time.c (renamed samsung-time.c). > I added some #define for s3c24xx and s5pc100 case. > > > Naour Romain (5): > Rename s5p-time to samsung-time > Add samsung-time support for s3c24xx > Add samsung-time support for s3c64xx > Add samsung-time support for s5pc100 > Remove unused plat-samsung/time.c > > arch/arm/Kconfig | 9 +- > arch/arm/mach-exynos/Kconfig | 2 +- > arch/arm/mach-exynos/mach-universal_c210.c | 6 +- > arch/arm/mach-s3c24xx/Kconfig | 6 + > arch/arm/mach-s3c24xx/mach-amlm5900.c | 5 +- > arch/arm/mach-s3c24xx/mach-anubis.c | 4 +- > arch/arm/mach-s3c24xx/mach-at2440evb.c | 4 +- > arch/arm/mach-s3c24xx/mach-bast.c | 4 +- > arch/arm/mach-s3c24xx/mach-gta02.c | 4 +- > arch/arm/mach-s3c24xx/mach-h1940.c | 5 +- > arch/arm/mach-s3c24xx/mach-jive.c | 4 +- > arch/arm/mach-s3c24xx/mach-mini2440.c | 4 +- > arch/arm/mach-s3c24xx/mach-n30.c | 6 +- > arch/arm/mach-s3c24xx/mach-nexcoder.c | 4 +- > arch/arm/mach-s3c24xx/mach-osiris.c | 4 +- > arch/arm/mach-s3c24xx/mach-otom.c | 4 +- > arch/arm/mach-s3c24xx/mach-qt2410.c | 4 +- > arch/arm/mach-s3c24xx/mach-rx1950.c | 4 +- > arch/arm/mach-s3c24xx/mach-rx3715.c | 4 +- > arch/arm/mach-s3c24xx/mach-smdk2410.c | 4 +- > arch/arm/mach-s3c24xx/mach-smdk2413.c | 8 +- > arch/arm/mach-s3c24xx/mach-smdk2416.c | 4 +- > arch/arm/mach-s3c24xx/mach-smdk2440.c | 4 +- > arch/arm/mach-s3c24xx/mach-smdk2443.c | 4 +- > arch/arm/mach-s3c24xx/mach-tct_hammer.c | 4 +- > arch/arm/mach-s3c24xx/mach-vr1000.c | 4 +- > arch/arm/mach-s3c24xx/mach-vstms.c | 5 +- > arch/arm/mach-s3c64xx/Kconfig | 2 + > arch/arm/mach-s3c64xx/mach-anw6410.c | 4 +- > arch/arm/mach-s3c64xx/mach-crag6410.c | 4 +- > arch/arm/mach-s3c64xx/mach-hmt.c | 4 +- > arch/arm/mach-s3c64xx/mach-mini6410.c | 4 +- > arch/arm/mach-s3c64xx/mach-ncp.c | 4 +- > arch/arm/mach-s3c64xx/mach-real6410.c | 4 +- > arch/arm/mach-s3c64xx/mach-smartq.c | 2 + > arch/arm/mach-s3c64xx/mach-smartq5.c | 3 +- > arch/arm/mach-s3c64xx/mach-smartq7.c | 3 +- > arch/arm/mach-s3c64xx/mach-smdk6400.c | 4 +- > arch/arm/mach-s3c64xx/mach-smdk6410.c | 4 +- > arch/arm/mach-s5p64x0/Kconfig | 4 +- > arch/arm/mach-s5p64x0/mach-smdk6440.c | 6 +- > arch/arm/mach-s5p64x0/mach-smdk6450.c | 6 +- > arch/arm/mach-s5pc100/Kconfig | 1 + > arch/arm/mach-s5pc100/mach-smdkc100.c | 4 +- > arch/arm/mach-s5pv210/Kconfig | 2 +- > arch/arm/mach-s5pv210/mach-aquila.c | 6 +- > arch/arm/mach-s5pv210/mach-goni.c | 6 +- > arch/arm/mach-s5pv210/mach-smdkc110.c | 6 +- > arch/arm/mach-s5pv210/mach-smdkv210.c | 6 +- > arch/arm/mach-s5pv210/mach-torbreck.c | 6 +- > arch/arm/plat-samsung/Kconfig | 2 +- > arch/arm/plat-samsung/Makefile | 3 +- > arch/arm/plat-samsung/include/plat/cpu.h | 5 - > arch/arm/plat-samsung/include/plat/s5p-time.h | 40 --- > arch/arm/plat-samsung/include/plat/samsung-time.h | 51 ++++ > .../plat-samsung/{s5p-time.c => samsung-time.c} | 144 +++++------ > arch/arm/plat-samsung/time.c | 285 > --------------------- 57 files changed, 274 insertions(+), 475 > deletions(-) > delete mode 100644 arch/arm/plat-samsung/include/plat/s5p-time.h > create mode 100644 arch/arm/plat-samsung/include/plat/samsung-time.h > rename arch/arm/plat-samsung/{s5p-time.c => samsung-time.c} (68%) > delete mode 100644 arch/arm/plat-samsung/time.c The whole series looks good to me. Reviewed-by: Tomasz Figa <tomasz.figa@xxxxxxxxx> Also I tested it on a Tiny6410 (Mini6410-compatible) board. Tested-by: Tomasz Figa <tomasz.figa@xxxxxxxxx> Best regards, Tomasz Figa -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html