Skip to content

Commit 34281b4

Browse files
endriftJiri Kosina
authored andcommitted
HID: hid-steam: Avoid overwriting smoothing parameter
The original implementation of this driver incorrectly guessed the function of this register. It's not only unnecessary to write to this register for lizard mode but actually counter-productive since it overwrites whatever previous value was intentionally set, for example by Steam. Signed-off-by: Vicki Pfau <[email protected]> Signed-off-by: Jiri Kosina <[email protected]>
1 parent 3f10e21 commit 34281b4

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

drivers/hid/hid-steam.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -340,9 +340,6 @@ static void steam_set_lizard_mode(struct steam_device *steam, bool enable)
340340
steam_send_report_byte(steam, STEAM_CMD_DEFAULT_MAPPINGS);
341341
/* enable mouse */
342342
steam_send_report_byte(steam, STEAM_CMD_DEFAULT_MOUSE);
343-
steam_write_registers(steam,
344-
STEAM_REG_RPAD_MARGIN, 0x01, /* enable margin */
345-
0);
346343

347344
cancel_delayed_work_sync(&steam->heartbeat);
348345
} else {
@@ -351,7 +348,6 @@ static void steam_set_lizard_mode(struct steam_device *steam, bool enable)
351348

352349
if (steam->quirks & STEAM_QUIRK_DECK) {
353350
steam_write_registers(steam,
354-
STEAM_REG_RPAD_MARGIN, 0x00, /* disable margin */
355351
STEAM_REG_LPAD_MODE, 0x07, /* disable mouse */
356352
STEAM_REG_RPAD_MODE, 0x07, /* disable mouse */
357353
STEAM_REG_LPAD_CLICK_PRESSURE, 0xFFFF, /* disable clicky pad */
@@ -365,7 +361,6 @@ static void steam_set_lizard_mode(struct steam_device *steam, bool enable)
365361
schedule_delayed_work(&steam->heartbeat, 5 * HZ);
366362
} else {
367363
steam_write_registers(steam,
368-
STEAM_REG_RPAD_MARGIN, 0x00, /* disable margin */
369364
STEAM_REG_LPAD_MODE, 0x07, /* disable mouse */
370365
STEAM_REG_RPAD_MODE, 0x07, /* disable mouse */
371366
0);

0 commit comments

Comments
 (0)