omap init_early changes for irq and timer init
Subsystems (5)
Tony Lindgren
maintainercover omap init_early changes for irq and timer init
Hi all,
This series continues the work to only initialize minimal omap code in init_early and to cut down dependencies to code that should be initialized later. It also cleans up the omap2+ timer init code to prepare things for the later patches.
After this series further patches can make the dmtimer code a regular device driver. Later on we can also move the clockource init happen earlier to make sched_clock work properly with CONFIG_PRINTK_TIME.
Regards,
Tony
---
Tony Lindgren (10):
omap: Use separate init_irq functions to avoid cpu_is_omap tests early
omap: Set separate timer init functions to avoid cpu_is_omap tests
omap: Move dmtimer defines to dmtimer.h
omap: Make a subset of dmtimer functions into inline functions omap2+: Use dmtimer macros for clockevent
omap2+: Remove gptimer_wakekup for now
omap2+: Reserve clocksource and timesource and initialize dmtimer later omap2+: Use dmtimer macros for clocksource
omap2+: Remove omap2_gp_clockevent_set_gptimer
omap2+: Rename timer-gp.c into timer.c to combine timer init functions
arch/arm/mach-omap1/board-ams-delta.c | 4 arch/arm/mach-omap1/board-fsample.c | 4 arch/arm/mach-omap1/board-generic.c | 4 arch/arm/mach-omap1/board-h2.c | 4 arch/arm/mach-omap1/board-h3.c | 4 arch/arm/mach-omap1/board-htcherald.c | 4 arch/arm/mach-omap1/board-innovator.c | 4 arch/arm/mach-omap1/board-nokia770.c | 4 arch/arm/mach-omap1/board-osk.c | 4 arch/arm/mach-omap1/board-palmte.c | 4 arch/arm/mach-omap1/board-palmtt.c | 4 arch/arm/mach-omap1/board-palmz71.c | 4 arch/arm/mach-omap1/board-perseus2.c | 4 arch/arm/mach-omap1/board-sx1.c | 4 arch/arm/mach-omap1/board-voiceblue.c | 4 arch/arm/mach-omap1/irq.c | 2 arch/arm/mach-omap1/time.c | 6 arch/arm/mach-omap1/timer32k.c | 4 arch/arm/mach-omap2/Makefile | 2 arch/arm/mach-omap2/board-2430sdp.c | 4 arch/arm/mach-omap2/board-3430sdp.c | 4 arch/arm/mach-omap2/board-3630sdp.c | 4 arch/arm/mach-omap2/board-4430sdp.c | 6 arch/arm/mach-omap2/board-am3517crane.c | 4 arch/arm/mach-omap2/board-am3517evm.c | 4 arch/arm/mach-omap2/board-apollon.c | 4 arch/arm/mach-omap2/board-cm-t35.c | 4 arch/arm/mach-omap2/board-cm-t3517.c | 4 arch/arm/mach-omap2/board-devkit8000.c | 8 - arch/arm/mach-omap2/board-generic.c | 4 arch/arm/mach-omap2/board-h4.c | 4 arch/arm/mach-omap2/board-igep0020.c | 4 arch/arm/mach-omap2/board-igep0030.c | 4 arch/arm/mach-omap2/board-ldp.c | 4 arch/arm/mach-omap2/board-n8x0.c | 12 - arch/arm/mach-omap2/board-omap3beagle.c | 8 - arch/arm/mach-omap2/board-omap3evm.c | 4 arch/arm/mach-omap2/board-omap3logic.c | 9 - arch/arm/mach-omap2/board-omap3pandora.c | 4 arch/arm/mach-omap2/board-omap3stalker.c | 8 - arch/arm/mach-omap2/board-omap3touchbook.c | 8 - arch/arm/mach-omap2/board-omap4panda.c | 3 arch/arm/mach-omap2/board-overo.c | 4 arch/arm/mach-omap2/board-rm680.c | 4 arch/arm/mach-omap2/board-rx51.c | 4 arch/arm/mach-omap2/board-ti8168evm.c | 9 - arch/arm/mach-omap2/board-zoom.c | 8 - arch/arm/mach-omap2/io.c | 17 - arch/arm/mach-omap2/irq.c | 84 +++++-- arch/arm/mach-omap2/omap4-common.c | 6 arch/arm/mach-omap2/pm-debug.c | 27 -- arch/arm/mach-omap2/pm.h | 6 arch/arm/mach-omap2/pm34xx.c | 4 arch/arm/mach-omap2/timer-gp.c | 266 --------------------- arch/arm/mach-omap2/timer-gp.h | 16 - arch/arm/mach-omap2/timer.c | 361 ++++++++++++++++++++++++++++ arch/arm/plat-omap/counter_32k.c | 2 arch/arm/plat-omap/dmtimer.c | 219 ++++------------- arch/arm/plat-omap/include/plat/common.h | 7 - arch/arm/plat-omap/include/plat/dmtimer.h | 181 ++++++++++++++ arch/arm/plat-omap/include/plat/irqs.h | 6 61 files changed, 773 insertions(+), 654 deletions(-) delete mode 100644 arch/arm/mach-omap2/timer-gp.c delete mode 100644 arch/arm/mach-omap2/timer-gp.h create mode 100644 arch/arm/mach-omap2/timer.c
Signature · 1 line
Signature
Tony Lindgren
maintainerPATCH 1/10 omap: Use separate init_irq functions to avoid cpu_is_omap tests early
This allows us to remove cpu_is_omap calls from init_irq functions and the irq handler. There should not be any need for cpu_is_omap as at this point we only care about SoC generation, and not about subcategories.
The main reason for the patch is that we want to initialize only minimal omap specific code from the init_early call, and this and the following patches get us closer to that goal.
--- arch/arm/mach-omap1/board-ams-delta.c | 2 - arch/arm/mach-omap1/board-fsample.c | 2 - arch/arm/mach-omap1/board-generic.c | 2 - arch/arm/mach-omap1/board-h2.c | 2 - arch/arm/mach-omap1/board-h3.c | 2 - arch/arm/mach-omap1/board-htcherald.c | 2 - arch/arm/mach-omap1/board-innovator.c | 2 - arch/arm/mach-omap1/board-nokia770.c | 2 - arch/arm/mach-omap1/board-osk.c | 2 - arch/arm/mach-omap1/board-palmte.c | 2 - arch/arm/mach-omap1/board-palmtt.c | 2 - arch/arm/mach-omap1/board-palmz71.c | 2 - arch/arm/mach-omap1/board-perseus2.c | 2 - arch/arm/mach-omap1/board-sx1.c | 2 - arch/arm/mach-omap1/board-voiceblue.c | 2 - arch/arm/mach-omap1/irq.c | 2 - arch/arm/mach-omap2/board-2430sdp.c | 2 - arch/arm/mach-omap2/board-3430sdp.c | 2 - arch/arm/mach-omap2/board-3630sdp.c | 2 - arch/arm/mach-omap2/board-am3517crane.c | 2 - arch/arm/mach-omap2/board-am3517evm.c | 2 - arch/arm/mach-omap2/board-apollon.c | 2 - arch/arm/mach-omap2/board-cm-t35.c | 2 - arch/arm/mach-omap2/board-cm-t3517.c | 2 - arch/arm/mach-omap2/board-devkit8000.c | 2 - arch/arm/mach-omap2/board-generic.c | 2 - arch/arm/mach-omap2/board-h4.c | 2 - arch/arm/mach-omap2/board-igep0020.c | 2 - arch/arm/mach-omap2/board-igep0030.c | 2 - arch/arm/mach-omap2/board-ldp.c | 2 - arch/arm/mach-omap2/board-n8x0.c | 6 +- arch/arm/mach-omap2/board-omap3beagle.c | 2 - arch/arm/mach-omap2/board-omap3evm.c | 2 - arch/arm/mach-omap2/board-omap3logic.c | 4 + arch/arm/mach-omap2/board-omap3pandora.c | 2 - arch/arm/mach-omap2/board-omap3stalker.c | 2 - arch/arm/mach-omap2/board-omap3touchbook.c | 2 - arch/arm/mach-omap2/board-overo.c | 2 - arch/arm/mach-omap2/board-rm680.c | 2 - arch/arm/mach-omap2/board-rx51.c | 2 - arch/arm/mach-omap2/board-ti8168evm.c | 7 -- arch/arm/mach-omap2/board-zoom.c | 4 + arch/arm/mach-omap2/io.c | 17 ------ arch/arm/mach-omap2/irq.c | 84 ++++++++++++++++++++-------- arch/arm/mach-omap2/omap4-common.c | 6 ++ arch/arm/plat-omap/include/plat/irqs.h | 6 ++ 46 files changed, 117 insertions(+), 93 deletions(-)
arch/arm/mach-omap1/board-ams-delta.c +1 -1
@@ -138,7 +138,7 @@ void ams_delta_latch2_write(u16 mask, u16 value)static void __init ams_delta_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct map_desc ams_delta_io_desc[]__initdata = {
arch/arm/mach-omap1/board-fsample.c +1 -1
@@ -329,7 +329,7 @@ static void __init omap_fsample_init(void)static void __init omap_fsample_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */
arch/arm/mach-omap1/board-generic.c +1 -1
@@ -31,7 +31,7 @@static void __init omap_generic_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* assume no Mini-AB port */
arch/arm/mach-omap1/board-h2.c +1 -1
@@ -376,7 +376,7 @@ static struct i2c_board_info __initdata h2_i2c_board_info[] = {static void __init h2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config h2_usb_config__initdata = {
arch/arm/mach-omap1/board-h3.c +1 -1
@@ -439,7 +439,7 @@ static void __init h3_init(void)static void __init h3_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init h3_map_io (void )
arch/arm/mach-omap1/board-htcherald.c +1 -1
@@ -605,7 +605,7 @@ static void __init htcherald_init_irq(void) {printk (KERN_INFO "htcherald_init_irq.\n" );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }MACHINE_START (HERALD ,"HTC Herald" )
arch/arm/mach-omap1/board-innovator.c +1 -1
@@ -292,7 +292,7 @@ static void __init innovator_init_smc91x(void)static void __init innovator_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }#ifdef CONFIG_ARCH_OMAP15XX
arch/arm/mach-omap1/board-nokia770.c +1 -1
@@ -51,7 +51,7 @@ static void __init omap_nokia770_init_irq(void)omap_writew ((omap_readw (0xfffb5004 ) & ~2 ),0xfffb5004 );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int nokia770_keymap[] = {
arch/arm/mach-omap1/board-osk.c +1 -1
@@ -282,7 +282,7 @@ static void __init osk_init_cf(void)static void __init osk_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config osk_usb_config__initdata = {
arch/arm/mach-omap1/board-palmte.c +1 -1
@@ -62,7 +62,7 @@static void __init omap_palmte_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmte_keymap[] = {
arch/arm/mach-omap1/board-palmtt.c +1 -1
@@ -266,7 +266,7 @@ static struct spi_board_info __initdata palmtt_boardinfo[] = {static void __init omap_palmtt_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config palmtt_usb_config__initdata = {
arch/arm/mach-omap1/board-palmz71.c +1 -1
@@ -61,7 +61,7 @@ static void __initomap_palmz71_init_irq (void) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmz71_keymap[] = {
arch/arm/mach-omap1/board-perseus2.c +1 -1
@@ -297,7 +297,7 @@ static void __init omap_perseus2_init(void)static void __init omap_perseus2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */ static struct map_desc omap_perseus2_io_desc[]__initdata = {
arch/arm/mach-omap1/board-sx1.c +1 -1
@@ -411,7 +411,7 @@ static void __init omap_sx1_init(void)static void __init omap_sx1_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/*----------------------------------------*/
arch/arm/mach-omap1/board-voiceblue.c +1 -1
@@ -162,7 +162,7 @@ static struct omap_board_config_kernel voiceblue_config[] = {static void __init voiceblue_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init voiceblue_map_io (void )
arch/arm/mach-omap1/irq.c +1 -1
@@ -175,7 +175,7 @@ static struct irq_chip omap_irq_chip = { .irq_set_wake = omap_wake_irq, }; -void __init omap_init_irq (void ) +void __init omap1_init_irq (void ) {int i, j;
arch/arm/mach-omap2/board-2430sdp.c +1 -1
@@ -277,7 +277,7 @@ MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board") .reserve = omap_reserve, .map_io = omap_2430sdp_map_io, .init_early = omap_2430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_2430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3430sdp.c +1 -1
@@ -909,7 +909,7 @@ MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_3430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_3430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3630sdp.c +1 -1
@@ -219,7 +219,7 @@ MACHINE_START(OMAP_3630SDP, "OMAP 3630SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517crane.c +1 -1
@@ -110,7 +110,7 @@ MACHINE_START(CRANEBOARD, "AM3517/05 CRANEBOARD") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_crane_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_crane_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517evm.c +1 -1
@@ -516,7 +516,7 @@ MACHINE_START(OMAP3517EVM, "OMAP3517/AM3517 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-apollon.c +1 -1
@@ -357,7 +357,7 @@ MACHINE_START(OMAP_APOLLON, "OMAP24xx Apollon") .reserve = omap_reserve, .map_io = omap_apollon_map_io, .init_early = omap_apollon_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_apollon_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t35.c +1 -1
@@ -806,7 +806,7 @@ MACHINE_START(CM_T35, "Compulab CM-T35") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t35_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t35_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t3517.c +1 -1
@@ -306,7 +306,7 @@ MACHINE_START(CM_T3517, "Compulab CM-T3517") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t3517_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t3517_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-devkit8000.c +1 -1
@@ -457,7 +457,7 @@ static void __init devkit8000_init_early(void)static void __init devkit8000_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-generic.c +1 -1
@@ -70,7 +70,7 @@ MACHINE_START(OMAP_GENERIC, "Generic OMAP24xx") .reserve = omap_reserve, .map_io = omap_generic_map_io, .init_early = omap_generic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_generic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-h4.c +1 -1
@@ -298,7 +298,7 @@ static void __init omap_h4_init_early(void)static void __init omap_h4_init_irq (void ) { -omap_init_irq (); +omap2_init_irq (); }static struct at24_platform_data m24c01 = {
arch/arm/mach-omap2/board-igep0020.c +1 -1
@@ -708,7 +708,7 @@ MACHINE_START(IGEP0020, "IGEP v2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep2_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep2_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-igep0030.c +1 -1
@@ -452,7 +452,7 @@ MACHINE_START(IGEP0030, "IGEP OMAP3 module") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep3_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep3_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ldp.c +1 -1
@@ -466,7 +466,7 @@ MACHINE_START(OMAP_LDP, "OMAP LDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_ldp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_ldp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-n8x0.c +3 -3
@@ -705,7 +705,7 @@ MACHINE_START(NOKIA_N800, "Nokia N800") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -715,7 +715,7 @@ MACHINE_START(NOKIA_N810, "Nokia N810") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -725,7 +725,7 @@ MACHINE_START(NOKIA_N810_WIMAX, "Nokia N810 WiMAX") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3beagle.c +1 -1
@@ -540,7 +540,7 @@ static void __init omap3_beagle_init_early(void)static void __init omap3_beagle_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3evm.c +1 -1
@@ -894,7 +894,7 @@ MACHINE_START(OMAP3EVM, "OMAP3 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3logic.c +2 -2
@@ -225,7 +225,7 @@ MACHINE_START(OMAP3_TORPEDO, "Logic OMAP3 Torpedo board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
@@ -234,7 +234,7 @@ MACHINE_START(OMAP3530_LV_SOM, "OMAP Logic 3530 LV SOM board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3pandora.c +1 -1
@@ -720,7 +720,7 @@ MACHINE_START(OMAP3_PANDORA, "Pandora Handheld Console") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3pandora_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3pandora_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3stalker.c +1 -1
@@ -587,7 +587,7 @@ static void __init omap3_stalker_init_early(void)static void __init omap3_stalker_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3touchbook.c +1 -1
@@ -422,7 +422,7 @@ static void __init omap3_touchbook_init_early(void)static void __init omap3_touchbook_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-overo.c +1 -1
@@ -790,7 +790,7 @@ MACHINE_START(OVERO, "Gumstix Overo") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = overo_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = overo_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rm680.c +1 -1
@@ -178,7 +178,7 @@ MACHINE_START(NOKIA_RM680, "Nokia RM-680 board") .reserve = omap_reserve, .map_io = rm680_map_io, .init_early = rm680_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rm680_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rx51.c +1 -1
@@ -151,7 +151,7 @@ MACHINE_START(NOKIA_RX51, "Nokia RX-51 board") .reserve = omap_reserve, .map_io = rx51_map_io, .init_early = rx51_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rx51_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ti8168evm.c +1 -6
@@ -33,11 +33,6 @@ static void __init ti8168_init_early(void)omap2_init_common_devices (NULL ,NULL ); } -static void __init ti8168_evm_init_irq (void ) -{ -omap_init_irq (); -} -static void __init ti8168_evm_init (void ) {omap_serial_init ();
@@ -56,7 +51,7 @@ MACHINE_START(TI8168EVM, "ti8168evm") .boot_params = 0x80000100, .map_io = ti8168_evm_map_io, .init_early = ti8168_init_early, - .init_irq = ti8168_evm_init_irq, + .init_irq = ti816x_init_irq, .timer = &omap_timer, .init_machine = ti8168_evm_init,MACHINE_END
arch/arm/mach-omap2/board-zoom.c +2 -2
@@ -137,7 +137,7 @@ MACHINE_START(OMAP_ZOOM2, "OMAP Zoom2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
@@ -147,7 +147,7 @@ MACHINE_START(OMAP_ZOOM3, "OMAP Zoom3 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/io.c +1 -16
@@ -333,23 +333,9 @@ static int _set_hwmod_postsetup_state(struct omap_hwmod *oh, void *data)return omap_hwmod_set_postsetup_state (oh, *(u8 *)data); } +/* See irq.c, omap4-common.c and entry-macro.S */ void __iomem *omap_irq_base; -/* -* Initialize asm_irq_base for entry-macro.S -*/ -static inline void omap_irq_base_init (void ) -{ -if (cpu_is_omap24xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP24XX_IC_BASE ); -else if (cpu_is_omap34xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP34XX_IC_BASE ); -else if (cpu_is_omap44xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP44XX_GIC_CPU_BASE ); -else -pr_err ("Could not initialize omap_irq_base\n" ); -} -void __init omap2_init_common_infrastructure (void ) {u8 postsetup_state;
@@ -422,7 +408,6 @@ void __init omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,_omap2_init_reprogram_sdrc (); } -omap_irq_base_init (); }/*
arch/arm/mach-omap2/irq.c +59 -25
124 lines not on this page · raw message
arch/arm/mach-omap2/omap4-common.c +6 -0
@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
@@ -34,6 +36,10 @@ void __init gic_init_irq(void)void __iomem *gic_cpu_base;/* Static mapping, never released */ + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_4K ); +BUG_ON (!omap_irq_base); + +/* Static mapping, never released */ gic_dist_base_addr =ioremap (OMAP44XX_GIC_DIST_BASE ,SZ_4K );BUG_ON (!gic_dist_base_addr);
arch/arm/plat-omap/include/plat/irqs.h +5 -1
@@ -428,7 +428,11 @@#define INTCPS_NR_IRQS 96#ifndef __ASSEMBLY__ -extern void omap_init_irq (void ); +extern void __iomem *omap_irq_base; +void omap1_init_irq (void ); +void omap2_init_irq (void ); +void omap3_init_irq (void ); +void ti816x_init_irq (void );extern int omap_irq_pending (void );void omap_intc_save_context (void );void omap_intc_restore_context (void );
PATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
Quoted 20 lines
-----Original Message-----
From: linux-omap-owner at vger.kernel.org [mailto:linux-omap- owner at vger.kernel.org] On Behalf Of Tony Lindgren
Sent: Tuesday, March 29, 2011 3:51 AM
To: linux-arm-kernel at lists.infradead.org
Cc: linux-omap at vger.kernel.org
Subject: [PATCH 01/10] omap: Use separateinit_irqfunctions toavoidcpu_is_omaptests earlyThis allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>---
[....]
Quoted 15 lines
arch/arm/mach- renamed from arch/arm/mach-omap2/omap4-common.c +0
-0
omap2/omap4-common.c
index 9ef8c29..1c3bcf1 100644arch/arm/mach-omap2/omap4-common.c +2
-0@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 41
Quoted 6 lines
arch/arm/mach-omap2/omap4-common.c +3
-0@@ -34,6 +36,10 @@ void __init gic_init_irq(void)void __iomem *gic_cpu_base;/* Static mapping, never released */ + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_4K ); +BUG_ON (!omap_irq_base); +
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
(quoted diff) +0 -0
b/arch/arm/mach-omap2/omap4-common.c
index 9ef8c29..9bddc9e 100644
arch/arm/mach-omap2/omap4-common.c +2 -0
@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base); + omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }
Regards,
Santosh
PATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
In reply to Tony LindgrenQuoted 20 lines
-----Original Message-----
From: linux-omap-owner at vger.kernel.org [mailto:linux-omap- owner at vger.kernel.org] On Behalf Of Tony Lindgren
Sent: Tuesday, March 29, 2011 3:51 AM
To: linux-arm-kernel at lists.infradead.org
Cc: linux-omap at vger.kernel.org
Subject: [PATCH 01/10] omap: Use separateinit_irqfunctions toavoidcpu_is_omaptests earlyThis allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>---
[....]
Quoted 15 lines
arch/arm/mach- renamed from arch/arm/mach-omap2/omap4-common.c +0
-0
omap2/omap4-common.c
index 9ef8c29..1c3bcf1 100644arch/arm/mach-omap2/omap4-common.c +2
-0@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 41
Quoted 6 lines
arch/arm/mach-omap2/omap4-common.c +3
-0@@ -34,6 +36,10 @@ void __init gic_init_irq(void)void __iomem *gic_cpu_base;/* Static mapping, never released */ + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_4K ); +BUG_ON (!omap_irq_base); +
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
(quoted diff) +0 -0
b/arch/arm/mach-omap2/omap4-common.c
index 9ef8c29..9bddc9e 100644
arch/arm/mach-omap2/omap4-common.c +2 -0
@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base); + omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }
Regards,
Santosh
PATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
In reply to Tony LindgrenQuoted 20 lines
-----Original Message-----
From: linux-omap-owner at vger.kernel.org [mailto:linux-omap- owner at vger.kernel.org] On Behalf Of Tony Lindgren
Sent: Tuesday, March 29, 2011 3:51 AM
To: linux-arm-kernel at lists.infradead.org
Cc: linux-omap at vger.kernel.org
Subject: [PATCH 01/10] omap: Use separateinit_irqfunctions toavoidcpu_is_omaptests earlyThis allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>---
[....]
Quoted 15 lines
arch/arm/mach- renamed from arch/arm/mach-omap2/omap4-common.c +0
-0
omap2/omap4-common.c
index 9ef8c29..1c3bcf1 100644arch/arm/mach-omap2/omap4-common.c +2
-0@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 41
Quoted 6 lines
arch/arm/mach-omap2/omap4-common.c +3
-0@@ -34,6 +36,10 @@ void __init gic_init_irq(void)void __iomem *gic_cpu_base;/* Static mapping, never released */ + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_4K ); +BUG_ON (!omap_irq_base); +
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
(quoted diff) +0 -0
b/arch/arm/mach-omap2/omap4-common.c
index 9ef8c29..9bddc9e 100644
arch/arm/mach-omap2/omap4-common.c +2 -0
@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base); + omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }
Regards,
Santosh
PATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
In reply to Tony LindgrenQuoted 20 lines
-----Original Message-----
From: linux-omap-owner at vger.kernel.org [mailto:linux-omap- owner at vger.kernel.org] On Behalf Of Tony Lindgren
Sent: Tuesday, March 29, 2011 3:51 AM
To: linux-arm-kernel at lists.infradead.org
Cc: linux-omap at vger.kernel.org
Subject: [PATCH 01/10] omap: Use separateinit_irqfunctions toavoidcpu_is_omaptests earlyThis allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>---
[....]
Quoted 15 lines
arch/arm/mach- renamed from arch/arm/mach-omap2/omap4-common.c +0
-0
omap2/omap4-common.c
index 9ef8c29..1c3bcf1 100644arch/arm/mach-omap2/omap4-common.c +2
-0@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 41
Quoted 6 lines
arch/arm/mach-omap2/omap4-common.c +3
-0@@ -34,6 +36,10 @@ void __init gic_init_irq(void)void __iomem *gic_cpu_base;/* Static mapping, never released */ + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_4K ); +BUG_ON (!omap_irq_base); +
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
(quoted diff) +0 -0
b/arch/arm/mach-omap2/omap4-common.c
index 9ef8c29..9bddc9e 100644
arch/arm/mach-omap2/omap4-common.c +2 -0
@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base); + omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }
Regards,
Santosh
Tony Lindgren
maintainerPATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
Quoted 11 lines · Santosh Shilimkar
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
arch/arm/mach-omap2/omap4-common.c +0
-0
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 47
Quoted 7 lines
arch/arm/mach-omap2/omap4-common.c +2
-0@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base); + omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }
Thanks for cathing this.
Tony
Tony Lindgren
maintainerPATCH 1/10 omap: Use separate init_irq functions to avoidcpu_is_omap tests early
Quoted 15 lines · Tony Lindgren
Quoted 14 lines · Santosh Shilimkar
Above mapping is incorrect. The address space is of just 512 bytes. Infact you don't need to ioremap it again. Below should be enough.
arch/arm/mach-omap2/omap4-common.c +0
-0@@ -41,6 +41,8 @@ void __init gic_init_irq(void) gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 );BUG_ON (!gic_cpu_base);
Quoted 6 lines
arch/arm/mach-omap2/omap4-common.c void __init gic_init_irq(void) · line 47
Quoted 4 lines
arch/arm/mach-omap2/omap4-common.c +2
-0+ omap_irq_base = gic_cpu_base; +gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); }Thanks for cathing this.
Actually we can get rid of the local gic_cpu_base as we already have omap_irq_base that we need anyways. Updated patch below.
Regards,
Tony
From: Tony Lindgren <[email protected]>
Date: Tue, 29 Mar 2011 15:17:12 -0700
Subject: [PATCH] omap: Use separate init_irq functions to avoid cpu_is_omap tests early
This allows us to remove cpu_is_omap calls from init_irq functions and the irq handler. There should not be any need for cpu_is_omap as at this point we only care about SoC generation, and not about subcategories.
The main reason for the patch is that we want to initialize only minimal omap specific code from the init_early call, and this and the following patches get us closer to that goal.
arch/arm/mach-omap1/board-ams-delta.c +1 -1
@@ -138,7 +138,7 @@ void ams_delta_latch2_write(u16 mask, u16 value)static void __init ams_delta_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct map_desc ams_delta_io_desc[]__initdata = {
arch/arm/mach-omap1/board-fsample.c +1 -1
@@ -329,7 +329,7 @@ static void __init omap_fsample_init(void)static void __init omap_fsample_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */
arch/arm/mach-omap1/board-generic.c +1 -1
@@ -31,7 +31,7 @@static void __init omap_generic_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* assume no Mini-AB port */
arch/arm/mach-omap1/board-h2.c +1 -1
@@ -376,7 +376,7 @@ static struct i2c_board_info __initdata h2_i2c_board_info[] = {static void __init h2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config h2_usb_config__initdata = {
arch/arm/mach-omap1/board-h3.c +1 -1
@@ -439,7 +439,7 @@ static void __init h3_init(void)static void __init h3_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init h3_map_io (void )
arch/arm/mach-omap1/board-htcherald.c +1 -1
@@ -605,7 +605,7 @@ static void __init htcherald_init_irq(void) {printk (KERN_INFO "htcherald_init_irq.\n" );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }MACHINE_START (HERALD ,"HTC Herald" )
arch/arm/mach-omap1/board-innovator.c +1 -1
@@ -292,7 +292,7 @@ static void __init innovator_init_smc91x(void)static void __init innovator_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }#ifdef CONFIG_ARCH_OMAP15XX
arch/arm/mach-omap1/board-nokia770.c +1 -1
@@ -51,7 +51,7 @@ static void __init omap_nokia770_init_irq(void)omap_writew ((omap_readw (0xfffb5004 ) & ~2 ),0xfffb5004 );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int nokia770_keymap[] = {
arch/arm/mach-omap1/board-osk.c +1 -1
@@ -282,7 +282,7 @@ static void __init osk_init_cf(void)static void __init osk_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config osk_usb_config__initdata = {
arch/arm/mach-omap1/board-palmte.c +1 -1
@@ -62,7 +62,7 @@static void __init omap_palmte_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmte_keymap[] = {
arch/arm/mach-omap1/board-palmtt.c +1 -1
@@ -266,7 +266,7 @@ static struct spi_board_info __initdata palmtt_boardinfo[] = {static void __init omap_palmtt_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config palmtt_usb_config__initdata = {
arch/arm/mach-omap1/board-palmz71.c +1 -1
@@ -61,7 +61,7 @@ static void __initomap_palmz71_init_irq (void) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmz71_keymap[] = {
arch/arm/mach-omap1/board-perseus2.c +1 -1
@@ -297,7 +297,7 @@ static void __init omap_perseus2_init(void)static void __init omap_perseus2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */ static struct map_desc omap_perseus2_io_desc[]__initdata = {
arch/arm/mach-omap1/board-sx1.c +1 -1
@@ -411,7 +411,7 @@ static void __init omap_sx1_init(void)static void __init omap_sx1_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/*----------------------------------------*/
arch/arm/mach-omap1/board-voiceblue.c +1 -1
@@ -162,7 +162,7 @@ static struct omap_board_config_kernel voiceblue_config[] = {static void __init voiceblue_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init voiceblue_map_io (void )
arch/arm/mach-omap1/irq.c +1 -1
@@ -175,7 +175,7 @@ static struct irq_chip omap_irq_chip = { .irq_set_wake = omap_wake_irq, }; -void __init omap_init_irq (void ) +void __init omap1_init_irq (void ) {int i, j;
arch/arm/mach-omap2/board-2430sdp.c +1 -1
@@ -277,7 +277,7 @@ MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board") .reserve = omap_reserve, .map_io = omap_2430sdp_map_io, .init_early = omap_2430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_2430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3430sdp.c +1 -1
@@ -909,7 +909,7 @@ MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_3430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_3430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3630sdp.c +1 -1
@@ -219,7 +219,7 @@ MACHINE_START(OMAP_3630SDP, "OMAP 3630SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517crane.c +1 -1
@@ -110,7 +110,7 @@ MACHINE_START(CRANEBOARD, "AM3517/05 CRANEBOARD") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_crane_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_crane_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517evm.c +1 -1
@@ -516,7 +516,7 @@ MACHINE_START(OMAP3517EVM, "OMAP3517/AM3517 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-apollon.c +1 -1
@@ -357,7 +357,7 @@ MACHINE_START(OMAP_APOLLON, "OMAP24xx Apollon") .reserve = omap_reserve, .map_io = omap_apollon_map_io, .init_early = omap_apollon_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_apollon_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t35.c +1 -1
@@ -806,7 +806,7 @@ MACHINE_START(CM_T35, "Compulab CM-T35") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t35_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t35_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t3517.c +1 -1
@@ -306,7 +306,7 @@ MACHINE_START(CM_T3517, "Compulab CM-T3517") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t3517_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t3517_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-devkit8000.c +1 -1
@@ -457,7 +457,7 @@ static void __init devkit8000_init_early(void)static void __init devkit8000_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-generic.c +1 -1
@@ -70,7 +70,7 @@ MACHINE_START(OMAP_GENERIC, "Generic OMAP24xx") .reserve = omap_reserve, .map_io = omap_generic_map_io, .init_early = omap_generic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_generic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-h4.c +1 -1
@@ -298,7 +298,7 @@ static void __init omap_h4_init_early(void)static void __init omap_h4_init_irq (void ) { -omap_init_irq (); +omap2_init_irq (); }static struct at24_platform_data m24c01 = {
arch/arm/mach-omap2/board-igep0020.c +1 -1
@@ -708,7 +708,7 @@ MACHINE_START(IGEP0020, "IGEP v2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep2_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep2_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-igep0030.c +1 -1
@@ -452,7 +452,7 @@ MACHINE_START(IGEP0030, "IGEP OMAP3 module") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep3_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep3_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ldp.c +1 -1
@@ -466,7 +466,7 @@ MACHINE_START(OMAP_LDP, "OMAP LDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_ldp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_ldp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-n8x0.c +3 -3
@@ -705,7 +705,7 @@ MACHINE_START(NOKIA_N800, "Nokia N800") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -715,7 +715,7 @@ MACHINE_START(NOKIA_N810, "Nokia N810") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -725,7 +725,7 @@ MACHINE_START(NOKIA_N810_WIMAX, "Nokia N810 WiMAX") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3beagle.c +1 -1
@@ -540,7 +540,7 @@ static void __init omap3_beagle_init_early(void)static void __init omap3_beagle_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3evm.c +1 -1
@@ -894,7 +894,7 @@ MACHINE_START(OMAP3EVM, "OMAP3 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3logic.c +2 -2
@@ -225,7 +225,7 @@ MACHINE_START(OMAP3_TORPEDO, "Logic OMAP3 Torpedo board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
@@ -234,7 +234,7 @@ MACHINE_START(OMAP3530_LV_SOM, "OMAP Logic 3530 LV SOM board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3pandora.c +1 -1
@@ -720,7 +720,7 @@ MACHINE_START(OMAP3_PANDORA, "Pandora Handheld Console") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3pandora_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3pandora_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3stalker.c +1 -1
@@ -587,7 +587,7 @@ static void __init omap3_stalker_init_early(void)static void __init omap3_stalker_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3touchbook.c +1 -1
@@ -422,7 +422,7 @@ static void __init omap3_touchbook_init_early(void)static void __init omap3_touchbook_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-overo.c +1 -1
@@ -790,7 +790,7 @@ MACHINE_START(OVERO, "Gumstix Overo") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = overo_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = overo_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rm680.c +1 -1
@@ -178,7 +178,7 @@ MACHINE_START(NOKIA_RM680, "Nokia RM-680 board") .reserve = omap_reserve, .map_io = rm680_map_io, .init_early = rm680_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rm680_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rx51.c +1 -1
@@ -151,7 +151,7 @@ MACHINE_START(NOKIA_RX51, "Nokia RX-51 board") .reserve = omap_reserve, .map_io = rx51_map_io, .init_early = rx51_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rx51_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ti8168evm.c +1 -6
@@ -33,11 +33,6 @@ static void __init ti8168_init_early(void)omap2_init_common_devices (NULL ,NULL ); } -static void __init ti8168_evm_init_irq (void ) -{ -omap_init_irq (); -} -static void __init ti8168_evm_init (void ) {omap_serial_init ();
@@ -56,7 +51,7 @@ MACHINE_START(TI8168EVM, "ti8168evm") .boot_params = 0x80000100, .map_io = ti8168_evm_map_io, .init_early = ti8168_init_early, - .init_irq = ti8168_evm_init_irq, + .init_irq = ti816x_init_irq, .timer = &omap_timer, .init_machine = ti8168_evm_init,MACHINE_END
arch/arm/mach-omap2/board-zoom.c +2 -2
@@ -137,7 +137,7 @@ MACHINE_START(OMAP_ZOOM2, "OMAP Zoom2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
@@ -147,7 +147,7 @@ MACHINE_START(OMAP_ZOOM3, "OMAP Zoom3 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/io.c +1 -16
@@ -333,23 +333,9 @@ static int _set_hwmod_postsetup_state(struct omap_hwmod *oh, void *data)return omap_hwmod_set_postsetup_state (oh, *(u8 *)data); } +/* See irq.c, omap4-common.c and entry-macro.S */ void __iomem *omap_irq_base; -/* -* Initialize asm_irq_base for entry-macro.S -*/ -static inline void omap_irq_base_init (void ) -{ -if (cpu_is_omap24xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP24XX_IC_BASE ); -else if (cpu_is_omap34xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP34XX_IC_BASE ); -else if (cpu_is_omap44xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP44XX_GIC_CPU_BASE ); -else -pr_err ("Could not initialize omap_irq_base\n" ); -} -void __init omap2_init_common_infrastructure (void ) {u8 postsetup_state;
@@ -422,7 +408,6 @@ void __init omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,_omap2_init_reprogram_sdrc (); } -omap_irq_base_init (); }/*
arch/arm/mach-omap2/irq.c +59 -25
124 lines not on this page · raw message
arch/arm/mach-omap2/omap4-common.c +5 -5
@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
@@ -31,17 +33,15 @@ void __iomem *gic_dist_base_addr;void __init gic_init_irq (void ) { -void __iomem *gic_cpu_base; -/* Static mapping, never released */ gic_dist_base_addr =ioremap (OMAP44XX_GIC_DIST_BASE ,SZ_4K );BUG_ON (!gic_dist_base_addr);/* Static mapping, never released */ - gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 ); -BUG_ON (!gic_cpu_base); + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 ); +BUG_ON (!omap_irq_base); -gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); +gic_init (0 ,29 , gic_dist_base_addr, omap_irq_base); }#ifdef CONFIG_CACHE_L2X0
arch/arm/plat-omap/include/plat/irqs.h +5 -1
@@ -428,7 +428,11 @@#define INTCPS_NR_IRQS 96#ifndef __ASSEMBLY__ -extern void omap_init_irq (void ); +extern void __iomem *omap_irq_base; +void omap1_init_irq (void ); +void omap2_init_irq (void ); +void omap3_init_irq (void ); +void ti816x_init_irq (void );extern int omap_irq_pending (void );void omap_intc_save_context (void );void omap_intc_restore_context (void );
PATCH 1/10 omap: Use separate init_irq functions to avoid cpu_is_omap tests early
In reply to Tony LindgrenHi Tony,
Quoted 10 lines · Tony Lindgren
This allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>
[...]
A couple of minor comments on naming below...
Quoted 22 lines
arch/arm/mach-omap2/irq.c +7
-3@@ -103,28 +103,36 @@ static void omap_ack_irq(struct irq_data *d)intc_bank_write_reg (0x1 , &irq_banks[0 ],INTC_CONTROL ); } +static inline void _omap_mask_irq (int irq) +{ +int offset = irq & (~(IRQ_BITS_PER_REG -1 )); + irq &= (IRQ_BITS_PER_REG -1 ); +intc_bank_write_reg (1 << irq, &irq_banks[0 ],INTC_MIR_SET0 + offset); +} +static void omap_mask_irq (struct irq_data *d) {unsigned int irq = d->irq; -int offset = irq & (~(IRQ_BITS_PER_REG -1 )); -if (cpu_is_omap34xx () && !cpu_is_ti816x ()) { -int spurious =0 ;
arch/arm/mach-omap2/irq.c static void omap_ack_irq(struct irq_data *d) · line 120
Quoted 10 lines
arch/arm/mach-omap2/irq.c +3
-6+_omap_mask_irq (irq); +} -/* -* INT_34XX_GPT12_IRQ is also the spurious irq. Maybe because -* it is the highest irq number? -*/ -if (irq ==INT_34XX_GPT12_IRQ ) - spurious =omap_check_spurious (irq); +static void omap_mask_irq_omap3 (struct irq_data *d)
how about omap3_mask_irq().
In most other code we're using the SoC name as a prefix instead of suffix.
Quoted 24 lines
arch/arm/mach-omap2/irq.c +12
-5+{ +unsigned int irq = d->irq; +int spurious =0 ; -if (!spurious) - previous_irq = irq; - } +/* +* INT_34XX_GPT12_IRQ is also the spurious irq. Maybe because +* it is the highest irq number? +*/ +if (irq ==INT_34XX_GPT12_IRQ ) + spurious =omap_check_spurious (irq); - irq &= (IRQ_BITS_PER_REG -1 ); +if (!spurious) + previous_irq = irq; -intc_bank_write_reg (1 << irq, &irq_banks[0 ],INTC_MIR_SET0 + offset); +_omap_mask_irq (irq); }static void omap_unmask_irq (struct irq_data *d)
arch/arm/mach-omap2/irq.c static void omap_mask_ack_irq(struct irq_data *d) · line 154
Quoted 4 lines
arch/arm/mach-omap2/irq.c +1
-0@@ -143,6 +151,12 @@ static void omap_mask_ack_irq(struct irq_data *d)omap_ack_irq (d); } +static void omap_mask_ack_irq_omap3 (struct irq_data *d)
Also here: omap3_mask_ack_irq()
Kevin
Tony Lindgren
maintainerPATCH 1/10 omap: Use separate init_irq functions to avoid cpu_is_omap tests early
Quoted 79 lines · Kevin Hilman
Hi Tony,
Quoted 10 lines · Tony Lindgren
This allows us to remove
cpu_is_omapcalls frominit_irqfunctions and the irq handler. There should not be any need forcpu_is_omapas at this point we only care about SoC generation, and not about subcategories.The main reason for the patch is that we want to initialize only minimal omap specific code from the
init_earlycall, and this and the following patches get us closer to that goal.Signed-off-by Tony Lindgren <[email protected]>[...]
A couple of minor comments on naming below...
Quoted 22 lines
arch/arm/mach-omap2/irq.c +7
-3@@ -103,28 +103,36 @@ static void omap_ack_irq(struct irq_data *d)intc_bank_write_reg (0x1 , &irq_banks[0 ],INTC_CONTROL ); } +static inline void _omap_mask_irq (int irq) +{ +int offset = irq & (~(IRQ_BITS_PER_REG -1 )); + irq &= (IRQ_BITS_PER_REG -1 ); +intc_bank_write_reg (1 << irq, &irq_banks[0 ],INTC_MIR_SET0 + offset); +} +static void omap_mask_irq (struct irq_data *d) {unsigned int irq = d->irq; -int offset = irq & (~(IRQ_BITS_PER_REG -1 )); -if (cpu_is_omap34xx () && !cpu_is_ti816x ()) { -int spurious =0 ;arch/arm/mach-omap2/irq.c static void omap_ack_irq(struct irq_data *d) · line 120
Quoted 10 lines
arch/arm/mach-omap2/irq.c +3
-6+_omap_mask_irq (irq); +} -/* -* INT_34XX_GPT12_IRQ is also the spurious irq. Maybe because -* it is the highest irq number? -*/ -if (irq ==INT_34XX_GPT12_IRQ ) - spurious =omap_check_spurious (irq); +static void omap_mask_irq_omap3 (struct irq_data *d)how about
omap3_mask_irq().In most other code we're using the SoC name as a prefix instead of suffix.
Quoted 22 lines
arch/arm/mach-omap2/irq.c +12
-5+{ +unsigned int irq = d->irq; +int spurious =0 ; -if (!spurious) - previous_irq = irq; - } +/* +* INT_34XX_GPT12_IRQ is also the spurious irq. Maybe because +* it is the highest irq number? +*/ +if (irq ==INT_34XX_GPT12_IRQ ) + spurious =omap_check_spurious (irq); - irq &= (IRQ_BITS_PER_REG -1 ); +if (!spurious) + previous_irq = irq; -intc_bank_write_reg (1 << irq, &irq_banks[0 ],INTC_MIR_SET0 + offset); +_omap_mask_irq (irq); }
Quoted 8 lines
arch/arm/mach-omap2/irq.c static void omap_mask_ack_irq(struct irq_data *d) · line 154
Quoted 6 lines
arch/arm/mach-omap2/irq.c +1
-0static void omap_unmask_irq (struct irq_data *d)@@ -143,6 +151,12 @@ static void omap_mask_ack_irq(struct irq_data *d)omap_ack_irq (d); } +static void omap_mask_ack_irq_omap3 (struct irq_data *d)Also here:
omap3_mask_ack_irq()
Sounds good to me.
Tony
Tony Lindgren
maintainerPATCH 1/10 omap: Use separate init_irq functions to avoid cpu_is_omap tests early
Quoted 5 lines · Tony Lindgren
Quoted 2 lines · Kevin Hilman
Also here:
omap3_mask_ack_irq()Sounds good to me.
Here's this one rebased on the genirq + omap2/3 genirq patch.
Now we've already removed the spurious handling with the genirq patches, so now we actually produce a slightly negative diffstat with this one too:
46 files changed, 78 insertions(+), 84 deletions(-)
Regards,
Tony
From: Tony Lindgren <[email protected]>
Date: Tue, 17 May 2011 03:51:26 -0700
Subject: [PATCH] omap: Use separate init_irq functions to avoid cpu_is_omap tests early
This allows us to remove cpu_is_omap calls from init_irq functions and the irq handler. There should not be any need for cpu_is_omap as at this point we only care about SoC generation, and not about subcategories.
The main reason for the patch is that we want to initialize only minimal omap specific code from the init_early call, and this and the following patches get us closer to that goal.
arch/arm/mach-omap1/board-ams-delta.c +1 -1
@@ -138,7 +138,7 @@ void ams_delta_latch2_write(u16 mask, u16 value)static void __init ams_delta_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct map_desc ams_delta_io_desc[]__initdata = {
arch/arm/mach-omap1/board-fsample.c +1 -1
@@ -329,7 +329,7 @@ static void __init omap_fsample_init(void)static void __init omap_fsample_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */
arch/arm/mach-omap1/board-generic.c +1 -1
@@ -31,7 +31,7 @@static void __init omap_generic_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* assume no Mini-AB port */
arch/arm/mach-omap1/board-h2.c +1 -1
@@ -376,7 +376,7 @@ static struct i2c_board_info __initdata h2_i2c_board_info[] = {static void __init h2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config h2_usb_config__initdata = {
arch/arm/mach-omap1/board-h3.c +1 -1
@@ -439,7 +439,7 @@ static void __init h3_init(void)static void __init h3_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init h3_map_io (void )
arch/arm/mach-omap1/board-htcherald.c +1 -1
@@ -605,7 +605,7 @@ static void __init htcherald_init_irq(void) {printk (KERN_INFO "htcherald_init_irq.\n" );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }MACHINE_START (HERALD ,"HTC Herald" )
arch/arm/mach-omap1/board-innovator.c +1 -1
@@ -292,7 +292,7 @@ static void __init innovator_init_smc91x(void)static void __init innovator_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }#ifdef CONFIG_ARCH_OMAP15XX
arch/arm/mach-omap1/board-nokia770.c +1 -1
@@ -51,7 +51,7 @@ static void __init omap_nokia770_init_irq(void)omap_writew ((omap_readw (0xfffb5004 ) & ~2 ),0xfffb5004 );omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int nokia770_keymap[] = {
arch/arm/mach-omap1/board-osk.c +1 -1
@@ -282,7 +282,7 @@ static void __init osk_init_cf(void)static void __init osk_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config osk_usb_config__initdata = {
arch/arm/mach-omap1/board-palmte.c +1 -1
@@ -62,7 +62,7 @@static void __init omap_palmte_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmte_keymap[] = {
arch/arm/mach-omap1/board-palmtt.c +1 -1
@@ -266,7 +266,7 @@ static struct spi_board_info __initdata palmtt_boardinfo[] = {static void __init omap_palmtt_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static struct omap_usb_config palmtt_usb_config__initdata = {
arch/arm/mach-omap1/board-palmz71.c +1 -1
@@ -61,7 +61,7 @@ static void __initomap_palmz71_init_irq (void) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static const unsigned int palmz71_keymap[] = {
arch/arm/mach-omap1/board-perseus2.c +1 -1
@@ -297,7 +297,7 @@ static void __init omap_perseus2_init(void)static void __init omap_perseus2_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/* Only FPGA needs to be mapped here. All others are done with ioremap */ static struct map_desc omap_perseus2_io_desc[]__initdata = {
arch/arm/mach-omap1/board-sx1.c +1 -1
@@ -411,7 +411,7 @@ static void __init omap_sx1_init(void)static void __init omap_sx1_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }/*----------------------------------------*/
arch/arm/mach-omap1/board-voiceblue.c +1 -1
@@ -162,7 +162,7 @@ static struct omap_board_config_kernel voiceblue_config[] = {static void __init voiceblue_init_irq (void ) {omap1_init_common_hw (); -omap_init_irq (); +omap1_init_irq (); }static void __init voiceblue_map_io (void )
arch/arm/mach-omap1/irq.c +1 -1
@@ -175,7 +175,7 @@ static struct irq_chip omap_irq_chip = { .irq_set_wake = omap_wake_irq, }; -void __init omap_init_irq (void ) +void __init omap1_init_irq (void ) {int i, j;
arch/arm/mach-omap2/board-2430sdp.c +1 -1
@@ -277,7 +277,7 @@ MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board") .reserve = omap_reserve, .map_io = omap_2430sdp_map_io, .init_early = omap_2430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_2430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3430sdp.c +1 -1
@@ -909,7 +909,7 @@ MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_3430sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_3430sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-3630sdp.c +1 -1
@@ -219,7 +219,7 @@ MACHINE_START(OMAP_3630SDP, "OMAP 3630SDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_sdp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_sdp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517crane.c +1 -1
@@ -110,7 +110,7 @@ MACHINE_START(CRANEBOARD, "AM3517/05 CRANEBOARD") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_crane_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_crane_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517evm.c +1 -1
@@ -516,7 +516,7 @@ MACHINE_START(OMAP3517EVM, "OMAP3517/AM3517 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = am3517_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = am3517_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-apollon.c +1 -1
@@ -357,7 +357,7 @@ MACHINE_START(OMAP_APOLLON, "OMAP24xx Apollon") .reserve = omap_reserve, .map_io = omap_apollon_map_io, .init_early = omap_apollon_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_apollon_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t35.c +1 -1
@@ -806,7 +806,7 @@ MACHINE_START(CM_T35, "Compulab CM-T35") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t35_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t35_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t3517.c +1 -1
@@ -306,7 +306,7 @@ MACHINE_START(CM_T3517, "Compulab CM-T3517") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = cm_t3517_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = cm_t3517_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-devkit8000.c +1 -1
@@ -457,7 +457,7 @@ static void __init devkit8000_init_early(void)static void __init devkit8000_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-generic.c +1 -1
@@ -70,7 +70,7 @@ MACHINE_START(OMAP_GENERIC, "Generic OMAP24xx") .reserve = omap_reserve, .map_io = omap_generic_map_io, .init_early = omap_generic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = omap_generic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-h4.c +1 -1
@@ -298,7 +298,7 @@ static void __init omap_h4_init_early(void)static void __init omap_h4_init_irq (void ) { -omap_init_irq (); +omap2_init_irq (); }static struct at24_platform_data m24c01 = {
arch/arm/mach-omap2/board-igep0020.c +1 -1
@@ -708,7 +708,7 @@ MACHINE_START(IGEP0020, "IGEP v2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep2_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep2_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-igep0030.c +1 -1
@@ -452,7 +452,7 @@ MACHINE_START(IGEP0030, "IGEP OMAP3 module") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = igep3_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = igep3_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ldp.c +1 -1
@@ -466,7 +466,7 @@ MACHINE_START(OMAP_LDP, "OMAP LDP board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_ldp_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_ldp_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-n8x0.c +3 -3
@@ -705,7 +705,7 @@ MACHINE_START(NOKIA_N800, "Nokia N800") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -715,7 +715,7 @@ MACHINE_START(NOKIA_N810, "Nokia N810") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
@@ -725,7 +725,7 @@ MACHINE_START(NOKIA_N810_WIMAX, "Nokia N810 WiMAX") .reserve = omap_reserve, .map_io = n8x0_map_io, .init_early = n8x0_init_early, - .init_irq = omap_init_irq, + .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3beagle.c +1 -1
@@ -540,7 +540,7 @@ static void __init omap3_beagle_init_early(void)static void __init omap3_beagle_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3evm.c +1 -1
@@ -894,7 +894,7 @@ MACHINE_START(OMAP3EVM, "OMAP3 EVM") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3_evm_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3_evm_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3logic.c +2 -2
@@ -225,7 +225,7 @@ MACHINE_START(OMAP3_TORPEDO, "Logic OMAP3 Torpedo board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
@@ -234,7 +234,7 @@ MACHINE_START(OMAP3530_LV_SOM, "OMAP Logic 3530 LV SOM board") .boot_params = 0x80000100, .map_io = omap3_map_io, .init_early = omap3logic_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3logic_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3pandora.c +1 -1
@@ -720,7 +720,7 @@ MACHINE_START(OMAP3_PANDORA, "Pandora Handheld Console") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap3pandora_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap3pandora_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3stalker.c +1 -1
@@ -587,7 +587,7 @@ static void __init omap3_stalker_init_early(void)static void __init omap3_stalker_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-omap3touchbook.c +1 -1
@@ -422,7 +422,7 @@ static void __init omap3_touchbook_init_early(void)static void __init omap3_touchbook_init_irq (void ) { -omap_init_irq (); +omap3_init_irq ();#ifdef CONFIG_OMAP_32K_TIMER omap2_gp_clockevent_set_gptimer (12 );#endif
arch/arm/mach-omap2/board-overo.c +1 -1
@@ -790,7 +790,7 @@ MACHINE_START(OVERO, "Gumstix Overo") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = overo_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = overo_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rm680.c +1 -1
@@ -178,7 +178,7 @@ MACHINE_START(NOKIA_RM680, "Nokia RM-680 board") .reserve = omap_reserve, .map_io = rm680_map_io, .init_early = rm680_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rm680_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-rx51.c +1 -1
@@ -151,7 +151,7 @@ MACHINE_START(NOKIA_RX51, "Nokia RX-51 board") .reserve = omap_reserve, .map_io = rx51_map_io, .init_early = rx51_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = rx51_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/board-ti8168evm.c +1 -6
@@ -33,11 +33,6 @@ static void __init ti8168_init_early(void)omap2_init_common_devices (NULL ,NULL ); } -static void __init ti8168_evm_init_irq (void ) -{ -omap_init_irq (); -} -static void __init ti8168_evm_init (void ) {omap_serial_init ();
@@ -56,7 +51,7 @@ MACHINE_START(TI8168EVM, "ti8168evm") .boot_params = 0x80000100, .map_io = ti8168_evm_map_io, .init_early = ti8168_init_early, - .init_irq = ti8168_evm_init_irq, + .init_irq = ti816x_init_irq, .timer = &omap_timer, .init_machine = ti8168_evm_init,MACHINE_END
arch/arm/mach-omap2/board-zoom.c +2 -2
@@ -137,7 +137,7 @@ MACHINE_START(OMAP_ZOOM2, "OMAP Zoom2 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
@@ -147,7 +147,7 @@ MACHINE_START(OMAP_ZOOM3, "OMAP Zoom3 board") .reserve = omap_reserve, .map_io = omap3_map_io, .init_early = omap_zoom_init_early, - .init_irq = omap_init_irq, + .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, .timer = &omap_timer,MACHINE_END
arch/arm/mach-omap2/io.c +1 -16
@@ -333,23 +333,9 @@ static int _set_hwmod_postsetup_state(struct omap_hwmod *oh, void *data)return omap_hwmod_set_postsetup_state (oh, *(u8 *)data); } +/* See irq.c, omap4-common.c and entry-macro.S */ void __iomem *omap_irq_base; -/* -* Initialize asm_irq_base for entry-macro.S -*/ -static inline void omap_irq_base_init (void ) -{ -if (cpu_is_omap24xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP24XX_IC_BASE ); -else if (cpu_is_omap34xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP34XX_IC_BASE ); -else if (cpu_is_omap44xx ()) - omap_irq_base =OMAP2_L4_IO_ADDRESS (OMAP44XX_GIC_CPU_BASE ); -else -pr_err ("Could not initialize omap_irq_base\n" ); -} -void __init omap2_init_common_infrastructure (void ) {u8 postsetup_state;
@@ -422,7 +408,6 @@ void __init omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,_omap2_init_reprogram_sdrc (); } -omap_irq_base_init (); }/*
arch/arm/mach-omap2/irq.c +21 -11
@@ -141,25 +141,20 @@ omap_alloc_gc(void __iomem *base, unsigned int irq_start, unsigned int num) IRQ_NOREQUEST | IRQ_NOPROBE, 0); } -void __init omap_init_irq (void ) +void __init omap_init_irq (u32 base,int nr_irqs) {unsigned long nr_of_irqs =0 ;unsigned int nr_banks =0 ;int i, j; + omap_irq_base =ioremap (base,SZ_4K ); +if (WARN_ON (!omap_irq_base)) +return ; +for (i =0 ; i <ARRAY_SIZE (irq_banks); i++) { -unsigned long base =0 ;struct omap_irq_bank *bank = irq_banks + i; -if (cpu_is_omap24xx ()) - base =OMAP24XX_IC_BASE ; -else if (cpu_is_omap34xx ()) - base =OMAP34XX_IC_BASE ; - -BUG_ON (!base); - -if (cpu_is_ti816x ()) - bank->nr_irqs =128 ; + bank->nr_irqs = nr_irqs;/* Static mapping, never released */ bank->base_reg =ioremap (base,SZ_4K );
@@ -181,6 +176,21 @@ void __init omap_init_irq(void) nr_of_irqs, nr_banks, nr_banks > 1 ? "s" : ""); } +void __init omap2_init_irq (void ) +{ +omap_init_irq (OMAP24XX_IC_BASE ,96 ); +} + +void __init omap3_init_irq (void ) +{ +omap_init_irq (OMAP34XX_IC_BASE ,96 ); +} + +void __init ti816x_init_irq (void ) +{ +omap_init_irq (OMAP34XX_IC_BASE ,128 ); +} +#ifdef CONFIG_ARCH_OMAP3 static struct omap3_intc_regs intc_context[ARRAY_SIZE (irq_banks)];
arch/arm/mach-omap2/omap4-common.c +5 -5
@@ -19,6 +19,8 @@#include <asm/hardware/gic.h> #include <asm/hardware/cache-l2x0.h> +#include <plat/irqs.h> +#include <mach/hardware.h> #include <mach/omap4-common.h>
@@ -31,17 +33,15 @@ void __iomem *gic_dist_base_addr;void __init gic_init_irq (void ) { -void __iomem *gic_cpu_base; -/* Static mapping, never released */ gic_dist_base_addr =ioremap (OMAP44XX_GIC_DIST_BASE ,SZ_4K );BUG_ON (!gic_dist_base_addr);/* Static mapping, never released */ - gic_cpu_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 ); -BUG_ON (!gic_cpu_base); + omap_irq_base =ioremap (OMAP44XX_GIC_CPU_BASE ,SZ_512 ); +BUG_ON (!omap_irq_base); -gic_init (0 ,29 , gic_dist_base_addr, gic_cpu_base); +gic_init (0 ,29 , gic_dist_base_addr, omap_irq_base); }#ifdef CONFIG_CACHE_L2X0
arch/arm/plat-omap/include/plat/irqs.h +5 -1
@@ -428,7 +428,11 @@#define INTCPS_NR_IRQS 96#ifndef __ASSEMBLY__ -extern void omap_init_irq (void ); +extern void __iomem *omap_irq_base; +void omap1_init_irq (void ); +void omap2_init_irq (void ); +void omap3_init_irq (void ); +void ti816x_init_irq (void );extern int omap_irq_pending (void );void omap_intc_save_context (void );void omap_intc_restore_context (void );
Tony Lindgren
maintainerPATCH 2/10 omap: Set separate timer init functions to avoid cpu_is_omap tests
In reply to Tony LindgrenThis is needed for the following patches to initialize non-sys_timer hardware later on. Again, there's no need to do cpu_is_omap calls as we only care about the major omap generation. We can do this with the .timer entry alone.
Note that for now we just set stubs for the various sys_timer entries that will get populated in a later patch. Also the following patches will remove the omap_dm_timer_init calls and make it an arch_initcall.
--- arch/arm/mach-omap1/board-ams-delta.c | 2 - arch/arm/mach-omap1/board-fsample.c | 2 - arch/arm/mach-omap1/board-generic.c | 2 - arch/arm/mach-omap1/board-h2.c | 2 - arch/arm/mach-omap1/board-h3.c | 2 - arch/arm/mach-omap1/board-htcherald.c | 2 - arch/arm/mach-omap1/board-innovator.c | 2 - arch/arm/mach-omap1/board-nokia770.c | 2 - arch/arm/mach-omap1/board-osk.c | 2 - arch/arm/mach-omap1/board-palmte.c | 2 - arch/arm/mach-omap1/board-palmtt.c | 2 - arch/arm/mach-omap1/board-palmz71.c | 2 - arch/arm/mach-omap1/board-perseus2.c | 2 - arch/arm/mach-omap1/board-sx1.c | 2 - arch/arm/mach-omap1/board-voiceblue.c | 2 - arch/arm/mach-omap1/time.c | 6 +-- arch/arm/mach-omap2/board-2430sdp.c | 2 - arch/arm/mach-omap2/board-3430sdp.c | 2 - arch/arm/mach-omap2/board-3630sdp.c | 2 - arch/arm/mach-omap2/board-4430sdp.c | 2 - arch/arm/mach-omap2/board-am3517crane.c | 2 - arch/arm/mach-omap2/board-am3517evm.c | 2 - arch/arm/mach-omap2/board-apollon.c | 2 - arch/arm/mach-omap2/board-cm-t35.c | 2 - arch/arm/mach-omap2/board-cm-t3517.c | 2 - arch/arm/mach-omap2/board-devkit8000.c | 2 - arch/arm/mach-omap2/board-generic.c | 2 - arch/arm/mach-omap2/board-h4.c | 2 - arch/arm/mach-omap2/board-igep0020.c | 2 - arch/arm/mach-omap2/board-igep0030.c | 2 - arch/arm/mach-omap2/board-ldp.c | 2 - arch/arm/mach-omap2/board-n8x0.c | 6 +-- arch/arm/mach-omap2/board-omap3beagle.c | 2 - arch/arm/mach-omap2/board-omap3evm.c | 2 - arch/arm/mach-omap2/board-omap3logic.c | 4 +- arch/arm/mach-omap2/board-omap3pandora.c | 2 - arch/arm/mach-omap2/board-omap3stalker.c | 2 - arch/arm/mach-omap2/board-omap3touchbook.c | 2 - arch/arm/mach-omap2/board-omap4panda.c | 2 - arch/arm/mach-omap2/board-overo.c | 2 - arch/arm/mach-omap2/board-rm680.c | 2 - arch/arm/mach-omap2/board-rx51.c | 2 - arch/arm/mach-omap2/board-ti8168evm.c | 2 - arch/arm/mach-omap2/board-zoom.c | 4 +- arch/arm/mach-omap2/timer-gp.c | 63 ++++++++++++++++++++++++---- arch/arm/plat-omap/include/plat/common.h | 7 +++ arch/arm/plat-omap/include/plat/dmtimer.h | 1 47 files changed, 111 insertions(+), 60 deletions(-)
arch/arm/mach-omap1/board-ams-delta.c +1 -1
@@ -391,7 +391,7 @@ MACHINE_START(AMS_DELTA, "Amstrad E3 (Delta)") .reserve = omap_reserve, .init_irq = ams_delta_init_irq, .init_machine = ams_delta_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END EXPORT_SYMBOL (ams_delta_latch1_write );
arch/arm/mach-omap1/board-fsample.c +1 -1
@@ -394,5 +394,5 @@ MACHINE_START(OMAP_FSAMPLE, "OMAP730 F-Sample") .reserve = omap_reserve, .init_irq = omap_fsample_init_irq, .init_machine = omap_fsample_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-generic.c +1 -1
@@ -99,5 +99,5 @@ MACHINE_START(OMAP_GENERIC, "Generic OMAP1510/1610/1710") .reserve = omap_reserve, .init_irq = omap_generic_init_irq, .init_machine = omap_generic_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-h2.c +1 -1
@@ -466,5 +466,5 @@ MACHINE_START(OMAP_H2, "TI-H2") .reserve = omap_reserve, .init_irq = h2_init_irq, .init_machine = h2_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-h3.c +1 -1
@@ -454,5 +454,5 @@ MACHINE_START(OMAP_H3, "TI OMAP1710 H3 board") .reserve = omap_reserve, .init_irq = h3_init_irq, .init_machine = h3_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-htcherald.c +1 -1
@@ -616,5 +616,5 @@ MACHINE_START(HERALD, "HTC Herald") .reserve = omap_reserve, .init_irq = htcherald_init_irq, .init_machine = htcherald_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-innovator.c +1 -1
@@ -464,5 +464,5 @@ MACHINE_START(OMAP_INNOVATOR, "TI-Innovator") .reserve = omap_reserve, .init_irq = innovator_init_irq, .init_machine = innovator_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-nokia770.c +1 -1
@@ -269,5 +269,5 @@ MACHINE_START(NOKIA770, "Nokia 770") .reserve = omap_reserve, .init_irq = omap_nokia770_init_irq, .init_machine = omap_nokia770_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-osk.c +1 -1
@@ -588,5 +588,5 @@ MACHINE_START(OMAP_OSK, "TI-OSK") .reserve = omap_reserve, .init_irq = osk_init_irq, .init_machine = osk_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-palmte.c +1 -1
@@ -280,5 +280,5 @@ MACHINE_START(OMAP_PALMTE, "OMAP310 based Palm Tungsten E") .reserve = omap_reserve, .init_irq = omap_palmte_init_irq, .init_machine = omap_palmte_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-palmtt.c +1 -1
@@ -326,5 +326,5 @@ MACHINE_START(OMAP_PALMTT, "OMAP1510 based Palm Tungsten|T") .reserve = omap_reserve, .init_irq = omap_palmtt_init_irq, .init_machine = omap_palmtt_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-palmz71.c +1 -1
@@ -346,5 +346,5 @@ MACHINE_START(OMAP_PALMZ71, "OMAP310 based Palm Zire71") .reserve = omap_reserve, .init_irq = omap_palmz71_init_irq, .init_machine = omap_palmz71_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-perseus2.c +1 -1
@@ -355,5 +355,5 @@ MACHINE_START(OMAP_PERSEUS2, "OMAP730 Perseus2") .reserve = omap_reserve, .init_irq = omap_perseus2_init_irq, .init_machine = omap_perseus2_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-sx1.c +1 -1
@@ -426,5 +426,5 @@ MACHINE_START(SX1, "OMAP310 based Siemens SX1") .reserve = omap_reserve, .init_irq = omap_sx1_init_irq, .init_machine = omap_sx1_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/board-voiceblue.c +1 -1
@@ -306,5 +306,5 @@ MACHINE_START(VOICEBLUE, "VoiceBlue OMAP5910") .reserve = omap_reserve, .init_irq = voiceblue_init_irq, .init_machine = voiceblue_init, - .timer = &omap_timer, + .timer = &omap1_timer,MACHINE_END
arch/arm/mach-omap1/time.c +3 -3
@@ -322,7 +322,7 @@ static inline int omap_32k_timer_usable(void)* Timer initialization * --------------------------------------------------------------------------- */ -static void __init omap_timer_init (void ) +static void __init omap1_timer_init (void ) {if (omap_32k_timer_usable ()) {preferred_sched_clock_init (1 );
@@ -332,6 +332,6 @@ static void __init omap_timer_init(void) } } -struct sys_timer omap_timer = { - .init = omap_timer_init, +struct sys_timer omap1_timer = { + .init = omap1_timer_init, };
arch/arm/mach-omap2/board-2430sdp.c +1 -1
@@ -279,5 +279,5 @@ MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board") .init_early = omap_2430sdp_init_early, .init_irq = omap2_init_irq, .init_machine = omap_2430sdp_init, - .timer = &omap_timer, + .timer = &omap243x_timer,MACHINE_END
arch/arm/mach-omap2/board-3430sdp.c +1 -1
@@ -911,5 +911,5 @@ MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board") .init_early = omap_3430sdp_init_early, .init_irq = omap3_init_irq, .init_machine = omap_3430sdp_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-3630sdp.c +1 -1
@@ -221,5 +221,5 @@ MACHINE_START(OMAP_3630SDP, "OMAP 3630SDP board") .init_early = omap_sdp_init_early, .init_irq = omap3_init_irq, .init_machine = omap_sdp_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-4430sdp.c +1 -1
@@ -827,5 +827,5 @@ MACHINE_START(OMAP_4430SDP, "OMAP4430 4430SDP board") .init_early = omap_4430sdp_init_early, .init_irq = gic_init_irq, .init_machine = omap_4430sdp_init, - .timer = &omap_timer, + .timer = &omap4_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517crane.c +1 -1
@@ -112,5 +112,5 @@ MACHINE_START(CRANEBOARD, "AM3517/05 CRANEBOARD") .init_early = am3517_crane_init_early, .init_irq = omap3_init_irq, .init_machine = am3517_crane_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-am3517evm.c +1 -1
@@ -518,5 +518,5 @@ MACHINE_START(OMAP3517EVM, "OMAP3517/AM3517 EVM") .init_early = am3517_evm_init_early, .init_irq = omap3_init_irq, .init_machine = am3517_evm_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-apollon.c +1 -1
@@ -359,5 +359,5 @@ MACHINE_START(OMAP_APOLLON, "OMAP24xx Apollon") .init_early = omap_apollon_init_early, .init_irq = omap2_init_irq, .init_machine = omap_apollon_init, - .timer = &omap_timer, + .timer = &omap242x_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t35.c +1 -1
@@ -808,5 +808,5 @@ MACHINE_START(CM_T35, "Compulab CM-T35") .init_early = cm_t35_init_early, .init_irq = omap3_init_irq, .init_machine = cm_t35_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-cm-t3517.c +1 -1
@@ -308,5 +308,5 @@ MACHINE_START(CM_T3517, "Compulab CM-T3517") .init_early = cm_t3517_init_early, .init_irq = omap3_init_irq, .init_machine = cm_t3517_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-devkit8000.c +1 -1
@@ -816,5 +816,5 @@ MACHINE_START(DEVKIT8000, "OMAP3 Devkit8000") .init_early = devkit8000_init_early, .init_irq = devkit8000_init_irq, .init_machine = devkit8000_init, - .timer = &omap_timer, + .timer = &omap3_beagle_timer,MACHINE_END
arch/arm/mach-omap2/board-generic.c +1 -1
@@ -72,5 +72,5 @@ MACHINE_START(OMAP_GENERIC, "Generic OMAP24xx") .init_early = omap_generic_init_early, .init_irq = omap2_init_irq, .init_machine = omap_generic_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-h4.c +1 -1
@@ -388,5 +388,5 @@ MACHINE_START(OMAP_H4, "OMAP2420 H4 board") .init_early = omap_h4_init_early, .init_irq = omap_h4_init_irq, .init_machine = omap_h4_init, - .timer = &omap_timer, + .timer = &omap242x_timer,MACHINE_END
arch/arm/mach-omap2/board-igep0020.c +1 -1
@@ -710,5 +710,5 @@ MACHINE_START(IGEP0020, "IGEP v2 board") .init_early = igep2_init_early, .init_irq = omap3_init_irq, .init_machine = igep2_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-igep0030.c +1 -1
@@ -454,5 +454,5 @@ MACHINE_START(IGEP0030, "IGEP OMAP3 module") .init_early = igep3_init_early, .init_irq = omap3_init_irq, .init_machine = igep3_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-ldp.c +1 -1
@@ -468,5 +468,5 @@ MACHINE_START(OMAP_LDP, "OMAP LDP board") .init_early = omap_ldp_init_early, .init_irq = omap3_init_irq, .init_machine = omap_ldp_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-n8x0.c +3 -3
@@ -707,7 +707,7 @@ MACHINE_START(NOKIA_N800, "Nokia N800") .init_early = n8x0_init_early, .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, - .timer = &omap_timer, + .timer = &omap242x_timer, MACHINE_ENDMACHINE_START (NOKIA_N810 ,"Nokia N810" )
@@ -717,7 +717,7 @@ MACHINE_START(NOKIA_N810, "Nokia N810") .init_early = n8x0_init_early, .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, - .timer = &omap_timer, + .timer = &omap242x_timer, MACHINE_ENDMACHINE_START (NOKIA_N810_WIMAX ,"Nokia N810 WiMAX" )
@@ -727,5 +727,5 @@ MACHINE_START(NOKIA_N810_WIMAX, "Nokia N810 WiMAX") .init_early = n8x0_init_early, .init_irq = omap2_init_irq, .init_machine = n8x0_init_machine, - .timer = &omap_timer, + .timer = &omap242x_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3beagle.c +1 -1
@@ -689,5 +689,5 @@ MACHINE_START(OMAP3_BEAGLE, "OMAP3 Beagle Board") .init_early = omap3_beagle_init_early, .init_irq = omap3_beagle_init_irq, .init_machine = omap3_beagle_init, - .timer = &omap_timer, + .timer = &omap3_beagle_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3evm.c +1 -1
@@ -896,5 +896,5 @@ MACHINE_START(OMAP3EVM, "OMAP3 EVM") .init_early = omap3_evm_init_early, .init_irq = omap3_init_irq, .init_machine = omap3_evm_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3logic.c +2 -2
@@ -227,7 +227,7 @@ MACHINE_START(OMAP3_TORPEDO, "Logic OMAP3 Torpedo board") .init_early = omap3logic_init_early, .init_irq = omap3_init_irq, .init_machine = omap3logic_init, - .timer = &omap_timer, + .timer = &omap3_timer, MACHINE_ENDMACHINE_START (OMAP3530_LV_SOM ,"OMAP Logic 3530 LV SOM board" )
@@ -236,5 +236,5 @@ MACHINE_START(OMAP3530_LV_SOM, "OMAP Logic 3530 LV SOM board") .init_early = omap3logic_init_early, .init_irq = omap3_init_irq, .init_machine = omap3logic_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3pandora.c +1 -1
@@ -722,5 +722,5 @@ MACHINE_START(OMAP3_PANDORA, "Pandora Handheld Console") .init_early = omap3pandora_init_early, .init_irq = omap3_init_irq, .init_machine = omap3pandora_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3stalker.c +1 -1
@@ -661,5 +661,5 @@ MACHINE_START(SBC3530, "OMAP3 STALKER") .init_early = omap3_stalker_init_early, .init_irq = omap3_stalker_init_irq, .init_machine = omap3_stalker_init, - .timer = &omap_timer, + .timer = &omap3_beagle_timer,MACHINE_END
arch/arm/mach-omap2/board-omap3touchbook.c +1 -1
@@ -546,5 +546,5 @@ MACHINE_START(TOUCHBOOK, "OMAP3 touchbook Board") .init_early = omap3_touchbook_init_early, .init_irq = omap3_touchbook_init_irq, .init_machine = omap3_touchbook_init, - .timer = &omap_timer, + .timer = &omap3_beagle_timer,MACHINE_END
arch/arm/mach-omap2/board-omap4panda.c +1 -1
@@ -749,5 +749,5 @@ MACHINE_START(OMAP4_PANDA, "OMAP4 Panda board") .init_early = omap4_panda_init_early, .init_irq = gic_init_irq, .init_machine = omap4_panda_init, - .timer = &omap_timer, + .timer = &omap4_timer,MACHINE_END
arch/arm/mach-omap2/board-overo.c +1 -1
@@ -792,5 +792,5 @@ MACHINE_START(OVERO, "Gumstix Overo") .init_early = overo_init_early, .init_irq = omap3_init_irq, .init_machine = overo_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-rm680.c +1 -1
@@ -180,5 +180,5 @@ MACHINE_START(NOKIA_RM680, "Nokia RM-680 board") .init_early = rm680_init_early, .init_irq = omap3_init_irq, .init_machine = rm680_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-rx51.c +1 -1
@@ -153,5 +153,5 @@ MACHINE_START(NOKIA_RX51, "Nokia RX-51 board") .init_early = rx51_init_early, .init_irq = omap3_init_irq, .init_machine = rx51_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/board-ti8168evm.c +1 -1
@@ -52,6 +52,6 @@ MACHINE_START(TI8168EVM, "ti8168evm") .map_io = ti8168_evm_map_io, .init_early = ti8168_init_early, .init_irq = ti816x_init_irq, - .timer = &omap_timer, + .timer = &omap3_timer, .init_machine = ti8168_evm_init,MACHINE_END
arch/arm/mach-omap2/board-zoom.c +2 -2
@@ -139,7 +139,7 @@ MACHINE_START(OMAP_ZOOM2, "OMAP Zoom2 board") .init_early = omap_zoom_init_early, .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, - .timer = &omap_timer, + .timer = &omap3_timer, MACHINE_ENDMACHINE_START (OMAP_ZOOM3 ,"OMAP Zoom3 board" )
@@ -149,5 +149,5 @@ MACHINE_START(OMAP_ZOOM3, "OMAP Zoom3 board") .init_early = omap_zoom_init_early, .init_irq = omap3_init_irq, .init_machine = omap_zoom_init, - .timer = &omap_timer, + .timer = &omap3_timer,MACHINE_END
arch/arm/mach-omap2/timer-gp.c +55 -8
75 lines not on this page · raw message
arch/arm/plat-omap/include/plat/common.h +6 -1
@@ -34,7 +34,12 @@struct sys_timer ;extern void omap_map_common_io (void ); -extern struct sys_timer omap_timer; +extern struct sys_timer omap1_timer; +extern struct sys_timer omap242x_timer; +extern struct sys_timer omap243x_timer; +extern struct sys_timer omap3_timer; +extern struct sys_timer omap3_beagle_timer; +extern struct sys_timer omap4_timer;extern bool omap_32k_timer_init (void );extern int __init omap_init_clocksource_32k (void );extern unsigned long long notrace omap_32k_sched_clock (void );
arch/arm/plat-omap/include/plat/dmtimer.h +0 -1
@@ -57,7 +57,6 @@#define OMAP_TIMER_IP_VERSION_1 0x1struct omap_dm_timer ;extern struct omap_dm_timer *gptimer_wakeup; -extern struct sys_timer omap_timer;struct clk ;int omap_dm_timer_init (void );
Tony Lindgren
maintainerPATCH 3/10 omap: Move dmtimer defines to dmtimer.h
In reply to Tony LindgrenThese will be needed when dmtimer platform init code gets split for omap1 and omap2+. These will also be needed for inline macros for sys_timer in the following patches. No functional changes.
--- arch/arm/plat-omap/dmtimer.c | 121 ---------------------------- arch/arm/plat-omap/include/plat/dmtimer.h | 125 +++++++++++++++++++++++++++++ 2 files changed, 125 insertions(+), 121 deletions(-)
arch/arm/plat-omap/dmtimer.c +0 -121
@@ -41,127 +41,6 @@#include <plat/dmtimer.h> #include <mach/irqs.h> -/* register offsets */ -#define _OMAP_TIMER_ID_OFFSET 0x00 -#define _OMAP_TIMER_OCP_CFG_OFFSET 0x10 -#define _OMAP_TIMER_SYS_STAT_OFFSET 0x14 -#define _OMAP_TIMER_STAT_OFFSET 0x18 -#define _OMAP_TIMER_INT_EN_OFFSET 0x1c -#define _OMAP_TIMER_WAKEUP_EN_OFFSET 0x20 -#define _OMAP_TIMER_CTRL_OFFSET 0x24 -#define OMAP_TIMER_CTRL_GPOCFG (1 << 14) -#define OMAP_TIMER_CTRL_CAPTMODE (1 << 13) -#define OMAP_TIMER_CTRL_PT (1 << 12) -#define OMAP_TIMER_CTRL_TCM_LOWTOHIGH (0x1 << 8) -#define OMAP_TIMER_CTRL_TCM_HIGHTOLOW (0x2 << 8) -#define OMAP_TIMER_CTRL_TCM_BOTHEDGES (0x3 << 8) -#define OMAP_TIMER_CTRL_SCPWM (1 << 7) -#define OMAP_TIMER_CTRL_CE (1 << 6)/* compare enable */ -#define OMAP_TIMER_CTRL_PRE (1 << 5)/* prescaler enable */ -#define OMAP_TIMER_CTRL_PTV_SHIFT 2/* prescaler value shift */ -#define OMAP_TIMER_CTRL_POSTED (1 << 2) -#define OMAP_TIMER_CTRL_AR (1 << 1)/* auto-reload enable */ -#define OMAP_TIMER_CTRL_ST (1 << 0)/* start timer */ -#define _OMAP_TIMER_COUNTER_OFFSET 0x28 -#define _OMAP_TIMER_LOAD_OFFSET 0x2c -#define _OMAP_TIMER_TRIGGER_OFFSET 0x30 -#define _OMAP_TIMER_WRITE_PEND_OFFSET 0x34 -#define WP_NONE 0/* no write pending bit */ -#define WP_TCLR (1 << 0) -#define WP_TCRR (1 << 1) -#define WP_TLDR (1 << 2) -#define WP_TTGR (1 << 3) -#define WP_TMAR (1 << 4) -#define WP_TPIR (1 << 5) -#define WP_TNIR (1 << 6) -#define WP_TCVR (1 << 7) -#define WP_TOCR (1 << 8) -#define WP_TOWR (1 << 9) -#define _OMAP_TIMER_MATCH_OFFSET 0x38 -#define _OMAP_TIMER_CAPTURE_OFFSET 0x3c -#define _OMAP_TIMER_IF_CTRL_OFFSET 0x40 -#define _OMAP_TIMER_CAPTURE2_OFFSET 0x44/* TCAR2, 34xx only */ -#define _OMAP_TIMER_TICK_POS_OFFSET 0x48/* TPIR, 34xx only */ -#define _OMAP_TIMER_TICK_NEG_OFFSET 0x4c/* TNIR, 34xx only */ -#define _OMAP_TIMER_TICK_COUNT_OFFSET 0x50/* TCVR, 34xx only */ -#define _OMAP_TIMER_TICK_INT_MASK_SET_OFFSET 0x54/* TOCR, 34xx only */ -#define _OMAP_TIMER_TICK_INT_MASK_COUNT_OFFSET 0x58/* TOWR, 34xx only */ - -/* register offsets with the write pending bit encoded */ -#define WPSHIFT 16 - -#define OMAP_TIMER_ID_REG (_OMAP_TIMER_ID_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_OCP_CFG_REG (_OMAP_TIMER_OCP_CFG_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_SYS_STAT_REG (_OMAP_TIMER_SYS_STAT_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_STAT_REG (_OMAP_TIMER_STAT_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_INT_EN_REG (_OMAP_TIMER_INT_EN_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_WAKEUP_EN_REG (_OMAP_TIMER_WAKEUP_EN_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_CTRL_REG (_OMAP_TIMER_CTRL_OFFSET \ - | (WP_TCLR << WPSHIFT)) - -#define OMAP_TIMER_COUNTER_REG (_OMAP_TIMER_COUNTER_OFFSET \ - | (WP_TCRR << WPSHIFT)) - -#define OMAP_TIMER_LOAD_REG (_OMAP_TIMER_LOAD_OFFSET \ - | (WP_TLDR << WPSHIFT)) - -#define OMAP_TIMER_TRIGGER_REG (_OMAP_TIMER_TRIGGER_OFFSET \ - | (WP_TTGR << WPSHIFT)) - -#define OMAP_TIMER_WRITE_PEND_REG (_OMAP_TIMER_WRITE_PEND_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_MATCH_REG (_OMAP_TIMER_MATCH_OFFSET \ - | (WP_TMAR << WPSHIFT)) - -#define OMAP_TIMER_CAPTURE_REG (_OMAP_TIMER_CAPTURE_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_IF_CTRL_REG (_OMAP_TIMER_IF_CTRL_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_CAPTURE2_REG (_OMAP_TIMER_CAPTURE2_OFFSET \ - | (WP_NONE << WPSHIFT)) - -#define OMAP_TIMER_TICK_POS_REG (_OMAP_TIMER_TICK_POS_OFFSET \ - | (WP_TPIR << WPSHIFT)) - -#define OMAP_TIMER_TICK_NEG_REG (_OMAP_TIMER_TICK_NEG_OFFSET \ - | (WP_TNIR << WPSHIFT)) - -#define OMAP_TIMER_TICK_COUNT_REG (_OMAP_TIMER_TICK_COUNT_OFFSET \ - | (WP_TCVR << WPSHIFT)) - -#define OMAP_TIMER_TICK_INT_MASK_SET_REG \ - (_OMAP_TIMER_TICK_INT_MASK_SET_OFFSET | (WP_TOCR << WPSHIFT)) - -#define OMAP_TIMER_TICK_INT_MASK_COUNT_REG \ - (_OMAP_TIMER_TICK_INT_MASK_COUNT_OFFSET | (WP_TOWR << WPSHIFT)) - -struct omap_dm_timer { -unsigned long phys_base; -int irq; -#ifdef CONFIG_ARCH_OMAP2PLUS -struct clk *iclk, *fclk; -#endif -void __iomem *io_base; -unsigned reserved:1 ; -unsigned enabled:1 ; -unsigned posted:1 ; -}; -static int dm_timer_count;#ifdef CONFIG_ARCH_OMAP1
arch/arm/plat-omap/include/plat/dmtimer.h +125 -0
130 lines not on this page · raw message
PATCH 3/10 omap: Move dmtimer defines to dmtimer.h
Tony Lindgren <[email protected]> writes:
[...]
arch/arm/plat-omap/include/plat/dmtimer.h void omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value · line 96
Quoted 10 lines
arch/arm/plat-omap/include/plat/dmtimer.h +2
-0@@ -92,5 +92,130 @@ void omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int valueint omap_dm_timers_active (void ); +/* +* Do not the defines below, they are not needed. They should be only
Do not what? ;)
Kevin
Tony Lindgren
maintainerPATCH 3/10 omap: Move dmtimer defines to dmtimer.h
Quoted 10 lines · Kevin Hilman
Tony Lindgren <[email protected]> writes:
[...]
Quoted 6 lines
arch/arm/plat-omap/include/plat/dmtimer.h +0
-0@@ -92,5 +92,130 @@ void omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value
Quoted 6 lines
arch/arm/plat-omap/include/plat/dmtimer.h void omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value · line 96
Quoted 4 lines
arch/arm/plat-omap/include/plat/dmtimer.h +2
-0int omap_dm_timers_active (void ); +/* +* Do not the defines below, they are not needed. They should be only Do not what? ;)
Looks like the nasty parts got censored ;) Should say "Do not use".
Tony
Tony Lindgren
maintainerPATCH 4/10 omap: Make a subset of dmtimer functions into inline functions
In reply to Tony LindgrenThis will allow us to share the code between system timer and dmtimer device driver code without having to initialize all dmtimers early. It will also make the timer_set_next_event more efficient as the inline functions will optimize the code better in this case.
--- arch/arm/plat-omap/dmtimer.c | 90 ++++++++++++++++------------- arch/arm/plat-omap/include/plat/dmtimer.h | 53 +++++++++++++++++ 2 files changed, 102 insertions(+), 41 deletions(-)
arch/arm/plat-omap/dmtimer.c +49 -41
@@ -170,11 +170,7 @@ static spinlock_t dm_timer_lock;*/ static inline u32 omap_dm_timer_read_reg (struct omap_dm_timer *timer,u32 reg) { -if (timer->posted) -while (readl (timer->io_base + (OMAP_TIMER_WRITE_PEND_REG &0xff )) - & (reg >>WPSHIFT )) -cpu_relax (); -return readl (timer->io_base + (reg &0xff )); +return __omap_dm_timer_read (timer->io_base, reg, timer->posted); }/*
@@ -186,11 +182,7 @@ static inline u32 omap_dm_timer_read_reg(struct omap_dm_timer *timer, u32 reg)static void omap_dm_timer_write_reg (struct omap_dm_timer *timer,u32 reg,u32 value) { -if (timer->posted) -while (readl (timer->io_base + (OMAP_TIMER_WRITE_PEND_REG &0xff )) - & (reg >>WPSHIFT )) -cpu_relax (); -writel (value, timer->io_base + (reg &0xff )); +__omap_dm_timer_write (timer->io_base, reg, value, timer->posted); }static void omap_dm_timer_wait_for_reset (struct omap_dm_timer *timer)
@@ -207,38 +199,54 @@ static void omap_dm_timer_wait_for_reset(struct omap_dm_timer *timer) } } -static void omap_dm_timer_reset (struct omap_dm_timer *timer) +/* Assumes the source clock has been set by caller */ +void __omap_dm_timer_reset (struct omap_dm_timer *timer,int autoidle, +int wakeup) {u32 l; + l =omap_dm_timer_read_reg (timer,OMAP_TIMER_OCP_CFG_REG ); + l |=0x02 <<3 ;/* Set to smart-idle mode */ + l |=0x2 <<8 ;/* Set clock activity to perserve f-clock on idle */ + +if (autoidle) + l |=0x1 <<0 ; + +if (wakeup) + l |=1 <<2 ; + +omap_dm_timer_write_reg (timer,OMAP_TIMER_OCP_CFG_REG , l); + +/* Match hardware reset default of posted mode */ +omap_dm_timer_write_reg (timer,OMAP_TIMER_IF_CTRL_REG , +OMAP_TIMER_CTRL_POSTED ); + timer->posted =1 ; +} + +static void omap_dm_timer_reset (struct omap_dm_timer *timer) +{ +int autoidle =0 , wakeup =0 ; +if (!cpu_class_is_omap2 () || timer != &dm_timers[0 ]) {omap_dm_timer_write_reg (timer,OMAP_TIMER_IF_CTRL_REG ,0x06 );omap_dm_timer_wait_for_reset (timer); }omap_dm_timer_set_source (timer,OMAP_TIMER_SRC_32_KHZ ); - l =omap_dm_timer_read_reg (timer,OMAP_TIMER_OCP_CFG_REG ); - l |=0x02 <<3 ;/* Set to smart-idle mode */ - l |=0x2 <<8 ;/* Set clock activity to perserve f-clock on idle */ -/* Enable autoidle on OMAP2 / OMAP3 */ if (cpu_is_omap24xx () ||cpu_is_omap34xx ()) - l |=0x1 <<0 ; + autoidle =1 ;/* * Enable wake-up on OMAP2 CPUs. */ if (cpu_class_is_omap2 ()) - l |=1 <<2 ; -omap_dm_timer_write_reg (timer,OMAP_TIMER_OCP_CFG_REG , l); + wakeup =1 ; -/* Match hardware reset default of posted mode */ -omap_dm_timer_write_reg (timer,OMAP_TIMER_IF_CTRL_REG , -OMAP_TIMER_CTRL_POSTED ); - timer->posted =1 ; +__omap_dm_timer_reset (timer, autoidle, wakeup); } -static void omap_dm_timer_prepare (struct omap_dm_timer *timer) +void omap_dm_timer_prepare (struct omap_dm_timer *timer) {omap_dm_timer_enable (timer);omap_dm_timer_reset (timer);
@@ -449,16 +457,13 @@ EXPORT_SYMBOL_GPL(omap_dm_timer_set_source);#else -int omap_dm_timer_set_source (struct omap_dm_timer *timer,int source) +int __omap_dm_timer_set_source (struct clk *timer_fck,struct clk *parent) { -int ret = -EINVAL ; - -if (source <0 || source >=3 ) -return -EINVAL ; +int ret; -clk_disable (timer->fclk); - ret =clk_set_parent (timer->fclk, dm_source_clocks[source]); -clk_enable (timer->fclk); +clk_disable (timer_fck); + ret =clk_set_parent (timer_fck, parent); +clk_enable (timer_fck);/* * When the functional clock disappears, too quick writes seem
@@ -468,6 +473,15 @@ int omap_dm_timer_set_source(struct omap_dm_timer *timer, int source) return ret; } + +int omap_dm_timer_set_source (struct omap_dm_timer *timer,int source) +{ +if (source <0 || source >=3 ) +return -EINVAL ; + +return __omap_dm_timer_set_source (timer->fclk, + dm_source_clocks[source]); +}EXPORT_SYMBOL_GPL (omap_dm_timer_set_source);#endif
@@ -504,8 +518,7 @@ void omap_dm_timer_set_load_start(struct omap_dm_timer *timer, int autoreload, } l |=OMAP_TIMER_CTRL_ST ; -omap_dm_timer_write_reg (timer,OMAP_TIMER_COUNTER_REG , load); -omap_dm_timer_write_reg (timer,OMAP_TIMER_CTRL_REG , l); +__omap_dm_timer_load_start (timer->io_base, l, load, timer->posted); }EXPORT_SYMBOL_GPL (omap_dm_timer_set_load_start);
@@ -558,8 +571,7 @@ EXPORT_SYMBOL_GPL(omap_dm_timer_set_prescaler);void omap_dm_timer_set_int_enable (struct omap_dm_timer *timer,unsigned int value) { -omap_dm_timer_write_reg (timer,OMAP_TIMER_INT_EN_REG , value); -omap_dm_timer_write_reg (timer,OMAP_TIMER_WAKEUP_EN_REG , value); +__omap_dm_timer_int_enable (timer->io_base, value); }EXPORT_SYMBOL_GPL (omap_dm_timer_set_int_enable);
@@ -575,17 +587,13 @@ EXPORT_SYMBOL_GPL(omap_dm_timer_read_status);void omap_dm_timer_write_status (struct omap_dm_timer *timer,unsigned int value) { -omap_dm_timer_write_reg (timer,OMAP_TIMER_STAT_REG , value); +__omap_dm_timer_write_status (timer->io_base, value); }EXPORT_SYMBOL_GPL (omap_dm_timer_write_status);unsigned int omap_dm_timer_read_counter (struct omap_dm_timer *timer) { -unsigned int l; - - l =omap_dm_timer_read_reg (timer,OMAP_TIMER_COUNTER_REG ); - -return l; +return __omap_dm_timer_read_counter (timer->io_base, timer->posted); }EXPORT_SYMBOL_GPL (omap_dm_timer_read_counter);
arch/arm/plat-omap/include/plat/dmtimer.h +53 -0
57 lines not on this page · raw message
PATCH 4/10 omap: Make a subset of dmtimer functions into inline functions
Quoted 7 lines · Tony Lindgren
This will allow us to share the code between system timer and dmtimer device driver code without having to initialize all dmtimers early. It will also make the
timer_set_next_eventmore efficient as the inline functions will optimize the code better in this case.Signed-off-by Tony Lindgren <[email protected]>
[...]
Quoted 14 lines
arch/arm/plat-omap/dmtimer.c +10
-1-static void omap_dm_timer_reset (struct omap_dm_timer *timer) +/* Assumes the source clock has been set by caller */ +void __omap_dm_timer_reset (struct omap_dm_timer *timer,int autoidle, +int wakeup) {u32 l; + l =omap_dm_timer_read_reg (timer,OMAP_TIMER_OCP_CFG_REG ); + l |=0x02 <<3 ;/* Set to smart-idle mode */ + l |=0x2 <<8 ;/* Set clock activity to perserve f-clock on idle */ + +if (autoidle) + l |=0x1 <<0 ; +
arch/arm/plat-omap/dmtimer.c static void omap_dm_timer_wait_for_reset(struct omap_dm_timer *timer) · line 224
Quoted 10 lines
arch/arm/plat-omap/dmtimer.c +10
-0+if (wakeup) + l |=1 <<2 ; + +omap_dm_timer_write_reg (timer,OMAP_TIMER_OCP_CFG_REG , l); + +/* Match hardware reset default of posted mode */ +omap_dm_timer_write_reg (timer,OMAP_TIMER_IF_CTRL_REG , +OMAP_TIMER_CTRL_POSTED ); + timer->posted =1 ; +}
The reset code is an example of something that will not be able to be shared between a system timer driver and a real device driver. In the real driver, the reset (as well as smart-idle, autoidle, wakeup capability, etc.) will all be handled by the hwmod.
With a hwmod conversion, the system timer will have to have duplicate/alternate compared to the real timer.
Ideally, what we need is a way for the system timer to be early_init only. When the real driver is available, it switches to that. This could probably be done pretty easily by using the 'rating' field of the clockevent so when the "real" timer driver becomes available with a higher rating, the clockevent code would switch to it.
Kevin
Tony Lindgren
maintainerPATCH 4/10 omap: Make a subset of dmtimer functions into inline functions
Quoted 8 lines · Kevin Hilman
The reset code is an example of something that will not be able to be shared between a system timer driver and a real device driver. In the real driver, the reset (as well as smart-idle, autoidle, wakeup capability, etc.) will all be handled by the hwmod.
With a hwmod conversion, the system timer will have to have duplicate/alternate compared to the real timer.
Well the hwmod is already there also for system timer, so that should not be a problem. See the init_one function in the next patch in the series.
Quoted 5 lines
Ideally, what we need is a way for the system timer to be
early_initonly. When the real driver is available, it switches to that. This could probably be done pretty easily by using the 'rating' field of the clockevent so when the "real" timer driver becomes available with a higher rating, the clockevent code would switch to it.
I don't think this is needed. When the dmtimer driver code initializes it just picks up the already initialized struct omap_dm_timer entries but does not reset them. At that point the struct dev entry can be created, but from system timer point of view nothing changes.
Regards,
Tony
PATCH 4/10 omap: Make a subset of dmtimer functions into inline functions
Quoted 7 lines · Tony Lindgren
Quoted 6 lines · Kevin Hilman
The reset code is an example of something that will not be able to be shared between a system timer driver and a real device driver. In the real driver, the reset (as well as smart-idle, autoidle, wakeup capability, etc.) will all be handled by the hwmod.
Quoted 6 lines
Quoted 2 lines
With a hwmod conversion, the system timer will have to have duplicate/alternate compared to the real timer.
Well the hwmod is already there also for system timer, so that should not be a problem. See the
init_onefunction in the next patch in the series.
OK, I see that now. Guess I should get through the whole series before responding.
So Tarun's series should convert then convert the reset to using the equivalent hwmod functions.
Kevin
Tony Lindgren
maintainerPATCH 4/10 omap: Make a subset of dmtimer functions into inline functions
Quoted 16 lines · Kevin Hilman
Quoted 7 lines · Tony Lindgren
Quoted 6 lines · Kevin Hilman
The reset code is an example of something that will not be able to be shared between a system timer driver and a real device driver. In the real driver, the reset (as well as smart-idle, autoidle, wakeup capability, etc.) will all be handled by the hwmod.
Quoted 6 lines
Quoted 2 lines
With a hwmod conversion, the system timer will have to have duplicate/alternate compared to the real timer.
Well the hwmod is already there also for system timer, so that should not be a problem. See the
init_onefunction in the next patch in the series.
Quoted 5 lines
OK, I see that now. Guess I should get through the whole series before responding.
So Tarun's series should convert then convert the reset to using the equivalent hwmod functions.
Yeh, that should also get rid of the clock names in the init_one function.
Regards,
Tony
Tony Lindgren
maintainerPATCH 5/10 omap2+: Use dmtimer macros for clockevent
In reply to Tony LindgrenThis patch makes timer-gp.c to use only a subset of dmtimer functions without the need to initialize dmtimer code early.
Note that omap_dmtimer_init_one can eventually be moved to omap2+ specific dmtimer.c.
Also note that now with the inline functions, timer_set_next_event becomes more efficient in the lines of assembly code.
--- arch/arm/mach-omap2/timer-gp.c | 145 ++++++++++++++++++++++------- arch/arm/plat-omap/include/plat/dmtimer.h | 1 2 files changed, 111 insertions(+), 35 deletions(-)
arch/arm/mach-omap2/timer-gp.c +110 -35
@@ -45,10 +45,33 @@#include "timer-gp.h" +/* Parent clocks, eventually these will come from the clock framework */ + +#define OMAP2_MPU_SOURCE "sys_ck" +#define OMAP3_MPU_SOURCE OMAP2_MPU_SOURCE +#define OMAP4_MPU_SOURCE "sys_clkin_ck" +#define OMAP2_32K_SOURCE "func_32k_ck" +#define OMAP3_32K_SOURCE "omap_32k_fck" +#define OMAP4_32K_SOURCE "sys_32k_ck" + +#ifdef CONFIG_OMAP_32K_TIMER +#define OMAP2_CLKEV_SOURCE OMAP2_32K_SOURCE +#define OMAP3_CLKEV_SOURCE OMAP3_32K_SOURCE +#define OMAP4_CLKEV_SOURCE OMAP4_32K_SOURCE +#define OMAP3_BEAGLE_TIMER 12 +#else +#define OMAP2_CLKEV_SOURCE OMAP2_MPU_SOURCE +#define OMAP3_CLKEV_SOURCE OMAP3_MPU_SOURCE +#define OMAP4_CLKEV_SOURCE OMAP4_MPU_SOURCE +#define OMAP3_BEAGLE_TIMER 1 +#endif /* MAX_GPTIMER_ID: number of GPTIMERs on the chip */ #define MAX_GPTIMER_ID 12 +/* Clockevent code */ + +static struct omap_dm_timer clkev;static struct omap_dm_timer *gptimer;static struct clock_event_device clockevent_gpt;static u8 __initdata gptimer_id =1 ;
@@ -57,10 +80,9 @@ struct omap_dm_timer *gptimer_wakeup;static irqreturn_t omap2_gp_timer_interrupt (int irq,void *dev_id) { -struct omap_dm_timer *gpt = (struct omap_dm_timer *)dev_id;struct clock_event_device *evt = &clockevent_gpt; -omap_dm_timer_write_status (gpt,OMAP_TIMER_INT_OVERFLOW ); +__omap_dm_timer_write_status (clkev.io_base,OMAP_TIMER_INT_OVERFLOW ); evt->event_handler (evt);return IRQ_HANDLED ;
@@ -75,7 +97,8 @@ static struct irqaction omap2_gp_timer_irq = {static int omap2_gp_timer_set_next_event (unsigned long cycles,struct clock_event_device *evt) { -omap_dm_timer_set_load_start (gptimer,0 ,0xffffffff - cycles); +__omap_dm_timer_load_start (clkev.io_base,OMAP_TIMER_CTRL_ST , +0xffffffff - cycles,1 );return 0 ; }
@@ -85,13 +108,13 @@ static void omap2_gp_timer_set_mode(enum clock_event_mode mode, {u32 period; -omap_dm_timer_stop (gptimer); +omap_dm_timer_stop (&clkev);switch (mode) {case CLOCK_EVT_MODE_PERIODIC : - period =clk_get_rate (omap_dm_timer_get_fclk (gptimer)) /HZ ; + period = clkev.rate /HZ ; period -=1 ; -omap_dm_timer_set_load_start (gptimer,1 ,0xffffffff - period); +omap_dm_timer_set_load_start (&clkev,1 ,0xffffffff - period);break ;case CLOCK_EVT_MODE_ONESHOT :break ;
@@ -130,43 +153,88 @@ int __init omap2_gp_clockevent_set_gptimer(u8 id)return 0 ; } -static void __init omap2_gp_clockevent_init (void ) +static int __init omap_dm_timer_init_one (struct omap_dm_timer *timer, +int gptimer_id, +const char *fck_source) { -u32 tick_rate; -int src; -char clockevent_hwmod_name[8 ];/* 8 = sizeof("timerXX0") */ - - inited =1 ; +char name[10 ];/* 10 = sizeof("gptXX_Xck0") */ +struct omap_hwmod *oh; +size_t size; +int res =0 ; + +sprintf (name,"timer%d" , gptimer_id); +omap_hwmod_setup_one (name); + oh =omap_hwmod_lookup (name); +if (!oh) +return -ENODEV ; + + timer->irq = oh->mpu_irqs[0 ].irq; + timer->phys_base = oh->slaves[0 ]->addr->pa_start; + size = oh->slaves[0 ]->addr->pa_end - timer->phys_base; + +/* Static mapping, never released */ + timer->io_base =ioremap (timer->phys_base, size); +if (!timer->io_base) +return -ENXIO ; + +/* After the dmtimer is using hwmod these clocks won't be needed */ +sprintf (name,"gpt%d_fck" , gptimer_id); + timer->fclk =clk_get (NULL , name); +if (IS_ERR (timer->fclk)) +return -ENODEV ; + +sprintf (name,"gpt%d_ick" , gptimer_id); + timer->iclk =clk_get (NULL , name); +if (IS_ERR (timer->iclk)) { +clk_put (timer->fclk); +return -ENODEV ; + } -sprintf (clockevent_hwmod_name,"timer%d" , gptimer_id); -omap_hwmod_setup_one (clockevent_hwmod_name); +omap_hwmod_enable (oh); gptimer =omap_dm_timer_request_specific (gptimer_id);BUG_ON (gptimer ==NULL ); gptimer_wakeup = gptimer; -#if defined(CONFIG_OMAP_32K_TIMER ) - src =OMAP_TIMER_SRC_32_KHZ ; -#else - src =OMAP_TIMER_SRC_SYS_CLK ; -WARN (gptimer_id ==12 ,"WARNING: GPTIMER12 can only use the " -"secure 32KiHz clock source\n" ); -#endif +if (gptimer_id !=12 ) { +struct clk *src; + + src =clk_get (NULL , fck_source); +if (IS_ERR (src)) { + res = -EINVAL ; + }else { + res =__omap_dm_timer_set_source (timer->fclk, src); +if (IS_ERR_VALUE (res)) +pr_warning ("%s: timer%i cannot set source\n" , + __func__, gptimer_id); +clk_put (src); + } + } +__omap_dm_timer_reset (timer,1 ,1 ); + + timer->rate =clk_get_rate (timer->fclk); + + timer->reserved =1 ; -if (gptimer_id !=12 ) -WARN (IS_ERR_VALUE (omap_dm_timer_set_source (gptimer, src)), -"timer-gp: omap_dm_timer_set_source() failed\n" ); +return res; +} - tick_rate =clk_get_rate (omap_dm_timer_get_fclk (gptimer)); +static void __init omap2_gp_clockevent_init (int gptimer_id, +const char *fck_source) +{ +int res; -pr_info ("OMAP clockevent source: GPTIMER%d at %u Hz\n" , - gptimer_id, tick_rate); + inited =1 ; + + res =omap_dm_timer_init_one (&clkev, gptimer_id, fck_source); +BUG_ON (res); omap2_gp_timer_irq.dev_id = (void *)gptimer; -setup_irq (omap_dm_timer_get_irq (gptimer), &omap2_gp_timer_irq); -omap_dm_timer_set_int_enable (gptimer,OMAP_TIMER_INT_OVERFLOW ); +setup_irq (clkev.irq, &omap2_gp_timer_irq); + +__omap_dm_timer_int_enable (clkev.io_base,OMAP_TIMER_INT_OVERFLOW ); - clockevent_gpt.mult =div_sc (tick_rate,NSEC_PER_SEC , + clockevent_gpt.mult =div_sc (clkev.rate,NSEC_PER_SEC , clockevent_gpt.shift); clockevent_gpt.max_delta_ns =clockevent_delta2ns (0xffffffff , &clockevent_gpt);
@@ -176,6 +244,9 @@ static void __init omap2_gp_clockevent_init(void) clockevent_gpt.cpumask =cpumask_of (0 );clockevents_register_device (&clockevent_gpt); + +pr_info ("OMAP clockevent source: GPTIMER%d at %lu Hz\n" , + gptimer_id, clkev.rate); }/* Clocksource code */
@@ -251,7 +322,7 @@ static void __init omap2_gp_clocksource_init(void)static void __init omap242x_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -262,7 +333,7 @@ struct sys_timer omap242x_timer = {static void __init omap243x_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -275,7 +346,7 @@ struct sys_timer omap243x_timer = {static void __init omap3_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (1 ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -283,10 +354,14 @@ struct sys_timer omap3_timer = { .init = omap3_timer_init, }; +/* +* Beagle based designs typically have an issue with gptimer1. Also note +* that GPTIMER12 can only use the secure 32KiHz clock source. +*/ static void __init omap3_beagle_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (OMAP3_BEAGLE_TIMER ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -303,7 +378,7 @@ static void __init omap4_timer_init(void)BUG_ON (!twd_base);#endif omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (1 ,OMAP4_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
arch/arm/plat-omap/include/plat/dmtimer.h +1 -0
@@ -213,6 +213,7 @@ struct omap_dm_timer {struct clk *iclk, *fclk;#endif void __iomem *io_base; +unsigned long rate; unsigned reserved:1; unsigned enabled:1; unsigned posted:1;
Tony Lindgren
maintainerPATCH 5/10 omap2+: Use dmtimer macros for clockevent
Quoted 3 lines · Tony Lindgren
Note that
omap_dmtimer_init_onecan eventually be moved to omap2+ specificdmtimer.c.
Actually after this series we'll have just timer.c, will remove this comment.
Tony
PATCH 5/10 omap2+: Use dmtimer macros for clockevent
In reply to Tony LindgrenQuoted 10 lines · Tony Lindgren
This patch makes
timer-gp.cto use only a subset of dmtimer functions without the need to initialize dmtimer code early.Note that
omap_dmtimer_init_onecan eventually be moved to omap2+ specificdmtimer.c.Also note that now with the inline functions,
timer_set_next_eventbecomes more efficient in the lines of assembly code.Signed-off-by Tony Lindgren <[email protected]>
In looking closer at this patch, I have a few questions.
[...]
arch/arm/mach-omap2/timer-gp.c static struct irqaction omap2_gp_timer_irq = { · line 103
Quoted 9 lines
arch/arm/mach-omap2/timer-gp.c +2
-1@@ -75,7 +97,8 @@ static struct irqaction omap2_gp_timer_irq = {static int omap2_gp_timer_set_next_event (unsigned long cycles,struct clock_event_device *evt) { -omap_dm_timer_set_load_start (gptimer,0 ,0xffffffff - cycles); +__omap_dm_timer_load_start (clkev.io_base,OMAP_TIMER_CTRL_ST , +0xffffffff - cycles,1 );return 0 ;
The creation of macros is causing some readability concern, at least for me...
In creating the macro versions of some of the functions, the macro version actually has different behavior which makes reading the code a little confusing.
I just noticed that the macro version of _load_start() doesn't actually do the "start", so you added the OMAP_TIMER_CTRL_ST when calling it. To do this, the macro version takes a 'ctrl' argument where as the "real" version only takes the autoreload argument.
If you're going to keep the same function name (but just add the __ prefix, I would expect that the functionality of the function doesn't change.
If the functionality of the macro is different from the "real" function,
it should probably just be given a different name. In this case,
probably dropping the _start suffix is probably enough.
Quoted 5 lines
arch/arm/mach-omap2/timer-gp.c +0
-0}@@ -85,13 +108,13 @@ static void omap2_gp_timer_set_mode(enum clock_event_mode mode, {u32 period;
arch/arm/mach-omap2/timer-gp.c static void omap2_gp_timer_set_mode(enum clock_event_mode mode, · line 117
Quoted 10 lines
arch/arm/mach-omap2/timer-gp.c +3
-3-omap_dm_timer_stop (gptimer); +omap_dm_timer_stop (&clkev);switch (mode) {case CLOCK_EVT_MODE_PERIODIC : - period =clk_get_rate (omap_dm_timer_get_fclk (gptimer)) /HZ ; + period = clkev.rate /HZ ; period -=1 ; -omap_dm_timer_set_load_start (gptimer,1 ,0xffffffff - period); +omap_dm_timer_set_load_start (&clkev,1 ,0xffffffff - period);
Hmm, you're using the driver function here not the macro. Is that intended?
Why not use the macro version here with OMAP_TIMER_CTRL_AR | OMAP_TIMER_CTRL_ST.
Kevin
Tony Lindgren
maintainerPATCH 5/10 omap2+: Use dmtimer macros for clockevent
Quoted 39 lines · Kevin Hilman
Quoted 10 lines · Tony Lindgren
This patch makes
timer-gp.cto use only a subset of dmtimer functions without the need to initialize dmtimer code early.Note that
omap_dmtimer_init_onecan eventually be moved to omap2+ specificdmtimer.c.Also note that now with the inline functions,
timer_set_next_eventbecomes more efficient in the lines of assembly code.Signed-off-by Tony Lindgren <[email protected]>In looking closer at this patch, I have a few questions.
[...]
arch/arm/mach-omap2/timer-gp.c static struct irqaction omap2_gp_timer_irq = { · line 103
Quoted 9 lines
arch/arm/mach-omap2/timer-gp.c +2
-1@@ -75,7 +97,8 @@ static struct irqaction omap2_gp_timer_irq = {static int omap2_gp_timer_set_next_event (unsigned long cycles,struct clock_event_device *evt) { -omap_dm_timer_set_load_start (gptimer,0 ,0xffffffff - cycles); +__omap_dm_timer_load_start (clkev.io_base,OMAP_TIMER_CTRL_ST , +0xffffffff - cycles,1 );return 0 ;The creation of macros is causing some readability concern, at least for me...
In creating the macro versions of some of the functions, the macro version actually has different behavior which makes reading the code a little confusing.
I just noticed that the macro version of
_load_start()doesn't actually do the "start", so you added theOMAP_TIMER_CTRL_STwhen calling it. To do this, the macro version takes a 'ctrl' argument where as the "real" version only takes the autoreload argument.
Quoted 7 lines
If you're going to keep the same function name (but just add the __ prefix, I would expect that the functionality of the function doesn't change.
If the functionality of the macro is different from the "real" function,
it should probably just be given a different name. In this case,
probably dropping the_startsuffix is probably enough.
OK good point.
Quoted 19 lines
Quoted 5 lines
arch/arm/mach-omap2/timer-gp.c +0
-0}@@ -85,13 +108,13 @@ static void omap2_gp_timer_set_mode(enum clock_event_mode mode, {u32 period;arch/arm/mach-omap2/timer-gp.c static void omap2_gp_timer_set_mode(enum clock_event_mode mode, · line 117
Quoted 10 lines
arch/arm/mach-omap2/timer-gp.c +3
-3-omap_dm_timer_stop (gptimer); +omap_dm_timer_stop (&clkev);switch (mode) {case CLOCK_EVT_MODE_PERIODIC : - period =clk_get_rate (omap_dm_timer_get_fclk (gptimer)) /HZ ; + period = clkev.rate /HZ ; period -=1 ; -omap_dm_timer_set_load_start (gptimer,1 ,0xffffffff - period); +omap_dm_timer_set_load_start (&clkev,1 ,0xffffffff - period);Hmm, you're using the driver function here not the macro. Is that intended?
Quoted 2 lines
Why not use the macro version here with
OMAP_TIMER_CTRL_AR|OMAP_TIMER_CTRL_ST.
Hmm I guess I tried to change as little as possible here when I noticed that there's some weirdness with the autoreload mode in the original omap_dm_timer_set_load_start function that requires writing twice in the autoreload case. Looks like that's not needed for the one-shot case.
Anyways, you're right, this should not be using the driver function to move the rest of the timers to be under drivers/ eventually. Will post an updated version after taking another look at the dmtimer hwmod series.
Regards,
Tony
Tony Lindgren
maintainerPATCH 6/10 omap2+: Remove gptimer_wakekup for now
In reply to Tony LindgrenThis removes the support for setting the wake-up timer for debugging.
Later on we can reserve gptimer1 for PM code only and have similar functionality.
--- arch/arm/mach-omap2/pm-debug.c | 27 --------------------------- arch/arm/mach-omap2/pm.h | 6 ------ arch/arm/mach-omap2/pm34xx.c | 4 ---- arch/arm/mach-omap2/timer-gp.c | 8 +------- 4 files changed, 1 insertions(+), 44 deletions(-)
arch/arm/mach-omap2/pm-debug.c +0 -27
@@ -41,8 +41,6 @@int omap2_pm_debug;u32 enable_off_mode;u32 sleep_while_idle; -u32 wakeup_timer_seconds; -u32 wakeup_timer_milliseconds;#define DUMP_PRM_MOD_REG (mod, reg) \ regs[reg_count].name = #mod "." #reg; \
@@ -162,23 +160,6 @@ void omap2_pm_dump(int mode, int resume, unsigned int us)printk (KERN_INFO "%-20s: 0x%08x\n" , regs[i].name, regs[i].val); } -void omap2_pm_wakeup_on_timer (u32 seconds,u32 milliseconds) -{ -u32 tick_rate, cycles; - -if (!seconds && !milliseconds) -return ; - - tick_rate =clk_get_rate (omap_dm_timer_get_fclk (gptimer_wakeup)); - cycles = tick_rate * seconds + tick_rate * milliseconds /1000 ; -omap_dm_timer_stop (gptimer_wakeup); -omap_dm_timer_set_load_start (gptimer_wakeup,0 ,0xffffffff - cycles); - -pr_info ("PM: Resume timer in %u.%03u secs" -" (%d ticks at %d ticks/sec.)\n" , - seconds, milliseconds, cycles, tick_rate); -} -#ifdef CONFIG_DEBUG_FS #include <linux/debugfs.h> #include <linux/seq_file.h>
@@ -576,9 +557,6 @@ static int option_set(void *data, u64 val) {u32 *option = data; -if (option == &wakeup_timer_milliseconds && val >=1000 ) -return -EINVAL ; - *option = val;if (option == &enable_off_mode) {
@@ -641,11 +619,6 @@ static int __init pm_dbg_init(void) &enable_off_mode, &pm_dbg_option_fops); (void )debugfs_create_file ("sleep_while_idle" ,S_IRUGO |S_IWUSR , d, &sleep_while_idle, &pm_dbg_option_fops); - (void )debugfs_create_file ("wakeup_timer_seconds" ,S_IRUGO |S_IWUSR , d, - &wakeup_timer_seconds, &pm_dbg_option_fops); - (void )debugfs_create_file ("wakeup_timer_milliseconds" , -S_IRUGO |S_IWUSR , d, &wakeup_timer_milliseconds, - &pm_dbg_option_fops); pm_dbg_init_done =1 ;return 0 ;
arch/arm/mach-omap2/pm.h +0 -6
@@ -55,19 +55,13 @@ inline void omap3_pm_init_cpuidle(struct cpuidle_params *cpuidle_board_params)extern int omap3_pm_get_suspend_state (struct powerdomain *pwrdm);extern int omap3_pm_set_suspend_state (struct powerdomain *pwrdm,int state); -extern u32 wakeup_timer_seconds; -extern u32 wakeup_timer_milliseconds; -extern struct omap_dm_timer *gptimer_wakeup; -#ifdef CONFIG_PM_DEBUG extern void omap2_pm_dump (int mode,int resume,unsigned int us); -extern void omap2_pm_wakeup_on_timer (u32 seconds,u32 milliseconds);extern int omap2_pm_debug;extern u32 enable_off_mode;extern u32 sleep_while_idle;#else #define omap2_pm_dump (mode, resume, us) do {} while (0); -#define omap2_pm_wakeup_on_timer (seconds, milliseconds) do {} while (0);#define omap2_pm_debug 0#define enable_off_mode 0#define sleep_while_idle 0
arch/arm/mach-omap2/pm34xx.c +0 -4
@@ -534,10 +534,6 @@ static int omap3_pm_suspend(void)struct power_state *pwrst;int state, ret =0 ; -if (wakeup_timer_seconds || wakeup_timer_milliseconds) -omap2_pm_wakeup_on_timer (wakeup_timer_seconds, - wakeup_timer_milliseconds); -/* Read current next_pwrsts */ list_for_each_entry (pwrst, &pwrst_list, node) pwrst->saved_state =pwrdm_read_next_pwrst (pwrst->pwrdm);
arch/arm/mach-omap2/timer-gp.c +1 -7
@@ -72,11 +72,9 @@/* Clockevent code */ static struct omap_dm_timer clkev; -static struct omap_dm_timer *gptimer;static struct clock_event_device clockevent_gpt;static u8 __initdata gptimer_id =1 ;static u8 __initdata inited; -struct omap_dm_timer *gptimer_wakeup;static irqreturn_t omap2_gp_timer_interrupt (int irq,void *dev_id) {
@@ -192,10 +190,6 @@ static int __init omap_dm_timer_init_one(struct omap_dm_timer *timer,omap_hwmod_enable (oh); - gptimer =omap_dm_timer_request_specific (gptimer_id); -BUG_ON (gptimer ==NULL ); - gptimer_wakeup = gptimer; -if (gptimer_id !=12 ) {struct clk *src;
@@ -229,7 +223,7 @@ static void __init omap2_gp_clockevent_init(int gptimer_id, res =omap_dm_timer_init_one (&clkev, gptimer_id, fck_source);BUG_ON (res); - omap2_gp_timer_irq.dev_id = (void *)gptimer; + omap2_gp_timer_irq.dev_id = (void *)&clkev;setup_irq (clkev.irq, &omap2_gp_timer_irq);__omap_dm_timer_int_enable (clkev.io_base,OMAP_TIMER_INT_OVERFLOW );
PATCH 6/10 omap2+: Remove gptimer_wakekup for now
Quoted 6 lines · Tony Lindgren
This removes the support for setting the wake-up timer for debugging.
Later on we can reserve gptimer1 for PM code only and have similar functionality.
Signed-off-by Tony Lindgren <[email protected]>
While we work on an alternative, rather than completely remove this functionality, below is a very small patch (replaces $SUBJECT patch) that will keep the current wakeup-from-suspend timer for PM debug working.
Note that GPT1 fs not just used for wakeups from suspend. GPT1 needs to also be the clockevent (at least during idle) so that next-timer interrupts during idle are also programmed for GPT1.
Here is what I see as a possible "real" solution. Let's see if we're on the same page.
- GPT1 reserved for "special" PM wakeup
- GPT2 used as high-resolution clockevent (using sys_clk, but stops during idle)
- GPT3 (or counter_32k) used as clocksource depending on Kconfig
Whenever we're going idle (or suspend), we have to effectively switch the clockevent from GPT2 to GPT1. I assume this is what you have in mind as well.
We'll need to dig into the clockevent (and tick broadcast) code to get this to work on UP. On SMP, the C3STOP flag is used to signify that at clockevent will stop during specific power states, so an alternate clockevent is used, but IIUC, this doesn't currently work the same on UP. I think Santosh has looked into this more recently than I have. Santosh, if you have any recent status on this, could you share? I'll gladly work on the clockevent layer if necessary for this.
Tony, in the mean time, can you use something like the patch below instead of $SUBJECT patch? At least we won't loose functionality while working on this feature.
Kevin
PATCH 6/10 omap2+: Remove gptimer_wakekup for now
Kevin,
Quoted 29 lines · Kevin Hilman
Quoted 6 lines · Tony Lindgren
This removes the support for setting the wake-up timer for debugging.
Later on we can reserve gptimer1 for PM code only and have similar functionality.
Signed-off-by Tony Lindgren<[email protected]>While we work on an alternative, rather than completely remove this functionality, below is a very small patch (replaces $SUBJECT patch) that will keep the current wakeup-from-suspend timer for PM debug working.
Note that GPT1 fs not just used for wakeups from suspend. GPT1 needs to also be the clockevent (at least during idle) so that next-timer interrupts during idle are also programmed for GPT1.
Here is what I see as a possible "real" solution. Let's see if we're on the same page.
- GPT1 reserved for "special" PM wakeup
- GPT2 used as high-resolution clockevent (usingsys_clk, but stops during idle)
- GPT3 (orcounter_32k) used as clocksource depending onKconfigWhenever we're going idle (or suspend), we have to effectively switch the clockevent from GPT2 to GPT1. I assume this is what you have in mind as well.
Quoted 8 lines
We'll need to dig into the clockevent (and tick broadcast) code to get this to work on UP. On SMP, the C3STOP flag is used to signify that at clockevent will stop during specific power states, so an alternate clockevent is used, but IIUC, this doesn't currently work the same on UP. I think Santosh has looked into this more recently than I have. Santosh, if you have any recent status on this, could you share? I'll gladly work on the clockevent layer if necessary for this.
For OMAP4, it's already part of OMAP4 PM series as you know. For OMAP3 as well I had cooked up couple of patches some time back to get clock event switching working.
Will find and post them soon.
Regards
Santosh
Tony Lindgren
maintainerPATCH 7/10 omap2+: Reserve clocksource and timesource and initialize dmtimer later
In reply to Tony LindgrenThere's no need to initialize the dmtimer framework early. Just mark the clocksource and timesource as reserved, and initialize dmtimer with an arch_initcall.
--- arch/arm/mach-omap1/timer32k.c | 4 ---- arch/arm/mach-omap2/timer-gp.c | 9 ++++----- arch/arm/plat-omap/dmtimer.c | 8 ++++++++ arch/arm/plat-omap/include/plat/dmtimer.h | 1 + 4 files changed, 13 insertions(+), 9 deletions(-)
arch/arm/mach-omap1/timer32k.c +0 -4
@@ -183,10 +183,6 @@ static __init void omap_init_32k_timer(void)bool __init omap_32k_timer_init (void ) {omap_init_clocksource_32k (); - -#ifdef CONFIG_OMAP_DM_TIMER -omap_dm_timer_init (); -#endif omap_init_32k_timer ();return true;
arch/arm/mach-omap2/timer-gp.c +4 -5
@@ -69,6 +69,8 @@/* MAX_GPTIMER_ID: number of GPTIMERs on the chip */ #define MAX_GPTIMER_ID 12 +u32 sys_timer_reserved; +/* Clockevent code */ static struct omap_dm_timer clkev;
@@ -190,6 +192,8 @@ static int __init omap_dm_timer_init_one(struct omap_dm_timer *timer,omap_hwmod_enable (oh); + sys_timer_reserved |= (1 << (gptimer_id -1 )); +if (gptimer_id !=12 ) {struct clk *src;
@@ -315,7 +319,6 @@ static void __init omap2_gp_clocksource_init(void)#ifdef CONFIG_ARCH_OMAP2 static void __init omap242x_timer_init (void ) { -omap_dm_timer_init ();omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -326,7 +329,6 @@ struct sys_timer omap242x_timer = {static void __init omap243x_timer_init (void ) { -omap_dm_timer_init ();omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -339,7 +341,6 @@ struct sys_timer omap243x_timer = {#ifdef CONFIG_ARCH_OMAP3 static void __init omap3_timer_init (void ) { -omap_dm_timer_init ();omap2_gp_clockevent_init (1 ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -354,7 +355,6 @@ struct sys_timer omap3_timer = {*/ static void __init omap3_beagle_timer_init (void ) { -omap_dm_timer_init ();omap2_gp_clockevent_init (OMAP3_BEAGLE_TIMER ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
@@ -371,7 +371,6 @@ static void __init omap4_timer_init(void) twd_base =ioremap (OMAP44XX_LOCAL_TWD_BASE ,SZ_256 );BUG_ON (!twd_base);#endif -omap_dm_timer_init ();omap2_gp_clockevent_init (1 ,OMAP4_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
arch/arm/plat-omap/dmtimer.c +8 -0
@@ -677,8 +677,16 @@ int __init omap_dm_timer_init(void)sprintf (clk_name,"gpt%d_fck" , i +1 ); timer->fclk = clk_get(NULL, clk_name); } + +/* One or two timers may be set up early for sys_timer */ +if (sys_timer_reserved & (1 << i)) { + timer->reserved =1 ; + timer->posted =1 ; + }#endif }return 0 ; } + +arch_initcall (omap_dm_timer_init);
arch/arm/plat-omap/include/plat/dmtimer.h +1 -0
@@ -219,6 +219,7 @@ struct omap_dm_timer { unsigned posted:1; }; +extern u32 sys_timer_reserved;void __omap_dm_timer_reset (struct omap_dm_timer *timer,int autoidle,int wakeup);void omap_dm_timer_prepare (struct omap_dm_timer *timer);
Tony Lindgren
maintainerPATCH 8/10 omap2+: Use dmtimer macros for clocksource
In reply to Tony LindgrenUse dmtimer macros for clocksource. As with the clockevent, this allows us to initialize the rest of dmtimer code later on.
Note that eventually we will be initializing the timesource from init_early so sched_clock will work properly for CONFIG_PRINTK_TIME.
--- arch/arm/mach-omap2/timer-gp.c | 60 +++++++++++++++++++++----------------- arch/arm/plat-omap/counter_32k.c | 2 + 2 files changed, 34 insertions(+), 28 deletions(-)
arch/arm/mach-omap2/timer-gp.c +33 -27
@@ -256,20 +256,22 @@ static void __init omap2_gp_clockevent_init(int gptimer_id,* sync counter. See clocksource setup in plat-omap/counter_32k.c */ -static void __init omap2_gp_clocksource_init (void ) +static void __init omap2_gp_clocksource_init (int unused,const char *dummy) {omap_init_clocksource_32k (); }#else + +static struct omap_dm_timer clksrc; +/* * clocksource */ static DEFINE_CLOCK_DATA (cd); -static struct omap_dm_timer *gpt_clocksource;static cycle_t clocksource_read_cycles (struct clocksource *cs) { -return (cycle_t )omap_dm_timer_read_counter (gpt_clocksource); +return (cycle_t )__omap_dm_timer_read_counter (clksrc.io_base,1 ); } static struct clocksource clocksource_gpt = {
@@ -284,35 +286,39 @@ static void notrace dmtimer_update_sched_clock(void) {u32 cyc; - cyc =omap_dm_timer_read_counter (gpt_clocksource); + cyc =__omap_dm_timer_read_counter (clksrc.io_base,1 );update_sched_clock (&cd, cyc, (u32 )~0 ); } -/* Setup free-running counter for clocksource */ -static void __init omap2_gp_clocksource_init (void ) +unsigned long long notrace sched_clock (void ) { -static struct omap_dm_timer *gpt; -u32 tick_rate; -static char err1[]__initdata =KERN_ERR -"%s: failed to request dm-timer\n" ; -static char err2[]__initdata =KERN_ERR -"%s: can't register clocksource!\n" ; +u32 cyc =0 ; - gpt =omap_dm_timer_request (); -if (!gpt) -printk (err1, clocksource_gpt.name); - gpt_clocksource = gpt; +if (clksrc.reserved) + cyc =__omap_dm_timer_read_counter (clksrc.io_base,1 ); -omap_dm_timer_set_source (gpt,OMAP_TIMER_SRC_SYS_CLK ); - tick_rate =clk_get_rate (omap_dm_timer_get_fclk (gpt)); +return cyc_to_sched_clock (&cd, cyc, (u32 )~0 ); +} + +/* Setup free-running counter for clocksource */ +static void __init omap2_gp_clocksource_init (int gptimer_id, +const char *fck_source) +{ +int res; + + res =omap_dm_timer_init_one (&clksrc, gptimer_id, fck_source); +BUG_ON (res); -omap_dm_timer_set_load_start (gpt,1 ,0 ); +pr_info ("OMAP clocksource: GPTIMER%d at %lu Hz\n" , + gptimer_id, clksrc.rate); -init_sched_clock (&cd, dmtimer_update_sched_clock,32 , tick_rate); +omap_dm_timer_set_load_start (&clksrc,1 ,0 ); +init_sched_clock (&cd, dmtimer_update_sched_clock,32 , clksrc.rate); -if (clocksource_register_hz (&clocksource_gpt, tick_rate)) -printk (err2, clocksource_gpt.name); +if (clocksource_register_hz (&clocksource_gpt, clksrc.rate)) +pr_err ("Could not register clocksource %s\n" , + clocksource_gpt.name); }#endif
@@ -320,7 +326,7 @@ static void __init omap2_gp_clocksource_init(void)static void __init omap242x_timer_init (void ) {omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE ); -omap2_gp_clocksource_init (); +omap2_gp_clocksource_init (2 ,OMAP2_MPU_SOURCE ); }struct sys_timer omap242x_timer = {
@@ -330,7 +336,7 @@ struct sys_timer omap242x_timer = {static void __init omap243x_timer_init (void ) {omap2_gp_clockevent_init (1 ,OMAP2_CLKEV_SOURCE ); -omap2_gp_clocksource_init (); +omap2_gp_clocksource_init (2 ,OMAP2_MPU_SOURCE ); }struct sys_timer omap243x_timer = {
@@ -342,7 +348,7 @@ struct sys_timer omap243x_timer = {static void __init omap3_timer_init (void ) {omap2_gp_clockevent_init (1 ,OMAP3_CLKEV_SOURCE ); -omap2_gp_clocksource_init (); +omap2_gp_clocksource_init (2 ,OMAP3_MPU_SOURCE ); }struct sys_timer omap3_timer = {
@@ -356,7 +362,7 @@ struct sys_timer omap3_timer = {static void __init omap3_beagle_timer_init (void ) {omap2_gp_clockevent_init (OMAP3_BEAGLE_TIMER ,OMAP3_CLKEV_SOURCE ); -omap2_gp_clocksource_init (); +omap2_gp_clocksource_init (2 ,OMAP3_MPU_SOURCE ); }struct sys_timer omap3_beagle_timer = {
@@ -372,7 +378,7 @@ static void __init omap4_timer_init(void)BUG_ON (!twd_base);#endif omap2_gp_clockevent_init (1 ,OMAP4_CLKEV_SOURCE ); -omap2_gp_clocksource_init (); +omap2_gp_clocksource_init (2 ,OMAP4_MPU_SOURCE ); }struct sys_timer omap4_timer = {
arch/arm/plat-omap/counter_32k.c +1 -1
@@ -126,7 +126,7 @@ static inline unsigned long long notrace _omap_32k_sched_clock(void)return cyc_to_fixed_sched_clock (&cd, cyc, (u32 )~0 ,SC_MULT ,SC_SHIFT ); } -#ifndef CONFIG_OMAP_MPU_TIMER +#if defined(CONFIG_OMAP_32K_TIMER ) && !defined(CONFIG_OMAP_MPU_TIMER )unsigned long long notrace sched_clock (void ) {return _omap_32k_sched_clock ();
Tony Lindgren
maintainerPATCH 9/10 omap2+: Remove omap2_gp_clockevent_set_gptimer
In reply to Tony LindgrenThis is no longer needed as we now just set the desired .timer in MACHINE_START. We can now also remove timer-gp.h.
--- arch/arm/mach-omap2/board-4430sdp.c | 4 ---- arch/arm/mach-omap2/board-devkit8000.c | 4 ---- arch/arm/mach-omap2/board-omap3beagle.c | 4 ---- arch/arm/mach-omap2/board-omap3logic.c | 1 - arch/arm/mach-omap2/board-omap3stalker.c | 4 ---- arch/arm/mach-omap2/board-omap3touchbook.c | 4 ---- arch/arm/mach-omap2/board-omap4panda.c | 1 - arch/arm/mach-omap2/timer-gp.c | 26 -------------------------- arch/arm/mach-omap2/timer-gp.h | 16 ---------------- 9 files changed, 0 insertions(+), 64 deletions(-) delete mode 100644 arch/arm/mach-omap2/timer-gp.h
arch/arm/mach-omap2/board-4430sdp.c +0 -4
@@ -40,7 +40,6 @@#include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #include "control.h" #define ETH_KS8851_IRQ 34
@@ -330,9 +329,6 @@ static void __init omap_4430sdp_init_early(void) {omap2_init_common_infrastructure ();omap2_init_common_devices (NULL ,NULL ); -#ifdef CONFIG_OMAP_32K_TIMER -omap2_gp_clockevent_set_gptimer (1 ); -#endif }static struct omap_musb_board_data musb_board_data = {
arch/arm/mach-omap2/board-devkit8000.c +0 -4
@@ -59,7 +59,6 @@#include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #define NAND_BLOCK_SIZE SZ_128K
@@ -458,9 +457,6 @@ static void __init devkit8000_init_early(void)static void __init devkit8000_init_irq (void ) {omap3_init_irq (); -#ifdef CONFIG_OMAP_32K_TIMER -omap2_gp_clockevent_set_gptimer (12 ); -#endif }static void __init devkit8000_ads7846_init (void )
arch/arm/mach-omap2/board-omap3beagle.c +0 -4
@@ -50,7 +50,6 @@#include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #include "pm.h" #define NAND_BLOCK_SIZE SZ_128K
@@ -541,9 +540,6 @@ static void __init omap3_beagle_init_early(void)static void __init omap3_beagle_init_irq (void ) {omap3_init_irq (); -#ifdef CONFIG_OMAP_32K_TIMER -omap2_gp_clockevent_set_gptimer (12 ); -#endif }static struct platform_device *omap3_beagle_devices[]__initdata = {
arch/arm/mach-omap2/board-omap3logic.c +0 -1
@@ -35,7 +35,6 @@#include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #include "control.h" #include <plat/mux.h>
arch/arm/mach-omap2/board-omap3stalker.c +0 -4
@@ -53,7 +53,6 @@#include "sdram-micron-mt46h32m32lf-6.h" #include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #if defined(CONFIG_SMSC911X ) || defined(CONFIG_SMSC911X_MODULE )#define OMAP3STALKER_ETHR_START 0x2c000000
@@ -588,9 +587,6 @@ static void __init omap3_stalker_init_early(void)static void __init omap3_stalker_init_irq (void ) {omap3_init_irq (); -#ifdef CONFIG_OMAP_32K_TIMER -omap2_gp_clockevent_set_gptimer (12 ); -#endif }static struct platform_device *omap3_stalker_devices[]__initdata = {
arch/arm/mach-omap2/board-omap3touchbook.c +0 -4
@@ -51,7 +51,6 @@#include "mux.h" #include "hsmmc.h" -#include "timer-gp.h" #include <asm/setup.h>
@@ -423,9 +422,6 @@ static void __init omap3_touchbook_init_early(void)static void __init omap3_touchbook_init_irq (void ) {omap3_init_irq (); -#ifdef CONFIG_OMAP_32K_TIMER -omap2_gp_clockevent_set_gptimer (12 ); -#endif }static struct platform_device *omap3_touchbook_devices[]__initdata = {
arch/arm/mach-omap2/board-omap4panda.c +0 -1
@@ -41,7 +41,6 @@#include <plat/usb.h> #include <plat/mmc.h> #include <plat/panel-generic-dpi.h> -#include "timer-gp.h" #include "hsmmc.h" #include "control.h"
arch/arm/mach-omap2/timer-gp.c +0 -26
@@ -43,8 +43,6 @@#include <plat/common.h> #include <plat/omap_hwmod.h> -#include "timer-gp.h" -/* Parent clocks, eventually these will come from the clock framework */ #define OMAP2_MPU_SOURCE "sys_ck"
@@ -75,8 +73,6 @@ u32 sys_timer_reserved;static struct omap_dm_timer clkev;static struct clock_event_device clockevent_gpt; -static u8 __initdata gptimer_id =1 ; -static u8 __initdata inited;static irqreturn_t omap2_gp_timer_interrupt (int irq,void *dev_id) {
@@ -133,26 +129,6 @@ static struct clock_event_device clockevent_gpt = { .set_mode = omap2_gp_timer_set_mode, }; -/** -* omap2_gp_clockevent_set_gptimer - set which GPTIMER is used for clockevents -* @id: GPTIMER to use (1..MAX_GPTIMER_ID) -* -* Define the GPTIMER that the system should use for the tick timer. -* Meant to be called from board-*.c files in the event that GPTIMER1, the -* default, is unsuitable. Returns -EINVAL on error or 0 on success. -*/ -int __init omap2_gp_clockevent_set_gptimer (u8 id) -{ -if (id <1 || id >MAX_GPTIMER_ID ) -return -EINVAL ; - -BUG_ON (inited); - - gptimer_id = id; - -return 0 ; -} -static int __init omap_dm_timer_init_one (struct omap_dm_timer *timer,int gptimer_id,const char *fck_source)
@@ -222,8 +198,6 @@ static void __init omap2_gp_clockevent_init(int gptimer_id, {int res; - inited =1 ; - res =omap_dm_timer_init_one (&clkev, gptimer_id, fck_source);BUG_ON (res);
arch/arm/mach-omap2/timer-gp.h deleted +0 -16
@@ -1,16 +0,0 @@ -/* -* OMAP2/3 GPTIMER support.headers -* -* Copyright (C) 2009 Nokia Corporation -* -* This file is subject to the terms and conditions of the GNU General Public -* License. See the file "COPYING" in the main directory of this archive -* for more details. -*/ - -#ifndef __ARCH_ARM_PLAT_OMAP_INCLUDE_MACH_TIMER_GP_H -#define __ARCH_ARM_PLAT_OMAP_INCLUDE_MACH_TIMER_GP_H - -extern int __init omap2_gp_clockevent_set_gptimer (u8 id); - -#endif
Tony Lindgren
maintainerPATCH 10/10 omap2+: Rename timer-gp.c into timer.c to combine timer init functions
In reply to Tony LindgrenWe can keep everything sys_timer and gptimer.c related code in timer.c as the code will be very minimal.
Later on we can also remove timer-mpu.c, as it can be called from omap4_timer_init function.
This allows us to get rid of confusing existing files. We currently have timer-gp.c, timer-mpu.c, and patches have been posted to add dmtimer.c. There's no need to have these multiple files, we can put everything into timer.c.
---
arch/arm/mach-omap2/Makefile | 2 +-
arch/arm/mach-omap2/timer.c | 0
2 files changed, 1 insertions(+), 1 deletions(-)
rename arch/arm/mach-omap2/{timer-gp.c => timer.c} (100%)arch/arm/mach-omap2/Makefile +1 -1
@@ -3,7 +3,7 @@ # # Common support -obj-y := id.o io.o control.o mux.o devices.o serial.o gpmc.o timer-gp.o pm.o \ +obj-y := id.o io.o control.o mux.o devices.o serial.o gpmc.o timer.o pm.o \ common.o gpio.o dma.o wd_timer.o omap-2-3-common = irq.o sdrc.o
arch/arm/mach-omap2/timer.c renamed from arch/arm/mach-omap2/timer-gp.c +0 -0
cover omap init_early changes for irq and timer init
In reply to Tony LindgrenQuoted 8 lines · Tony Lindgren
This series continues the work to only initialize minimal omap code in
init_earlyand to cut down dependencies to code that should be initialized later. It also cleans up the omap2+ timer init code to prepare things for the later patches.After this series further patches can make the dmtimer code a regular device driver. Later on we can also move the clockource init happen earlier to make
sched_clockwork properly withCONFIG_PRINTK_TIME.
This series looks OK to me.
My primary concerns were around the duplication of code between early/system timers and the "real" device driver. But it now looks like those early timers will also be doing individual init using hwmod and clock framework, so any duplication can be avoided.
Kevin
cover omap init_early changes for irq and timer init
In reply to Tony LindgrenTony,
Quoted 11 lines · Tony Lindgren
Hi all,
This series continues the work to only initialize minimal omap code in
init_earlyand to cut down dependencies to code that should be initialized later. It also cleans up the omap2+ timer init code to prepare things for the later patches.After this series further patches can make the dmtimer code a regular device driver. Later on we can also move the clockource init happen earlier to make
sched_clockwork properly withCONFIG_PRINTK_TIME.
Yesterday I started looking at this series but later because of my mailer horror show, I gave up.
After going through entire series again, it looks very good clean-up and right step towards moving rest of the timer to drivers/ directory.
I also realized that the discussion we had before this series was because of miss-understating about the 'wakeu_up' timer terminology. After this series I see that you were mentioning about the pm debug timer where as I was taling about clock-event wakeups from CPUidle point of view.
I still have few concerns about this series.
1)We have removed the flexibility if choosing the timer from
from board files and hard-coded them in platform timer header.
Below board related hard-coding in platform files looks more hacky and the interface done by Pual still has a merits of its own.
arch/arm/plat-omap/include/plat/common.h +6 -1
@@ -34,7 +34,12 @@struct sys_timer ;extern void omap_map_common_io (void ); -extern struct sys_timer omap_timer; +extern struct sys_timer omap1_timer; +extern struct sys_timer omap242x_timer; +extern struct sys_timer omap243x_timer; +extern struct sys_timer omap3_timer; +extern struct sys_timer omap3_beagle_timer; +extern struct sys_timer omap4_timer;extern bool omap_32k_timer_init (void );extern int __init omap_init_clocksource_32k (void );extern unsigned long long notrace omap_32k_sched_clock (void );
arch/arm/mach-omap2/timer-gp.c +20 -0
@@ -45,10 +45,33 @@#include "timer-gp.h" +/* Parent clocks, eventually these will come from the clock framework */ + +#define OMAP2_MPU_SOURCE "sys_ck" +#define OMAP3_MPU_SOURCE OMAP2_MPU_SOURCE +#define OMAP4_MPU_SOURCE "sys_clkin_ck" +#define OMAP2_32K_SOURCE "func_32k_ck" +#define OMAP3_32K_SOURCE "omap_32k_fck" +#define OMAP4_32K_SOURCE "sys_32k_ck" + +#ifdef CONFIG_OMAP_32K_TIMER +#define OMAP2_CLKEV_SOURCE OMAP2_32K_SOURCE +#define OMAP3_CLKEV_SOURCE OMAP3_32K_SOURCE +#define OMAP4_CLKEV_SOURCE OMAP4_32K_SOURCE +#define OMAP3_BEAGLE_TIMER 12 +#else +#define OMAP2_CLKEV_SOURCE OMAP2_MPU_SOURCE +#define OMAP3_CLKEV_SOURCE OMAP3_MPU_SOURCE +#define OMAP4_CLKEV_SOURCE OMAP4_MPU_SOURCE +#define OMAP3_BEAGLE_TIMER 1 +#endif
[.....]
+/*
+ * Beagle based designs typically have an issue with gptimer1. Also note
+ * that GPTIMER12 can only use the secure 32KiHz clock source.
+ */
static void __init omap3_beagle_timer_init(void)
{
omap_dm_timer_init();
- omap2_gp_clockevent_init();
+ omap2_gp_clockevent_init(OMAP3_BEAGLE_TIMER, OMAP3_CLKEV_SOURCE);
omap2_gp_clocksource_init();
}
2) In patch [6/10], you have removed wakeup timer debug capability with comment "Later on we can reserve gptimer1 for PM code only and have similar functionality".
I don't know how this will work on OMAP. GPTIMER1 is the only timer which is in always on power domain and wakeup OMAP from deeper power states like CORE OSWR, CORE OFF, DEVICE OFF.
We do implement these C-state in CPUidle as well and hence the clock-event is expected to be of wakeup-capable. Hence GPTIMER1 is already reserved for clock-event.
The wakeup timer used was only in suspend scenario's and that point of time the timer system is already suspended. So during that even if TIMER1 is re-used for wakeup (which is the case), I don't see any issue.
At least I don't see other solution than using GPT1 for wakeup.
3) You have created few functions so that they can be used between system timer and dmtimer driver code. When we move the dmtimer driver to say some drivers/ directory. Is it allowed to share such functions from device drivers
Apart from above comments, I really liked this series. If you like you can add my,
Regards
Santosh
Tony Lindgren
maintainercover omap init_early changes for irq and timer init
Quoted 10 lines · Santosh Shilimkar
After going through entire series again, it looks very good clean-up and right step towards moving rest of the timer to
drivers/directory.I also realized that the discussion we had before this series was because of miss-understating about the '
wakeu_up' timer terminology. After this series I see that you were mentioning about the pm debug timer where as I was taling about clock-event wakeups from CPUidle point of view.
Well this patcheset keeps the current behaviour, except for the PM debug hack to program gpt1.
What it does not have is the code to dedicate gpt1 for PM code, which can be done later once all the other dmtimer changes are done.
So the earlier discussion is still a separate issue from the PM debug hack removed in this series. I'll post some patches later on for that.
Quoted 5 lines
I still have few concerns about this series.
1)We have removed the flexibility if choosing the timer from
from board files and hard-coded them in platform timer header.
Well it's not removed, you can still select the timer setup by setting .timer entry in the board-*.c file.
So for example, we have omap3_timer and omap3_beagle_timer for the beagle rev a & b based boards.
Quoted 10 lines
Below board related hard-coding in platform files looks more hacky and the interface done by Pual still has a merits of its own.
arch/arm/plat-omap/include/plat/common.h +0
-0@@ -34,7 +34,12 @@struct sys_timer ;extern void omap_map_common_io (void );
arch/arm/plat-omap/include/plat/common.h line 45
Quoted 10 lines
arch/arm/plat-omap/include/plat/common.h +6
-1-extern struct sys_timer omap_timer; +extern struct sys_timer omap1_timer; +extern struct sys_timer omap242x_timer; +extern struct sys_timer omap243x_timer; +extern struct sys_timer omap3_timer; +extern struct sys_timer omap3_beagle_timer; +extern struct sys_timer omap4_timer;extern bool omap_32k_timer_init (void );extern int __init omap_init_clocksource_32k (void );extern unsigned long long notrace omap_32k_sched_clock (void );
Hmm the only reason for omap3_beagle_timer naming is the hardware bug in rev a & b beagle boards.
arch/arm/plat-omap/include/plat/common.h
Quoted 11 lines
arch/arm/plat-omap/include/plat/common.h +5
-1+/* +* Beagle based designs typically have an issue with gptimer1. Also note +* that GPTIMER12 can only use the secure 32KiHz clock source. +*/ static void __init omap3_beagle_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (OMAP3_BEAGLE_TIMER ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }
Got any better name in mind for that timer?
For removing the old interface, I don't see any reason to select timer combinations on omap3 other than omap3_timer and omap3_beagle_timer.
If there's need, any new valid sane combinations can be esily added, although I seriously doubt that we'll need more for omap3.
Quoted 8 lines
2) In patch [6/10], you have removed wakeup timer debug capability with comment "Later on we can reserve gptimer1 for PM code only and have similar functionality".
I don't know how this will work on OMAP. GPTIMER1 is the only timer which is in always on power domain and wakeup OMAP from deeper power states like CORE OSWR, CORE OFF, DEVICE OFF.
Right, that's why the PM code should manage it for the wake-up events. For the clockevent and clocksource we just want to use something fast for the runtime.
Then when we hit idle, we can just let PM code program gpt1 for the wake-up event.
Quoted 3 lines
We do implement these C-state in CPUidle as well and hence the clock-event is expected to be of wakeup-capable. Hence GPTIMER1 is already reserved for clock-event.
The PM code can easily deal with gpt1 by either calling next_timer_interrupt, or just by cloning the clockevent timer value. But again, that's a separate patch series to come..
Quoted 4 lines
The wakeup timer used was only in suspend scenario's and that point of time the timer system is already suspended. So during that even if TIMER1 is re-used for wakeup (which is the case), I don't see any issue.
The reason is that for performance and latency reasons we want to use localtimers for runtime on omap4+, and need only gpt1 for PM wake-up.
The gpt1 using 32KiHz source clock is a very slow clock to reprogram as it's on the external bus.
Quoted 2 lines
At least I don't see other solution than using GPT1 for wakeup.
Right, there's no other way to wake except gpt1 or wake-up enabled gpio lines. But we don't need to use gpt1 during runtime at all.
Quoted 5 lines
3) You have created few functions so that they can be used between system timer and dmtimer driver code. When we move the dmtimer driver to say some
drivers/directory. Is it allowed to share such functions from device drivers
The only function we for clockevent and clocksource is the init_one function, which will be implemented in in arch/arm/mach-omap2/timer.c anyways. The function to reprogram the timer is an inline function so dmtimer can then be a loadable driver module :)
Quoted 3 lines
Apart from above comments, I really liked this series. If you like you can add my,
Reviewed-by Santosh Shilimkar <[email protected]>
Thanks for looking, will add.
Regards,
Tony
cover omap init_early changes for irq and timer init
Quoted 6 lines · Tony Lindgren
Quoted 5 lines · Santosh Shilimkar
After going through entire series again, it looks very good clean-up and right step towards moving rest of the timer to
drivers/directory.
Quoted 9 lines
Quoted 5 lines
I also realized that the discussion we had before this series was because of miss-understating about the '
wakeu_up' timer terminology. After this series I see that you were mentioning about the pm debug timer where as I was taling about clock-event wakeups from CPUidle point of view.Well this patcheset keeps the current behaviour, except for the PM debug hack to program gpt1.
This is the problem what I see.
Quoted 4 lines
What it does not have is the code to dedicate gpt1 for PM code, which can be done later once all the other dmtimer changes are done.
Which not possible to do unless you plan to hack generic timer framework or waste additional timer hardware for this.
Quoted 4 lines
So the earlier discussion is still a separate issue from the PM debug hack removed in this series. I'll post some patches later on for that.
No that discussion become relevant with this series now.
Quoted 6 lines
Quoted 5 lines
I still have few concerns about this series.
1)We have removed the flexibility if choosing the timer from
from board files and hard-coded them in platform timer header.
Quoted 6 lines
Well it's not removed, you can still select the timer setup by setting .timer entry in the board-*.c file.
So for example, we have
omap3_timerandomap3_beagle_timerfor the beagle rev a& b based boards.
This is exactly my point. This is the board specific data which is now getting moved to generic headers.
Quoted 38 lines
Quoted 10 lines
Below board related hard-coding in platform files looks more hacky and the interface done by Pual still has a merits of its own.
arch/arm/plat-omap/include/plat/common.h +0
-0@@ -34,7 +34,12 @@struct sys_timer ;extern void omap_map_common_io (void );arch/arm/plat-omap/include/plat/common.h line 45
Quoted 10 lines
arch/arm/plat-omap/include/plat/common.h +6
-1-extern struct sys_timer omap_timer; +extern struct sys_timer omap1_timer; +extern struct sys_timer omap242x_timer; +extern struct sys_timer omap243x_timer; +extern struct sys_timer omap3_timer; +extern struct sys_timer omap3_beagle_timer; +extern struct sys_timer omap4_timer;extern bool omap_32k_timer_init (void );extern int __init omap_init_clocksource_32k (void );extern unsigned long long notrace omap_32k_sched_clock (void );Hmm the only reason for
omap3_beagle_timernaming is the hardware bug in rev a& b beagle boards.arch/arm/plat-omap/include/plat/common.h
Quoted 11 lines
arch/arm/plat-omap/include/plat/common.h +5
-1+/* +* Beagle based designs typically have an issue with gptimer1. Also note +* that GPTIMER12 can only use the secure 32KiHz clock source. +*/ static void __init omap3_beagle_timer_init (void ) {omap_dm_timer_init (); -omap2_gp_clockevent_init (); +omap2_gp_clockevent_init (OMAP3_BEAGLE_TIMER ,OMAP3_CLKEV_SOURCE );omap2_gp_clocksource_init (); }Got any better name in mind for that timer?
Quoted 8 lines
For removing the old interface, I don't see any reason to select timer combinations on omap3 other than
omap3_timerandomap3_beagle_timer.If there's need, any new valid sane combinations can be esily added, although I seriously doubt that we'll need more for omap3.
May be I am wrong but the point is about the merit of the solution even if there are only couple of board files where we use that interface.
It much cleaner and simpler to say timerid=X, from board file rather than creating a "struct sys_timer" instance and putting that in timer code.
Quoted 25 lines
Quoted 8 lines
2) In patch [6/10], you have removed wakeup timer debug capability with comment "Later on we can reserve gptimer1 for PM code only and have similar functionality".
I don't know how this will work on OMAP. GPTIMER1 is the only timer which is in always on power domain and wakeup OMAP from deeper power states like CORE OSWR, CORE OFF, DEVICE OFF.
Right, that's why the PM code should manage it for the wake-up events. For the clockevent and clocksource we just want to use something fast for the runtime.
Then when we hit idle, we can just let PM code program gpt1 for the wake-up event.
Quoted 3 lines
We do implement these C-state in CPUidle as well and hence the clock-event is expected to be of wakeup-capable. Hence GPTIMER1 is already reserved for clock-event.
The PM code can easily deal with gpt1 by either calling
next_timer_interrupt, or just by cloning the clockevent timer value. But again, that's a separate patch series to come..
Quoted 9 lines
Quoted 4 lines
The wakeup timer used was only in suspend scenario's and that point of time the timer system is already suspended. So during that even if TIMER1 is re-used for wakeup (which is the case), I don't see any issue.
The reason is that for performance and latency reasons we want to use localtimers for runtime on omap4+, and need only gpt1 for PM wake-up.
This is already the case with or without this series.
Quoted 3 lines
The gpt1 using 32KiHz source clock is a very slow clock to reprogram as it's on the external bus.
Even its slow, to make CPUIDLE work in low power modes and you need this clock-event as well. This is where the generic timer framework with C3STOP helps.
Quoted 7 lines
Quoted 2 lines
At least I don't see other solution than using GPT1 for wakeup.
Right, there's no other way to wake except gpt1 or wake-up enabled gpio lines. But we don't need to use gpt1 during runtime at all.
This is not entirely correct and I think this is the point where we are not on same page. During runtime, gpt1 clock event is not used for tick generation but it's kept programmed because low power state switch via
get triggered any time and on any CPU.
This is the same problem as X86 APIC timer + HPET switching and I worked with Thomas G and Russell to get this working on ARM platforms using generic timer framework. No hacking is needed in PM code for this.
Quoted 6 lines
Quoted 5 lines
3) You have created few functions so that they can be used between system timer and dmtimer driver code. When we move the dmtimer driver to say some
drivers/directory. Is it allowed to share such functions from device drivers
Quoted 6 lines
The only function we for clockevent and clocksource is the
init_onefunction, which will be implemented in inarch/arm/mach-omap2/timer.canyways. The function to reprogram the timer is an inline function so dmtimer can then be a loadable driver module :)
Great. This was more of question for my undertanding.
For me, with current set of patches, suspend wakeup using GPT1 is not possible.
It's not a big problem because it's just debug aid and production kernel doesn't use this as such.
Regards
Santosh
Tony Lindgren
maintainercover omap init_early changes for irq and timer init
Quoted 9 lines · Santosh Shilimkar
Quoted 4 lines · Tony Lindgren
What it does not have is the code to dedicate gpt1 for PM code, which can be done later once all the other dmtimer changes are done.
Which not possible to do unless you plan to hack generic timer framework or waste additional timer hardware for this.
Well an extra timer hardware would only be needed on omap2 & 3.
But hey, if it makes sense to do or not to do is a different set of patches. At least we now have an option to play with it.
Quoted 8 lines
Quoted 8 lines
For removing the old interface, I don't see any reason to select timer combinations on omap3 other than
omap3_timerandomap3_beagle_timer.If there's need, any new valid sane combinations can be esily added, although I seriously doubt that we'll need more for omap3.
Quoted 7 lines
May be I am wrong but the point is about the merit of the solution even if there are only couple of board files where we use that interface.
It much cleaner and simpler to say timerid=X, from board file rather than creating a "struct
sys_timer" instance and putting that in timer code.
Well the timerid=X adds yet another interface and more calls from board-*.c to the common code. And it requires more changes if beagle boards want to use the system clock as the source clock instead of the 32KiHZ source.
Maybe let's call the omap3_beagle_timer omap3_secure_timer instead?
That should solve your issue of having the board name show up in the generic code, no?
Quoted 7 lines
Quoted 7 lines
Quoted 2 lines
At least I don't see other solution than using GPT1 for wakeup.
Right, there's no other way to wake except gpt1 or wake-up enabled gpio lines. But we don't need to use gpt1 during runtime at all.
Quoted 5 lines
This is not entirely correct and I think this is the point where we are not on same page. During runtime, gpt1 clock event is not used for tick generation but it's kept programmed because low power state switch via
get triggered any time and on any CPU.
Well ideally we would not program it during runtime at all because it's slow to program. I don't think that can be currently done with the sys_timer.
Quoted 5 lines
This is the same problem as X86 APIC timer + HPET switching and I worked with Thomas G and Russell to get this working on ARM platforms using generic timer framework. No hacking is needed in PM code for this.
Except we should improve things eventually where we don't need to program the slow external timer during runtime if we have local timers.
Hmm maybe I'm wrong and you got that working already?
Regards,
Tony
cover omap init_early changes for irq and timer init
Quoted 37 lines · Tony Lindgren
Quoted 9 lines · Santosh Shilimkar
Quoted 4 lines · Tony Lindgren
What it does not have is the code to dedicate gpt1 for PM code, which can be done later once all the other dmtimer changes are done.
Which not possible to do unless you plan to hack generic timer framework or waste additional timer hardware for this.
Well an extra timer hardware would only be needed on omap2& 3.
But hey, if it makes sense to do or not to do is a different set of patches. At least we now have an option to play with it.
Quoted 8 lines
Quoted 8 lines
For removing the old interface, I don't see any reason to select timer combinations on omap3 other than
omap3_timerandomap3_beagle_timer.If there's need, any new valid sane combinations can be esily added, although I seriously doubt that we'll need more for omap3.
Quoted 7 lines
May be I am wrong but the point is about the merit of the solution even if there are only couple of board files where we use that interface.
It much cleaner and simpler to say timerid=X, from board file rather than creating a "struct
sys_timer" instance and putting that in timer code.Well the timerid=X adds yet another interface and more calls from board-*.c to the common code. And it requires more changes if beagle boards want to use the system clock as the source clock instead of the 32KiHZ source.
Quoted 5 lines
Maybe let's call the
omap3_beagle_timeromap3_secure_timerinstead?That should solve your issue of having the board name show up in the generic code, no?
Sorry about picking up on names but that was not my point. I agree with you on reducing interfaces so I step back on this point.
Quoted 23 lines
Quoted 7 lines
Quoted 7 lines
Quoted 2 lines
At least I don't see other solution than using GPT1 for wakeup.
Right, there's no other way to wake except gpt1 or wake-up enabled gpio lines. But we don't need to use gpt1 during runtime at all.
Quoted 5 lines
This is not entirely correct and I think this is the point where we are not on same page. During runtime, gpt1 clock event is not used for tick generation but it's kept programmed because low power state switch via
get triggered any time and on any CPU.Well ideally we would not program it during runtime at all because it's slow to program. I don't think that can be currently done with the
sys_timer.Quoted 5 lines
This is the same problem as X86 APIC timer + HPET switching and I worked with Thomas G and Russell to get this working on ARM platforms using generic timer framework. No hacking is needed in PM code for this.
Quoted 4 lines
Except we should improve things eventually where we don't need to program the slow external timer during runtime if we have local timers.
This is already the case now. On OMAP4 running system, CPU use their own local timers and rq's. There is no broadcasting. Whenever there is a need of it like the situations where local-timers die (low power states), timer system switches to broad-cast timer which is wakeup capable. GPT1 in our case. This is all managed by timer framework and works seamlessly.
Quoted 2 lines
Hmm maybe I'm wrong and you got that working already?
I don't think you are wrong. All your points are correct. The only missing point was the necessity of GPT1 registered as clock-event to allow dynamic switching between clock-events.
That's where I was saying that we are not left with GPT1 for PM debug feature.
Regards
Santosh