* [PATCH v4 12/35] platform/chromeos_tbmc: Move handler installing logic to driver
[not found] <20230601131739.300760-3-michal.wilczynski@intel.com>
@ 2023-06-01 13:17 ` Michal Wilczynski
2023-06-01 13:17 ` [PATCH v4 13/35] platform/wilco_ec: " Michal Wilczynski
1 sibling, 0 replies; 2+ messages in thread
From: Michal Wilczynski @ 2023-06-01 13:17 UTC (permalink / raw)
To: Benson Leung
Cc: Michal Wilczynski, chrome-platform, linux-acpi,
platform-driver-x86, rafael
Currently logic for installing notifications from ACPI devices is
implemented using notify callback in struct acpi_driver. Preparations
are being made to replace acpi_driver with more generic struct
platform_driver, which doesn't contain notify callback. Furthermore
as of now handlers are being called indirectly through
acpi_notify_device(), which decreases performance.
Call acpi_device_install_event_handler() at the end of .add() callback.
Call acpi_device_remove_event_handler() at the beginning of .remove()
callback. Change arguments passed to the notify callback to match with
what's required by acpi_device_install_event_handler().
Signed-off-by: Michal Wilczynski <michal.wilczynski@intel.com>
---
drivers/platform/chrome/chromeos_tbmc.c | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/chrome/chromeos_tbmc.c b/drivers/platform/chrome/chromeos_tbmc.c
index d1cf8f3463ce..ac0f43fc530a 100644
--- a/drivers/platform/chrome/chromeos_tbmc.c
+++ b/drivers/platform/chrome/chromeos_tbmc.c
@@ -45,8 +45,10 @@ static __maybe_unused int chromeos_tbmc_resume(struct device *dev)
return chromeos_tbmc_query_switch(adev, adev->driver_data);
}
-static void chromeos_tbmc_notify(struct acpi_device *adev, u32 event)
+static void chromeos_tbmc_notify(acpi_handle handle, u32 event, void *data)
{
+ struct acpi_device *adev = data;
+
acpi_pm_wakeup_event(&adev->dev);
switch (event) {
case 0x80:
@@ -92,7 +94,13 @@ static int chromeos_tbmc_add(struct acpi_device *adev)
return ret;
}
device_init_wakeup(dev, true);
- return 0;
+
+ return acpi_device_install_event_handler(adev, ACPI_DEVICE_NOTIFY, chromeos_tbmc_notify);
+}
+
+static void chromeos_tbmc_remove(struct acpi_device *adev)
+{
+ acpi_device_remove_event_handler(adev, ACPI_DEVICE_NOTIFY, chromeos_tbmc_notify);
}
static const struct acpi_device_id chromeos_tbmc_acpi_device_ids[] = {
@@ -110,7 +118,7 @@ static struct acpi_driver chromeos_tbmc_driver = {
.ids = chromeos_tbmc_acpi_device_ids,
.ops = {
.add = chromeos_tbmc_add,
- .notify = chromeos_tbmc_notify,
+ .remove = chromeos_tbmc_remove,
},
.drv.pm = &chromeos_tbmc_pm_ops,
};
--
2.40.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH v4 13/35] platform/wilco_ec: Move handler installing logic to driver
[not found] <20230601131739.300760-3-michal.wilczynski@intel.com>
2023-06-01 13:17 ` [PATCH v4 12/35] platform/chromeos_tbmc: Move handler installing logic to driver Michal Wilczynski
@ 2023-06-01 13:17 ` Michal Wilczynski
1 sibling, 0 replies; 2+ messages in thread
From: Michal Wilczynski @ 2023-06-01 13:17 UTC (permalink / raw)
To: Benson Leung
Cc: Michal Wilczynski, chrome-platform, linux-acpi,
platform-driver-x86, rafael
Currently logic for installing notifications from ACPI devices is
implemented using notify callback in struct acpi_driver. Preparations
are being made to replace acpi_driver with more generic struct
platform_driver, which doesn't contain notify callback. Furthermore
as of now handlers are being called indirectly through
acpi_notify_device(), which decreases performance.
Call acpi_device_install_event_handler() at the end of .add() callback.
Call acpi_device_remove_event_handler() at the beginning of .remove()
callback. Change arguments passed to the notify callback to match with
what's required by acpi_device_install_event_handler().
Signed-off-by: Michal Wilczynski <michal.wilczynski@intel.com>
---
drivers/platform/chrome/wilco_ec/event.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/platform/chrome/wilco_ec/event.c b/drivers/platform/chrome/wilco_ec/event.c
index a40f60bcefb6..23fd6de623f4 100644
--- a/drivers/platform/chrome/wilco_ec/event.c
+++ b/drivers/platform/chrome/wilco_ec/event.c
@@ -251,16 +251,10 @@ static int enqueue_events(struct acpi_device *adev, const u8 *buf, u32 length)
return 0;
}
-/**
- * event_device_notify() - Callback when EC generates an event over ACPI.
- * @adev: The device that the event is coming from.
- * @value: Value passed to Notify() in ACPI.
- *
- * This function will read the events from the device and enqueue them.
- */
-static void event_device_notify(struct acpi_device *adev, u32 value)
+static void event_device_notify(acpi_handle handle, u32 value, void *data)
{
struct acpi_buffer event_buffer = { ACPI_ALLOCATE_BUFFER, NULL };
+ struct acpi_device *adev = data;
union acpi_object *obj;
acpi_status status;
@@ -490,6 +484,11 @@ static int event_device_add(struct acpi_device *adev)
if (error)
goto free_dev_data;
+ error = acpi_device_install_event_handler(adev, ACPI_DEVICE_NOTIFY,
+ event_device_notify);
+ if (error)
+ goto free_dev_data;
+
return 0;
free_dev_data:
@@ -503,6 +502,7 @@ static void event_device_remove(struct acpi_device *adev)
{
struct event_device_data *dev_data = adev->driver_data;
+ acpi_device_remove_event_handler(adev, ACPI_DEVICE_NOTIFY, event_device_notify);
cdev_device_del(&dev_data->cdev, &dev_data->dev);
ida_simple_remove(&event_ida, MINOR(dev_data->dev.devt));
hangup_device(dev_data);
@@ -520,7 +520,6 @@ static struct acpi_driver event_driver = {
.ids = event_acpi_ids,
.ops = {
.add = event_device_add,
- .notify = event_device_notify,
.remove = event_device_remove,
},
.owner = THIS_MODULE,
--
2.40.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-06-01 13:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20230601131739.300760-3-michal.wilczynski@intel.com>
2023-06-01 13:17 ` [PATCH v4 12/35] platform/chromeos_tbmc: Move handler installing logic to driver Michal Wilczynski
2023-06-01 13:17 ` [PATCH v4 13/35] platform/wilco_ec: " Michal Wilczynski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).