[PATCH] ARM: mvebu: fix compilation warning on Armada 370 (i.e. non-SMP)

Arnaud Ebalard arno at natisbad.org
Mon Jan 20 16:42:18 EST 2014


Hi Arnd,

Arnd Bergmann <arnd at arndb.de> writes:

> On Monday 20 January 2014 21:14:13 Arnaud Ebalard wrote:
>> diff --git a/drivers/irqchip/irq-armada-370-xp.c b/drivers/irqchip/irq-armada-370-xp.c
>> index 433cc8568dec..499c5070f841 100644
>> --- a/drivers/irqchip/irq-armada-370-xp.c
>> +++ b/drivers/irqchip/irq-armada-370-xp.c
>> @@ -59,7 +59,9 @@
>>  #define PCI_MSI_DOORBELL_END                    (32)
>>  #define PCI_MSI_DOORBELL_MASK                   0xFFFF0000
>>  
>> +#ifdef CONFIG_SMP
>>  static DEFINE_RAW_SPINLOCK(irq_controller_lock);
>> +#endif
>>  
>>  static void __iomem *per_cpu_
>
> How about moving it inside of the existing #ifdef?
>
> That also brings it closer to the one place it's used.

I also hesitated but decided to let the declaration of this global
at the beginning of the file. I will send a v2 with the declaration
moved inside the existing #ifdef.

Cheers,

a+



More information about the linux-arm-kernel mailing list