mbox series

[v1,00/34] Remove .notify callback in acpi_device_ops

Message ID 20230512140222.124868-1-michal.wilczynski@intel.com
Headers show
Series Remove .notify callback in acpi_device_ops | expand

Message

Michal Wilczynski May 12, 2023, 2:01 p.m. UTC 
Currently drivers support ACPI event handlers by defining .notify
callback in acpi_device_ops. This solution is suboptimal as event
handler installer installs intermediary function acpi_notify_device as a
handler in every driver. Also this approach requires extra variable
'flags' for specifying event types that the driver want to subscribe to.
Additionally this is a pre-work required to align acpi_driver with
platform_driver and eventually replace acpi_driver with platform_driver.

Remove .notify callback from the acpi_device_ops. Replace it with each
driver installing and removing it's event handlers.

Michal Wilczynski (34):
  acpi: Adjust functions installing bus event handlers
  acpi/ac: Move handler installing logic to driver
  acpi/video: Move handler installing logic to driver
  acpi/battery: Move handler installing logic to driver
  acpi/button: Move handler installing logic to driver
  acpi/hed: Move handler installing logic to driver
  acpi/nfit: Move handler installing logic to driver
  acpi/thermal: Move handler installing logic to driver
  acpi/tiny-power-button: Move handler installing logic to driver
  hwmon: Move handler installing logic to driver
  iio/acpi-als: Move handler installing logic to driver
  platform/chromeos_tbmc: Move handler installing logic to driver
  platform/wilco_ec: Move handler installing logic to driver
  platform/surface/button: Move handler installing logic to driver
  platform/x86/acer-wireless: Move handler installing logic to driver
  platform/x86/asus-laptop: Move handler installing logic to driver
  platform/x86/asus-wireless: Move handler installing logic to driver
  platform/x86/classmate-laptop: Move handler installing logic to driver
  platform/x86/dell/dell-rbtn: Move handler installing logic to driver
  platform/x86/eeepc-laptop: Move handler installing logic to driver
  platform/x86/fujitsu-laptop: Move handler installing logic to driver
  platform/x86/lg-laptop: Move handler installing logic to driver
  platform/x86/panasonic-laptop: Move handler installing logic to driver
  platform/x86/system76_acpi: Move handler installing logic to driver
  platform/x86/topstar-laptop: Move handler installing logic to driver
  platform/x86/toshiba_acpi: Move handler installing logic to driver
  platform/x86/toshiba_bluetooth: Move handler installing logic to
    driver
  platform/x86/toshiba_haps: Move handler installing logic to driver
  platform/x86/wireless-hotkey: Move handler installing logic to driver
  platform/x86/xo15-ebook: Move handler installing logic to driver
  virt/vmgenid: Move handler installing logic to driver
  acpi/bus: Remove installing/removing notify handlers from probe/remove
  acpi/bus: Remove redundant functions
  acpi/bus: Remove notify callback and flags

 drivers/acpi/ac.c                             | 14 +++--
 drivers/acpi/acpi_video.c                     | 18 +++++--
 drivers/acpi/battery.c                        | 14 +++--
 drivers/acpi/bus.c                            | 53 ++++++-------------
 drivers/acpi/button.c                         | 18 +++++--
 drivers/acpi/hed.c                            |  6 +--
 drivers/acpi/nfit/core.c                      | 11 ++--
 drivers/acpi/thermal.c                        | 20 ++++---
 drivers/acpi/tiny-power-button.c              | 16 +++---
 drivers/hwmon/acpi_power_meter.c              | 15 +++---
 drivers/iio/light/acpi-als.c                  | 21 ++++++--
 drivers/platform/chrome/chromeos_tbmc.c       | 14 +++--
 drivers/platform/chrome/wilco_ec/event.c      | 21 ++++----
 drivers/platform/surface/surfacepro3_button.c | 19 +++++--
 drivers/platform/x86/acer-wireless.c          | 22 ++++++--
 drivers/platform/x86/asus-laptop.c            | 16 ++++--
 drivers/platform/x86/asus-wireless.c          | 24 +++++----
 drivers/platform/x86/classmate-laptop.c       | 12 +++--
 drivers/platform/x86/dell/dell-rbtn.c         | 15 ++++--
 drivers/platform/x86/eeepc-laptop.c           | 18 +++++--
 drivers/platform/x86/fujitsu-laptop.c         | 17 ++++--
 drivers/platform/x86/lg-laptop.c              | 12 +++--
 drivers/platform/x86/panasonic-laptop.c       | 17 ++++--
 drivers/platform/x86/system76_acpi.c          | 28 ++++++----
 drivers/platform/x86/topstar-laptop.c         | 16 ++++--
 drivers/platform/x86/toshiba_acpi.c           | 17 ++++--
 drivers/platform/x86/toshiba_bluetooth.c      | 27 +++++++---
 drivers/platform/x86/toshiba_haps.c           |  9 ++--
 drivers/platform/x86/wireless-hotkey.c        | 23 +++++---
 drivers/platform/x86/xo15-ebook.c             | 11 ++--
 drivers/virt/vmgenid.c                        | 16 ++++--
 include/acpi/acpi_bus.h                       | 10 ++--
 32 files changed, 380 insertions(+), 190 deletions(-)