From: Aaron Armstrong Skomra <sko...@gmail.com> Commit a50aac7193f1 introduces 'led.groups' and adds EKR support for these groups. However, unlike the other devices with LEDs, the EKR's LEDs are read-only and we shouldn't attempt to control them in wacom_led_control().
See bug: https://sourceforge.net/p/linuxwacom/bugs/342/ Fixes: a50aac7193f1 ("backport: HID: wacom: leds: dynamically allocate LED groups") Cc: stable <sta...@vger.kernel.org> # 4.9 Signed-off-by: Aaron Armstrong Skomra <aaron.sko...@wacom.com> Reviewed-by: Jason Gerecke <jason.gere...@wacom.com> Signed-off-by: Jiri Kosina <jkos...@suse.cz> [jason.gere...@wacom.com: Imported into input-wacom repository (74aebed6dc)] Signed-off-by: Jason Gerecke <jason.gere...@wacom.com> [jason.gere...@wacom.com: Backported from input-wacom repository (7a69536)] Signed-off-by: Jason Gerecke <jason.gere...@wacom.com> --- 3.17/wacom_sys.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/3.17/wacom_sys.c b/3.17/wacom_sys.c index 2996d0d..8fd0b78 100644 --- a/3.17/wacom_sys.c +++ b/3.17/wacom_sys.c @@ -794,6 +794,9 @@ static int wacom_led_control(struct wacom *wacom) if (!wacom->led.groups) return -ENOTSUPP; + if (wacom->wacom_wac.features.type == REMOTE) + return -ENOTSUPP; + if (wacom->wacom_wac.pid) { /* wireless connected */ report_id = WAC_CMD_WL_LED_CONTROL; buf_size = 13; -- 2.14.1 ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linuxwacom-devel mailing list Linuxwacom-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel
