On 08/12, Nicolas Pitre wrote: > On Tue, 12 Aug 2014, Stephen Boyd wrote: > > > > diff --git a/drivers/irqchip/irq-gic.c b/drivers/irqchip/irq-gic.c > > index 7c131cf7cc13..c1dc65209899 100644 > > --- a/drivers/irqchip/irq-gic.c > > +++ b/drivers/irqchip/irq-gic.c > > @@ -81,6 +81,11 @@ static DEFINE_RAW_SPINLOCK(irq_controller_lock); > > #define NR_GIC_CPU_IF 8 > > static u8 gic_cpu_map[NR_GIC_CPU_IF] __read_mostly; > > > > +#ifdef CONFIG_BL_SWITCHER > > +/* Synchronize switching CPU interface and sending SGIs */ > > +static DEFINE_RAW_SPINLOCK(gic_sgi_lock); > > What about ... > > #define sgi_map_lock(flags) raw_spin_lock_irqsave(&gic_sgi_lock, flags) > #define sgi_map_unlock(flags) raw_spin_unlock_irqrestore(&gic_sgi_lock, > flags) > #else sgi_map_lock(flags) (void)(flags) > #define sgi_map_unlock(flags) (void)(flags) > > That would avoid some #ifdefs in the main code. Sure. v4 shortly. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
