diff options
| author |   Johan Hovold <johan@kernel.org> | 2019-01-18 15:03:07 +0100 |
|---|---|---|
| committer |   Palmer Dabbelt <palmer@sifive.com> | 2019-02-11 15:35:33 -0800 |
| commit | e3d794d555cda31d48c89bdbc96ce862857be93f (patch) | |
| tree | a024ce1610facb6f5f54ffdd2824024c478d3d6b /arch/riscv | |
| parent | riscv: fix riscv_of_processor_hartid() comment (diff) | |
| download | linux-dev-e3d794d555cda31d48c89bdbc96ce862857be93f.tar.xz linux-dev-e3d794d555cda31d48c89bdbc96ce862857be93f.zip | |
riscv: treat cpu devicetree nodes without status as enabled
Follow the Linux convention and treat devicetree nodes without a status
property as enabled rather than disabled, while also allowing "ok" as a
shorthand for "okay".
Signed-off-by: Johan Hovold <johan@kernel.org>
Reviewed-by: Paul Walmsley <paul.walmsley@sifive.com>
Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
Diffstat (limited to 'arch/riscv')
| -rw-r--r-- | arch/riscv/kernel/cpu.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/arch/riscv/kernel/cpu.c b/arch/riscv/kernel/cpu.c index 974d374fd36b..d1d9bfd5a89f 100644 --- a/arch/riscv/kernel/cpu.c +++ b/arch/riscv/kernel/cpu.c @@ -22,7 +22,7 @@ */ int riscv_of_processor_hartid(struct device_node *node) { - const char *isa, *status; + const char *isa; u32 hart; if (!of_device_is_compatible(node, "riscv")) { @@ -39,12 +39,8 @@ int riscv_of_processor_hartid(struct device_node *node) return -ENODEV; } - if (of_property_read_string(node, "status", &status)) { - pr_warn("CPU with hartid=%d has no \"status\" property\n", hart); - return -ENODEV; - } - if (strcmp(status, "okay")) { - pr_info("CPU with hartid=%d has a non-okay status of \"%s\"\n", hart, status); + if (!of_device_is_available(node)) { + pr_info("CPU with hartid=%d is not available\n", hart); return -ENODEV; } |