forked from Minki/linux
2b329f5694
The driver core ignores the return value of struct bus_type::remove() (and so wmi_dev_remove()) because there is only little that can be done. To simplify the quest to make this function return void, let struct wmi_driver::remove() return void, too. All implementers of this callback return 0 already and this way it should be obvious to driver authors that returning an error code is a bad idea. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Reviewed-by: Pali Rohár <pali@kernel.org> Link: https://lore.kernel.org/r/20210301160404.1677064-1-u.kleine-koenig@pengutronix.de Signed-off-by: Hans de Goede <hdegoede@redhat.com>
95 lines
2.3 KiB
C
95 lines
2.3 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
/*
|
|
* WMI Thunderbolt driver
|
|
*
|
|
* Copyright (C) 2017 Dell Inc. All Rights Reserved.
|
|
*/
|
|
|
|
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
|
|
|
#include <linux/acpi.h>
|
|
#include <linux/device.h>
|
|
#include <linux/fs.h>
|
|
#include <linux/kernel.h>
|
|
#include <linux/module.h>
|
|
#include <linux/string.h>
|
|
#include <linux/sysfs.h>
|
|
#include <linux/types.h>
|
|
#include <linux/wmi.h>
|
|
|
|
#define INTEL_WMI_THUNDERBOLT_GUID "86CCFD48-205E-4A77-9C48-2021CBEDE341"
|
|
|
|
static ssize_t force_power_store(struct device *dev,
|
|
struct device_attribute *attr,
|
|
const char *buf, size_t count)
|
|
{
|
|
struct acpi_buffer input;
|
|
acpi_status status;
|
|
u8 mode;
|
|
|
|
input.length = sizeof(u8);
|
|
input.pointer = &mode;
|
|
mode = hex_to_bin(buf[0]);
|
|
dev_dbg(dev, "force_power: storing %#x\n", mode);
|
|
if (mode == 0 || mode == 1) {
|
|
status = wmi_evaluate_method(INTEL_WMI_THUNDERBOLT_GUID, 0, 1,
|
|
&input, NULL);
|
|
if (ACPI_FAILURE(status)) {
|
|
dev_dbg(dev, "force_power: failed to evaluate ACPI method\n");
|
|
return -ENODEV;
|
|
}
|
|
} else {
|
|
dev_dbg(dev, "force_power: unsupported mode\n");
|
|
return -EINVAL;
|
|
}
|
|
return count;
|
|
}
|
|
|
|
static DEVICE_ATTR_WO(force_power);
|
|
|
|
static struct attribute *tbt_attrs[] = {
|
|
&dev_attr_force_power.attr,
|
|
NULL
|
|
};
|
|
|
|
static const struct attribute_group tbt_attribute_group = {
|
|
.attrs = tbt_attrs,
|
|
};
|
|
|
|
static int intel_wmi_thunderbolt_probe(struct wmi_device *wdev,
|
|
const void *context)
|
|
{
|
|
int ret;
|
|
|
|
ret = sysfs_create_group(&wdev->dev.kobj, &tbt_attribute_group);
|
|
kobject_uevent(&wdev->dev.kobj, KOBJ_CHANGE);
|
|
return ret;
|
|
}
|
|
|
|
static void intel_wmi_thunderbolt_remove(struct wmi_device *wdev)
|
|
{
|
|
sysfs_remove_group(&wdev->dev.kobj, &tbt_attribute_group);
|
|
kobject_uevent(&wdev->dev.kobj, KOBJ_CHANGE);
|
|
}
|
|
|
|
static const struct wmi_device_id intel_wmi_thunderbolt_id_table[] = {
|
|
{ .guid_string = INTEL_WMI_THUNDERBOLT_GUID },
|
|
{ },
|
|
};
|
|
|
|
static struct wmi_driver intel_wmi_thunderbolt_driver = {
|
|
.driver = {
|
|
.name = "intel-wmi-thunderbolt",
|
|
},
|
|
.probe = intel_wmi_thunderbolt_probe,
|
|
.remove = intel_wmi_thunderbolt_remove,
|
|
.id_table = intel_wmi_thunderbolt_id_table,
|
|
};
|
|
|
|
module_wmi_driver(intel_wmi_thunderbolt_driver);
|
|
|
|
MODULE_DEVICE_TABLE(wmi, intel_wmi_thunderbolt_id_table);
|
|
MODULE_AUTHOR("Mario Limonciello <mario.limonciello@dell.com>");
|
|
MODULE_DESCRIPTION("Intel WMI Thunderbolt force power driver");
|
|
MODULE_LICENSE("GPL v2");
|