Commit 31c025b5 authored by Yaowei Bai's avatar Yaowei Bai Committed by Linus Torvalds

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: default avatarYaowei Bai <baiyaowei@cmss.chinamobile.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent df0108c5
...@@ -164,10 +164,10 @@ static const char *panic_later, *panic_param; ...@@ -164,10 +164,10 @@ static const char *panic_later, *panic_param;
extern const struct obs_kernel_param __setup_start[], __setup_end[]; 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; const struct obs_kernel_param *p;
int had_early_param = 0; bool had_early_param = false;
p = __setup_start; p = __setup_start;
do { do {
...@@ -179,13 +179,13 @@ static int __init obsolete_checksetup(char *line) ...@@ -179,13 +179,13 @@ static int __init obsolete_checksetup(char *line)
* Keep iterating, as we can have early * Keep iterating, as we can have early
* params and __setups of same names 8( */ * params and __setups of same names 8( */
if (line[n] == '\0' || line[n] == '=') if (line[n] == '\0' || line[n] == '=')
had_early_param = 1; had_early_param = true;
} else if (!p->setup_func) { } else if (!p->setup_func) {
pr_warn("Parameter %s is obsolete, ignored\n", pr_warn("Parameter %s is obsolete, ignored\n",
p->str); p->str);
return 1; return true;
} else if (p->setup_func(line + n)) } else if (p->setup_func(line + n))
return 1; return true;
} }
p++; p++;
} while (p < __setup_end); } while (p < __setup_end);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment