On Fri, Dec 21, 2012 at 10:02:01AM +0000, Santosh Shilimkar wrote: > On Tuesday 18 December 2012 05:36 PM, Mark Rutland wrote: > > Currently we only provide an implementation of smp_timer_broadcast in > > smp.c if GENERIC_CLOCKEVENTS_BROADCAST is selected. As > > smp_timer_broadcast is only used in smp.c, smp.c depends on SMP, and > > GENERIC_CLOCKEVENTS_BROADCAST is selected by SMP, this is unnecessary. > > > > This patch removes the redundant guard. > > > > Signed-off-by: Mark Rutland <mark.rutl...@arm.com> > > --- > CONFIG_GENERIC_CLOCKEVENTS_BROADCAST isn't mandatory to > have SMP kernel working so the below select in ARM arch > kconfig not seems to entirely accurate. SMP kernel will > still boot with !GENERIC_CLOCKEVENTS_BROADCAST. > > select GENERIC_CLOCKEVENTS_BROADCAST if SMP
Agreed, the selection is not entirely accurate. I'd be happy to see it made more fine-grained (i.e. selected by platforms) instead. If people want that I'm happy to drop this patch. > The issue comes only for deeper CPU power C-states. > Anyway, you patch is correct from the current code point of > view. > > Regards > Santosh Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/