* [PATCH] arm: omap2: timer: always define omap4_local_timer_init @ 2015-09-28 18:25 Felipe Balbi 2015-09-28 21:10 ` Nishanth Menon 0 siblings, 1 reply; 3+ messages in thread From: Felipe Balbi @ 2015-09-28 18:25 UTC (permalink / raw) To: linux-arm-kernel omap4_local_timer_init() can be used by other platforms as is. At least AM437x wants to use it. Instead of making omap4-only and providing a stub for builds without OMAP4, we can just always define that function. Reported-by: Nishanth Menon <nm@ti.com> Signed-off-by: Felipe Balbi <balbi@ti.com> --- arch/arm/mach-omap2/timer.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c index a55655127ef2..f9028582e962 100644 --- a/arch/arm/mach-omap2/timer.c +++ b/arch/arm/mach-omap2/timer.c @@ -642,20 +642,11 @@ static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", 2, "sys_clkin_ck", NULL); #endif -#ifdef CONFIG_ARCH_OMAP4 -#ifdef CONFIG_HAVE_ARM_TWD void __init omap4_local_timer_init(void) { omap4_sync32k_timer_init(); clocksource_of_init(); } -#else -void __init omap4_local_timer_init(void) -{ - omap4_sync32k_timer_init(); -} -#endif /* CONFIG_HAVE_ARM_TWD */ -#endif /* CONFIG_ARCH_OMAP4 */ #if defined(CONFIG_SOC_OMAP5) || defined(CONFIG_SOC_DRA7XX) void __init omap5_realtime_timer_init(void) -- 2.5.0.GIT ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] arm: omap2: timer: always define omap4_local_timer_init 2015-09-28 18:25 [PATCH] arm: omap2: timer: always define omap4_local_timer_init Felipe Balbi @ 2015-09-28 21:10 ` Nishanth Menon 2015-09-28 21:12 ` Felipe Balbi 0 siblings, 1 reply; 3+ messages in thread From: Nishanth Menon @ 2015-09-28 21:10 UTC (permalink / raw) To: linux-arm-kernel On 09/28/2015 01:25 PM, Felipe Balbi wrote: > omap4_local_timer_init() can be used by other > platforms as is. At least AM437x wants to use > it. Instead of making omap4-only and providing > a stub for builds without OMAP4, we can just > always define that function. > > Reported-by: Nishanth Menon <nm@ti.com> > Signed-off-by: Felipe Balbi <balbi@ti.com> > --- > arch/arm/mach-omap2/timer.c | 9 --------- > 1 file changed, 9 deletions(-) > > diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c > index a55655127ef2..f9028582e962 100644 > --- a/arch/arm/mach-omap2/timer.c > +++ b/arch/arm/mach-omap2/timer.c > @@ -642,20 +642,11 @@ static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", > 2, "sys_clkin_ck", NULL); > #endif > > -#ifdef CONFIG_ARCH_OMAP4 > -#ifdef CONFIG_HAVE_ARM_TWD > void __init omap4_local_timer_init(void) > { > omap4_sync32k_timer_init(); > clocksource_of_init(); > } > -#else > -void __init omap4_local_timer_init(void) > -{ > - omap4_sync32k_timer_init(); > -} > -#endif /* CONFIG_HAVE_ARM_TWD */ > -#endif /* CONFIG_ARCH_OMAP4 */ > > #if defined(CONFIG_SOC_OMAP5) || defined(CONFIG_SOC_DRA7XX) > void __init omap5_realtime_timer_init(void) > I am a little confused if this will build in a am437xx only build. #define OMAP_SYS_32K_TIMER_INIT(name, clkev_nr, clkev_src, clkev_prop, \ clksrc_nr, clksrc_src, clksrc_prop) \ void __init omap##name##_sync32k_timer_init(void) \ Would you like to consider this for OMAP4 only build as well? #if defined(CONFIG_ARCH_OMAP4) || defined(CONFIG_SOC_OMAP5) || \ defined(CONFIG_SOC_DRA7XX) static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", 2, "sys_clkin_ck", NULL); #endif -- Regards, Nishanth Menon ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] arm: omap2: timer: always define omap4_local_timer_init 2015-09-28 21:10 ` Nishanth Menon @ 2015-09-28 21:12 ` Felipe Balbi 0 siblings, 0 replies; 3+ messages in thread From: Felipe Balbi @ 2015-09-28 21:12 UTC (permalink / raw) To: linux-arm-kernel On Mon, Sep 28, 2015 at 04:10:40PM -0500, Nishanth Menon wrote: > On 09/28/2015 01:25 PM, Felipe Balbi wrote: > > omap4_local_timer_init() can be used by other > > platforms as is. At least AM437x wants to use > > it. Instead of making omap4-only and providing > > a stub for builds without OMAP4, we can just > > always define that function. > > > > Reported-by: Nishanth Menon <nm@ti.com> > > Signed-off-by: Felipe Balbi <balbi@ti.com> > > --- > > arch/arm/mach-omap2/timer.c | 9 --------- > > 1 file changed, 9 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c > > index a55655127ef2..f9028582e962 100644 > > --- a/arch/arm/mach-omap2/timer.c > > +++ b/arch/arm/mach-omap2/timer.c > > @@ -642,20 +642,11 @@ static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", > > 2, "sys_clkin_ck", NULL); > > #endif > > > > -#ifdef CONFIG_ARCH_OMAP4 > > -#ifdef CONFIG_HAVE_ARM_TWD > > void __init omap4_local_timer_init(void) > > { > > omap4_sync32k_timer_init(); > > clocksource_of_init(); > > } > > -#else > > -void __init omap4_local_timer_init(void) > > -{ > > - omap4_sync32k_timer_init(); > > -} > > -#endif /* CONFIG_HAVE_ARM_TWD */ > > -#endif /* CONFIG_ARCH_OMAP4 */ > > > > #if defined(CONFIG_SOC_OMAP5) || defined(CONFIG_SOC_DRA7XX) > > void __init omap5_realtime_timer_init(void) > > > I am a little confused if this will build in a am437xx only build. > > #define OMAP_SYS_32K_TIMER_INIT(name, clkev_nr, clkev_src, clkev_prop, \ > clksrc_nr, clksrc_src, clksrc_prop) \ > void __init omap##name##_sync32k_timer_init(void) \ > > Would you like to consider this for OMAP4 only build as well? > > #if defined(CONFIG_ARCH_OMAP4) || defined(CONFIG_SOC_OMAP5) || \ > > defined(CONFIG_SOC_DRA7XX) > static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", > 2, "sys_clkin_ck", NULL); > #endif heh, more of the ifdeferry to drop. Okay, I'll respin this -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150928/40131357/attachment.sig> ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-09-28 21:12 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-09-28 18:25 [PATCH] arm: omap2: timer: always define omap4_local_timer_init Felipe Balbi 2015-09-28 21:10 ` Nishanth Menon 2015-09-28 21:12 ` Felipe Balbi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox