diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-05-21 17:05:22 +0300 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-05-25 17:43:51 +0300 |
| commit | dd32e1966f42417ffad9b64b3837a0bbb0c2d035 (patch) | |
| tree | f82d0ea47a264e70aef77ea25e8802ad9d92a713 | |
| parent | d3f13e75bf2c781b23b8fa8320e5ecb3a0b4df2e (diff) | |
| download | linux-dd32e1966f42417ffad9b64b3837a0bbb0c2d035.tar.xz | |
ACPI: PAD: Rearrange acpi_pad_notify()
Use an if () in acpi_pad_notify() instead of a switch () statement to
make the code somewhat easier to follow and reduce its indentation
level.
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/3345485.5fSG56mABF@rafael.j.wysocki
| -rw-r--r-- | drivers/acpi/acpi_pad.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/acpi/acpi_pad.c b/drivers/acpi/acpi_pad.c index ec94b09bb747..91d32da76f8f 100644 --- a/drivers/acpi/acpi_pad.c +++ b/drivers/acpi/acpi_pad.c @@ -409,16 +409,13 @@ static void acpi_pad_notify(acpi_handle handle, u32 event, void *data) { struct acpi_device *adev = data; - switch (event) { - case ACPI_PROCESSOR_AGGREGATOR_NOTIFY: - acpi_pad_handle_notify(handle); - acpi_bus_generate_netlink_event("acpi_pad", - dev_name(&adev->dev), event, 0); - break; - default: + if (event != ACPI_PROCESSOR_AGGREGATOR_NOTIFY) { pr_warn("Unsupported event [0x%x]\n", event); - break; + return; } + + acpi_pad_handle_notify(handle); + acpi_bus_generate_netlink_event("acpi_pad", dev_name(&adev->dev), event, 0); } static int acpi_pad_probe(struct platform_device *pdev) |