diff options
| author | Chris Metcalf <cmetcalf@tilera.com> | 2010-11-01 20:25:16 +0100 |
|---|---|---|
| committer | Chris Metcalf <cmetcalf@tilera.com> | 2010-11-01 20:30:53 +0100 |
| commit | d59e609d6568ba5ab23c256f412ac5ec360722c1 (patch) | |
| tree | 2caad01eb530fc5ab31d189c8b9749021a596934 | |
| parent | arch/tile: bomb raw_local_irq_ to arch_local_irq_ (diff) | |
| download | linux-d59e609d6568ba5ab23c256f412ac5ec360722c1.tar.xz linux-d59e609d6568ba5ab23c256f412ac5ec360722c1.zip | |
arch/tile: avoid __must_check warning on one strict_strtol check
For the "initfree" boot argument it's not that big a deal, but
to avoid warnings in the code, we check for a valid value before
allowing the specified argument to override the kernel default.
Signed-off-by: Chris Metcalf <cmetcalf@tilera.com>
| -rw-r--r-- | arch/tile/mm/init.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/arch/tile/mm/init.c b/arch/tile/mm/init.c index 78e1982cb6c9..0b9ce69b0ee5 100644 --- a/arch/tile/mm/init.c +++ b/arch/tile/mm/init.c @@ -988,8 +988,12 @@ static long __write_once initfree = 1; /* Select whether to free (1) or mark unusable (0) the __init pages. */ static int __init set_initfree(char *str) { - strict_strtol(str, 0, &initfree); - pr_info("initfree: %s free init pages\n", initfree ? "will" : "won't"); + long val; + if (strict_strtol(str, 0, &val)) { + initfree = val; + pr_info("initfree: %s free init pages\n", + initfree ? "will" : "won't"); + } return 1; } __setup("initfree=", set_initfree); |