[PATCH v1 08/17] ACPI: PAD: Rearrange acpi_pad_notify()

From: Rafael J. Wysocki

Date: Thu May 21 2026 - 10:37:35 EST


From: "Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx>

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@xxxxxxxxx>
---
drivers/acpi/acpi_pad.c | 13 +++++--------
1 file 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)
--
2.51.0