Support Vivobook S 16 (M5606K) with dynamic report_id configuration

[owsma]

Summary

If the relevant HID on the Vivobook S 16 (M5606K) is detected, the correct firmware and color report_ids are set.

Tried to keep the patch as minimal as possible - sorry for setting global variables inside a function. Wasn't sure what to bump the version to or how I should edit the README as I'm just getting started with open source.

Tested on my M5606K only.

Debug logs before merge

vrgb --debug status:

[debug] hidraw0: hid_id=0018:0000093A:00003017 hid_name=ASCP1200:00 093A:3017 score=-1 reason=no match
[debug] hidraw1: hid_id=0018:00000B05:00005570 hid_name=ITE5570:00 0B05:5570 score=-1 reason=no match
Error: VRGB HID device not found

Debug logs after merge

vrgb --debug status:

[debug] hidraw0: hid_id=0018:0000093A:00003017 hid_name=ASCP1200:00 093A:3017 score=-1 reason=no match
[debug] hidraw1: hid_id=0018:00000B05:00005570 hid_name=ITE5570:00 0B05:5570 score=100 reason=exact HID_ID match
[debug] Selected device: /dev/hidraw1 (exact HID_ID match)
[debug] Using report IDs: firmware=0x46 color=0x45
Device: /dev/hidraw1
Saved color: #f7630c
Saved brightness: 100 %
Last-on brightness: 100 %
Saved mode: host/static
[debug] status complete mode=host/static

vrgb --debug set FF0000 100:

[debug] hidraw0: hid_id=0018:0000093A:00003017 hid_name=ASCP1200:00 093A:3017 score=-1 reason=no match
[debug] hidraw1: hid_id=0018:00000B05:00005570 hid_name=ITE5570:00 0B05:5570 score=100 reason=exact HID_ID match
[debug] Selected device: /dev/hidraw1 (exact HID_ID match)
[debug] Using report IDs: firmware=0x46 color=0x45
[debug] cmd_set color=FF0000 percent=100 intensity=255
[debug] set_firmware_mode enabled=False
[debug] hid_set_feature dev=/dev/hidraw1 report=0x46 payload=00
[debug] set_color r=255 g=0 b=0 intensity=255
[debug] hid_set_feature dev=/dev/hidraw1 report=0x45 payload=0100000000ff0000ff

[vrgb-dev]

Really appreciate you taking the time to dig into this and put together a patch on your S 16. This is exactly the kind of hardware feedback I was hoping to get early on. The logs and notes you included are super helpful, especially confirming behavior on a different Vivobook model.

I’m going to review this over the next few days (I'm new to the open source world as well, it's all good) and adapt the approach into the main branch in a way that keeps the current path stable while expanding support for your model.

Thanks again for contributing and for testing on your hardware. It helps a lot, and is exciting to see that this script can work on other laptops.

[vrgb-dev]

Thanks again for your patch and the logs. I've now integrated your report ID findings into the main script in a slightly cleaner way by moving VRGB to a supported-device mapping, while keeping the existing profile support intact. I've validated the updated code on my S5406SA S14, and the new code is now on main (v.0.3.1).

When you get a chance, could you pull the latest v.0.3.1 from main and test it on your M5606K S16? I'd love confirmation that everything works (status, set, brightness, off, restore, auto on/off, etc) on your system.

You taking the time to test your system out and contribute really made this device support possible, so I really appreciate your help.