Re: [PATCH 1/3] Rename s5p-time to samsung-time

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Romain,

Am Freitag, 30. November 2012, 00:18:08 schrieb Romain Naour:
> Hi Heiko,
> 
> Le 28/11/2012 00:57, Heiko Stübner a écrit :
> > Hi Romain,
> > 
> > Am Mittwoch, 28. November 2012, 00:27:36 schrieb Romain Naour:
> >> Hi Tomasz, Kgene
> >> 
> >>> I would also suggest splitting this huge patch into a series of several
> >>> smaller, possibly:
> >>> 1) Rename s5p-time to samsung-time (and correct any platforms using it
> >>> currently)
> >>> 2) Add samsung-time support for s3c24xx
> >>> 3) Add samsung-time support for s3c64xx
> >>> 
> >>> What do you think?
> >>> 
> >>> Best regards,
> >>> Tomasz Figa
> >> 
> >> I have separated the patch into three parts, as you said.
> >> And also, I corrected some oversights ;-)
> >> 
> >> These patches are for 3.7-rc6.
> > 
> >> Here is the first one:
> > Comments like this should either be in the comment section (below the
> > three dashes) or in a cover-letter (git format-patch --cover-letter),
> > but not in the commit message of the patch itself.
> 
> Thank you for your advice :)
> 
> > Otherwise the patches look very cool. Hopefully I'll find a bit of time
> > to test them tomorrow.
> 
> On which processor can you test these patches ?

A S3C2416. It should just work, but I'm very curious to see a tick-less kernel 
on it ;-)

> > But I do have some quick comments:
> > 
> > What happens to s3c24xx_timer? It seems you changed every user of it but
> > did not remove the old timer code. Might be interesting as a 4th patch,
> > or is it still used somewhere?
> 
> s3c24xx_timer is still used by s5pc100 CPUs.
> That's why I didn't remove plat-samsung/time.c
> I do not have the datasheet, but s5pc100 timers seem to be similar to
> s3c24xx timers. (16bis) Can someone test on a processor s5pc100 ?
> 
> > Please use the -M option to "git format-patch" which minimizes the impact
> > of moved files in the patch itself.
> 
> Do you want I resends these patches, plus one for s5pc100 ?

That might be helpful. Especially to get tests on the s5pc100. If you provide 
a test-ready patch, chances are a lot higher someone finds the time to test 
it.

So, as you wrote you could resend the series with
- the three original patches
- a 4th converting s5pc100
- a 5th removing the then unused s3c24xx-timer


Heiko

