On 2023/6/5 18:35, Sudeep Holla wrote:
Move all of the ARM-specific initialization into one function namely acpi_arm_init(), so it is not necessary to modify/update bus.c every time a new piece of it is added. Cc: Lorenzo Pieralisi <lpieralisi@xxxxxxxxxx> Cc: Hanjun Guo <guohanjun@xxxxxxxxxx> Cc: Rafael J. Wysocki <rafael@xxxxxxxxxx> Suggested-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> Link: https://lore.kernel.org/r/CAJZ5v0iBZRZmV_oU+VurqxnVMbFN_ttqrL=cLh0sUH+=u0PYsw@xxxxxxxxxxxxxx Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx> --- drivers/acpi/arm64/Makefile | 2 +- drivers/acpi/arm64/agdi.c | 2 +- drivers/acpi/arm64/apmt.c | 2 +- drivers/acpi/arm64/init.c | 10 ++++++++++ drivers/acpi/arm64/init.h | 20 ++++++++++++++++++++ drivers/acpi/arm64/iort.c | 1 + drivers/acpi/bus.c | 7 +------ include/linux/acpi.h | 6 ++++++ include/linux/acpi_agdi.h | 13 ------------- include/linux/acpi_apmt.h | 19 ------------------- include/linux/acpi_iort.h | 2 -- 11 files changed, 41 insertions(+), 43 deletions(-) create mode 100644 drivers/acpi/arm64/init.c create mode 100644 drivers/acpi/arm64/init.h delete mode 100644 include/linux/acpi_agdi.h delete mode 100644 include/linux/acpi_apmt.h
I like this consolidation a lot, Reviewed-by: Hanjun Guo <guohanjun@xxxxxxxxxx>