github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-07-27 09:36:22 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

platform/x86: xo15-ebook: Convert ACPI driver to a platform one

Browse Source 
In all cases in which a struct acpi_driver is used for binding a driver
to an ACPI device object, a corresponding platform device is created by
the ACPI core and that device is regarded as a proper representation of
underlying hardware.  Accordingly, a struct platform_driver should be
used by driver code to bind to that device.  There are multiple reasons
why drivers should not bind directly to ACPI device objects [1].

Overall, it is better to bind drivers to platform devices than to their
ACPI companions, so convert the OLPC XO-1.5 ebook switch driver from an
ACPI driver to a platform one.

After this change, the input device registered by the driver will appear
under the platform device used for driver binding.

While this is not expected to alter functionality, it changes sysfs
layout and so it will be visible to user space.

Link: https://lore.kernel.org/all/2396510.ElGaqSPkdT@rafael.j.wysocki/ [1]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/8688586.T7Z3S40VBb@rafael.j.wysocki
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
This commit is contained in:
Rafael J. Wysocki 2026-05-08 19:44:58 +02:00 committed by Ilpo Järvinen
parent 60e68011d8
commit b82c2e30cf
No known key found for this signature in database
GPG Key ID: 59AC4F6153E5CE31
1 changed files with 23 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
drivers/platform/x86/xo15-ebook.c
View File

@ -15,6 +15,7 @@
#include <linux/types.h>
#include <linux/input.h>
#include <linux/acpi.h>
#include <linux/platform_device.h>
#define MODULE_NAME "xo15-ebook"
@ -41,13 +42,13 @@ struct ebook_switch {
bool gpe_enabled;
};
static int ebook_send_state(struct acpi_device *device)
static int ebook_send_state(struct device *dev)
{
struct ebook_switch *button = acpi_driver_data(device);
struct ebook_switch *button = dev_get_drvdata(dev);
unsigned long long state;
acpi_status status;
status = acpi_evaluate_integer(device->handle, "EBK", NULL, &state);
status = acpi_evaluate_integer(ACPI_HANDLE(dev), "EBK", NULL, &state);
if (ACPI_FAILURE(status))
return -EIO;
@ -73,14 +74,15 @@ static void ebook_switch_notify(acpi_handle handle, u32 event, void *data)
#ifdef CONFIG_PM_SLEEP
static int ebook_switch_resume(struct device *dev)
{
return ebook_send_state(to_acpi_device(dev));
return ebook_send_state(dev);
}
#endif
static SIMPLE_DEV_PM_OPS(ebook_switch_pm, NULL, ebook_switch_resume);
static int ebook_switch_add(struct acpi_device *device)
static int ebook_switch_probe(struct platform_device *pdev)
{
struct acpi_device *device = ACPI_COMPANION(&pdev->dev);
const struct acpi_device_id *id;
struct ebook_switch *button;
struct input_dev *input;
@ -90,7 +92,7 @@ static int ebook_switch_add(struct acpi_device *device)
if (!button)
return -ENOMEM;
device->driver_data = button;
platform_set_drvdata(pdev, button);
button->input = input = input_allocate_device();
if (!input) {
@ -100,7 +102,7 @@ static int ebook_switch_add(struct acpi_device *device)
id = acpi_match_acpi_device(ebook_device_ids, device);
if (!id) {
dev_err(&device->dev, "Unsupported hid\n");
dev_err(&pdev->dev, "Unsupported hid\n");
error = -ENODEV;
goto err_free_input;
}
@ -113,7 +115,7 @@ static int ebook_switch_add(struct acpi_device *device)
input->name = acpi_device_name(device);
input->phys = button->phys;
input->id.bustype = BUS_HOST;
input->dev.parent = &device->dev;
input->dev.parent = &pdev->dev;
input->evbit[0] = BIT_MASK(EV_SW);
set_bit(SW_TABLET_MODE, input->swbit);
@ -123,11 +125,11 @@ static int ebook_switch_add(struct acpi_device *device)
goto err_free_input;
error = acpi_dev_install_notify_handler(device, ACPI_DEVICE_NOTIFY,
ebook_switch_notify, device);
ebook_switch_notify, &pdev->dev);
if (error)
goto err_unregister_input;
ebook_send_state(device);
ebook_send_state(&pdev->dev);
if (device->wakeup.flags.valid) {
/* Button's GPE is run-wake GPE */
@ -149,9 +151,10 @@ static int ebook_switch_add(struct acpi_device *device)
goto err_free_button;
}
static void ebook_switch_remove(struct acpi_device *device)
static void ebook_switch_remove(struct platform_device *pdev)
{
struct ebook_switch *button = acpi_driver_data(device);
struct ebook_switch *button = platform_get_drvdata(pdev);
struct acpi_device *device = ACPI_COMPANION(&pdev->dev);
if (button->gpe_enabled)
acpi_disable_gpe(device->wakeup.gpe_device,
@ -163,14 +166,13 @@ static void ebook_switch_remove(struct acpi_device *device)
kfree(button);
}
static struct acpi_driver xo15_ebook_driver = {
.name = MODULE_NAME,
.class = XO15_EBOOK_CLASS,
.ids = ebook_device_ids,
.ops = {
.add = ebook_switch_add,
.remove = ebook_switch_remove,
static struct platform_driver xo15_ebook_driver = {
.probe = ebook_switch_probe,
.remove = ebook_switch_remove,
.driver = {
.name = MODULE_NAME,
.acpi_match_table = ebook_device_ids,
.pm = &ebook_switch_pm,
},
.drv.pm = &ebook_switch_pm,
};
module_acpi_driver(xo15_ebook_driver);
module_platform_driver(xo15_ebook_driver);