diff options
| author | Ingo Molnar <mingo@kernel.org> | 2015-03-05 02:28:48 -0500 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2015-03-05 03:23:12 -0500 |
| commit | c281b94570ab711fdf21e81bdfb3d79764492bcf (patch) | |
| tree | 2316728b6d6312d7729a4335206d80a0d18a91a1 /include/linux/init.h | |
| parent | e61980a70245715ab39cbee2b9d6e6afc1ec37d4 (diff) | |
init.h: Clean up the __setup()/early_param() macros
Make it all a bit easier on the eyes:
- Move the __setup_param() lines right after their init functions
- Use consistent vertical spacing
- Use more horizontal spacing to make it look like regular C code
- Use standard comment style
Cc: Luis R. Rodriguez <mcgrof@suse.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Borislav Petkov <bp@suse.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: David Vrabel <david.vrabel@citrix.com>
Cc: Dexuan Cui <decui@microsoft.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Jan Beulich <JBeulich@suse.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Cc: Juergen Gross <jgross@suse.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Toshi Kani <toshi.kani@hp.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: Xishi Qiu <qiuxishi@huawei.com>
Cc: julia.lawall@lip6.fr
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'include/linux/init.h')
| -rw-r--r-- | include/linux/init.h | 49 |
1 files changed, 27 insertions, 22 deletions
diff --git a/include/linux/init.h b/include/linux/init.h index bc11ff96f336..21b6d768edd7 100644 --- a/include/linux/init.h +++ b/include/linux/init.h | |||
| @@ -253,34 +253,39 @@ struct obs_kernel_param { | |||
| 253 | * obs_kernel_param "array" too far apart in .init.setup. | 253 | * obs_kernel_param "array" too far apart in .init.setup. |
| 254 | */ | 254 | */ |
| 255 | #define __setup_param(str, unique_id, fn, early) \ | 255 | #define __setup_param(str, unique_id, fn, early) \ |
| 256 | static const char __setup_str_##unique_id[] __initconst \ | 256 | static const char __setup_str_##unique_id[] __initconst \ |
| 257 | __aligned(1) = str; \ | 257 | __aligned(1) = str; \ |
| 258 | static struct obs_kernel_param __setup_##unique_id \ | 258 | static struct obs_kernel_param __setup_##unique_id \ |
| 259 | __used __section(.init.setup) \ | 259 | __used __section(.init.setup) \ |
| 260 | __attribute__((aligned((sizeof(long))))) \ | 260 | __attribute__((aligned((sizeof(long))))) \ |
| 261 | = { __setup_str_##unique_id, fn, early } | 261 | = { __setup_str_##unique_id, fn, early } |
| 262 | 262 | ||
| 263 | #define __setup(str, fn) \ | 263 | #define __setup(str, fn) \ |
| 264 | __setup_param(str, fn, fn, 0) | 264 | __setup_param(str, fn, fn, 0) |
| 265 | 265 | ||
| 266 | /* NOTE: fn is as per module_param, not __setup! Emits warning if fn | 266 | /* |
| 267 | * returns non-zero. */ | 267 | * NOTE: fn is as per module_param, not __setup! |
| 268 | #define early_param(str, fn) \ | 268 | * Emits warning if fn returns non-zero. |
| 269 | */ | ||
| 270 | #define early_param(str, fn) \ | ||
| 269 | __setup_param(str, fn, fn, 1) | 271 | __setup_param(str, fn, fn, 1) |
| 270 | 272 | ||
| 271 | #define early_param_on_off(str_on, str_off, var, config) \ | 273 | #define early_param_on_off(str_on, str_off, var, config) \ |
| 272 | int var = IS_ENABLED(config); \ | 274 | \ |
| 273 | static int __init parse_##var##_on(char *arg) \ | 275 | int var = IS_ENABLED(config); \ |
| 274 | { \ | 276 | \ |
| 275 | var = 1; \ | 277 | static int __init parse_##var##_on(char *arg) \ |
| 276 | return 0; \ | 278 | { \ |
| 277 | } \ | 279 | var = 1; \ |
| 278 | static int __init parse_##var##_off(char *arg) \ | 280 | return 0; \ |
| 279 | { \ | 281 | } \ |
| 280 | var = 0; \ | 282 | __setup_param(str_on, parse_##var##_on, parse_##var##_on, 1); \ |
| 281 | return 0; \ | 283 | \ |
| 282 | } \ | 284 | static int __init parse_##var##_off(char *arg) \ |
| 283 | __setup_param(str_on, parse_##var##_on, parse_##var##_on, 1); \ | 285 | { \ |
| 286 | var = 0; \ | ||
| 287 | return 0; \ | ||
| 288 | } \ | ||
| 284 | __setup_param(str_off, parse_##var##_off, parse_##var##_off, 1) | 289 | __setup_param(str_off, parse_##var##_off, parse_##var##_off, 1) |
| 285 | 290 | ||
| 286 | /* Relies on boot_command_line being set */ | 291 | /* Relies on boot_command_line being set */ |
