From: davidb@codeaurora•org (David Brown)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v2 14/15] ARM: local timers: convert MSM to runtime registration interface
Date: Fri, 13 Jan 2012 15:33:34 -0800 [thread overview]
Message-ID: <20120113233334.GA21168@codeaurora.org> (raw)
In-Reply-To: <20120113224709.GH1068@n2100.arm.linux.org.uk>
On Fri, Jan 13, 2012 at 10:47:09PM +0000, Russell King - ARM Linux wrote:
> On Fri, Jan 13, 2012 at 02:37:40PM -0800, David Brown wrote:
> > On Fri, Jan 13, 2012 at 06:28:41PM +0000, Marc Zyngier wrote:
> > > Convert the MSM timers to the runtime registration interface.
> > >
> > > Cc: Stephen Boyd <sboyd@codeaurora•org>
> > > Tested-by: David Brown <davidb@codeaurora•org>
> > > Acked-by: David Brown <davidb@codeaurora•org>
> > > Signed-off-by: Marc Zyngier <marc.zyngier@arm•com>
> >
> > Ok, this is definitely now consistenly causing cpu hotplug to fail.
> > I'll look in to this further, but for now, please removed my Tested-by
> > and Acked-by.
>
> That'll be because the local_timer_ops structure is marked __initdata
> but is used after init has completed. It should be marked __cpuinitdata
> if it's used for CPU bringup (which it is.)
:-) I guess I should be checking my email while looking for
problems. But, I found it anyway.
Thanks,
David
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2012-01-13 23:33 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-13 18:28 [PATCH v2 00/15] Runtime registration for local timers Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 01/15] ARM: smp_twd: make local_timer_stop a symbol instead of a #define Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 02/15] ARM: local timers: introduce a new registration interface Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 03/15] ARM: smp_twd: add runtime registration support Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 04/15] ARM: smp_twd: add device tree support Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 05/15] ARM: OMAP4: convert to twd_local_timer_register() interface Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 06/15] ARM: plat-versatile: " Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 07/15] ARM: tegra: " Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 08/15] ARM: shmobile: " Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 09/15] ARM: ux500: " Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 10/15] ARM: highbank: " Marc Zyngier
2012-01-13 19:07 ` Rob Herring
2012-01-14 2:41 ` Shawn Guo
2012-01-14 8:46 ` Russell King - ARM Linux
2012-01-14 15:06 ` Rob Herring
2012-01-13 18:28 ` [PATCH v2 11/15] ARM: imx6q: " Marc Zyngier
2012-01-14 2:47 ` Shawn Guo
2012-01-13 18:28 ` [PATCH v2 12/15] ARM: smp_twd: remove old local timer interface Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 13/15] ARM: local timers: convert exynos to runtime registration interface Marc Zyngier
2012-01-13 23:29 ` David Brown
2012-01-13 18:28 ` [PATCH v2 14/15] ARM: local timers: convert MSM " Marc Zyngier
2012-01-13 22:37 ` David Brown
2012-01-13 22:47 ` Russell King - ARM Linux
2012-01-13 23:33 ` David Brown [this message]
2012-01-13 23:26 ` David Brown
2012-01-13 23:29 ` Russell King - ARM Linux
2012-01-14 16:27 ` Marc Zyngier
2012-01-13 18:28 ` [PATCH v2 15/15] ARM: local timers: make the runtime registration interface mandatory Marc Zyngier
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=20120113233334.GA21168@codeaurora.org \
--to=davidb@codeaurora$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox