Messages in this thread | | | Date | Mon, 28 Mar 2005 13:43:41 -0800 | From | "H. Peter Anvin" <> | Subject | Re: [RFC 2.6.11.2 1/1] Add reboot fixup for gx1/cs5530a |
| |
H. Peter Anvin wrote: > Sam Ravnborg wrote: > >> >> And hide this ifdef in same hederfile. >> >> #ifndef CONFIG_X86_REBOOTFIXUPS >> #define mach_reboot_fixups do {} while (0); >> #endif >> > > No semicolon; besides: > > #define mach_reboot_fixups(x) ((void)0) > > ... is better all around. >
Correction, that should have been:
#define mach_reboot_fixups(x) ((void)(x))
... in order to preserve side effects.
-hpa - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |