[PATCH] ACPI: bus: Consolidate all arm specific initialisation into acpi_arm_init()

Hanjun Guo guohanjun at huawei.com
Mon Jun 5 23:57:58 PDT 2023


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 at kernel.org>
> Cc: Hanjun Guo <guohanjun at huawei.com>
> Cc: Rafael J. Wysocki <rafael at kernel.org>
> Suggested-by: Rafael J. Wysocki <rafael.j.wysocki at intel.com>
> Link: https://lore.kernel.org/r/CAJZ5v0iBZRZmV_oU+VurqxnVMbFN_ttqrL=cLh0sUH+=u0PYsw@mail.gmail.com
> Signed-off-by: Sudeep Holla <sudeep.holla at arm.com>
> ---
>   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 at huawei.com>



More information about the linux-arm-kernel mailing list