|
@@ -25,7 +25,6 @@
|
|
|
#define WAC_CMD_RETRIES 10
|
|
|
#define WAC_CMD_DELETE_PAIRING 0x20
|
|
|
#define WAC_CMD_UNPAIR_ALL 0xFF
|
|
|
-#define WAC_REMOTE_SERIAL_MAX_STRLEN 9
|
|
|
|
|
|
#define DEV_ATTR_RW_PERM (S_IRUGO | S_IWUSR | S_IWGRP)
|
|
|
#define DEV_ATTR_WO_PERM (S_IWUSR | S_IWGRP)
|
|
@@ -91,7 +90,12 @@ static void wacom_close(struct input_dev *dev)
|
|
|
{
|
|
|
struct wacom *wacom = input_get_drvdata(dev);
|
|
|
|
|
|
- hid_hw_close(wacom->hdev);
|
|
|
+ /*
|
|
|
+ * wacom->hdev should never be null, but surprisingly, I had the case
|
|
|
+ * once while unplugging the Wacom Wireless Receiver.
|
|
|
+ */
|
|
|
+ if (wacom->hdev)
|
|
|
+ hid_hw_close(wacom->hdev);
|
|
|
}
|
|
|
|
|
|
/*
|
|
@@ -523,36 +527,95 @@ struct wacom_hdev_data {
|
|
|
static LIST_HEAD(wacom_udev_list);
|
|
|
static DEFINE_MUTEX(wacom_udev_list_lock);
|
|
|
|
|
|
+static bool compare_device_paths(struct hid_device *hdev_a,
|
|
|
+ struct hid_device *hdev_b, char separator)
|
|
|
+{
|
|
|
+ int n1 = strrchr(hdev_a->phys, separator) - hdev_a->phys;
|
|
|
+ int n2 = strrchr(hdev_b->phys, separator) - hdev_b->phys;
|
|
|
+
|
|
|
+ if (n1 != n2 || n1 <= 0 || n2 <= 0)
|
|
|
+ return false;
|
|
|
+
|
|
|
+ return !strncmp(hdev_a->phys, hdev_b->phys, n1);
|
|
|
+}
|
|
|
+
|
|
|
static bool wacom_are_sibling(struct hid_device *hdev,
|
|
|
struct hid_device *sibling)
|
|
|
{
|
|
|
struct wacom *wacom = hid_get_drvdata(hdev);
|
|
|
struct wacom_features *features = &wacom->wacom_wac.features;
|
|
|
- int vid = features->oVid;
|
|
|
- int pid = features->oPid;
|
|
|
- int n1,n2;
|
|
|
+ struct wacom *sibling_wacom = hid_get_drvdata(sibling);
|
|
|
+ struct wacom_features *sibling_features = &sibling_wacom->wacom_wac.features;
|
|
|
+ __u32 oVid = features->oVid ? features->oVid : hdev->vendor;
|
|
|
+ __u32 oPid = features->oPid ? features->oPid : hdev->product;
|
|
|
+
|
|
|
+ /* The defined oVid/oPid must match that of the sibling */
|
|
|
+ if (features->oVid != HID_ANY_ID && sibling->vendor != oVid)
|
|
|
+ return false;
|
|
|
+ if (features->oPid != HID_ANY_ID && sibling->product != oPid)
|
|
|
+ return false;
|
|
|
|
|
|
- if (vid == 0 && pid == 0) {
|
|
|
- vid = hdev->vendor;
|
|
|
- pid = hdev->product;
|
|
|
+ /*
|
|
|
+ * Devices with the same VID/PID must share the same physical
|
|
|
+ * device path, while those with different VID/PID must share
|
|
|
+ * the same physical parent device path.
|
|
|
+ */
|
|
|
+ if (hdev->vendor == sibling->vendor && hdev->product == sibling->product) {
|
|
|
+ if (!compare_device_paths(hdev, sibling, '/'))
|
|
|
+ return false;
|
|
|
+ } else {
|
|
|
+ if (!compare_device_paths(hdev, sibling, '.'))
|
|
|
+ return false;
|
|
|
}
|
|
|
|
|
|
- if (vid != sibling->vendor || pid != sibling->product)
|
|
|
+ /* Skip the remaining heuristics unless you are a HID_GENERIC device */
|
|
|
+ if (features->type != HID_GENERIC)
|
|
|
+ return true;
|
|
|
+
|
|
|
+ /*
|
|
|
+ * Direct-input devices may not be siblings of indirect-input
|
|
|
+ * devices.
|
|
|
+ */
|
|
|
+ if ((features->device_type & WACOM_DEVICETYPE_DIRECT) &&
|
|
|
+ !(sibling_features->device_type & WACOM_DEVICETYPE_DIRECT))
|
|
|
return false;
|
|
|
|
|
|
- /* Compare the physical path. */
|
|
|
- n1 = strrchr(hdev->phys, '.') - hdev->phys;
|
|
|
- n2 = strrchr(sibling->phys, '.') - sibling->phys;
|
|
|
- if (n1 != n2 || n1 <= 0 || n2 <= 0)
|
|
|
+ /*
|
|
|
+ * Indirect-input devices may not be siblings of direct-input
|
|
|
+ * devices.
|
|
|
+ */
|
|
|
+ if (!(features->device_type & WACOM_DEVICETYPE_DIRECT) &&
|
|
|
+ (sibling_features->device_type & WACOM_DEVICETYPE_DIRECT))
|
|
|
+ return false;
|
|
|
+
|
|
|
+ /* Pen devices may only be siblings of touch devices */
|
|
|
+ if ((features->device_type & WACOM_DEVICETYPE_PEN) &&
|
|
|
+ !(sibling_features->device_type & WACOM_DEVICETYPE_TOUCH))
|
|
|
return false;
|
|
|
|
|
|
- return !strncmp(hdev->phys, sibling->phys, n1);
|
|
|
+ /* Touch devices may only be siblings of pen devices */
|
|
|
+ if ((features->device_type & WACOM_DEVICETYPE_TOUCH) &&
|
|
|
+ !(sibling_features->device_type & WACOM_DEVICETYPE_PEN))
|
|
|
+ return false;
|
|
|
+
|
|
|
+ /*
|
|
|
+ * No reason could be found for these two devices to NOT be
|
|
|
+ * siblings, so there's a good chance they ARE siblings
|
|
|
+ */
|
|
|
+ return true;
|
|
|
}
|
|
|
|
|
|
static struct wacom_hdev_data *wacom_get_hdev_data(struct hid_device *hdev)
|
|
|
{
|
|
|
struct wacom_hdev_data *data;
|
|
|
|
|
|
+ /* Try to find an already-probed interface from the same device */
|
|
|
+ list_for_each_entry(data, &wacom_udev_list, list) {
|
|
|
+ if (compare_device_paths(hdev, data->dev, '/'))
|
|
|
+ return data;
|
|
|
+ }
|
|
|
+
|
|
|
+ /* Fallback to finding devices that appear to be "siblings" */
|
|
|
list_for_each_entry(data, &wacom_udev_list, list) {
|
|
|
if (wacom_are_sibling(hdev, data->dev)) {
|
|
|
kref_get(&data->kref);
|
|
@@ -563,6 +626,38 @@ static struct wacom_hdev_data *wacom_get_hdev_data(struct hid_device *hdev)
|
|
|
return NULL;
|
|
|
}
|
|
|
|
|
|
+static void wacom_release_shared_data(struct kref *kref)
|
|
|
+{
|
|
|
+ struct wacom_hdev_data *data =
|
|
|
+ container_of(kref, struct wacom_hdev_data, kref);
|
|
|
+
|
|
|
+ mutex_lock(&wacom_udev_list_lock);
|
|
|
+ list_del(&data->list);
|
|
|
+ mutex_unlock(&wacom_udev_list_lock);
|
|
|
+
|
|
|
+ kfree(data);
|
|
|
+}
|
|
|
+
|
|
|
+static void wacom_remove_shared_data(void *res)
|
|
|
+{
|
|
|
+ struct wacom *wacom = res;
|
|
|
+ struct wacom_hdev_data *data;
|
|
|
+ struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
|
+
|
|
|
+ if (wacom_wac->shared) {
|
|
|
+ data = container_of(wacom_wac->shared, struct wacom_hdev_data,
|
|
|
+ shared);
|
|
|
+
|
|
|
+ if (wacom_wac->shared->touch == wacom->hdev)
|
|
|
+ wacom_wac->shared->touch = NULL;
|
|
|
+ else if (wacom_wac->shared->pen == wacom->hdev)
|
|
|
+ wacom_wac->shared->pen = NULL;
|
|
|
+
|
|
|
+ kref_put(&data->kref, wacom_release_shared_data);
|
|
|
+ wacom_wac->shared = NULL;
|
|
|
+ }
|
|
|
+}
|
|
|
+
|
|
|
static int wacom_add_shared_data(struct hid_device *hdev)
|
|
|
{
|
|
|
struct wacom *wacom = hid_get_drvdata(hdev);
|
|
@@ -587,6 +682,13 @@ static int wacom_add_shared_data(struct hid_device *hdev)
|
|
|
|
|
|
wacom_wac->shared = &data->shared;
|
|
|
|
|
|
+ retval = devm_add_action(&hdev->dev, wacom_remove_shared_data, wacom);
|
|
|
+ if (retval) {
|
|
|
+ mutex_unlock(&wacom_udev_list_lock);
|
|
|
+ wacom_remove_shared_data(wacom);
|
|
|
+ return retval;
|
|
|
+ }
|
|
|
+
|
|
|
if (wacom_wac->features.device_type & WACOM_DEVICETYPE_TOUCH)
|
|
|
wacom_wac->shared->touch = hdev;
|
|
|
else if (wacom_wac->features.device_type & WACOM_DEVICETYPE_PEN)
|
|
@@ -597,37 +699,6 @@ out:
|
|
|
return retval;
|
|
|
}
|
|
|
|
|
|
-static void wacom_release_shared_data(struct kref *kref)
|
|
|
-{
|
|
|
- struct wacom_hdev_data *data =
|
|
|
- container_of(kref, struct wacom_hdev_data, kref);
|
|
|
-
|
|
|
- mutex_lock(&wacom_udev_list_lock);
|
|
|
- list_del(&data->list);
|
|
|
- mutex_unlock(&wacom_udev_list_lock);
|
|
|
-
|
|
|
- kfree(data);
|
|
|
-}
|
|
|
-
|
|
|
-static void wacom_remove_shared_data(struct wacom *wacom)
|
|
|
-{
|
|
|
- struct wacom_hdev_data *data;
|
|
|
- struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
|
-
|
|
|
- if (wacom_wac->shared) {
|
|
|
- data = container_of(wacom_wac->shared, struct wacom_hdev_data,
|
|
|
- shared);
|
|
|
-
|
|
|
- if (wacom_wac->shared->touch == wacom->hdev)
|
|
|
- wacom_wac->shared->touch = NULL;
|
|
|
- else if (wacom_wac->shared->pen == wacom->hdev)
|
|
|
- wacom_wac->shared->pen = NULL;
|
|
|
-
|
|
|
- kref_put(&data->kref, wacom_release_shared_data);
|
|
|
- wacom_wac->shared = NULL;
|
|
|
- }
|
|
|
-}
|
|
|
-
|
|
|
static int wacom_led_control(struct wacom *wacom)
|
|
|
{
|
|
|
unsigned char *buf;
|
|
@@ -635,6 +706,12 @@ static int wacom_led_control(struct wacom *wacom)
|
|
|
unsigned char report_id = WAC_CMD_LED_CONTROL;
|
|
|
int buf_size = 9;
|
|
|
|
|
|
+ if (!hid_get_drvdata(wacom->hdev))
|
|
|
+ return -ENODEV;
|
|
|
+
|
|
|
+ if (!wacom->led.groups)
|
|
|
+ return -ENOTSUPP;
|
|
|
+
|
|
|
if (wacom->wacom_wac.pid) { /* wireless connected */
|
|
|
report_id = WAC_CMD_WL_LED_CONTROL;
|
|
|
buf_size = 13;
|
|
@@ -650,7 +727,7 @@ static int wacom_led_control(struct wacom *wacom)
|
|
|
* one of four values:
|
|
|
* 0 = Low; 1 = Medium; 2 = High; 3 = Off
|
|
|
*/
|
|
|
- int ring_led = wacom->led.select[0] & 0x03;
|
|
|
+ int ring_led = wacom->led.groups[0].select & 0x03;
|
|
|
int ring_lum = (((wacom->led.llv & 0x60) >> 5) - 1) & 0x03;
|
|
|
int crop_lum = 0;
|
|
|
unsigned char led_bits = (crop_lum << 4) | (ring_lum << 2) | (ring_led);
|
|
@@ -665,11 +742,11 @@ static int wacom_led_control(struct wacom *wacom)
|
|
|
buf[1] = led_bits;
|
|
|
}
|
|
|
else {
|
|
|
- int led = wacom->led.select[0] | 0x4;
|
|
|
+ int led = wacom->led.groups[0].select | 0x4;
|
|
|
|
|
|
if (wacom->wacom_wac.features.type == WACOM_21UX2 ||
|
|
|
wacom->wacom_wac.features.type == WACOM_24HD)
|
|
|
- led |= (wacom->led.select[1] << 4) | 0x40;
|
|
|
+ led |= (wacom->led.groups[1].select << 4) | 0x40;
|
|
|
|
|
|
buf[0] = report_id;
|
|
|
buf[1] = led;
|
|
@@ -741,7 +818,7 @@ static ssize_t wacom_led_select_store(struct device *dev, int set_id,
|
|
|
|
|
|
mutex_lock(&wacom->lock);
|
|
|
|
|
|
- wacom->led.select[set_id] = id & 0x3;
|
|
|
+ wacom->led.groups[set_id].select = id & 0x3;
|
|
|
err = wacom_led_control(wacom);
|
|
|
|
|
|
mutex_unlock(&wacom->lock);
|
|
@@ -761,7 +838,7 @@ static ssize_t wacom_led##SET_ID##_select_show(struct device *dev, \
|
|
|
struct hid_device *hdev = to_hid_device(dev);\
|
|
|
struct wacom *wacom = hid_get_drvdata(hdev); \
|
|
|
return scnprintf(buf, PAGE_SIZE, "%d\n", \
|
|
|
- wacom->led.select[SET_ID]); \
|
|
|
+ wacom->led.groups[SET_ID].select); \
|
|
|
} \
|
|
|
static DEVICE_ATTR(status_led##SET_ID##_select, DEV_ATTR_RW_PERM, \
|
|
|
wacom_led##SET_ID##_select_show, \
|
|
@@ -904,6 +981,327 @@ static struct attribute_group intuos5_led_attr_group = {
|
|
|
.attrs = intuos5_led_attrs,
|
|
|
};
|
|
|
|
|
|
+struct wacom_sysfs_group_devres {
|
|
|
+ struct attribute_group *group;
|
|
|
+ struct kobject *root;
|
|
|
+};
|
|
|
+
|
|
|
+static void wacom_devm_sysfs_group_release(struct device *dev, void *res)
|
|
|
+{
|
|
|
+ struct wacom_sysfs_group_devres *devres = res;
|
|
|
+ struct kobject *kobj = devres->root;
|
|
|
+
|
|
|
+ dev_dbg(dev, "%s: dropping reference to %s\n",
|
|
|
+ __func__, devres->group->name);
|
|
|
+ sysfs_remove_group(kobj, devres->group);
|
|
|
+}
|
|
|
+
|
|
|
+static int __wacom_devm_sysfs_create_group(struct wacom *wacom,
|
|
|
+ struct kobject *root,
|
|
|
+ struct attribute_group *group)
|
|
|
+{
|
|
|
+ struct wacom_sysfs_group_devres *devres;
|
|
|
+ int error;
|
|
|
+
|
|
|
+ devres = devres_alloc(wacom_devm_sysfs_group_release,
|
|
|
+ sizeof(struct wacom_sysfs_group_devres),
|
|
|
+ GFP_KERNEL);
|
|
|
+ if (!devres)
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ devres->group = group;
|
|
|
+ devres->root = root;
|
|
|
+
|
|
|
+ error = sysfs_create_group(devres->root, group);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
+ devres_add(&wacom->hdev->dev, devres);
|
|
|
+
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_devm_sysfs_create_group(struct wacom *wacom,
|
|
|
+ struct attribute_group *group)
|
|
|
+{
|
|
|
+ return __wacom_devm_sysfs_create_group(wacom, &wacom->hdev->dev.kobj,
|
|
|
+ group);
|
|
|
+}
|
|
|
+
|
|
|
+enum led_brightness wacom_leds_brightness_get(struct wacom_led *led)
|
|
|
+{
|
|
|
+ struct wacom *wacom = led->wacom;
|
|
|
+
|
|
|
+ if (wacom->led.max_hlv)
|
|
|
+ return led->hlv * LED_FULL / wacom->led.max_hlv;
|
|
|
+
|
|
|
+ if (wacom->led.max_llv)
|
|
|
+ return led->llv * LED_FULL / wacom->led.max_llv;
|
|
|
+
|
|
|
+ /* device doesn't support brightness tuning */
|
|
|
+ return LED_FULL;
|
|
|
+}
|
|
|
+
|
|
|
+static enum led_brightness __wacom_led_brightness_get(struct led_classdev *cdev)
|
|
|
+{
|
|
|
+ struct wacom_led *led = container_of(cdev, struct wacom_led, cdev);
|
|
|
+ struct wacom *wacom = led->wacom;
|
|
|
+
|
|
|
+ if (wacom->led.groups[led->group].select != led->id)
|
|
|
+ return LED_OFF;
|
|
|
+
|
|
|
+ return wacom_leds_brightness_get(led);
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_led_brightness_set(struct led_classdev *cdev,
|
|
|
+ enum led_brightness brightness)
|
|
|
+{
|
|
|
+ struct wacom_led *led = container_of(cdev, struct wacom_led, cdev);
|
|
|
+ struct wacom *wacom = led->wacom;
|
|
|
+ int error;
|
|
|
+
|
|
|
+ mutex_lock(&wacom->lock);
|
|
|
+
|
|
|
+ if (!wacom->led.groups || (brightness == LED_OFF &&
|
|
|
+ wacom->led.groups[led->group].select != led->id)) {
|
|
|
+ error = 0;
|
|
|
+ goto out;
|
|
|
+ }
|
|
|
+
|
|
|
+ led->llv = wacom->led.llv = wacom->led.max_llv * brightness / LED_FULL;
|
|
|
+ led->hlv = wacom->led.hlv = wacom->led.max_hlv * brightness / LED_FULL;
|
|
|
+
|
|
|
+ wacom->led.groups[led->group].select = led->id;
|
|
|
+
|
|
|
+ error = wacom_led_control(wacom);
|
|
|
+
|
|
|
+out:
|
|
|
+ mutex_unlock(&wacom->lock);
|
|
|
+
|
|
|
+ return error;
|
|
|
+}
|
|
|
+
|
|
|
+static void wacom_led_readonly_brightness_set(struct led_classdev *cdev,
|
|
|
+ enum led_brightness brightness)
|
|
|
+{
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_led_register_one(struct device *dev, struct wacom *wacom,
|
|
|
+ struct wacom_led *led, unsigned int group,
|
|
|
+ unsigned int id, bool read_only)
|
|
|
+{
|
|
|
+ int error;
|
|
|
+ char *name;
|
|
|
+
|
|
|
+ name = devm_kasprintf(dev, GFP_KERNEL,
|
|
|
+ "%s::wacom-%d.%d",
|
|
|
+ dev_name(dev),
|
|
|
+ group,
|
|
|
+ id);
|
|
|
+ if (!name)
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ if (!read_only) {
|
|
|
+ led->trigger.name = name;
|
|
|
+ error = devm_led_trigger_register(dev, &led->trigger);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "failed to register LED trigger %s: %d\n",
|
|
|
+ led->cdev.name, error);
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+ }
|
|
|
+
|
|
|
+ led->group = group;
|
|
|
+ led->id = id;
|
|
|
+ led->wacom = wacom;
|
|
|
+ led->llv = wacom->led.llv;
|
|
|
+ led->hlv = wacom->led.hlv;
|
|
|
+ led->cdev.name = name;
|
|
|
+ led->cdev.max_brightness = LED_FULL;
|
|
|
+ led->cdev.flags = LED_HW_PLUGGABLE;
|
|
|
+ led->cdev.brightness_get = __wacom_led_brightness_get;
|
|
|
+ if (!read_only) {
|
|
|
+ led->cdev.brightness_set_blocking = wacom_led_brightness_set;
|
|
|
+ led->cdev.default_trigger = led->cdev.name;
|
|
|
+ } else {
|
|
|
+ led->cdev.brightness_set = wacom_led_readonly_brightness_set;
|
|
|
+ }
|
|
|
+
|
|
|
+ error = devm_led_classdev_register(dev, &led->cdev);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "failed to register LED %s: %d\n",
|
|
|
+ led->cdev.name, error);
|
|
|
+ led->cdev.name = NULL;
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
+static void wacom_led_groups_release_one(void *data)
|
|
|
+{
|
|
|
+ struct wacom_group_leds *group = data;
|
|
|
+
|
|
|
+ devres_release_group(group->dev, group);
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_led_groups_alloc_and_register_one(struct device *dev,
|
|
|
+ struct wacom *wacom,
|
|
|
+ int group_id, int count,
|
|
|
+ bool read_only)
|
|
|
+{
|
|
|
+ struct wacom_led *leds;
|
|
|
+ int i, error;
|
|
|
+
|
|
|
+ if (group_id >= wacom->led.count || count <= 0)
|
|
|
+ return -EINVAL;
|
|
|
+
|
|
|
+ if (!devres_open_group(dev, &wacom->led.groups[group_id], GFP_KERNEL))
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ leds = devm_kzalloc(dev, sizeof(struct wacom_led) * count, GFP_KERNEL);
|
|
|
+ if (!leds) {
|
|
|
+ error = -ENOMEM;
|
|
|
+ goto err;
|
|
|
+ }
|
|
|
+
|
|
|
+ wacom->led.groups[group_id].leds = leds;
|
|
|
+ wacom->led.groups[group_id].count = count;
|
|
|
+
|
|
|
+ for (i = 0; i < count; i++) {
|
|
|
+ error = wacom_led_register_one(dev, wacom, &leds[i],
|
|
|
+ group_id, i, read_only);
|
|
|
+ if (error)
|
|
|
+ goto err;
|
|
|
+ }
|
|
|
+
|
|
|
+ wacom->led.groups[group_id].dev = dev;
|
|
|
+
|
|
|
+ devres_close_group(dev, &wacom->led.groups[group_id]);
|
|
|
+
|
|
|
+ /*
|
|
|
+ * There is a bug (?) in devm_led_classdev_register() in which its
|
|
|
+ * increments the refcount of the parent. If the parent is an input
|
|
|
+ * device, that means the ref count never reaches 0 when
|
|
|
+ * devm_input_device_release() gets called.
|
|
|
+ * This means that the LEDs are still there after disconnect.
|
|
|
+ * Manually force the release of the group so that the leds are released
|
|
|
+ * once we are done using them.
|
|
|
+ */
|
|
|
+ error = devm_add_action_or_reset(&wacom->hdev->dev,
|
|
|
+ wacom_led_groups_release_one,
|
|
|
+ &wacom->led.groups[group_id]);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
+ return 0;
|
|
|
+
|
|
|
+err:
|
|
|
+ devres_release_group(dev, &wacom->led.groups[group_id]);
|
|
|
+ return error;
|
|
|
+}
|
|
|
+
|
|
|
+struct wacom_led *wacom_led_find(struct wacom *wacom, unsigned int group_id,
|
|
|
+ unsigned int id)
|
|
|
+{
|
|
|
+ struct wacom_group_leds *group;
|
|
|
+
|
|
|
+ if (group_id >= wacom->led.count)
|
|
|
+ return NULL;
|
|
|
+
|
|
|
+ group = &wacom->led.groups[group_id];
|
|
|
+
|
|
|
+ if (!group->leds)
|
|
|
+ return NULL;
|
|
|
+
|
|
|
+ id %= group->count;
|
|
|
+
|
|
|
+ return &group->leds[id];
|
|
|
+}
|
|
|
+
|
|
|
+/**
|
|
|
+ * wacom_led_next: gives the next available led with a wacom trigger.
|
|
|
+ *
|
|
|
+ * returns the next available struct wacom_led which has its default trigger
|
|
|
+ * or the current one if none is available.
|
|
|
+ */
|
|
|
+struct wacom_led *wacom_led_next(struct wacom *wacom, struct wacom_led *cur)
|
|
|
+{
|
|
|
+ struct wacom_led *next_led;
|
|
|
+ int group, next;
|
|
|
+
|
|
|
+ if (!wacom || !cur)
|
|
|
+ return NULL;
|
|
|
+
|
|
|
+ group = cur->group;
|
|
|
+ next = cur->id;
|
|
|
+
|
|
|
+ do {
|
|
|
+ next_led = wacom_led_find(wacom, group, ++next);
|
|
|
+ if (!next_led || next_led == cur)
|
|
|
+ return next_led;
|
|
|
+ } while (next_led->cdev.trigger != &next_led->trigger);
|
|
|
+
|
|
|
+ return next_led;
|
|
|
+}
|
|
|
+
|
|
|
+static void wacom_led_groups_release(void *data)
|
|
|
+{
|
|
|
+ struct wacom *wacom = data;
|
|
|
+
|
|
|
+ wacom->led.groups = NULL;
|
|
|
+ wacom->led.count = 0;
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_led_groups_allocate(struct wacom *wacom, int count)
|
|
|
+{
|
|
|
+ struct device *dev = &wacom->hdev->dev;
|
|
|
+ struct wacom_group_leds *groups;
|
|
|
+ int error;
|
|
|
+
|
|
|
+ groups = devm_kzalloc(dev, sizeof(struct wacom_group_leds) * count,
|
|
|
+ GFP_KERNEL);
|
|
|
+ if (!groups)
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ error = devm_add_action_or_reset(dev, wacom_led_groups_release, wacom);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
+ wacom->led.groups = groups;
|
|
|
+ wacom->led.count = count;
|
|
|
+
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_leds_alloc_and_register(struct wacom *wacom, int group_count,
|
|
|
+ int led_per_group, bool read_only)
|
|
|
+{
|
|
|
+ struct device *dev;
|
|
|
+ int i, error;
|
|
|
+
|
|
|
+ if (!wacom->wacom_wac.pad_input)
|
|
|
+ return -EINVAL;
|
|
|
+
|
|
|
+ dev = &wacom->wacom_wac.pad_input->dev;
|
|
|
+
|
|
|
+ error = wacom_led_groups_allocate(wacom, group_count);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
+ for (i = 0; i < group_count; i++) {
|
|
|
+ error = wacom_led_groups_alloc_and_register_one(dev, wacom, i,
|
|
|
+ led_per_group,
|
|
|
+ read_only);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
static int wacom_initialize_leds(struct wacom *wacom)
|
|
|
{
|
|
|
int error;
|
|
@@ -917,25 +1315,38 @@ static int wacom_initialize_leds(struct wacom *wacom)
|
|
|
case INTUOS4:
|
|
|
case INTUOS4WL:
|
|
|
case INTUOS4L:
|
|
|
- wacom->led.select[0] = 0;
|
|
|
- wacom->led.select[1] = 0;
|
|
|
wacom->led.llv = 10;
|
|
|
wacom->led.hlv = 20;
|
|
|
+ wacom->led.max_llv = 127;
|
|
|
+ wacom->led.max_hlv = 127;
|
|
|
wacom->led.img_lum = 10;
|
|
|
- error = sysfs_create_group(&wacom->hdev->dev.kobj,
|
|
|
- &intuos4_led_attr_group);
|
|
|
+
|
|
|
+ error = wacom_leds_alloc_and_register(wacom, 1, 4, false);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "cannot create leds err: %d\n", error);
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+
|
|
|
+ error = wacom_devm_sysfs_create_group(wacom,
|
|
|
+ &intuos4_led_attr_group);
|
|
|
break;
|
|
|
|
|
|
case WACOM_24HD:
|
|
|
case WACOM_21UX2:
|
|
|
- wacom->led.select[0] = 0;
|
|
|
- wacom->led.select[1] = 0;
|
|
|
wacom->led.llv = 0;
|
|
|
wacom->led.hlv = 0;
|
|
|
wacom->led.img_lum = 0;
|
|
|
|
|
|
- error = sysfs_create_group(&wacom->hdev->dev.kobj,
|
|
|
- &cintiq_led_attr_group);
|
|
|
+ error = wacom_leds_alloc_and_register(wacom, 2, 4, false);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "cannot create leds err: %d\n", error);
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+
|
|
|
+ error = wacom_devm_sysfs_create_group(wacom,
|
|
|
+ &cintiq_led_attr_group);
|
|
|
break;
|
|
|
|
|
|
case INTUOS5S:
|
|
@@ -944,16 +1355,31 @@ static int wacom_initialize_leds(struct wacom *wacom)
|
|
|
case INTUOSPS:
|
|
|
case INTUOSPM:
|
|
|
case INTUOSPL:
|
|
|
- wacom->led.select[0] = 0;
|
|
|
- wacom->led.select[1] = 0;
|
|
|
wacom->led.llv = 32;
|
|
|
- wacom->led.hlv = 0;
|
|
|
- wacom->led.img_lum = 0;
|
|
|
+ wacom->led.max_llv = 96;
|
|
|
+
|
|
|
+ error = wacom_leds_alloc_and_register(wacom, 1, 4, false);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "cannot create leds err: %d\n", error);
|
|
|
+ return error;
|
|
|
+ }
|
|
|
|
|
|
- error = sysfs_create_group(&wacom->hdev->dev.kobj,
|
|
|
- &intuos5_led_attr_group);
|
|
|
+ error = wacom_devm_sysfs_create_group(wacom,
|
|
|
+ &intuos5_led_attr_group);
|
|
|
break;
|
|
|
|
|
|
+ case REMOTE:
|
|
|
+ wacom->led.llv = 255;
|
|
|
+ wacom->led.max_llv = 255;
|
|
|
+ error = wacom_led_groups_allocate(wacom, 5);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "cannot create leds err: %d\n", error);
|
|
|
+ return error;
|
|
|
+ }
|
|
|
+ return 0;
|
|
|
+
|
|
|
default:
|
|
|
return 0;
|
|
|
}
|
|
@@ -964,86 +1390,45 @@ static int wacom_initialize_leds(struct wacom *wacom)
|
|
|
return error;
|
|
|
}
|
|
|
wacom_led_control(wacom);
|
|
|
- wacom->led_initialized = true;
|
|
|
|
|
|
return 0;
|
|
|
}
|
|
|
|
|
|
-static void wacom_destroy_leds(struct wacom *wacom)
|
|
|
-{
|
|
|
- if (!wacom->led_initialized)
|
|
|
- return;
|
|
|
-
|
|
|
- if (!(wacom->wacom_wac.features.device_type & WACOM_DEVICETYPE_PAD))
|
|
|
- return;
|
|
|
-
|
|
|
- wacom->led_initialized = false;
|
|
|
-
|
|
|
- switch (wacom->wacom_wac.features.type) {
|
|
|
- case INTUOS4S:
|
|
|
- case INTUOS4:
|
|
|
- case INTUOS4WL:
|
|
|
- case INTUOS4L:
|
|
|
- sysfs_remove_group(&wacom->hdev->dev.kobj,
|
|
|
- &intuos4_led_attr_group);
|
|
|
- break;
|
|
|
-
|
|
|
- case WACOM_24HD:
|
|
|
- case WACOM_21UX2:
|
|
|
- sysfs_remove_group(&wacom->hdev->dev.kobj,
|
|
|
- &cintiq_led_attr_group);
|
|
|
- break;
|
|
|
-
|
|
|
- case INTUOS5S:
|
|
|
- case INTUOS5:
|
|
|
- case INTUOS5L:
|
|
|
- case INTUOSPS:
|
|
|
- case INTUOSPM:
|
|
|
- case INTUOSPL:
|
|
|
- sysfs_remove_group(&wacom->hdev->dev.kobj,
|
|
|
- &intuos5_led_attr_group);
|
|
|
- break;
|
|
|
- }
|
|
|
-}
|
|
|
-
|
|
|
static enum power_supply_property wacom_battery_props[] = {
|
|
|
+ POWER_SUPPLY_PROP_MODEL_NAME,
|
|
|
POWER_SUPPLY_PROP_PRESENT,
|
|
|
POWER_SUPPLY_PROP_STATUS,
|
|
|
POWER_SUPPLY_PROP_SCOPE,
|
|
|
POWER_SUPPLY_PROP_CAPACITY
|
|
|
};
|
|
|
|
|
|
-static enum power_supply_property wacom_ac_props[] = {
|
|
|
- POWER_SUPPLY_PROP_PRESENT,
|
|
|
- POWER_SUPPLY_PROP_ONLINE,
|
|
|
- POWER_SUPPLY_PROP_SCOPE,
|
|
|
-};
|
|
|
-
|
|
|
static int wacom_battery_get_property(struct power_supply *psy,
|
|
|
enum power_supply_property psp,
|
|
|
union power_supply_propval *val)
|
|
|
{
|
|
|
- struct wacom *wacom = power_supply_get_drvdata(psy);
|
|
|
+ struct wacom_battery *battery = power_supply_get_drvdata(psy);
|
|
|
int ret = 0;
|
|
|
|
|
|
switch (psp) {
|
|
|
+ case POWER_SUPPLY_PROP_MODEL_NAME:
|
|
|
+ val->strval = battery->wacom->wacom_wac.name;
|
|
|
+ break;
|
|
|
case POWER_SUPPLY_PROP_PRESENT:
|
|
|
- val->intval = wacom->wacom_wac.bat_connected;
|
|
|
+ val->intval = battery->bat_connected;
|
|
|
break;
|
|
|
case POWER_SUPPLY_PROP_SCOPE:
|
|
|
val->intval = POWER_SUPPLY_SCOPE_DEVICE;
|
|
|
break;
|
|
|
case POWER_SUPPLY_PROP_CAPACITY:
|
|
|
- val->intval =
|
|
|
- wacom->wacom_wac.battery_capacity;
|
|
|
+ val->intval = battery->battery_capacity;
|
|
|
break;
|
|
|
case POWER_SUPPLY_PROP_STATUS:
|
|
|
- if (wacom->wacom_wac.bat_charging)
|
|
|
+ if (battery->bat_charging)
|
|
|
val->intval = POWER_SUPPLY_STATUS_CHARGING;
|
|
|
- else if (wacom->wacom_wac.battery_capacity == 100 &&
|
|
|
- wacom->wacom_wac.ps_connected)
|
|
|
+ else if (battery->battery_capacity == 100 &&
|
|
|
+ battery->ps_connected)
|
|
|
val->intval = POWER_SUPPLY_STATUS_FULL;
|
|
|
- else if (wacom->wacom_wac.ps_connected)
|
|
|
+ else if (battery->ps_connected)
|
|
|
val->intval = POWER_SUPPLY_STATUS_NOT_CHARGING;
|
|
|
else
|
|
|
val->intval = POWER_SUPPLY_STATUS_DISCHARGING;
|
|
@@ -1056,84 +1441,64 @@ static int wacom_battery_get_property(struct power_supply *psy,
|
|
|
return ret;
|
|
|
}
|
|
|
|
|
|
-static int wacom_ac_get_property(struct power_supply *psy,
|
|
|
- enum power_supply_property psp,
|
|
|
- union power_supply_propval *val)
|
|
|
+static int __wacom_initialize_battery(struct wacom *wacom,
|
|
|
+ struct wacom_battery *battery)
|
|
|
{
|
|
|
- struct wacom *wacom = power_supply_get_drvdata(psy);
|
|
|
- int ret = 0;
|
|
|
+ static atomic_t battery_no = ATOMIC_INIT(0);
|
|
|
+ struct device *dev = &wacom->hdev->dev;
|
|
|
+ struct power_supply_config psy_cfg = { .drv_data = battery, };
|
|
|
+ struct power_supply *ps_bat;
|
|
|
+ struct power_supply_desc *bat_desc = &battery->bat_desc;
|
|
|
+ unsigned long n;
|
|
|
+ int error;
|
|
|
|
|
|
- switch (psp) {
|
|
|
- case POWER_SUPPLY_PROP_PRESENT:
|
|
|
- /* fall through */
|
|
|
- case POWER_SUPPLY_PROP_ONLINE:
|
|
|
- val->intval = wacom->wacom_wac.ps_connected;
|
|
|
- break;
|
|
|
- case POWER_SUPPLY_PROP_SCOPE:
|
|
|
- val->intval = POWER_SUPPLY_SCOPE_DEVICE;
|
|
|
- break;
|
|
|
- default:
|
|
|
- ret = -EINVAL;
|
|
|
- break;
|
|
|
+ if (!devres_open_group(dev, bat_desc, GFP_KERNEL))
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ battery->wacom = wacom;
|
|
|
+
|
|
|
+ n = atomic_inc_return(&battery_no) - 1;
|
|
|
+
|
|
|
+ bat_desc->properties = wacom_battery_props;
|
|
|
+ bat_desc->num_properties = ARRAY_SIZE(wacom_battery_props);
|
|
|
+ bat_desc->get_property = wacom_battery_get_property;
|
|
|
+ sprintf(battery->bat_name, "wacom_battery_%ld", n);
|
|
|
+ bat_desc->name = battery->bat_name;
|
|
|
+ bat_desc->type = POWER_SUPPLY_TYPE_USB;
|
|
|
+ bat_desc->use_for_apm = 0;
|
|
|
+
|
|
|
+ ps_bat = devm_power_supply_register(dev, bat_desc, &psy_cfg);
|
|
|
+ if (IS_ERR(ps_bat)) {
|
|
|
+ error = PTR_ERR(ps_bat);
|
|
|
+ goto err;
|
|
|
}
|
|
|
- return ret;
|
|
|
+
|
|
|
+ power_supply_powers(ps_bat, &wacom->hdev->dev);
|
|
|
+
|
|
|
+ battery->battery = ps_bat;
|
|
|
+
|
|
|
+ devres_close_group(dev, bat_desc);
|
|
|
+ return 0;
|
|
|
+
|
|
|
+err:
|
|
|
+ devres_release_group(dev, bat_desc);
|
|
|
+ return error;
|
|
|
}
|
|
|
|
|
|
static int wacom_initialize_battery(struct wacom *wacom)
|
|
|
{
|
|
|
- static atomic_t battery_no = ATOMIC_INIT(0);
|
|
|
- struct power_supply_config psy_cfg = { .drv_data = wacom, };
|
|
|
- unsigned long n;
|
|
|
-
|
|
|
- if (wacom->wacom_wac.features.quirks & WACOM_QUIRK_BATTERY) {
|
|
|
- struct power_supply_desc *bat_desc = &wacom->battery_desc;
|
|
|
- struct power_supply_desc *ac_desc = &wacom->ac_desc;
|
|
|
- n = atomic_inc_return(&battery_no) - 1;
|
|
|
-
|
|
|
- bat_desc->properties = wacom_battery_props;
|
|
|
- bat_desc->num_properties = ARRAY_SIZE(wacom_battery_props);
|
|
|
- bat_desc->get_property = wacom_battery_get_property;
|
|
|
- sprintf(wacom->wacom_wac.bat_name, "wacom_battery_%ld", n);
|
|
|
- bat_desc->name = wacom->wacom_wac.bat_name;
|
|
|
- bat_desc->type = POWER_SUPPLY_TYPE_BATTERY;
|
|
|
- bat_desc->use_for_apm = 0;
|
|
|
-
|
|
|
- ac_desc->properties = wacom_ac_props;
|
|
|
- ac_desc->num_properties = ARRAY_SIZE(wacom_ac_props);
|
|
|
- ac_desc->get_property = wacom_ac_get_property;
|
|
|
- sprintf(wacom->wacom_wac.ac_name, "wacom_ac_%ld", n);
|
|
|
- ac_desc->name = wacom->wacom_wac.ac_name;
|
|
|
- ac_desc->type = POWER_SUPPLY_TYPE_MAINS;
|
|
|
- ac_desc->use_for_apm = 0;
|
|
|
-
|
|
|
- wacom->battery = power_supply_register(&wacom->hdev->dev,
|
|
|
- &wacom->battery_desc, &psy_cfg);
|
|
|
- if (IS_ERR(wacom->battery))
|
|
|
- return PTR_ERR(wacom->battery);
|
|
|
-
|
|
|
- power_supply_powers(wacom->battery, &wacom->hdev->dev);
|
|
|
-
|
|
|
- wacom->ac = power_supply_register(&wacom->hdev->dev,
|
|
|
- &wacom->ac_desc,
|
|
|
- &psy_cfg);
|
|
|
- if (IS_ERR(wacom->ac)) {
|
|
|
- power_supply_unregister(wacom->battery);
|
|
|
- return PTR_ERR(wacom->ac);
|
|
|
- }
|
|
|
-
|
|
|
- power_supply_powers(wacom->ac, &wacom->hdev->dev);
|
|
|
- }
|
|
|
+ if (wacom->wacom_wac.features.quirks & WACOM_QUIRK_BATTERY)
|
|
|
+ return __wacom_initialize_battery(wacom, &wacom->battery);
|
|
|
|
|
|
return 0;
|
|
|
}
|
|
|
|
|
|
static void wacom_destroy_battery(struct wacom *wacom)
|
|
|
{
|
|
|
- if (wacom->battery) {
|
|
|
- power_supply_unregister(wacom->battery);
|
|
|
- wacom->battery = NULL;
|
|
|
- power_supply_unregister(wacom->ac);
|
|
|
- wacom->ac = NULL;
|
|
|
+ if (wacom->battery.battery) {
|
|
|
+ devres_release_group(&wacom->hdev->dev,
|
|
|
+ &wacom->battery.bat_desc);
|
|
|
+ wacom->battery.battery = NULL;
|
|
|
}
|
|
|
}
|
|
|
|
|
@@ -1179,7 +1544,7 @@ static ssize_t wacom_show_remote_mode(struct kobject *kobj,
|
|
|
struct wacom *wacom = hid_get_drvdata(hdev);
|
|
|
u8 mode;
|
|
|
|
|
|
- mode = wacom->led.select[index];
|
|
|
+ mode = wacom->led.groups[index].select;
|
|
|
if (mode >= 0 && mode < 3)
|
|
|
return snprintf(buf, PAGE_SIZE, "%d\n", mode);
|
|
|
else
|
|
@@ -1212,54 +1577,30 @@ DEVICE_EKR_ATTR_GROUP(2);
|
|
|
DEVICE_EKR_ATTR_GROUP(3);
|
|
|
DEVICE_EKR_ATTR_GROUP(4);
|
|
|
|
|
|
-int wacom_remote_create_attr_group(struct wacom *wacom, __u32 serial, int index)
|
|
|
+static int wacom_remote_create_attr_group(struct wacom *wacom, __u32 serial,
|
|
|
+ int index)
|
|
|
{
|
|
|
int error = 0;
|
|
|
- char *buf;
|
|
|
- struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
|
|
|
- wacom_wac->serial[index] = serial;
|
|
|
-
|
|
|
- buf = kzalloc(WAC_REMOTE_SERIAL_MAX_STRLEN, GFP_KERNEL);
|
|
|
- if (!buf)
|
|
|
+ remote->remotes[index].group.name = devm_kasprintf(&wacom->hdev->dev,
|
|
|
+ GFP_KERNEL,
|
|
|
+ "%d", serial);
|
|
|
+ if (!remote->remotes[index].group.name)
|
|
|
return -ENOMEM;
|
|
|
- snprintf(buf, WAC_REMOTE_SERIAL_MAX_STRLEN, "%d", serial);
|
|
|
- wacom->remote_group[index].name = buf;
|
|
|
|
|
|
- error = sysfs_create_group(wacom->remote_dir,
|
|
|
- &wacom->remote_group[index]);
|
|
|
+ error = __wacom_devm_sysfs_create_group(wacom, remote->remote_dir,
|
|
|
+ &remote->remotes[index].group);
|
|
|
if (error) {
|
|
|
+ remote->remotes[index].group.name = NULL;
|
|
|
hid_err(wacom->hdev,
|
|
|
"cannot create sysfs group err: %d\n", error);
|
|
|
- kobject_put(wacom->remote_dir);
|
|
|
return error;
|
|
|
}
|
|
|
|
|
|
return 0;
|
|
|
}
|
|
|
|
|
|
-void wacom_remote_destroy_attr_group(struct wacom *wacom, __u32 serial)
|
|
|
-{
|
|
|
- struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
|
- int i;
|
|
|
-
|
|
|
- if (!serial)
|
|
|
- return;
|
|
|
-
|
|
|
- for (i = 0; i < WACOM_MAX_REMOTES; i++) {
|
|
|
- if (wacom_wac->serial[i] == serial) {
|
|
|
- wacom_wac->serial[i] = 0;
|
|
|
- wacom->led.select[i] = WACOM_STATUS_UNKNOWN;
|
|
|
- if (wacom->remote_group[i].name) {
|
|
|
- sysfs_remove_group(wacom->remote_dir,
|
|
|
- &wacom->remote_group[i]);
|
|
|
- kfree(wacom->remote_group[i].name);
|
|
|
- wacom->remote_group[i].name = NULL;
|
|
|
- }
|
|
|
- }
|
|
|
- }
|
|
|
-}
|
|
|
-
|
|
|
static int wacom_cmd_unpair_remote(struct wacom *wacom, unsigned char selector)
|
|
|
{
|
|
|
const size_t buf_size = 2;
|
|
@@ -1316,27 +1657,57 @@ static const struct attribute *remote_unpair_attrs[] = {
|
|
|
NULL
|
|
|
};
|
|
|
|
|
|
-static int wacom_initialize_remote(struct wacom *wacom)
|
|
|
+static void wacom_remotes_destroy(void *data)
|
|
|
+{
|
|
|
+ struct wacom *wacom = data;
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
+
|
|
|
+ if (!remote)
|
|
|
+ return;
|
|
|
+
|
|
|
+ kobject_put(remote->remote_dir);
|
|
|
+ kfifo_free(&remote->remote_fifo);
|
|
|
+ wacom->remote = NULL;
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_initialize_remotes(struct wacom *wacom)
|
|
|
{
|
|
|
int error = 0;
|
|
|
- struct wacom_wac *wacom_wac = &(wacom->wacom_wac);
|
|
|
+ struct wacom_remote *remote;
|
|
|
int i;
|
|
|
|
|
|
if (wacom->wacom_wac.features.type != REMOTE)
|
|
|
return 0;
|
|
|
|
|
|
- wacom->remote_group[0] = remote0_serial_group;
|
|
|
- wacom->remote_group[1] = remote1_serial_group;
|
|
|
- wacom->remote_group[2] = remote2_serial_group;
|
|
|
- wacom->remote_group[3] = remote3_serial_group;
|
|
|
- wacom->remote_group[4] = remote4_serial_group;
|
|
|
+ remote = devm_kzalloc(&wacom->hdev->dev, sizeof(*wacom->remote),
|
|
|
+ GFP_KERNEL);
|
|
|
+ if (!remote)
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ wacom->remote = remote;
|
|
|
|
|
|
- wacom->remote_dir = kobject_create_and_add("wacom_remote",
|
|
|
- &wacom->hdev->dev.kobj);
|
|
|
- if (!wacom->remote_dir)
|
|
|
+ spin_lock_init(&remote->remote_lock);
|
|
|
+
|
|
|
+ error = kfifo_alloc(&remote->remote_fifo,
|
|
|
+ 5 * sizeof(struct wacom_remote_data),
|
|
|
+ GFP_KERNEL);
|
|
|
+ if (error) {
|
|
|
+ hid_err(wacom->hdev, "failed allocating remote_fifo\n");
|
|
|
return -ENOMEM;
|
|
|
+ }
|
|
|
|
|
|
- error = sysfs_create_files(wacom->remote_dir, remote_unpair_attrs);
|
|
|
+ remote->remotes[0].group = remote0_serial_group;
|
|
|
+ remote->remotes[1].group = remote1_serial_group;
|
|
|
+ remote->remotes[2].group = remote2_serial_group;
|
|
|
+ remote->remotes[3].group = remote3_serial_group;
|
|
|
+ remote->remotes[4].group = remote4_serial_group;
|
|
|
+
|
|
|
+ remote->remote_dir = kobject_create_and_add("wacom_remote",
|
|
|
+ &wacom->hdev->dev.kobj);
|
|
|
+ if (!remote->remote_dir)
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ error = sysfs_create_files(remote->remote_dir, remote_unpair_attrs);
|
|
|
|
|
|
if (error) {
|
|
|
hid_err(wacom->hdev,
|
|
@@ -1345,10 +1716,15 @@ static int wacom_initialize_remote(struct wacom *wacom)
|
|
|
}
|
|
|
|
|
|
for (i = 0; i < WACOM_MAX_REMOTES; i++) {
|
|
|
- wacom->led.select[i] = WACOM_STATUS_UNKNOWN;
|
|
|
- wacom_wac->serial[i] = 0;
|
|
|
+ wacom->led.groups[i].select = WACOM_STATUS_UNKNOWN;
|
|
|
+ remote->remotes[i].serial = 0;
|
|
|
}
|
|
|
|
|
|
+ error = devm_add_action_or_reset(&wacom->hdev->dev,
|
|
|
+ wacom_remotes_destroy, wacom);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
return 0;
|
|
|
}
|
|
|
|
|
@@ -1358,7 +1734,7 @@ static struct input_dev *wacom_allocate_input(struct wacom *wacom)
|
|
|
struct hid_device *hdev = wacom->hdev;
|
|
|
struct wacom_wac *wacom_wac = &(wacom->wacom_wac);
|
|
|
|
|
|
- input_dev = input_allocate_device();
|
|
|
+ input_dev = devm_input_allocate_device(&hdev->dev);
|
|
|
if (!input_dev)
|
|
|
return NULL;
|
|
|
|
|
@@ -1377,36 +1753,6 @@ static struct input_dev *wacom_allocate_input(struct wacom *wacom)
|
|
|
return input_dev;
|
|
|
}
|
|
|
|
|
|
-static void wacom_clean_inputs(struct wacom *wacom)
|
|
|
-{
|
|
|
- if (wacom->wacom_wac.pen_input) {
|
|
|
- if (wacom->wacom_wac.pen_registered)
|
|
|
- input_unregister_device(wacom->wacom_wac.pen_input);
|
|
|
- else
|
|
|
- input_free_device(wacom->wacom_wac.pen_input);
|
|
|
- }
|
|
|
- if (wacom->wacom_wac.touch_input) {
|
|
|
- if (wacom->wacom_wac.touch_registered)
|
|
|
- input_unregister_device(wacom->wacom_wac.touch_input);
|
|
|
- else
|
|
|
- input_free_device(wacom->wacom_wac.touch_input);
|
|
|
- }
|
|
|
- if (wacom->wacom_wac.pad_input) {
|
|
|
- if (wacom->wacom_wac.pad_registered)
|
|
|
- input_unregister_device(wacom->wacom_wac.pad_input);
|
|
|
- else
|
|
|
- input_free_device(wacom->wacom_wac.pad_input);
|
|
|
- }
|
|
|
- kobject_put(wacom->remote_dir);
|
|
|
- wacom->wacom_wac.pen_input = NULL;
|
|
|
- wacom->wacom_wac.touch_input = NULL;
|
|
|
- wacom->wacom_wac.pad_input = NULL;
|
|
|
- wacom->wacom_wac.pen_registered = false;
|
|
|
- wacom->wacom_wac.touch_registered = false;
|
|
|
- wacom->wacom_wac.pad_registered = false;
|
|
|
- wacom_destroy_leds(wacom);
|
|
|
-}
|
|
|
-
|
|
|
static int wacom_allocate_inputs(struct wacom *wacom)
|
|
|
{
|
|
|
struct wacom_wac *wacom_wac = &(wacom->wacom_wac);
|
|
@@ -1414,10 +1760,10 @@ static int wacom_allocate_inputs(struct wacom *wacom)
|
|
|
wacom_wac->pen_input = wacom_allocate_input(wacom);
|
|
|
wacom_wac->touch_input = wacom_allocate_input(wacom);
|
|
|
wacom_wac->pad_input = wacom_allocate_input(wacom);
|
|
|
- if (!wacom_wac->pen_input || !wacom_wac->touch_input || !wacom_wac->pad_input) {
|
|
|
- wacom_clean_inputs(wacom);
|
|
|
+ if (!wacom_wac->pen_input ||
|
|
|
+ !wacom_wac->touch_input ||
|
|
|
+ !wacom_wac->pad_input)
|
|
|
return -ENOMEM;
|
|
|
- }
|
|
|
|
|
|
wacom_wac->pen_input->name = wacom_wac->pen_name;
|
|
|
wacom_wac->touch_input->name = wacom_wac->touch_name;
|
|
@@ -1448,8 +1794,7 @@ static int wacom_register_inputs(struct wacom *wacom)
|
|
|
} else {
|
|
|
error = input_register_device(pen_input_dev);
|
|
|
if (error)
|
|
|
- goto fail_register_pen_input;
|
|
|
- wacom_wac->pen_registered = true;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
error = wacom_setup_touch_input_capabilities(touch_input_dev, wacom_wac);
|
|
@@ -1461,8 +1806,7 @@ static int wacom_register_inputs(struct wacom *wacom)
|
|
|
} else {
|
|
|
error = input_register_device(touch_input_dev);
|
|
|
if (error)
|
|
|
- goto fail_register_touch_input;
|
|
|
- wacom_wac->touch_registered = true;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
error = wacom_setup_pad_input_capabilities(pad_input_dev, wacom_wac);
|
|
@@ -1474,37 +1818,15 @@ static int wacom_register_inputs(struct wacom *wacom)
|
|
|
} else {
|
|
|
error = input_register_device(pad_input_dev);
|
|
|
if (error)
|
|
|
- goto fail_register_pad_input;
|
|
|
- wacom_wac->pad_registered = true;
|
|
|
-
|
|
|
- error = wacom_initialize_leds(wacom);
|
|
|
- if (error)
|
|
|
- goto fail_leds;
|
|
|
-
|
|
|
- error = wacom_initialize_remote(wacom);
|
|
|
- if (error)
|
|
|
- goto fail_remote;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
return 0;
|
|
|
|
|
|
-fail_remote:
|
|
|
- wacom_destroy_leds(wacom);
|
|
|
-fail_leds:
|
|
|
- input_unregister_device(pad_input_dev);
|
|
|
- pad_input_dev = NULL;
|
|
|
- wacom_wac->pad_registered = false;
|
|
|
-fail_register_pad_input:
|
|
|
- if (touch_input_dev)
|
|
|
- input_unregister_device(touch_input_dev);
|
|
|
+fail:
|
|
|
+ wacom_wac->pad_input = NULL;
|
|
|
wacom_wac->touch_input = NULL;
|
|
|
- wacom_wac->touch_registered = false;
|
|
|
-fail_register_touch_input:
|
|
|
- if (pen_input_dev)
|
|
|
- input_unregister_device(pen_input_dev);
|
|
|
wacom_wac->pen_input = NULL;
|
|
|
- wacom_wac->pen_registered = false;
|
|
|
-fail_register_pen_input:
|
|
|
return error;
|
|
|
}
|
|
|
|
|
@@ -1543,14 +1865,14 @@ static void wacom_calculate_res(struct wacom_features *features)
|
|
|
|
|
|
void wacom_battery_work(struct work_struct *work)
|
|
|
{
|
|
|
- struct wacom *wacom = container_of(work, struct wacom, work);
|
|
|
+ struct wacom *wacom = container_of(work, struct wacom, battery_work);
|
|
|
|
|
|
if ((wacom->wacom_wac.features.quirks & WACOM_QUIRK_BATTERY) &&
|
|
|
- !wacom->battery) {
|
|
|
+ !wacom->battery.battery) {
|
|
|
wacom_initialize_battery(wacom);
|
|
|
}
|
|
|
else if (!(wacom->wacom_wac.features.quirks & WACOM_QUIRK_BATTERY) &&
|
|
|
- wacom->battery) {
|
|
|
+ wacom->battery.battery) {
|
|
|
wacom_destroy_battery(wacom);
|
|
|
}
|
|
|
}
|
|
@@ -1606,6 +1928,9 @@ static void wacom_update_name(struct wacom *wacom, const char *suffix)
|
|
|
strlcpy(name, features->name, sizeof(name));
|
|
|
}
|
|
|
|
|
|
+ snprintf(wacom_wac->name, sizeof(wacom_wac->name), "%s%s",
|
|
|
+ name, suffix);
|
|
|
+
|
|
|
/* Append the device type to the name */
|
|
|
snprintf(wacom_wac->pen_name, sizeof(wacom_wac->pen_name),
|
|
|
"%s%s Pen", name, suffix);
|
|
@@ -1615,6 +1940,22 @@ static void wacom_update_name(struct wacom *wacom, const char *suffix)
|
|
|
"%s%s Pad", name, suffix);
|
|
|
}
|
|
|
|
|
|
+static void wacom_release_resources(struct wacom *wacom)
|
|
|
+{
|
|
|
+ struct hid_device *hdev = wacom->hdev;
|
|
|
+
|
|
|
+ if (!wacom->resources)
|
|
|
+ return;
|
|
|
+
|
|
|
+ devres_release_group(&hdev->dev, wacom);
|
|
|
+
|
|
|
+ wacom->resources = false;
|
|
|
+
|
|
|
+ wacom->wacom_wac.pen_input = NULL;
|
|
|
+ wacom->wacom_wac.touch_input = NULL;
|
|
|
+ wacom->wacom_wac.pad_input = NULL;
|
|
|
+}
|
|
|
+
|
|
|
static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
{
|
|
|
struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
@@ -1627,9 +1968,14 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
if (features->pktlen > WACOM_PKGLEN_MAX)
|
|
|
return -EINVAL;
|
|
|
|
|
|
+ if (!devres_open_group(&hdev->dev, wacom, GFP_KERNEL))
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ wacom->resources = true;
|
|
|
+
|
|
|
error = wacom_allocate_inputs(wacom);
|
|
|
if (error)
|
|
|
- return error;
|
|
|
+ goto fail;
|
|
|
|
|
|
/*
|
|
|
* Bamboo Pad has a generic hid handling for the Pen, and we switch it
|
|
@@ -1642,7 +1988,7 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
} else if ((features->pktlen != WACOM_PKGLEN_BPAD_TOUCH) &&
|
|
|
(features->pktlen != WACOM_PKGLEN_BPAD_TOUCH_USB)) {
|
|
|
error = -ENODEV;
|
|
|
- goto fail_allocate_inputs;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
}
|
|
|
|
|
@@ -1662,7 +2008,7 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
error ? "Ignoring" : "Assuming pen");
|
|
|
|
|
|
if (error)
|
|
|
- goto fail_parsed;
|
|
|
+ goto fail;
|
|
|
|
|
|
features->device_type |= WACOM_DEVICETYPE_PEN;
|
|
|
}
|
|
@@ -1673,18 +2019,28 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
|
|
|
error = wacom_add_shared_data(hdev);
|
|
|
if (error)
|
|
|
- goto fail_shared_data;
|
|
|
+ goto fail;
|
|
|
|
|
|
if (!(features->device_type & WACOM_DEVICETYPE_WL_MONITOR) &&
|
|
|
(features->quirks & WACOM_QUIRK_BATTERY)) {
|
|
|
error = wacom_initialize_battery(wacom);
|
|
|
if (error)
|
|
|
- goto fail_battery;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
error = wacom_register_inputs(wacom);
|
|
|
if (error)
|
|
|
- goto fail_register_inputs;
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ if (wacom->wacom_wac.features.device_type & WACOM_DEVICETYPE_PAD) {
|
|
|
+ error = wacom_initialize_leds(wacom);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ error = wacom_initialize_remotes(wacom);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+ }
|
|
|
|
|
|
if (features->type == HID_GENERIC)
|
|
|
connect_mask |= HID_CONNECT_DRIVER;
|
|
@@ -1693,7 +2049,7 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
error = hid_hw_start(hdev, connect_mask);
|
|
|
if (error) {
|
|
|
hid_err(hdev, "hw start failed\n");
|
|
|
- goto fail_hw_start;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
if (!wireless) {
|
|
@@ -1705,7 +2061,7 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
if ((features->type == BAMBOO_TOUCH) &&
|
|
|
(features->device_type & WACOM_DEVICETYPE_PEN)) {
|
|
|
error = -ENODEV;
|
|
|
- goto fail_hw_start;
|
|
|
+ goto fail_quirks;
|
|
|
}
|
|
|
|
|
|
/* pen only Bamboo neither support touch nor pad */
|
|
@@ -1713,37 +2069,33 @@ static int wacom_parse_and_register(struct wacom *wacom, bool wireless)
|
|
|
((features->device_type & WACOM_DEVICETYPE_TOUCH) ||
|
|
|
(features->device_type & WACOM_DEVICETYPE_PAD))) {
|
|
|
error = -ENODEV;
|
|
|
- goto fail_hw_start;
|
|
|
+ goto fail_quirks;
|
|
|
}
|
|
|
|
|
|
if (features->device_type & WACOM_DEVICETYPE_WL_MONITOR)
|
|
|
error = hid_hw_open(hdev);
|
|
|
|
|
|
if ((wacom_wac->features.type == INTUOSHT ||
|
|
|
- wacom_wac->features.type == INTUOSHT2) &&
|
|
|
+ wacom_wac->features.type == INTUOSHT2) &&
|
|
|
(wacom_wac->features.device_type & WACOM_DEVICETYPE_TOUCH)) {
|
|
|
- wacom_wac->shared->touch_input = wacom_wac->touch_input;
|
|
|
+ wacom_wac->shared->type = wacom_wac->features.type;
|
|
|
+ wacom_wac->shared->touch_input = wacom_wac->touch_input;
|
|
|
}
|
|
|
|
|
|
+ devres_close_group(&hdev->dev, wacom);
|
|
|
+
|
|
|
return 0;
|
|
|
|
|
|
-fail_hw_start:
|
|
|
+fail_quirks:
|
|
|
hid_hw_stop(hdev);
|
|
|
-fail_register_inputs:
|
|
|
- wacom_clean_inputs(wacom);
|
|
|
- wacom_destroy_battery(wacom);
|
|
|
-fail_battery:
|
|
|
- wacom_remove_shared_data(wacom);
|
|
|
-fail_shared_data:
|
|
|
-fail_parsed:
|
|
|
-fail_allocate_inputs:
|
|
|
- wacom_clean_inputs(wacom);
|
|
|
+fail:
|
|
|
+ wacom_release_resources(wacom);
|
|
|
return error;
|
|
|
}
|
|
|
|
|
|
static void wacom_wireless_work(struct work_struct *work)
|
|
|
{
|
|
|
- struct wacom *wacom = container_of(work, struct wacom, work);
|
|
|
+ struct wacom *wacom = container_of(work, struct wacom, wireless_work);
|
|
|
struct usb_device *usbdev = wacom->usbdev;
|
|
|
struct wacom_wac *wacom_wac = &wacom->wacom_wac;
|
|
|
struct hid_device *hdev1, *hdev2;
|
|
@@ -1762,17 +2114,16 @@ static void wacom_wireless_work(struct work_struct *work)
|
|
|
hdev1 = usb_get_intfdata(usbdev->config->interface[1]);
|
|
|
wacom1 = hid_get_drvdata(hdev1);
|
|
|
wacom_wac1 = &(wacom1->wacom_wac);
|
|
|
- wacom_clean_inputs(wacom1);
|
|
|
+ wacom_release_resources(wacom1);
|
|
|
|
|
|
/* Touch interface */
|
|
|
hdev2 = usb_get_intfdata(usbdev->config->interface[2]);
|
|
|
wacom2 = hid_get_drvdata(hdev2);
|
|
|
wacom_wac2 = &(wacom2->wacom_wac);
|
|
|
- wacom_clean_inputs(wacom2);
|
|
|
+ wacom_release_resources(wacom2);
|
|
|
|
|
|
if (wacom_wac->pid == 0) {
|
|
|
hid_info(wacom->hdev, "wireless tablet disconnected\n");
|
|
|
- wacom_wac1->shared->type = 0;
|
|
|
} else {
|
|
|
const struct hid_device_id *id = wacom_ids;
|
|
|
|
|
@@ -1814,6 +2165,8 @@ static void wacom_wireless_work(struct work_struct *work)
|
|
|
goto fail;
|
|
|
}
|
|
|
|
|
|
+ strlcpy(wacom_wac->name, wacom_wac1->name,
|
|
|
+ sizeof(wacom_wac->name));
|
|
|
error = wacom_initialize_battery(wacom);
|
|
|
if (error)
|
|
|
goto fail;
|
|
@@ -1822,11 +2175,177 @@ static void wacom_wireless_work(struct work_struct *work)
|
|
|
return;
|
|
|
|
|
|
fail:
|
|
|
- wacom_clean_inputs(wacom1);
|
|
|
- wacom_clean_inputs(wacom2);
|
|
|
+ wacom_release_resources(wacom1);
|
|
|
+ wacom_release_resources(wacom2);
|
|
|
return;
|
|
|
}
|
|
|
|
|
|
+static void wacom_remote_destroy_one(struct wacom *wacom, unsigned int index)
|
|
|
+{
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
+ u32 serial = remote->remotes[index].serial;
|
|
|
+ int i;
|
|
|
+ unsigned long flags;
|
|
|
+
|
|
|
+ spin_lock_irqsave(&remote->remote_lock, flags);
|
|
|
+ remote->remotes[index].registered = false;
|
|
|
+ spin_unlock_irqrestore(&remote->remote_lock, flags);
|
|
|
+
|
|
|
+ if (remote->remotes[index].battery.battery)
|
|
|
+ devres_release_group(&wacom->hdev->dev,
|
|
|
+ &remote->remotes[index].battery.bat_desc);
|
|
|
+
|
|
|
+ if (remote->remotes[index].group.name)
|
|
|
+ devres_release_group(&wacom->hdev->dev,
|
|
|
+ &remote->remotes[index]);
|
|
|
+
|
|
|
+ for (i = 0; i < WACOM_MAX_REMOTES; i++) {
|
|
|
+ if (remote->remotes[i].serial == serial) {
|
|
|
+ remote->remotes[i].serial = 0;
|
|
|
+ remote->remotes[i].group.name = NULL;
|
|
|
+ remote->remotes[i].registered = false;
|
|
|
+ remote->remotes[i].battery.battery = NULL;
|
|
|
+ wacom->led.groups[i].select = WACOM_STATUS_UNKNOWN;
|
|
|
+ }
|
|
|
+ }
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_remote_create_one(struct wacom *wacom, u32 serial,
|
|
|
+ unsigned int index)
|
|
|
+{
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
+ struct device *dev = &wacom->hdev->dev;
|
|
|
+ int error, k;
|
|
|
+
|
|
|
+ /* A remote can pair more than once with an EKR,
|
|
|
+ * check to make sure this serial isn't already paired.
|
|
|
+ */
|
|
|
+ for (k = 0; k < WACOM_MAX_REMOTES; k++) {
|
|
|
+ if (remote->remotes[k].serial == serial)
|
|
|
+ break;
|
|
|
+ }
|
|
|
+
|
|
|
+ if (k < WACOM_MAX_REMOTES) {
|
|
|
+ remote->remotes[index].serial = serial;
|
|
|
+ return 0;
|
|
|
+ }
|
|
|
+
|
|
|
+ if (!devres_open_group(dev, &remote->remotes[index], GFP_KERNEL))
|
|
|
+ return -ENOMEM;
|
|
|
+
|
|
|
+ error = wacom_remote_create_attr_group(wacom, serial, index);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ remote->remotes[index].input = wacom_allocate_input(wacom);
|
|
|
+ if (!remote->remotes[index].input) {
|
|
|
+ error = -ENOMEM;
|
|
|
+ goto fail;
|
|
|
+ }
|
|
|
+ remote->remotes[index].input->uniq = remote->remotes[index].group.name;
|
|
|
+ remote->remotes[index].input->name = wacom->wacom_wac.pad_name;
|
|
|
+
|
|
|
+ if (!remote->remotes[index].input->name) {
|
|
|
+ error = -EINVAL;
|
|
|
+ goto fail;
|
|
|
+ }
|
|
|
+
|
|
|
+ error = wacom_setup_pad_input_capabilities(remote->remotes[index].input,
|
|
|
+ &wacom->wacom_wac);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ remote->remotes[index].serial = serial;
|
|
|
+
|
|
|
+ error = input_register_device(remote->remotes[index].input);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ error = wacom_led_groups_alloc_and_register_one(
|
|
|
+ &remote->remotes[index].input->dev,
|
|
|
+ wacom, index, 3, true);
|
|
|
+ if (error)
|
|
|
+ goto fail;
|
|
|
+
|
|
|
+ remote->remotes[index].registered = true;
|
|
|
+
|
|
|
+ devres_close_group(dev, &remote->remotes[index]);
|
|
|
+ return 0;
|
|
|
+
|
|
|
+fail:
|
|
|
+ devres_release_group(dev, &remote->remotes[index]);
|
|
|
+ remote->remotes[index].serial = 0;
|
|
|
+ return error;
|
|
|
+}
|
|
|
+
|
|
|
+static int wacom_remote_attach_battery(struct wacom *wacom, int index)
|
|
|
+{
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
+ int error;
|
|
|
+
|
|
|
+ if (!remote->remotes[index].registered)
|
|
|
+ return 0;
|
|
|
+
|
|
|
+ if (remote->remotes[index].battery.battery)
|
|
|
+ return 0;
|
|
|
+
|
|
|
+ if (wacom->led.groups[index].select == WACOM_STATUS_UNKNOWN)
|
|
|
+ return 0;
|
|
|
+
|
|
|
+ error = __wacom_initialize_battery(wacom,
|
|
|
+ &wacom->remote->remotes[index].battery);
|
|
|
+ if (error)
|
|
|
+ return error;
|
|
|
+
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
+static void wacom_remote_work(struct work_struct *work)
|
|
|
+{
|
|
|
+ struct wacom *wacom = container_of(work, struct wacom, remote_work);
|
|
|
+ struct wacom_remote *remote = wacom->remote;
|
|
|
+ struct wacom_remote_data data;
|
|
|
+ unsigned long flags;
|
|
|
+ unsigned int count;
|
|
|
+ u32 serial;
|
|
|
+ int i;
|
|
|
+
|
|
|
+ spin_lock_irqsave(&remote->remote_lock, flags);
|
|
|
+
|
|
|
+ count = kfifo_out(&remote->remote_fifo, &data, sizeof(data));
|
|
|
+
|
|
|
+ if (count != sizeof(data)) {
|
|
|
+ hid_err(wacom->hdev,
|
|
|
+ "workitem triggered without status available\n");
|
|
|
+ spin_unlock_irqrestore(&remote->remote_lock, flags);
|
|
|
+ return;
|
|
|
+ }
|
|
|
+
|
|
|
+ if (!kfifo_is_empty(&remote->remote_fifo))
|
|
|
+ wacom_schedule_work(&wacom->wacom_wac, WACOM_WORKER_REMOTE);
|
|
|
+
|
|
|
+ spin_unlock_irqrestore(&remote->remote_lock, flags);
|
|
|
+
|
|
|
+ for (i = 0; i < WACOM_MAX_REMOTES; i++) {
|
|
|
+ serial = data.remote[i].serial;
|
|
|
+ if (data.remote[i].connected) {
|
|
|
+
|
|
|
+ if (remote->remotes[i].serial == serial) {
|
|
|
+ wacom_remote_attach_battery(wacom, i);
|
|
|
+ continue;
|
|
|
+ }
|
|
|
+
|
|
|
+ if (remote->remotes[i].serial)
|
|
|
+ wacom_remote_destroy_one(wacom, i);
|
|
|
+
|
|
|
+ wacom_remote_create_one(wacom, serial, i);
|
|
|
+
|
|
|
+ } else if (remote->remotes[i].serial) {
|
|
|
+ wacom_remote_destroy_one(wacom, i);
|
|
|
+ }
|
|
|
+ }
|
|
|
+}
|
|
|
+
|
|
|
static int wacom_probe(struct hid_device *hdev,
|
|
|
const struct hid_device_id *id)
|
|
|
{
|
|
@@ -1845,7 +2364,7 @@ static int wacom_probe(struct hid_device *hdev,
|
|
|
/* hid-core sets this quirk for the boot interface */
|
|
|
hdev->quirks &= ~HID_QUIRK_NOGET;
|
|
|
|
|
|
- wacom = kzalloc(sizeof(struct wacom), GFP_KERNEL);
|
|
|
+ wacom = devm_kzalloc(&hdev->dev, sizeof(struct wacom), GFP_KERNEL);
|
|
|
if (!wacom)
|
|
|
return -ENOMEM;
|
|
|
|
|
@@ -1858,7 +2377,7 @@ static int wacom_probe(struct hid_device *hdev,
|
|
|
|
|
|
if (features->check_for_hid_type && features->hid_type != hdev->type) {
|
|
|
error = -ENODEV;
|
|
|
- goto fail_type;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
wacom_wac->hid_data.inputmode = -1;
|
|
@@ -1867,18 +2386,20 @@ static int wacom_probe(struct hid_device *hdev,
|
|
|
wacom->usbdev = dev;
|
|
|
wacom->intf = intf;
|
|
|
mutex_init(&wacom->lock);
|
|
|
- INIT_WORK(&wacom->work, wacom_wireless_work);
|
|
|
+ INIT_WORK(&wacom->wireless_work, wacom_wireless_work);
|
|
|
+ INIT_WORK(&wacom->battery_work, wacom_battery_work);
|
|
|
+ INIT_WORK(&wacom->remote_work, wacom_remote_work);
|
|
|
|
|
|
/* ask for the report descriptor to be loaded by HID */
|
|
|
error = hid_parse(hdev);
|
|
|
if (error) {
|
|
|
hid_err(hdev, "parse failed\n");
|
|
|
- goto fail_parse;
|
|
|
+ goto fail;
|
|
|
}
|
|
|
|
|
|
error = wacom_parse_and_register(wacom, false);
|
|
|
if (error)
|
|
|
- goto fail_parse;
|
|
|
+ goto fail;
|
|
|
|
|
|
if (hdev->bus == BUS_BLUETOOTH) {
|
|
|
error = device_create_file(&hdev->dev, &dev_attr_speed);
|
|
@@ -1890,9 +2411,7 @@ static int wacom_probe(struct hid_device *hdev,
|
|
|
|
|
|
return 0;
|
|
|
|
|
|
-fail_type:
|
|
|
-fail_parse:
|
|
|
- kfree(wacom);
|
|
|
+fail:
|
|
|
hid_set_drvdata(hdev, NULL);
|
|
|
return error;
|
|
|
}
|
|
@@ -1908,15 +2427,13 @@ static void wacom_remove(struct hid_device *hdev)
|
|
|
|
|
|
hid_hw_stop(hdev);
|
|
|
|
|
|
- cancel_work_sync(&wacom->work);
|
|
|
- wacom_clean_inputs(wacom);
|
|
|
+ cancel_work_sync(&wacom->wireless_work);
|
|
|
+ cancel_work_sync(&wacom->battery_work);
|
|
|
+ cancel_work_sync(&wacom->remote_work);
|
|
|
if (hdev->bus == BUS_BLUETOOTH)
|
|
|
device_remove_file(&hdev->dev, &dev_attr_speed);
|
|
|
- wacom_destroy_battery(wacom);
|
|
|
- wacom_remove_shared_data(wacom);
|
|
|
|
|
|
hid_set_drvdata(hdev, NULL);
|
|
|
- kfree(wacom);
|
|
|
}
|
|
|
|
|
|
#ifdef CONFIG_PM
|