[V3 PATCH] arm: add global early device init

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Fri May 6 02:32:02 EDT 2011


Hi,

	Paul, Magnus any feed back?

Best Regards,
J.
On 06:03 Tue 03 May     , Jean-Christophe PLAGNIOL-VILLARD wrote:
> this will init earlytimer and will try to probe 1 or 2 timers for clockevents
> and clocksource
> 
> This will also allow to share timer cross arch such as sh-mobile and sh
> 
> all arm timers will be update to early device step by step
> 
> on ARM sh-mobile we do not need to use the late_time_init
> as the irq are setup before the probe of the timers
> 
> tested on at91sam9263-ek, at91rm9200-ek
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> Cc: Russell King <linux at arm.linux.org.uk>
> Cc: Paul Mundt <lethal at linux-sh.org>
> Cc: Magnus Damm <magnus.damm at gmail.com>
> Cc: Nicolas Ferre <nicolas.ferre at atmel.com>
> Cc: Patrice Vilchez <patrice.vilchez at atmel.com>
> ---
> V3:
> 	now we can do not specify the .timer at all
> 	and just realy on the early device
> 
> 	on ARM sh-mobile we do not need to use the late_time_init
> 	as the irq are setup before the probe of the timers
> 
> 	the only remaing question is is ok to move the clock_init in the
> 	.init_early on sh-mobile
> 
> 	this patch depends on [PATCH] shmobile: fix map_io init usage
> 
> Best Regards,
> J.
> 
>  arch/arm/kernel/time.c                  |   28 +++++++++++++++++--
>  arch/arm/mach-shmobile/Makefile         |    2 +-
>  arch/arm/mach-shmobile/board-ag5evm.c   |   13 +--------
>  arch/arm/mach-shmobile/board-ap4evb.c   |   18 +++---------
>  arch/arm/mach-shmobile/board-g3evm.c    |   12 +-------
>  arch/arm/mach-shmobile/board-g4evm.c    |   12 +-------
>  arch/arm/mach-shmobile/board-mackerel.c |   18 +++---------
>  arch/arm/mach-shmobile/timer.c          |   46 -------------------------------
>  8 files changed, 37 insertions(+), 112 deletions(-)
>  delete mode 100644 arch/arm/mach-shmobile/timer.c
> 
> diff --git a/arch/arm/kernel/time.c b/arch/arm/kernel/time.c
> index 1ff46ca..12cee50 100644
> --- a/arch/arm/kernel/time.c
> +++ b/arch/arm/kernel/time.c
> @@ -13,6 +13,7 @@
>   */
>  #include <linux/module.h>
>  #include <linux/kernel.h>
> +#include <linux/platform_device.h>
>  #include <linux/interrupt.h>
>  #include <linux/time.h>
>  #include <linux/init.h>
> @@ -77,6 +78,9 @@ EXPORT_SYMBOL(profile_pc);
>  #ifdef CONFIG_ARCH_USES_GETTIMEOFFSET
>  u32 arch_gettimeoffset(void)
>  {
> +	if (!system_timer)
> +		return 0;
> +
>  	if (system_timer->offset != NULL)
>  		return system_timer->offset() * 1000;
>  
> @@ -147,7 +151,12 @@ static struct sysdev_class timer_sysclass = {
>  
>  static int __init timer_init_sysfs(void)
>  {
> -	int ret = sysdev_class_register(&timer_sysclass);
> +	int ret;
> +
> +	if (!system_timer)
> +		return 0;
> +
> +	ret = sysdev_class_register(&timer_sysclass);
>  	if (ret == 0) {
>  		system_timer->dev.cls = &timer_sysclass;
>  		ret = sysdev_register(&system_timer->dev);
> @@ -161,9 +170,22 @@ device_initcall(timer_init_sysfs);
>  void __init time_init(void)
>  {
>  	system_timer = machine_desc->timer;
> -	system_timer->init();
> +	if (system_timer && system_timer->init)
> +		system_timer->init();
> +
> +	/*
> +	 * Make sure all compiled-in early timers register themselves.
> +	 *
> +	 * Run probe() for two "earlytimer" devices, these will be the
> +	 * clockevents and clocksource devices respectively. In the event
> +	 * that only a clockevents device is available, we -ENODEV on the
> +	 * clocksource and the jiffies clocksource is used transparently
> +	 * instead. No error handling is necessary here.
> +	 */
> +	early_platform_driver_register_all("earlytimer");
> +	early_platform_driver_probe("earlytimer", 2, 0);
> +
>  #ifdef CONFIG_HAVE_SCHED_CLOCK
>  	sched_clock_postinit();
>  #endif
>  }
> -
> diff --git a/arch/arm/mach-shmobile/Makefile b/arch/arm/mach-shmobile/Makefile
> index 260e09d..6a1cc9e 100644
> --- a/arch/arm/mach-shmobile/Makefile
> +++ b/arch/arm/mach-shmobile/Makefile
> @@ -3,7 +3,7 @@
>  #
>  
>  # Common objects
> -obj-y				:= map_io.o timer.o console.o clock.o pm_runtime.o
> +obj-y				:= map_io.o console.o clock.o pm_runtime.o
>  
>  # CPU objects
>  obj-$(CONFIG_ARCH_SH7367)	+= setup-sh7367.o clock-sh7367.o intc-sh7367.o
> diff --git a/arch/arm/mach-shmobile/board-ag5evm.c b/arch/arm/mach-shmobile/board-ag5evm.c
> index b30b11d..cd9ba29 100644
> --- a/arch/arm/mach-shmobile/board-ag5evm.c
> +++ b/arch/arm/mach-shmobile/board-ag5evm.c
> @@ -339,6 +339,7 @@ static void __init ag5evm_init_early(void)
>  	/* setup early devices and console here as well */
>  	sh73a0_add_early_devices();
>  	shmobile_setup_console();
> +	sh73a0_clock_init();
>  }
>  
>  #define PINTC_ADDR	0xe6900000
> @@ -447,22 +448,10 @@ static void __init ag5evm_init(void)
>  	platform_add_devices(ag5evm_devices, ARRAY_SIZE(ag5evm_devices));
>  }
>  
> -static void __init ag5evm_timer_init(void)
> -{
> -	sh73a0_clock_init();
> -	shmobile_timer.init();
> -	return;
> -}
> -
> -struct sys_timer ag5evm_timer = {
> -	.init	= ag5evm_timer_init,
> -};
> -
>  MACHINE_START(AG5EVM, "ag5evm")
>  	.map_io		= shmobile_map_io,
>  	.init_irq	= ag5evm_init_irq,
>  	.handle_irq	= shmobile_handle_irq_gic,
>  	.init_early	= ag5evm_init_early,
>  	.init_machine	= ag5evm_init,
> -	.timer		= &ag5evm_timer,
>  MACHINE_END
> diff --git a/arch/arm/mach-shmobile/board-ap4evb.c b/arch/arm/mach-shmobile/board-ap4evb.c
> index ed55c01..f800cae 100644
> --- a/arch/arm/mach-shmobile/board-ap4evb.c
> +++ b/arch/arm/mach-shmobile/board-ap4evb.c
> @@ -1096,6 +1096,10 @@ static void __init ap4evb_init_early(void)
>  	/* setup early devices and console here as well */
>  	sh7372_add_early_devices();
>  	shmobile_setup_console();
> +	sh7372_clock_init();
> +
> +	/* External clock source */
> +	clk_set_rate(&sh7372_dv_clki_clk, 27000000);
>  }
>  
>  #define GPIO_PORT9CR	0xE6051009
> @@ -1330,24 +1334,10 @@ static void __init ap4evb_init(void)
>  	fsi_init_pm_clock();
>  }
>  
> -static void __init ap4evb_timer_init(void)
> -{
> -	sh7372_clock_init();
> -	shmobile_timer.init();
> -
> -	/* External clock source */
> -	clk_set_rate(&sh7372_dv_clki_clk, 27000000);
> -}
> -
> -static struct sys_timer ap4evb_timer = {
> -	.init		= ap4evb_timer_init,
> -};
> -
>  MACHINE_START(AP4EVB, "ap4evb")
>  	.map_io		= shmobile_map_io,
>  	.init_irq	= sh7372_init_irq,
>  	.handle_irq	= shmobile_handle_irq_intc,
>  	.init_early	= ap4evb_init_early,
>  	.init_machine	= ap4evb_init,
> -	.timer		= &ap4evb_timer,
>  MACHINE_END
> diff --git a/arch/arm/mach-shmobile/board-g3evm.c b/arch/arm/mach-shmobile/board-g3evm.c
> index 20eeacf..63a6c35 100644
> --- a/arch/arm/mach-shmobile/board-g3evm.c
> +++ b/arch/arm/mach-shmobile/board-g3evm.c
> @@ -249,6 +249,7 @@ static void __init g3evm_init_early(void)
>  	/* setup early devices and console here as well */
>  	sh7367_add_early_devices();
>  	shmobile_setup_console();
> +	sh7367_clock_init();
>  }
>  
>  static void __init g3evm_init(void)
> @@ -338,21 +339,10 @@ static void __init g3evm_init(void)
>  	platform_add_devices(g3evm_devices, ARRAY_SIZE(g3evm_devices));
>  }
>  
> -static void __init g3evm_timer_init(void)
> -{
> -	sh7367_clock_init();
> -	shmobile_timer.init();
> -}
> -
> -static struct sys_timer g3evm_timer = {
> -	.init		= g3evm_timer_init,
> -};
> -
>  MACHINE_START(G3EVM, "g3evm")
>  	.map_io		= shmobile_map_io,
>  	.init_irq	= sh7367_init_irq,
>  	.handle_irq	= shmobile_handle_irq_intc,
>  	.init_early	= g3evm_init_early,
>  	.init_machine	= g3evm_init,
> -	.timer		= &g3evm_timer,
>  MACHINE_END
> diff --git a/arch/arm/mach-shmobile/board-g4evm.c b/arch/arm/mach-shmobile/board-g4evm.c
> index 079dd35..46e1b0c 100644
> --- a/arch/arm/mach-shmobile/board-g4evm.c
> +++ b/arch/arm/mach-shmobile/board-g4evm.c
> @@ -263,6 +263,7 @@ static void __init g4evm_init_early(void)
>  	/* setup early devices and console here as well */
>  	sh7377_add_early_devices();
>  	shmobile_setup_console();
> +	sh7377_clock_init();
>  }
>  
>  #define GPIO_SDHID0_D0	0xe60520fc
> @@ -381,21 +382,10 @@ static void __init g4evm_init(void)
>  	platform_add_devices(g4evm_devices, ARRAY_SIZE(g4evm_devices));
>  }
>  
> -static void __init g4evm_timer_init(void)
> -{
> -	sh7377_clock_init();
> -	shmobile_timer.init();
> -}
> -
> -static struct sys_timer g4evm_timer = {
> -	.init		= g4evm_timer_init,
> -};
> -
>  MACHINE_START(G4EVM, "g4evm")
>  	.map_io		= shmobile_map_io,
>  	.init_irq	= sh7377_init_irq,
>  	.handle_irq	= shmobile_handle_irq_intc,
>  	.init_early	= g4evm_init_early,
>  	.init_machine	= g4evm_init,
> -	.timer		= &g4evm_timer,
>  MACHINE_END
> diff --git a/arch/arm/mach-shmobile/board-mackerel.c b/arch/arm/mach-shmobile/board-mackerel.c
> index f05a673..d52f323 100644
> --- a/arch/arm/mach-shmobile/board-mackerel.c
> +++ b/arch/arm/mach-shmobile/board-mackerel.c
> @@ -1011,6 +1011,10 @@ static void __init mackerel_init_early(void)
>  	/* setup early devices and console here as well */
>  	sh7372_add_early_devices();
>  	shmobile_setup_console();
> +	sh7372_clock_init();
> +
> +	/* External clock source */
> +	clk_set_rate(&sh7372_dv_clki_clk, 27000000);
>  }
>  
>  #define GPIO_PORT9CR	0xE6051009
> @@ -1203,24 +1207,10 @@ static void __init mackerel_init(void)
>  	hdmi_init_pm_clock();
>  }
>  
> -static void __init mackerel_timer_init(void)
> -{
> -	sh7372_clock_init();
> -	shmobile_timer.init();
> -
> -	/* External clock source */
> -	clk_set_rate(&sh7372_dv_clki_clk, 27000000);
> -}
> -
> -static struct sys_timer mackerel_timer = {
> -	.init		= mackerel_timer_init,
> -};
> -
>  MACHINE_START(MACKEREL, "mackerel")
>  	.map_io		= shmobile_map_io,
>  	.init_irq	= sh7372_init_irq,
>  	.handle_irq	= shmobile_handle_irq_intc,
>  	.init_early	= mackerel_init_early,
>  	.init_machine	= mackerel_init,
> -	.timer		= &mackerel_timer,
>  MACHINE_END
> diff --git a/arch/arm/mach-shmobile/timer.c b/arch/arm/mach-shmobile/timer.c
> deleted file mode 100644
> index 895794b..0000000
> --- a/arch/arm/mach-shmobile/timer.c
> +++ /dev/null
> @@ -1,46 +0,0 @@
> -/*
> - * SH-Mobile Timer
> - *
> - * Copyright (C) 2010  Magnus Damm
> - * Copyright (C) 2002 - 2009  Paul Mundt
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; version 2 of the License.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
> - *
> - */
> -#include <linux/platform_device.h>
> -#include <asm/mach/time.h>
> -
> -static void __init shmobile_late_time_init(void)
> -{
> -	/*
> -	 * Make sure all compiled-in early timers register themselves.
> -	 *
> -	 * Run probe() for two "earlytimer" devices, these will be the
> -	 * clockevents and clocksource devices respectively. In the event
> -	 * that only a clockevents device is available, we -ENODEV on the
> -	 * clocksource and the jiffies clocksource is used transparently
> -	 * instead. No error handling is necessary here.
> -	 */
> -	early_platform_driver_register_all("earlytimer");
> -	early_platform_driver_probe("earlytimer", 2, 0);
> -}
> -
> -static void __init shmobile_timer_init(void)
> -{
> -	late_time_init = shmobile_late_time_init;
> -}
> -
> -struct sys_timer shmobile_timer = {
> -	.init		= shmobile_timer_init,
> -};
> -- 
> 1.7.4.1



More information about the linux-arm-kernel mailing list