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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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>



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux