[PATCH v6 02/15] ARM: SoC: Add per SoC SMP and CPU hotplug operations

Marc Zyngier marc.zyngier at arm.com
Wed Feb 8 20:00:48 EST 2012


On Wed, 8 Feb 2012 15:42:53 -0800, David Brown <davidb at codeaurora.org>
wrote:
> On Wed, Feb 08, 2012 at 12:30:46AM +0000, Marc Zyngier wrote:
> 
>> +#ifdef CONFIG_SMP
>> +#define soc_smp_init_ops(ops)		.smp_init_ops = &(ops),
>> +#define soc_smp_ops(ops)		.smp_ops = &(ops),
>> +extern void soc_smp_ops_register(struct arm_soc_smp_init_ops *,
>> +				 struct arm_soc_smp_ops *);
>> +#else
>> +#define soc_smp_init_ops(ops)		/* empty */
>> +#define soc_smp_ops(ops)		/* empty */
>> +#define soc_smp_ops_register(a,b)	do {} while(0)
>> +#endif
> 
> I'm not sure if this bothers anyone else, but these macros mess up
> editors that know how to indent C code, since the comma ends up inside
> the macro.  I'm not sure I have a better idea, though.

Emacs seem quite happy when I wrote the code... Shall we re-open the
editor war? ;-)

        M.
-- 
Fast, cheap, reliable. Pick two.



More information about the linux-arm-kernel mailing list