> > Thanks for doing this change
> > Heiko
> 
> Thank you for your comments :)
> Romain
> 
> >> This patch rename s5p-time to samsung-time.
> >> There is no functional change.
> >> 
> >> Signed-off-by: Naour Romain <romain.naour@xxxxxxxxxxx>
> >> ---
> >> 
> >>  arch/arm/mach-exynos/Kconfig                      |   2 +-
> >>  arch/arm/mach-exynos/mach-universal_c210.c        |   6 +-
> >>  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-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                    |   2 +-
> >>  arch/arm/plat-samsung/include/plat/s5p-time.h     |  40 ---
> >>  arch/arm/plat-samsung/include/plat/samsung-time.h |  40 +++
> >>  arch/arm/plat-samsung/s5p-time.c                  | 405
> >> 
> >> ---------------------- arch/arm/plat-samsung/samsung-time.c             
> >> | 405 ++++++++++++++++++++++ 17 files changed, 475 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
> >>  delete mode 100644 arch/arm/plat-samsung/s5p-time.c
> >>  create mode 100644 arch/arm/plat-samsung/samsung-time.c
> >> 
> >> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> >> index da55107..20edfa3 100644
> >> --- a/arch/arm/mach-exynos/Kconfig
> >> +++ b/arch/arm/mach-exynos/Kconfig
> >> @@ -274,7 +274,7 @@ config MACH_UNIVERSAL_C210
> >> 
> >>  	select S5P_DEV_ONENAND
> >>  	select S5P_DEV_TV
> >>  	select S5P_GPIO_INT
> >> 
> >> -	select S5P_HRT
> >> +	select SAMSUNG_HRT
> >> 
> >>  	select S5P_SETUP_MIPIPHY
> >>  	help
> >>  	
> >>  	  Machine support for Samsung Mobile Universal S5PC210 Reference
> >> 
> >> diff --git a/arch/arm/mach-exynos/mach-universal_c210.c
> >> b/arch/arm/mach-exynos/mach-universal_c210.c index ebc9dd3..325bfe9
> >> 100644 --- a/arch/arm/mach-exynos/mach-universal_c210.c
> >> +++ b/arch/arm/mach-exynos/mach-universal_c210.c
> >> @@ -41,7 +41,7 @@
> >> 
> >>  #include <plat/mfc.h>
> >>  #include <plat/sdhci.h>
> >>  #include <plat/fimc-core.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/camport.h>
> >>  #include <linux/platform_data/mipi-csis.h>
> >> 
> >> @@ -1099,7 +1099,7 @@ static void __init universal_map_io(void)
> >> 
> >>  	exynos_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(clk_xusbxti.rate);
> >>  	s3c24xx_init_uarts(universal_uartcfgs,
> >>  	ARRAY_SIZE(universal_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM2, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM2, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void s5p_tv_setup(void)
> >> 
> >> @@ -1158,7 +1158,7 @@ MACHINE_START(UNIVERSAL_C210, "UNIVERSAL_C210")
> >> 
> >>  	.handle_irq	= gic_handle_irq,
> >>  	.init_machine	= universal_machine_init,
> >>  	.init_late	= exynos_init_late,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.reserve        = &universal_reserve,
> >>  	.restart	= exynos4_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5p64x0/Kconfig
> >> b/arch/arm/mach-s5p64x0/Kconfig index e8742cb..f0ec535 100644
> >> --- a/arch/arm/mach-s5p64x0/Kconfig
> >> +++ b/arch/arm/mach-s5p64x0/Kconfig
> >> @@ -9,7 +9,7 @@ if ARCH_S5P64X0
> >> 
> >>  config CPU_S5P6440
> >>  
> >>  	bool
> >> 
> >> -	select S5P_HRT
> >> +	select SAMSUNG_HRT
> >> 
> >>  	select S5P_SLEEP if PM
> >>  	select SAMSUNG_DMADEV
> >>  	select SAMSUNG_WAKEMASK if PM
> >> 
> >> @@ -18,7 +18,7 @@ config CPU_S5P6440
> >> 
> >>  config CPU_S5P6450
> >>  
> >>  	bool
> >> 
> >> -	select S5P_HRT
> >> +	select SAMSUNG_HRT
> >> 
> >>  	select S5P_SLEEP if PM
> >>  	select SAMSUNG_DMADEV
> >>  	select SAMSUNG_WAKEMASK if PM
> >> 
> >> diff --git a/arch/arm/mach-s5p64x0/mach-smdk6440.c
> >> b/arch/arm/mach-s5p64x0/mach-smdk6440.c index 96ea1fe..587fec5 100644
> >> --- a/arch/arm/mach-s5p64x0/mach-smdk6440.c
> >> +++ b/arch/arm/mach-s5p64x0/mach-smdk6440.c
> >> @@ -50,7 +50,7 @@
> >> 
> >>  #include <plat/pll.h>
> >>  #include <plat/adc.h>
> >>  #include <linux/platform_data/touchscreen-s3c2410.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/backlight.h>
> >>  #include <plat/fb.h>
> >>  #include <plat/sdhci.h>
> >> 
> >> @@ -231,7 +231,7 @@ static void __init smdk6440_map_io(void)
> >> 
> >>  	s5p64x0_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(12000000);
> >>  	s3c24xx_init_uarts(smdk6440_uartcfgs, ARRAY_SIZE(smdk6440_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void s5p6440_set_lcd_interface(void)
> >> 
> >> @@ -276,6 +276,6 @@ MACHINE_START(SMDK6440, "SMDK6440")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= smdk6440_map_io,
> >>  	.init_machine	= smdk6440_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5p64x0_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5p64x0/mach-smdk6450.c
> >> b/arch/arm/mach-s5p64x0/mach-smdk6450.c index 12748b6..714cd8a 100644
> >> --- a/arch/arm/mach-s5p64x0/mach-smdk6450.c
> >> +++ b/arch/arm/mach-s5p64x0/mach-smdk6450.c
> >> @@ -50,7 +50,7 @@
> >> 
> >>  #include <plat/pll.h>
> >>  #include <plat/adc.h>
> >>  #include <linux/platform_data/touchscreen-s3c2410.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/backlight.h>
> >>  #include <plat/fb.h>
> >>  #include <plat/sdhci.h>
> >> 
> >> @@ -250,7 +250,7 @@ static void __init smdk6450_map_io(void)
> >> 
> >>  	s5p64x0_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(19200000);
> >>  	s3c24xx_init_uarts(smdk6450_uartcfgs, ARRAY_SIZE(smdk6450_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void s5p6450_set_lcd_interface(void)
> >> 
> >> @@ -295,6 +295,6 @@ MACHINE_START(SMDK6450, "SMDK6450")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= smdk6450_map_io,
> >>  	.init_machine	= smdk6450_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5p64x0_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/Kconfig
> >> b/arch/arm/mach-s5pv210/Kconfig index 92ad72f..01018ef 100644
> >> --- a/arch/arm/mach-s5pv210/Kconfig
> >> +++ b/arch/arm/mach-s5pv210/Kconfig
> >> @@ -12,7 +12,7 @@ if ARCH_S5PV210
> >> 
> >>  config CPU_S5PV210
> >>  
> >>  	bool
> >>  	select S5P_EXT_INT
> >> 
> >> -	select S5P_HRT
> >> +	select SAMSUNG_HRT
> >> 
> >>  	select S5P_PM if PM
> >>  	select S5P_SLEEP if PM
> >>  	select SAMSUNG_DMADEV
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/mach-aquila.c
> >> b/arch/arm/mach-s5pv210/mach-aquila.c index ee9fa5c..7c7d89b 100644
> >> --- a/arch/arm/mach-s5pv210/mach-aquila.c
> >> +++ b/arch/arm/mach-s5pv210/mach-aquila.c
> >> @@ -39,7 +39,7 @@
> >> 
> >>  #include <plat/fb.h>
> >>  #include <plat/fimc-core.h>
> >>  #include <plat/sdhci.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include "common.h"
> >> 
> >> @@ -652,7 +652,7 @@ static void __init aquila_map_io(void)
> >> 
> >>  	s5pv210_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(24000000);
> >>  	s3c24xx_init_uarts(aquila_uartcfgs, ARRAY_SIZE(aquila_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void __init aquila_machine_init(void)
> >> 
> >> @@ -688,6 +688,6 @@ MACHINE_START(AQUILA, "Aquila")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= aquila_map_io,
> >>  	.init_machine	= aquila_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5pv210_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/mach-goni.c
> >> b/arch/arm/mach-s5pv210/mach-goni.c index 55e1dba..2740001 100644
> >> --- a/arch/arm/mach-s5pv210/mach-goni.c
> >> +++ b/arch/arm/mach-s5pv210/mach-goni.c
> >> @@ -48,7 +48,7 @@
> >> 
> >>  #include <plat/keypad.h>
> >>  #include <plat/sdhci.h>
> >>  #include <plat/clock.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/mfc.h>
> >>  #include <plat/camport.h>
> >> 
> >> @@ -910,7 +910,7 @@ static void __init goni_map_io(void)
> >> 
> >>  	s5pv210_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(clk_xusbxti.rate);
> >>  	s3c24xx_init_uarts(goni_uartcfgs, ARRAY_SIZE(goni_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void __init goni_reserve(void)
> >> 
> >> @@ -976,7 +976,7 @@ MACHINE_START(GONI, "GONI")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= goni_map_io,
> >>  	.init_machine	= goni_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.reserve	= &goni_reserve,
> >>  	.restart	= s5pv210_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/mach-smdkc110.c
> >> b/arch/arm/mach-s5pv210/mach-smdkc110.c index d9c99fc..d2e93b7 100644
> >> --- a/arch/arm/mach-s5pv210/mach-smdkc110.c
> >> +++ b/arch/arm/mach-s5pv210/mach-smdkc110.c
> >> @@ -30,7 +30,7 @@
> >> 
> >>  #include <linux/platform_data/ata-samsung_cf.h>
> >>  #include <linux/platform_data/i2c-s3c2410.h>
> >>  #include <plat/pm.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/mfc.h>
> >>  
> >>  #include "common.h"
> >> 
> >> @@ -122,7 +122,7 @@ static void __init smdkc110_map_io(void)
> >> 
> >>  	s5pv210_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(24000000);
> >>  	s3c24xx_init_uarts(smdkv210_uartcfgs, ARRAY_SIZE(smdkv210_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void __init smdkc110_reserve(void)
> >> 
> >> @@ -156,7 +156,7 @@ MACHINE_START(SMDKC110, "SMDKC110")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= smdkc110_map_io,
> >>  	.init_machine	= smdkc110_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5pv210_restart,
> >>  	.reserve	= &smdkc110_reserve,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/mach-smdkv210.c
> >> b/arch/arm/mach-s5pv210/mach-smdkv210.c index 4cdb5bb..cd28725 100644
> >> --- a/arch/arm/mach-s5pv210/mach-smdkv210.c
> >> +++ b/arch/arm/mach-s5pv210/mach-smdkv210.c
> >> @@ -45,7 +45,7 @@
> >> 
> >>  #include <plat/keypad.h>
> >>  #include <plat/pm.h>
> >>  #include <plat/fb.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include <plat/backlight.h>
> >>  #include <plat/mfc.h>
> >>  #include <plat/clock.h>
> >> 
> >> @@ -287,7 +287,7 @@ static void __init smdkv210_map_io(void)
> >> 
> >>  	s5pv210_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(clk_xusbxti.rate);
> >>  	s3c24xx_init_uarts(smdkv210_uartcfgs, ARRAY_SIZE(smdkv210_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM2, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM2, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void __init smdkv210_reserve(void)
> >> 
> >> @@ -332,7 +332,7 @@ MACHINE_START(SMDKV210, "SMDKV210")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= smdkv210_map_io,
> >>  	.init_machine	= smdkv210_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5pv210_restart,
> >>  	.reserve	= &smdkv210_reserve,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/mach-s5pv210/mach-torbreck.c
> >> b/arch/arm/mach-s5pv210/mach-torbreck.c index 18785cb..aec668c 100644
> >> --- a/arch/arm/mach-s5pv210/mach-torbreck.c
> >> +++ b/arch/arm/mach-s5pv210/mach-torbreck.c
> >> @@ -27,7 +27,7 @@
> >> 
> >>  #include <plat/devs.h>
> >>  #include <plat/cpu.h>
> >>  #include <linux/platform_data/i2c-s3c2410.h>
> >> 
> >> -#include <plat/s5p-time.h>
> >> +#include <plat/samsung-time.h>
> >> 
> >>  #include "common.h"
> >> 
> >> @@ -107,7 +107,7 @@ static void __init torbreck_map_io(void)
> >> 
> >>  	s5pv210_init_io(NULL, 0);
> >>  	s3c24xx_init_clocks(24000000);
> >>  	s3c24xx_init_uarts(torbreck_uartcfgs, ARRAY_SIZE(torbreck_uartcfgs));
> >> 
> >> -	s5p_set_timer_source(S5P_PWM3, S5P_PWM4);
> >> +	samsung_set_timer_source(SAMSUNG_PWM3, SAMSUNG_PWM4);
> >> 
> >>  }
> >>  
> >>  static void __init torbreck_machine_init(void)
> >> 
> >> @@ -132,6 +132,6 @@ MACHINE_START(TORBRECK, "TORBRECK")
> >> 
> >>  	.handle_irq	= vic_handle_irq,
> >>  	.map_io		= torbreck_map_io,
> >>  	.init_machine	= torbreck_machine_init,
> >> 
> >> -	.timer		= &s5p_timer,
> >> +	.timer		= &samsung_timer,
> >> 
> >>  	.restart	= s5pv210_restart,
> >>  
> >>  MACHINE_END
> >> 
> >> diff --git a/arch/arm/plat-samsung/Kconfig
> >> b/arch/arm/plat-samsung/Kconfig index 59401e1..5278795 100644
> >> --- a/arch/arm/plat-samsung/Kconfig
> >> +++ b/arch/arm/plat-samsung/Kconfig
> >> @@ -70,7 +70,7 @@ config S3C_LOWLEVEL_UART_PORT
> >> 
> >>  # timer options
> >> 
> >> -config S5P_HRT
> >> +config SAMSUNG_HRT
> >> 
> >>  	bool
> >>  	select SAMSUNG_DEV_PWM
> >>  	help
> >> 
> >> diff --git a/arch/arm/plat-samsung/Makefile
> >> b/arch/arm/plat-samsung/Makefile index 9e40e8d..06f2312 100644
> >> --- a/arch/arm/plat-samsung/Makefile
> >> +++ b/arch/arm/plat-samsung/Makefile
> >> @@ -13,7 +13,7 @@ obj-				:=
> >> 
> >>  obj-y				+= init.o cpu.o
> >>  obj-$(CONFIG_ARCH_USES_GETTIMEOFFSET)   += time.o
> >> 
> >> -obj-$(CONFIG_S5P_HRT) 		+= s5p-time.o
> >> +obj-$(CONFIG_SAMSUNG_HRT) 		+= samsung-time.o
> >> 
> >>  obj-$(CONFIG_SAMSUNG_CLOCK)	+= clock.o
> >>  obj-$(CONFIG_SAMSUNG_CLOCK)	+= pwm-clock.o
> >> 
> >> diff --git a/arch/arm/plat-samsung/include/plat/s5p-time.h
> >> b/arch/arm/plat-samsung/include/plat/s5p-time.h deleted file mode 100644
> >> index 3a70aeb..0000000
> >> --- a/arch/arm/plat-samsung/include/plat/s5p-time.h
> >> +++ /dev/null
> >> @@ -1,40 +0,0 @@
> >> -/* linux/arch/arm/plat-samsung/include/plat/s5p-time.h
> >> - *
> >> - * Copyright 2011 Samsung Electronics Co., Ltd.
> >> - *		http://www.samsung.com/
> >> - *
> >> - * Header file for s5p time support
> >> - *
> >> - * This program is free software; you can redistribute it and/or modify
> >> - * it under the terms of the GNU General Public License version 2 as
> >> - * published by the Free Software Foundation.
> >> -*/
> >> -
> >> -#ifndef __ASM_PLAT_S5P_TIME_H
> >> -#define __ASM_PLAT_S5P_TIME_H __FILE__
> >> -
> >> -/* S5P HR-Timer Clock mode */
> >> -enum s5p_timer_mode {
> >> -	S5P_PWM0,
> >> -	S5P_PWM1,
> >> -	S5P_PWM2,
> >> -	S5P_PWM3,
> >> -	S5P_PWM4,
> >> -};
> >> -
> >> -struct s5p_timer_source {
> >> -	unsigned int event_id;
> >> -	unsigned int source_id;
> >> -};
> >> -
> >> -/* Be able to sleep for atleast 4 seconds (usually more) */
> >> -#define S5PTIMER_MIN_RANGE	4
> >> -
> >> -#define TCNT_MAX		0xffffffff
> >> -#define NON_PERIODIC		0
> >> -#define PERIODIC		1
> >> -
> >> -extern void __init s5p_set_timer_source(enum s5p_timer_mode event,
> >> -					enum s5p_timer_mode source);
> >> -extern	struct sys_timer s5p_timer;
> >> -#endif /* __ASM_PLAT_S5P_TIME_H */
> >> diff --git a/arch/arm/plat-samsung/include/plat/samsung-time.h
> >> b/arch/arm/plat-samsung/include/plat/samsung-time.h new file mode 100644
> >> index 0000000..9d6d622
> >> --- /dev/null
> >> +++ b/arch/arm/plat-samsung/include/plat/samsung-time.h
> >> @@ -0,0 +1,40 @@
> >> +/* linux/arch/arm/plat-samsung/include/plat/samsung-time.h
> >> + *
> >> + * Copyright 2011 Samsung Electronics Co., Ltd.
> >> + *		http://www.samsung.com/
> >> + *
> >> + * Header file for samsung s3c and s5p time support
> >> + *
> >> + * This program is free software; you can redistribute it and/or modify
> >> + * it under the terms of the GNU General Public License version 2 as
> >> + * published by the Free Software Foundation.
> >> +*/
> >> +
> >> +#ifndef __ASM_PLAT_SAMSUNG_TIME_H
> >> +#define __ASM_PLAT_SAMSUNG_TIME_H __FILE__
> >> +
> >> +/* SAMSUNG HR-Timer Clock mode */
> >> +enum samsung_timer_mode {
> >> +	SAMSUNG_PWM0,
> >> +	SAMSUNG_PWM1,
> >> +	SAMSUNG_PWM2,
> >> +	SAMSUNG_PWM3,
> >> +	SAMSUNG_PWM4,
> >> +};
> >> +
> >> +struct samsung_timer_source {
> >> +	unsigned int event_id;
> >> +	unsigned int source_id;
> >> +};
> >> +
> >> +/* Be able to sleep for atleast 4 seconds (usually more) */
> >> +#define SAMSUNG_TIMER_MIN_RANGE	4
> >> +
> >> +#define TCNT_MAX		0xffffffff
> >> +#define NON_PERIODIC		0
> >> +#define PERIODIC		1
> >> +
> >> +extern void __init samsung_set_timer_source(enum samsung_timer_mode
> >> event, +					enum samsung_timer_mode source);
> >> +extern	struct sys_timer samsung_timer;
> >> +#endif /* __ASM_PLAT_SAMSUNG_TIME_H */
> >> diff --git a/arch/arm/plat-samsung/s5p-time.c
> >> b/arch/arm/plat-samsung/s5p-time.c deleted file mode 100644
> >> index 028b6e8..0000000
> >> --- a/arch/arm/plat-samsung/s5p-time.c
> >> +++ /dev/null
> >> @@ -1,405 +0,0 @@
> >> -/*
> >> - * Copyright (c) 2011 Samsung Electronics Co., Ltd.
> >> - *		http://www.samsung.com/
> >> - *
> >> - * S5P - Common hr-timer support
> >> - *
> >> - * This program is free software; you can redistribute it and/or modify
> >> - * it under the terms of the GNU General Public License version 2 as
> >> - * published by the Free Software Foundation.
> >> -*/
> >> -
> >> -#include <linux/interrupt.h>
> >> -#include <linux/irq.h>
> >> -#include <linux/err.h>
> >> -#include <linux/clk.h>
> >> -#include <linux/clockchips.h>
> >> -#include <linux/platform_device.h>
> >> -
> >> -#include <asm/smp_twd.h>
> >> -#include <asm/mach/time.h>
> >> -#include <asm/mach/arch.h>
> >> -#include <asm/mach/map.h>
> >> -#include <asm/sched_clock.h>
> >> -
> >> -#include <mach/map.h>
> >> -#include <plat/devs.h>
> >> -#include <plat/regs-timer.h>
> >> -#include <plat/s5p-time.h>
> >> -
> >> -static struct clk *tin_event;
> >> -static struct clk *tin_source;
> >> -static struct clk *tdiv_event;
> >> -static struct clk *tdiv_source;
> >> -static struct clk *timerclk;
> >> -static struct s5p_timer_source timer_source;
> >> -static unsigned long clock_count_per_tick;
> >> -static void s5p_timer_resume(void);
> >> -
> >> -static void s5p_time_stop(enum s5p_timer_mode mode)
> >> -{
> >> -	unsigned long tcon;
> >> -
> >> -	tcon = __raw_readl(S3C2410_TCON);
> >> -
> >> -	switch (mode) {
> >> -	case S5P_PWM0:
> >> -		tcon &= ~S3C2410_TCON_T0START;
> >> -		break;
> >> -
> >> -	case S5P_PWM1:
> >> -		tcon &= ~S3C2410_TCON_T1START;
> >> -		break;
> >> -
> >> -	case S5P_PWM2:
> >> -		tcon &= ~S3C2410_TCON_T2START;
> >> -		break;
> >> -
> >> -	case S5P_PWM3:
> >> -		tcon &= ~S3C2410_TCON_T3START;
> >> -		break;
> >> -
> >> -	case S5P_PWM4:
> >> -		tcon &= ~S3C2410_TCON_T4START;
> >> -		break;
> >> -
> >> -	default:
> >> -		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> -		break;
> >> -	}
> >> -	__raw_writel(tcon, S3C2410_TCON);
> >> -}
> >> -
> >> -static void s5p_time_setup(enum s5p_timer_mode mode, unsigned long
> >> tcnt) -{
> >> -	unsigned long tcon;
> >> -
> >> -	tcon = __raw_readl(S3C2410_TCON);
> >> -
> >> -	tcnt--;
> >> -
> >> -	switch (mode) {
> >> -	case S5P_PWM0:
> >> -		tcon &= ~(0x0f << 0);
> >> -		tcon |= S3C2410_TCON_T0MANUALUPD;
> >> -		break;
> >> -
> >> -	case S5P_PWM1:
> >> -		tcon &= ~(0x0f << 8);
> >> -		tcon |= S3C2410_TCON_T1MANUALUPD;
> >> -		break;
> >> -
> >> -	case S5P_PWM2:
> >> -		tcon &= ~(0x0f << 12);
> >> -		tcon |= S3C2410_TCON_T2MANUALUPD;
> >> -		break;
> >> -
> >> -	case S5P_PWM3:
> >> -		tcon &= ~(0x0f << 16);
> >> -		tcon |= S3C2410_TCON_T3MANUALUPD;
> >> -		break;
> >> -
> >> -	case S5P_PWM4:
> >> -		tcon &= ~(0x07 << 20);
> >> -		tcon |= S3C2410_TCON_T4MANUALUPD;
> >> -		break;
> >> -
> >> -	default:
> >> -		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> -		break;
> >> -	}
> >> -
> >> -	__raw_writel(tcnt, S3C2410_TCNTB(mode));
> >> -	__raw_writel(tcnt, S3C2410_TCMPB(mode));
> >> -	__raw_writel(tcon, S3C2410_TCON);
> >> -}
> >> -
> >> -static void s5p_time_start(enum s5p_timer_mode mode, bool periodic)
> >> -{
> >> -	unsigned long tcon;
> >> -
> >> -	tcon  = __raw_readl(S3C2410_TCON);
> >> -
> >> -	switch (mode) {
> >> -	case S5P_PWM0:
> >> -		tcon |= S3C2410_TCON_T0START;
> >> -		tcon &= ~S3C2410_TCON_T0MANUALUPD;
> >> -
> >> -		if (periodic)
> >> -			tcon |= S3C2410_TCON_T0RELOAD;
> >> -		else
> >> -			tcon &= ~S3C2410_TCON_T0RELOAD;
> >> -		break;
> >> -
> >> -	case S5P_PWM1:
> >> -		tcon |= S3C2410_TCON_T1START;
> >> -		tcon &= ~S3C2410_TCON_T1MANUALUPD;
> >> -
> >> -		if (periodic)
> >> -			tcon |= S3C2410_TCON_T1RELOAD;
> >> -		else
> >> -			tcon &= ~S3C2410_TCON_T1RELOAD;
> >> -		break;
> >> -
> >> -	case S5P_PWM2:
> >> -		tcon |= S3C2410_TCON_T2START;
> >> -		tcon &= ~S3C2410_TCON_T2MANUALUPD;
> >> -
> >> -		if (periodic)
> >> -			tcon |= S3C2410_TCON_T2RELOAD;
> >> -		else
> >> -			tcon &= ~S3C2410_TCON_T2RELOAD;
> >> -		break;
> >> -
> >> -	case S5P_PWM3:
> >> -		tcon |= S3C2410_TCON_T3START;
> >> -		tcon &= ~S3C2410_TCON_T3MANUALUPD;
> >> -
> >> -		if (periodic)
> >> -			tcon |= S3C2410_TCON_T3RELOAD;
> >> -		else
> >> -			tcon &= ~S3C2410_TCON_T3RELOAD;
> >> -		break;
> >> -
> >> -	case S5P_PWM4:
> >> -		tcon |= S3C2410_TCON_T4START;
> >> -		tcon &= ~S3C2410_TCON_T4MANUALUPD;
> >> -
> >> -		if (periodic)
> >> -			tcon |= S3C2410_TCON_T4RELOAD;
> >> -		else
> >> -			tcon &= ~S3C2410_TCON_T4RELOAD;
> >> -		break;
> >> -
> >> -	default:
> >> -		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> -		break;
> >> -	}
> >> -	__raw_writel(tcon, S3C2410_TCON);
> >> -}
> >> -
> >> -static int s5p_set_next_event(unsigned long cycles,
> >> -				struct clock_event_device *evt)
> >> -{
> >> -	s5p_time_setup(timer_source.event_id, cycles);
> >> -	s5p_time_start(timer_source.event_id, NON_PERIODIC);
> >> -
> >> -	return 0;
> >> -}
> >> -
> >> -static void s5p_set_mode(enum clock_event_mode mode,
> >> -				struct clock_event_device *evt)
> >> -{
> >> -	s5p_time_stop(timer_source.event_id);
> >> -
> >> -	switch (mode) {
> >> -	case CLOCK_EVT_MODE_PERIODIC:
> >> -		s5p_time_setup(timer_source.event_id, clock_count_per_tick);
> >> -		s5p_time_start(timer_source.event_id, PERIODIC);
> >> -		break;
> >> -
> >> -	case CLOCK_EVT_MODE_ONESHOT:
> >> -		break;
> >> -
> >> -	case CLOCK_EVT_MODE_UNUSED:
> >> -	case CLOCK_EVT_MODE_SHUTDOWN:
> >> -		break;
> >> -
> >> -	case CLOCK_EVT_MODE_RESUME:
> >> -		s5p_timer_resume();
> >> -		break;
> >> -	}
> >> -}
> >> -
> >> -static void s5p_timer_resume(void)
> >> -{
> >> -	/* event timer restart */
> >> -	s5p_time_setup(timer_source.event_id, clock_count_per_tick);
> >> -	s5p_time_start(timer_source.event_id, PERIODIC);
> >> -
> >> -	/* source timer restart */
> >> -	s5p_time_setup(timer_source.source_id, TCNT_MAX);
> >> -	s5p_time_start(timer_source.source_id, PERIODIC);
> >> -}
> >> -
> >> -void __init s5p_set_timer_source(enum s5p_timer_mode event,
> >> -				 enum s5p_timer_mode source)
> >> -{
> >> -	s3c_device_timer[event].dev.bus = &platform_bus_type;
> >> -	s3c_device_timer[source].dev.bus = &platform_bus_type;
> >> -
> >> -	timer_source.event_id = event;
> >> -	timer_source.source_id = source;
> >> -}
> >> -
> >> -static struct clock_event_device time_event_device = {
> >> -	.name		= "s5p_event_timer",
> >> -	.features	= CLOCK_EVT_FEAT_PERIODIC | CLOCK_EVT_FEAT_ONESHOT,
> >> -	.rating		= 200,
> >> -	.set_next_event	= s5p_set_next_event,
> >> -	.set_mode	= s5p_set_mode,
> >> -};
> >> -
> >> -static irqreturn_t s5p_clock_event_isr(int irq, void *dev_id)
> >> -{
> >> -	struct clock_event_device *evt = dev_id;
> >> -
> >> -	evt->event_handler(evt);
> >> -
> >> -	return IRQ_HANDLED;
> >> -}
> >> -
> >> -static struct irqaction s5p_clock_event_irq = {
> >> -	.name		= "s5p_time_irq",
> >> -	.flags		= IRQF_DISABLED | IRQF_TIMER | IRQF_IRQPOLL,
> >> -	.handler	= s5p_clock_event_isr,
> >> -	.dev_id		= &time_event_device,
> >> -};
> >> -
> >> -static void __init s5p_clockevent_init(void)
> >> -{
> >> -	unsigned long pclk;
> >> -	unsigned long clock_rate;
> >> -	unsigned int irq_number;
> >> -	struct clk *tscaler;
> >> -
> >> -	pclk = clk_get_rate(timerclk);
> >> -
> >> -	tscaler = clk_get_parent(tdiv_event);
> >> -
> >> -	clk_set_rate(tscaler, pclk / 2);
> >> -	clk_set_rate(tdiv_event, pclk / 2);
> >> -	clk_set_parent(tin_event, tdiv_event);
> >> -
> >> -	clock_rate = clk_get_rate(tin_event);
> >> -	clock_count_per_tick = clock_rate / HZ;
> >> -
> >> -	clockevents_calc_mult_shift(&time_event_device,
> >> -				    clock_rate, S5PTIMER_MIN_RANGE);
> >> -	time_event_device.max_delta_ns =
> >> -		clockevent_delta2ns(-1, &time_event_device);
> >> -	time_event_device.min_delta_ns =
> >> -		clockevent_delta2ns(1, &time_event_device);
> >> -
> >> -	time_event_device.cpumask = cpumask_of(0);
> >> -	clockevents_register_device(&time_event_device);
> >> -
> >> -	irq_number = timer_source.event_id + IRQ_TIMER0;
> >> -	setup_irq(irq_number, &s5p_clock_event_irq);
> >> -}
> >> -
> >> -static void __iomem *s5p_timer_reg(void)
> >> -{
> >> -	unsigned long offset = 0;
> >> -
> >> -	switch (timer_source.source_id) {
> >> -	case S5P_PWM0:
> >> -	case S5P_PWM1:
> >> -	case S5P_PWM2:
> >> -	case S5P_PWM3:
> >> -		offset = (timer_source.source_id * 0x0c) + 0x14;
> >> -		break;
> >> -
> >> -	case S5P_PWM4:
> >> -		offset = 0x40;
> >> -		break;
> >> -
> >> -	default:
> >> -		printk(KERN_ERR "Invalid Timer %d\n", timer_source.source_id);
> >> -		return NULL;
> >> -	}
> >> -
> >> -	return S3C_TIMERREG(offset);
> >> -}
> >> -
> >> -/*
> >> - * Override the global weak sched_clock symbol with this
> >> - * local implementation which uses the clocksource to get some
> >> - * better resolution when scheduling the kernel. We accept that
> >> - * this wraps around for now, since it is just a relative time
> >> - * stamp. (Inspired by U300 implementation.)
> >> - */
> >> -static u32 notrace s5p_read_sched_clock(void)
> >> -{
> >> -	void __iomem *reg = s5p_timer_reg();
> >> -
> >> -	if (!reg)
> >> -		return 0;
> >> -
> >> -	return ~__raw_readl(reg);
> >> -}
> >> -
> >> -static void __init s5p_clocksource_init(void)
> >> -{
> >> -	unsigned long pclk;
> >> -	unsigned long clock_rate;
> >> -
> >> -	pclk = clk_get_rate(timerclk);
> >> -
> >> -	clk_set_rate(tdiv_source, pclk / 2);
> >> -	clk_set_parent(tin_source, tdiv_source);
> >> -
> >> -	clock_rate = clk_get_rate(tin_source);
> >> -
> >> -	s5p_time_setup(timer_source.source_id, TCNT_MAX);
> >> -	s5p_time_start(timer_source.source_id, PERIODIC);
> >> -
> >> -	setup_sched_clock(s5p_read_sched_clock, 32, clock_rate);
> >> -
> >> -	if (clocksource_mmio_init(s5p_timer_reg(), "s5p_clocksource_timer",
> >> -			clock_rate, 250, 32, clocksource_mmio_readl_down))
> >> -		panic("s5p_clocksource_timer: can't register clocksource\n");
> >> -}
> >> -
> >> -static void __init s5p_timer_resources(void)
> >> -{
> >> -
> >> -	unsigned long event_id = timer_source.event_id;
> >> -	unsigned long source_id = timer_source.source_id;
> >> -	char devname[15];
> >> -
> >> -	timerclk = clk_get(NULL, "timers");
> >> -	if (IS_ERR(timerclk))
> >> -		panic("failed to get timers clock for timer");
> >> -
> >> -	clk_enable(timerclk);
> >> -
> >> -	sprintf(devname, "s3c24xx-pwm.%lu", event_id);
> >> -	s3c_device_timer[event_id].id = event_id;
> >> -	s3c_device_timer[event_id].dev.init_name = devname;
> >> -
> >> -	tin_event = clk_get(&s3c_device_timer[event_id].dev, "pwm-tin");
> >> -	if (IS_ERR(tin_event))
> >> -		panic("failed to get pwm-tin clock for event timer");
> >> -
> >> -	tdiv_event = clk_get(&s3c_device_timer[event_id].dev, "pwm-tdiv");
> >> -	if (IS_ERR(tdiv_event))
> >> -		panic("failed to get pwm-tdiv clock for event timer");
> >> -
> >> -	clk_enable(tin_event);
> >> -
> >> -	sprintf(devname, "s3c24xx-pwm.%lu", source_id);
> >> -	s3c_device_timer[source_id].id = source_id;
> >> -	s3c_device_timer[source_id].dev.init_name = devname;
> >> -
> >> -	tin_source = clk_get(&s3c_device_timer[source_id].dev, "pwm-tin");
> >> -	if (IS_ERR(tin_source))
> >> -		panic("failed to get pwm-tin clock for source timer");
> >> -
> >> -	tdiv_source = clk_get(&s3c_device_timer[source_id].dev, "pwm-tdiv");
> >> -	if (IS_ERR(tdiv_source))
> >> -		panic("failed to get pwm-tdiv clock for source timer");
> >> -
> >> -	clk_enable(tin_source);
> >> -}
> >> -
> >> -static void __init s5p_timer_init(void)
> >> -{
> >> -	s5p_timer_resources();
> >> -	s5p_clockevent_init();
> >> -	s5p_clocksource_init();
> >> -}
> >> -
> >> -struct sys_timer s5p_timer = {
> >> -	.init		= s5p_timer_init,
> >> -};
> >> diff --git a/arch/arm/plat-samsung/samsung-time.c
> >> b/arch/arm/plat-samsung/samsung-time.c new file mode 100644
> >> index 0000000..91773bf
> >> --- /dev/null
> >> +++ b/arch/arm/plat-samsung/samsung-time.c
> >> @@ -0,0 +1,405 @@
> >> +/*
> >> + * Copyright (c) 2011 Samsung Electronics Co., Ltd.
> >> + *		http://www.samsung.com/
> >> + *
> >> + * SAMSUNG - Common hr-timer support
> >> + *
> >> + * This program is free software; you can redistribute it and/or modify
> >> + * it under the terms of the GNU General Public License version 2 as
> >> + * published by the Free Software Foundation.
> >> +*/
> >> +
> >> +#include <linux/interrupt.h>
> >> +#include <linux/irq.h>
> >> +#include <linux/err.h>
> >> +#include <linux/clk.h>
> >> +#include <linux/clockchips.h>
> >> +#include <linux/platform_device.h>
> >> +
> >> +#include <asm/smp_twd.h>
> >> +#include <asm/mach/time.h>
> >> +#include <asm/mach/arch.h>
> >> +#include <asm/mach/map.h>
> >> +#include <asm/sched_clock.h>
> >> +
> >> +#include <mach/map.h>
> >> +#include <plat/devs.h>
> >> +#include <plat/regs-timer.h>
> >> +#include <plat/samsung-time.h>
> >> +
> >> +static struct clk *tin_event;
> >> +static struct clk *tin_source;
> >> +static struct clk *tdiv_event;
> >> +static struct clk *tdiv_source;
> >> +static struct clk *timerclk;
> >> +static struct samsung_timer_source timer_source;
> >> +static unsigned long clock_count_per_tick;
> >> +static void samsung_timer_resume(void);
> >> +
> >> +static void samsung_time_stop(enum samsung_timer_mode mode)
> >> +{
> >> +	unsigned long tcon;
> >> +
> >> +	tcon = __raw_readl(S3C2410_TCON);
> >> +
> >> +	switch (mode) {
> >> +	case SAMSUNG_PWM0:
> >> +		tcon &= ~S3C2410_TCON_T0START;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM1:
> >> +		tcon &= ~S3C2410_TCON_T1START;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM2:
> >> +		tcon &= ~S3C2410_TCON_T2START;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM3:
> >> +		tcon &= ~S3C2410_TCON_T3START;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM4:
> >> +		tcon &= ~S3C2410_TCON_T4START;
> >> +		break;
> >> +
> >> +	default:
> >> +		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> +		break;
> >> +	}
> >> +	__raw_writel(tcon, S3C2410_TCON);
> >> +}
> >> +
> >> +static void samsung_time_setup(enum samsung_timer_mode mode, unsigned
> >> long tcnt) +{
> >> +	unsigned long tcon;
> >> +
> >> +	tcon = __raw_readl(S3C2410_TCON);
> >> +
> >> +	tcnt--;
> >> +
> >> +	switch (mode) {
> >> +	case SAMSUNG_PWM0:
> >> +		tcon &= ~(0x0f << 0);
> >> +		tcon |= S3C2410_TCON_T0MANUALUPD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM1:
> >> +		tcon &= ~(0x0f << 8);
> >> +		tcon |= S3C2410_TCON_T1MANUALUPD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM2:
> >> +		tcon &= ~(0x0f << 12);
> >> +		tcon |= S3C2410_TCON_T2MANUALUPD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM3:
> >> +		tcon &= ~(0x0f << 16);
> >> +		tcon |= S3C2410_TCON_T3MANUALUPD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM4:
> >> +		tcon &= ~(0x07 << 20);
> >> +		tcon |= S3C2410_TCON_T4MANUALUPD;
> >> +		break;
> >> +
> >> +	default:
> >> +		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> +		break;
> >> +	}
> >> +
> >> +	__raw_writel(tcnt, S3C2410_TCNTB(mode));
> >> +	__raw_writel(tcnt, S3C2410_TCMPB(mode));
> >> +	__raw_writel(tcon, S3C2410_TCON);
> >> +}
> >> +
> >> +static void samsung_time_start(enum samsung_timer_mode mode, bool
> >> periodic) +{
> >> +	unsigned long tcon;
> >> +
> >> +	tcon  = __raw_readl(S3C2410_TCON);
> >> +
> >> +	switch (mode) {
> >> +	case SAMSUNG_PWM0:
> >> +		tcon |= S3C2410_TCON_T0START;
> >> +		tcon &= ~S3C2410_TCON_T0MANUALUPD;
> >> +
> >> +		if (periodic)
> >> +			tcon |= S3C2410_TCON_T0RELOAD;
> >> +		else
> >> +			tcon &= ~S3C2410_TCON_T0RELOAD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM1:
> >> +		tcon |= S3C2410_TCON_T1START;
> >> +		tcon &= ~S3C2410_TCON_T1MANUALUPD;
> >> +
> >> +		if (periodic)
> >> +			tcon |= S3C2410_TCON_T1RELOAD;
> >> +		else
> >> +			tcon &= ~S3C2410_TCON_T1RELOAD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM2:
> >> +		tcon |= S3C2410_TCON_T2START;
> >> +		tcon &= ~S3C2410_TCON_T2MANUALUPD;
> >> +
> >> +		if (periodic)
> >> +			tcon |= S3C2410_TCON_T2RELOAD;
> >> +		else
> >> +			tcon &= ~S3C2410_TCON_T2RELOAD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM3:
> >> +		tcon |= S3C2410_TCON_T3START;
> >> +		tcon &= ~S3C2410_TCON_T3MANUALUPD;
> >> +
> >> +		if (periodic)
> >> +			tcon |= S3C2410_TCON_T3RELOAD;
> >> +		else
> >> +			tcon &= ~S3C2410_TCON_T3RELOAD;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM4:
> >> +		tcon |= S3C2410_TCON_T4START;
> >> +		tcon &= ~S3C2410_TCON_T4MANUALUPD;
> >> +
> >> +		if (periodic)
> >> +			tcon |= S3C2410_TCON_T4RELOAD;
> >> +		else
> >> +			tcon &= ~S3C2410_TCON_T4RELOAD;
> >> +		break;
> >> +
> >> +	default:
> >> +		printk(KERN_ERR "Invalid Timer %d\n", mode);
> >> +		break;
> >> +	}
> >> +	__raw_writel(tcon, S3C2410_TCON);
> >> +}
> >> +
> >> +static int samsung_set_next_event(unsigned long cycles,
> >> +				struct clock_event_device *evt)
> >> +{
> >> +	samsung_time_setup(timer_source.event_id, cycles);
> >> +	samsung_time_start(timer_source.event_id, NON_PERIODIC);
> >> +
> >> +	return 0;
> >> +}
> >> +
> >> +static void samsung_set_mode(enum clock_event_mode mode,
> >> +				struct clock_event_device *evt)
> >> +{
> >> +	samsung_time_stop(timer_source.event_id);
> >> +
> >> +	switch (mode) {
> >> +	case CLOCK_EVT_MODE_PERIODIC:
> >> +		samsung_time_setup(timer_source.event_id, clock_count_per_tick);
> >> +		samsung_time_start(timer_source.event_id, PERIODIC);
> >> +		break;
> >> +
> >> +	case CLOCK_EVT_MODE_ONESHOT:
> >> +		break;
> >> +
> >> +	case CLOCK_EVT_MODE_UNUSED:
> >> +	case CLOCK_EVT_MODE_SHUTDOWN:
> >> +		break;
> >> +
> >> +	case CLOCK_EVT_MODE_RESUME:
> >> +		samsung_timer_resume();
> >> +		break;
> >> +	}
> >> +}
> >> +
> >> +static void samsung_timer_resume(void)
> >> +{
> >> +	/* event timer restart */
> >> +	samsung_time_setup(timer_source.event_id, clock_count_per_tick);
> >> +	samsung_time_start(timer_source.event_id, PERIODIC);
> >> +
> >> +	/* source timer restart */
> >> +	samsung_time_setup(timer_source.source_id, TCNT_MAX);
> >> +	samsung_time_start(timer_source.source_id, PERIODIC);
> >> +}
> >> +
> >> +void __init samsung_set_timer_source(enum samsung_timer_mode event,
> >> +				 enum samsung_timer_mode source)
> >> +{
> >> +	s3c_device_timer[event].dev.bus = &platform_bus_type;
> >> +	s3c_device_timer[source].dev.bus = &platform_bus_type;
> >> +
> >> +	timer_source.event_id = event;
> >> +	timer_source.source_id = source;
> >> +}
> >> +
> >> +static struct clock_event_device time_event_device = {
> >> +	.name		= "samsung_event_timer",
> >> +	.features	= CLOCK_EVT_FEAT_PERIODIC | CLOCK_EVT_FEAT_ONESHOT,
> >> +	.rating		= 200,
> >> +	.set_next_event	= samsung_set_next_event,
> >> +	.set_mode	= samsung_set_mode,
> >> +};
> >> +
> >> +static irqreturn_t samsung_clock_event_isr(int irq, void *dev_id)
> >> +{
> >> +	struct clock_event_device *evt = dev_id;
> >> +
> >> +	evt->event_handler(evt);
> >> +
> >> +	return IRQ_HANDLED;
> >> +}
> >> +
> >> +static struct irqaction samsung_clock_event_irq = {
> >> +	.name		= "samsung_time_irq",
> >> +	.flags		= IRQF_DISABLED | IRQF_TIMER | IRQF_IRQPOLL,
> >> +	.handler	= samsung_clock_event_isr,
> >> +	.dev_id		= &time_event_device,
> >> +};
> >> +
> >> +static void __init samsung_clockevent_init(void)
> >> +{
> >> +	unsigned long pclk;
> >> +	unsigned long clock_rate;
> >> +	unsigned int irq_number;
> >> +	struct clk *tscaler;
> >> +
> >> +	pclk = clk_get_rate(timerclk);
> >> +
> >> +	tscaler = clk_get_parent(tdiv_event);
> >> +
> >> +	clk_set_rate(tscaler, pclk / 2);
> >> +	clk_set_rate(tdiv_event, pclk / 2);
> >> +	clk_set_parent(tin_event, tdiv_event);
> >> +
> >> +	clock_rate = clk_get_rate(tin_event);
> >> +	clock_count_per_tick = clock_rate / HZ;
> >> +
> >> +	clockevents_calc_mult_shift(&time_event_device,
> >> +				    clock_rate, SAMSUNG_TIMER_MIN_RANGE);
> >> +	time_event_device.max_delta_ns =
> >> +		clockevent_delta2ns(-1, &time_event_device);
> >> +	time_event_device.min_delta_ns =
> >> +		clockevent_delta2ns(1, &time_event_device);
> >> +
> >> +	time_event_device.cpumask = cpumask_of(0);
> >> +	clockevents_register_device(&time_event_device);
> >> +
> >> +	irq_number = timer_source.event_id + IRQ_TIMER0;
> >> +	setup_irq(irq_number, &samsung_clock_event_irq);
> >> +}
> >> +
> >> +static void __iomem *samsung_timer_reg(void)
> >> +{
> >> +	unsigned long offset = 0;
> >> +
> >> +	switch (timer_source.source_id) {
> >> +	case SAMSUNG_PWM0:
> >> +	case SAMSUNG_PWM1:
> >> +	case SAMSUNG_PWM2:
> >> +	case SAMSUNG_PWM3:
> >> +		offset = (timer_source.source_id * 0x0c) + 0x14;
> >> +		break;
> >> +
> >> +	case SAMSUNG_PWM4:
> >> +		offset = 0x40;
> >> +		break;
> >> +
> >> +	default:
> >> +		printk(KERN_ERR "Invalid Timer %d\n", timer_source.source_id);
> >> +		return NULL;
> >> +	}
> >> +
> >> +	return S3C_TIMERREG(offset);
> >> +}
> >> +
> >> +/*
> >> + * Override the global weak sched_clock symbol with this
> >> + * local implementation which uses the clocksource to get some
> >> + * better resolution when scheduling the kernel. We accept that
> >> + * this wraps around for now, since it is just a relative time
> >> + * stamp. (Inspired by U300 implementation.)
> >> + */
> >> +static u32 notrace samsung_read_sched_clock(void)
> >> +{
> >> +	void __iomem *reg = samsung_timer_reg();
> >> +
> >> +	if (!reg)
> >> +		return 0;
> >> +
> >> +	return ~__raw_readl(reg);
> >> +}
> >> +
> >> +static void __init samsung_clocksource_init(void)
> >> +{
> >> +	unsigned long pclk;
> >> +	unsigned long clock_rate;
> >> +
> >> +	pclk = clk_get_rate(timerclk);
> >> +
> >> +	clk_set_rate(tdiv_source, pclk / 2);
> >> +	clk_set_parent(tin_source, tdiv_source);
> >> +
> >> +	clock_rate = clk_get_rate(tin_source);
> >> +
> >> +	samsung_time_setup(timer_source.source_id, TCNT_MAX);
> >> +	samsung_time_start(timer_source.source_id, PERIODIC);
> >> +
> >> +	setup_sched_clock(samsung_read_sched_clock, 32, clock_rate);
> >> +
> >> +	if (clocksource_mmio_init(samsung_timer_reg(),
> >> "samsung_clocksource_timer", +			clock_rate, 250, 32,
> >> clocksource_mmio_readl_down))
> >> +		panic("samsung_clocksource_timer: can't register clocksource\n");
> >> +}
> >> +
> >> +static void __init samsung_timer_resources(void)
> >> +{
> >> +
> >> +	unsigned long event_id = timer_source.event_id;
> >> +	unsigned long source_id = timer_source.source_id;
> >> +	char devname[15];
> >> +
> >> +	timerclk = clk_get(NULL, "timers");
> >> +	if (IS_ERR(timerclk))
> >> +		panic("failed to get timers clock for timer");
> >> +
> >> +	clk_enable(timerclk);
> >> +
> >> +	sprintf(devname, "s3c24xx-pwm.%lu", event_id);
> >> +	s3c_device_timer[event_id].id = event_id;
> >> +	s3c_device_timer[event_id].dev.init_name = devname;
> >> +
> >> +	tin_event = clk_get(&s3c_device_timer[event_id].dev, "pwm-tin");
> >> +	if (IS_ERR(tin_event))
> >> +		panic("failed to get pwm-tin clock for event timer");
> >> +
> >> +	tdiv_event = clk_get(&s3c_device_timer[event_id].dev, "pwm-tdiv");
> >> +	if (IS_ERR(tdiv_event))
> >> +		panic("failed to get pwm-tdiv clock for event timer");
> >> +
> >> +	clk_enable(tin_event);
> >> +
> >> +	sprintf(devname, "s3c24xx-pwm.%lu", source_id);
> >> +	s3c_device_timer[source_id].id = source_id;
> >> +	s3c_device_timer[source_id].dev.init_name = devname;
> >> +
> >> +	tin_source = clk_get(&s3c_device_timer[source_id].dev, "pwm-tin");
> >> +	if (IS_ERR(tin_source))
> >> +		panic("failed to get pwm-tin clock for source timer");
> >> +
> >> +	tdiv_source = clk_get(&s3c_device_timer[source_id].dev, "pwm-tdiv");
> >> +	if (IS_ERR(tdiv_source))
> >> +		panic("failed to get pwm-tdiv clock for source timer");
> >> +
> >> +	clk_enable(tin_source);
> >> +}
> >> +
> >> +static void __init samsung_timer_init(void)
> >> +{
> >> +	samsung_timer_resources();
> >> +	samsung_clockevent_init();
> >> +	samsung_clocksource_init();
> >> +}
> >> +
> >> +struct sys_timer samsung_timer = {
> >> +	.init		= samsung_timer_init,
> >> +};

--
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


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux