diff options
author | Yaowei Bai <baiyaowei@cmss.chinamobile.com> | 2015-11-26 12:58:27 +1100 |
---|---|---|
committer | Stephen Rothwell <sfr@canb.auug.org.au> | 2015-11-26 12:58:27 +1100 |
commit | 0adc515b21a57a0e531b2d5d5e13ecfe95e71d6c (patch) | |
tree | 7d7af3ee406c339b6f1afcc3dc9ef9daad214782 | |
parent | 37721bfb1505cf0cd3741646ffa27717aee048ce (diff) |
init/main.c: obsolete_checksetup can be boolean
Make obsolete_checksetup() return bool due to this particular function
only using either one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyaowei@cmss.chinamobile.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r-- | init/main.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/init/main.c b/init/main.c index 9e64d7097f1a..23449b7dfe7a 100644 --- a/init/main.c +++ b/init/main.c @@ -164,10 +164,10 @@ static const char *panic_later, *panic_param; extern const struct obs_kernel_param __setup_start[], __setup_end[]; -static int __init obsolete_checksetup(char *line) +static bool __init obsolete_checksetup(char *line) { const struct obs_kernel_param *p; - int had_early_param = 0; + bool had_early_param = false; p = __setup_start; do { @@ -179,13 +179,13 @@ static int __init obsolete_checksetup(char *line) * Keep iterating, as we can have early * params and __setups of same names 8( */ if (line[n] == '\0' || line[n] == '=') - had_early_param = 1; + had_early_param = true; } else if (!p->setup_func) { pr_warn("Parameter %s is obsolete, ignored\n", p->str); - return 1; + return true; } else if (p->setup_func(line + n)) - return 1; + return true; } p++; } while (p < __setup_end); |