Commit f81a1295 authored by Benjamin Tissoires's avatar Benjamin Tissoires Committed by Dmitry Torokhov

Input: wacom - prepare the driver to include BT devices

Now that wacom is a hid driver, there is no point in having a separate
driver for bluetooth devices. This patch prepares the common paths of
Bluetooth devices in the common wacom driver. It also adds the sysfs file
"speed" used by Bluetooth devices.
Signed-off-by: default avatarBenjamin Tissoires <benjamin.tissoires@redhat.com>
Reviewed-by: default avatarPing Cheng <pingc@wacom.com>
Tested-by: default avatarPrzemo Firszt <przemo@firszt.eu>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent aeaf50d4
...@@ -262,6 +262,12 @@ static int wacom_set_device_mode(struct hid_device *hdev, int report_id, ...@@ -262,6 +262,12 @@ static int wacom_set_device_mode(struct hid_device *hdev, int report_id,
return error < 0 ? error : 0; return error < 0 ? error : 0;
} }
static int wacom_bt_query_tablet_data(struct hid_device *hdev, u8 speed,
struct wacom_features *features)
{
return 0;
}
/* /*
* Switch the tablet into its most-capable mode. Wacom tablets are * Switch the tablet into its most-capable mode. Wacom tablets are
* typically configured to power-up in a mode which sends mouse-like * typically configured to power-up in a mode which sends mouse-like
...@@ -272,6 +278,9 @@ static int wacom_set_device_mode(struct hid_device *hdev, int report_id, ...@@ -272,6 +278,9 @@ static int wacom_set_device_mode(struct hid_device *hdev, int report_id,
static int wacom_query_tablet_data(struct hid_device *hdev, static int wacom_query_tablet_data(struct hid_device *hdev,
struct wacom_features *features) struct wacom_features *features)
{ {
if (hdev->bus == BUS_BLUETOOTH)
return wacom_bt_query_tablet_data(hdev, 1, features);
if (features->device_type == BTN_TOOL_FINGER) { if (features->device_type == BTN_TOOL_FINGER) {
if (features->type > TABLETPC) { if (features->type > TABLETPC) {
/* MT Tablet PC touch */ /* MT Tablet PC touch */
...@@ -890,6 +899,38 @@ static void wacom_destroy_battery(struct wacom *wacom) ...@@ -890,6 +899,38 @@ static void wacom_destroy_battery(struct wacom *wacom)
} }
} }
static ssize_t wacom_show_speed(struct device *dev,
struct device_attribute
*attr, char *buf)
{
struct hid_device *hdev = container_of(dev, struct hid_device, dev);
struct wacom *wacom = hid_get_drvdata(hdev);
return snprintf(buf, PAGE_SIZE, "%i\n", wacom->wacom_wac.bt_high_speed);
}
static ssize_t wacom_store_speed(struct device *dev,
struct device_attribute *attr,
const char *buf, size_t count)
{
struct hid_device *hdev = container_of(dev, struct hid_device, dev);
struct wacom *wacom = hid_get_drvdata(hdev);
u8 new_speed;
if (kstrtou8(buf, 0, &new_speed))
return -EINVAL;
if (new_speed != 0 && new_speed != 1)
return -EINVAL;
wacom_bt_query_tablet_data(hdev, new_speed, &wacom->wacom_wac.features);
return count;
}
static DEVICE_ATTR(speed, S_IRUGO | S_IWUSR | S_IWGRP,
wacom_show_speed, wacom_store_speed);
static struct input_dev *wacom_allocate_input(struct wacom *wacom) static struct input_dev *wacom_allocate_input(struct wacom *wacom)
{ {
struct input_dev *input_dev; struct input_dev *input_dev;
...@@ -1210,6 +1251,9 @@ static int wacom_probe(struct hid_device *hdev, ...@@ -1210,6 +1251,9 @@ static int wacom_probe(struct hid_device *hdev,
features->y_max = 4096; features->y_max = 4096;
} }
if (hdev->bus == BUS_BLUETOOTH)
features->quirks |= WACOM_QUIRK_BATTERY;
wacom_setup_device_quirks(features); wacom_setup_device_quirks(features);
/* set unit to "100th of a mm" for devices not reported by HID */ /* set unit to "100th of a mm" for devices not reported by HID */
...@@ -1241,10 +1285,25 @@ static int wacom_probe(struct hid_device *hdev, ...@@ -1241,10 +1285,25 @@ static int wacom_probe(struct hid_device *hdev,
if (error) if (error)
goto fail2; goto fail2;
if (!(features->quirks & WACOM_QUIRK_MONITOR) &&
(features->quirks & WACOM_QUIRK_BATTERY)) {
error = wacom_initialize_battery(wacom);
if (error)
goto fail3;
}
if (!(features->quirks & WACOM_QUIRK_NO_INPUT)) { if (!(features->quirks & WACOM_QUIRK_NO_INPUT)) {
error = wacom_register_inputs(wacom); error = wacom_register_inputs(wacom);
if (error) if (error)
goto fail3; goto fail4;
}
if (hdev->bus == BUS_BLUETOOTH) {
error = device_create_file(&hdev->dev, &dev_attr_speed);
if (error)
hid_warn(hdev,
"can't create sysfs speed attribute err: %d\n",
error);
} }
/* Note that if query fails it is not a hard failure */ /* Note that if query fails it is not a hard failure */
...@@ -1254,7 +1313,7 @@ static int wacom_probe(struct hid_device *hdev, ...@@ -1254,7 +1313,7 @@ static int wacom_probe(struct hid_device *hdev,
error = hid_hw_start(hdev, HID_CONNECT_HIDRAW); error = hid_hw_start(hdev, HID_CONNECT_HIDRAW);
if (error) { if (error) {
hid_err(hdev, "hw start failed\n"); hid_err(hdev, "hw start failed\n");
goto fail4; goto fail5;
} }
if (features->quirks & WACOM_QUIRK_MONITOR) if (features->quirks & WACOM_QUIRK_MONITOR)
...@@ -1267,7 +1326,10 @@ static int wacom_probe(struct hid_device *hdev, ...@@ -1267,7 +1326,10 @@ static int wacom_probe(struct hid_device *hdev,
return 0; return 0;
fail4: wacom_unregister_inputs(wacom); fail5: if (hdev->bus == BUS_BLUETOOTH)
device_remove_file(&hdev->dev, &dev_attr_speed);
wacom_unregister_inputs(wacom);
fail4: wacom_destroy_battery(wacom);
fail3: wacom_destroy_leds(wacom); fail3: wacom_destroy_leds(wacom);
fail2: wacom_remove_shared_data(wacom_wac); fail2: wacom_remove_shared_data(wacom_wac);
fail1: kfree(wacom); fail1: kfree(wacom);
...@@ -1283,6 +1345,8 @@ static void wacom_remove(struct hid_device *hdev) ...@@ -1283,6 +1345,8 @@ static void wacom_remove(struct hid_device *hdev)
cancel_work_sync(&wacom->work); cancel_work_sync(&wacom->work);
wacom_unregister_inputs(wacom); wacom_unregister_inputs(wacom);
if (hdev->bus == BUS_BLUETOOTH)
device_remove_file(&hdev->dev, &dev_attr_speed);
wacom_destroy_battery(wacom); wacom_destroy_battery(wacom);
wacom_destroy_leds(wacom); wacom_destroy_leds(wacom);
wacom_remove_shared_data(&wacom->wacom_wac); wacom_remove_shared_data(&wacom->wacom_wac);
......
...@@ -169,6 +169,8 @@ struct wacom_wac { ...@@ -169,6 +169,8 @@ struct wacom_wac {
int num_contacts_left; int num_contacts_left;
int bat_charging; int bat_charging;
int ps_connected; int ps_connected;
u8 bt_features;
u8 bt_high_speed;
}; };
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment