From: Stephen Boyd <sboyd@codeaurora.org> To: Olof Johansson <olof@lixom.net> Cc: linux-omap@vger.kernel.org, Russell King - ARM Linux <linux@arm.linux.org.uk>, linux-arm-kernel@lists.infradead.org, Santosh Shilimkar <santosh.shilimkar@ti.com>, arm@kernel.org Subject: Re: [FAILURE] omap4430-sdp allnoconfig Date: Thu, 1 Aug 2013 11:59:56 -0700 [thread overview] Message-ID: <20130801185956.GK8868@codeaurora.org> (raw) In-Reply-To: <51FAAD73.3030103@ti.com> On 08/01, Santosh Shilimkar wrote: > This one looks fine for me. Can you send above in > a proper patch ? Let me know if you want me to > send it. > Here's a proper patch. Olof, can you please apply this on top of the merge (and maybe you can fix the merge again to handle the HAVE_CAN problem I mentioned earlier). ---8<--- Subject: [PATCH] ARM: OMAP2+: Fix fallout from localtimer divorce and SMP=n A recent patch ef3160c (ARM: OMAP2+: Divorce from local timer API, 2013-03-04) broke the omap build when SMP=n because the TWD functions are only compiled on SMP=y builds. Stub out the TWD calls when the TWD isn't built in to to keep everything building. arch/arm/mach-omap2/built-in.o: In function `omap4_local_timer_init': dss-common.c:(.init.text+0x1d90): undefined reference to `twd_local_timer_register' Reported-by: Russell King - ARM Linux <linux@arm.linux.org.uk> Cc: Santosh Shilimkar <santosh.shilimkar@ti.com> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- arch/arm/mach-omap2/timer.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c index 3b7caba..00dc53e 100644 --- a/arch/arm/mach-omap2/timer.c +++ b/arch/arm/mach-omap2/timer.c @@ -600,6 +600,7 @@ static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", #endif #ifdef CONFIG_ARCH_OMAP4 +#ifdef CONFIG_HAVE_ARM_TWD static DEFINE_TWD_LOCAL_TIMER(twd_local_timer, OMAP44XX_LOCAL_TWD_BASE, 29); void __init omap4_local_timer_init(void) { @@ -618,6 +619,12 @@ void __init omap4_local_timer_init(void) pr_err("twd_local_timer_register failed %d\n", err); } } +#else +void __init omap4_local_timer_init(void) +{ + omap4_sync32k_timer_init(); +} +#endif /* CONFIG_HAVE_ARM_TWD */ #endif /* CONFIG_ARCH_OMAP4 */ #ifdef CONFIG_SOC_OMAP5 -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation
WARNING: multiple messages have this Message-ID (diff)
From: sboyd@codeaurora.org (Stephen Boyd) To: linux-arm-kernel@lists.infradead.org Subject: [FAILURE] omap4430-sdp allnoconfig Date: Thu, 1 Aug 2013 11:59:56 -0700 [thread overview] Message-ID: <20130801185956.GK8868@codeaurora.org> (raw) In-Reply-To: <51FAAD73.3030103@ti.com> On 08/01, Santosh Shilimkar wrote: > This one looks fine for me. Can you send above in > a proper patch ? Let me know if you want me to > send it. > Here's a proper patch. Olof, can you please apply this on top of the merge (and maybe you can fix the merge again to handle the HAVE_CAN problem I mentioned earlier). ---8<--- Subject: [PATCH] ARM: OMAP2+: Fix fallout from localtimer divorce and SMP=n A recent patch ef3160c (ARM: OMAP2+: Divorce from local timer API, 2013-03-04) broke the omap build when SMP=n because the TWD functions are only compiled on SMP=y builds. Stub out the TWD calls when the TWD isn't built in to to keep everything building. arch/arm/mach-omap2/built-in.o: In function `omap4_local_timer_init': dss-common.c:(.init.text+0x1d90): undefined reference to `twd_local_timer_register' Reported-by: Russell King - ARM Linux <linux@arm.linux.org.uk> Cc: Santosh Shilimkar <santosh.shilimkar@ti.com> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- arch/arm/mach-omap2/timer.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c index 3b7caba..00dc53e 100644 --- a/arch/arm/mach-omap2/timer.c +++ b/arch/arm/mach-omap2/timer.c @@ -600,6 +600,7 @@ static OMAP_SYS_32K_TIMER_INIT(4, 1, "timer_32k_ck", "ti,timer-alwon", #endif #ifdef CONFIG_ARCH_OMAP4 +#ifdef CONFIG_HAVE_ARM_TWD static DEFINE_TWD_LOCAL_TIMER(twd_local_timer, OMAP44XX_LOCAL_TWD_BASE, 29); void __init omap4_local_timer_init(void) { @@ -618,6 +619,12 @@ void __init omap4_local_timer_init(void) pr_err("twd_local_timer_register failed %d\n", err); } } +#else +void __init omap4_local_timer_init(void) +{ + omap4_sync32k_timer_init(); +} +#endif /* CONFIG_HAVE_ARM_TWD */ #endif /* CONFIG_ARCH_OMAP4 */ #ifdef CONFIG_SOC_OMAP5 -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation
next prev parent reply other threads:[~2013-08-01 18:59 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-08-01 17:52 [FAILURE] omap4430-sdp allnoconfig Russell King - ARM Linux 2013-08-01 17:52 ` Russell King - ARM Linux 2013-08-01 18:11 ` Santosh Shilimkar 2013-08-01 18:11 ` Santosh Shilimkar 2013-08-01 18:27 ` Russell King - ARM Linux 2013-08-01 18:27 ` Russell King - ARM Linux 2013-08-01 18:31 ` Santosh Shilimkar 2013-08-01 18:31 ` Santosh Shilimkar 2013-08-01 18:34 ` Stephen Boyd 2013-08-01 18:34 ` Stephen Boyd 2013-08-01 18:37 ` Stephen Boyd 2013-08-01 18:37 ` Stephen Boyd 2013-08-01 18:48 ` Santosh Shilimkar 2013-08-01 18:48 ` Santosh Shilimkar 2013-08-01 18:59 ` Stephen Boyd [this message] 2013-08-01 18:59 ` Stephen Boyd 2013-08-01 19:01 ` Santosh Shilimkar 2013-08-01 19:01 ` Santosh Shilimkar 2013-08-01 19:02 ` Russell King - ARM Linux 2013-08-01 19:02 ` Russell King - ARM Linux 2013-08-01 20:06 ` Olof Johansson 2013-08-01 20:06 ` Olof Johansson 2013-08-02 20:44 ` Stephen Boyd 2013-08-02 20:44 ` Stephen Boyd 2013-08-02 20:50 ` Olof Johansson 2013-08-02 20:50 ` Olof Johansson 2013-08-02 20:56 ` Stephen Boyd 2013-08-02 20:56 ` Stephen Boyd 2013-08-02 20:59 ` Olof Johansson 2013-08-02 20:59 ` Olof Johansson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20130801185956.GK8868@codeaurora.org \ --to=sboyd@codeaurora.org \ --cc=arm@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=olof@lixom.net \ --cc=santosh.shilimkar@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.