diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2014-01-24 15:42:17 -0700 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-02-03 10:39:32 -0700 |
commit | 962fe9c91c5faa271e21c6ef589ff1df02fa13ed (patch) | |
tree | 102052ba5ff2b6835966e70a5b987bb014d623ff /drivers/acpi | |
parent | 486c79b5002e4a75c1c3614180702196ec6d904d (diff) | |
download | op-kernel-dev-962fe9c91c5faa271e21c6ef589ff1df02fa13ed.zip op-kernel-dev-962fe9c91c5faa271e21c6ef589ff1df02fa13ed.tar.gz |
ACPI / numa: Simplify acpi_get_node() style
Simplify control flow by removing local variable initialization and
returning a constant as soon as possible. No functional change.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi')
-rw-r--r-- | drivers/acpi/numa.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c index dab7dac..a7e779bf 100644 --- a/drivers/acpi/numa.c +++ b/drivers/acpi/numa.c @@ -333,12 +333,12 @@ int acpi_get_pxm(acpi_handle h) int acpi_get_node(acpi_handle handle) { - int pxm, node = NUMA_NO_NODE; + int pxm; pxm = acpi_get_pxm(handle); - if (pxm >= 0 && pxm < MAX_PXM_DOMAINS) - node = acpi_map_pxm_to_node(pxm); + if (pxm < 0 || pxm >= MAX_PXM_DOMAINS) + return NUMA_NO_NODE; - return node; + return acpi_map_pxm_to_node(pxm); } EXPORT_SYMBOL(acpi_get_node); |