[PATCH v5 00/10] Remove .notify callback in acpi_device_ops
From: Michal Wilczynski
Date: Fri Jun 16 2023 - 12:51:01 EST
*** IMPORTANT ***
This is part 1 - only drivers in acpi directory to ease up review
process. Rest of the drivers will be handled in separate patchsets.
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.
v5:
- rebased on top of Rafael changes [1], they're not merged yet
- fixed rollback in multiple drivers so they don't leak resources on
failure
- made this part 1, meaning only drivers in acpi directory, rest of
the drivers will be handled in separate patchsets to ease up review
v4:
- added one commit for previously missed driver sony-laptop,
refactored return statements, added NULL check for event installer
v3:
- lkp still reported some failures for eeepc, fujitsu and
toshiba_bluetooth, fix those
v2:
- fix compilation errors for drivers
[1]: https://lore.kernel.org/linux-acpi/1847933.atdPhlSkOF@kreacher/
Michal Wilczynski (10):
acpi/bus: Introduce wrappers for ACPICA event handler install/remove
acpi/bus: Set driver_data to NULL every time .add() fails
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/hed: Move handler installing logic to driver
acpi/nfit: Move acpi_nfit_notify() before acpi_nfit_add()
acpi/nfit: Improve terminator line in acpi_nfit_ids
acpi/nfit: Move handler installing logic to driver
acpi/thermal: Move handler installing logic to driver
drivers/acpi/ac.c | 33 ++++++++++++++++++++++++---------
drivers/acpi/acpi_video.c | 26 ++++++++++++++++++++++----
drivers/acpi/battery.c | 30 ++++++++++++++++++++++++------
drivers/acpi/bus.c | 30 +++++++++++++++++++++++++++++-
drivers/acpi/hed.c | 17 ++++++++++++++---
drivers/acpi/nfit/core.c | 32 ++++++++++++++++++++++----------
drivers/acpi/thermal.c | 28 ++++++++++++++++++++++------
include/acpi/acpi_bus.h | 6 ++++++
8 files changed, 163 insertions(+), 39 deletions(-)
--
2.41.0