On Mon, Aug 28, 2017 at 2:15 PM, Aaron Armstrong Skomra <skomra@xxxxxxxxx> wrote: > > 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 ("HID: wacom: leds: dynamically allocate LED groups") > Cc: stable <stable@xxxxxxxxxxxxxxx> # 4.9 > Signed-off-by: Aaron Armstrong Skomra <aaron.skomra@xxxxxxxxx> Looks good to me: Reviewed-by: Jason Gerecke <jason.gerecke@xxxxxxxxx> Jason --- Now instead of four in the eights place / you’ve got three, ‘Cause you added one / (That is to say, eight) to the two, / But you can’t take seven from three, / So you look at the sixty-fours.... > > --- > drivers/hid/wacom_sys.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c > index 838c1ebfffa9..db1f64dcb9ba 100644 > --- a/drivers/hid/wacom_sys.c > +++ b/drivers/hid/wacom_sys.c > @@ -766,6 +766,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.7.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
