Skip to content
Snippets Groups Projects

pocket-hid: add fwupd and reset capability

Merged Chris Hofstaedtler requested to merge zeha/pocket-reform:zeha-kbd-fwupd into main

This makes resetting using reform-mcu-tool work, and also fwupd works.

reform-mcu-tool shows a similar behavior to what was reported in !24 (merged), which I never could reproduce with the sysctl firmware. But the hid firmware shows the same issue!

reform-mcu-tool fix can happen later, resetting works after all.

% sudo reform-mcu-tool bootsel pocket-input-1.0
I: Found MNT Pocket Reform Input 1.0 on bus 3 address 16
I: Resetting device with serial DE625492032E7A32 into BOOTSEL
Traceback (most recent call last):
  File "/usr/sbin/reform-mcu-tool", line 246, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/sbin/reform-mcu-tool", line 242, in main
    return run(args, usb_context)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/sbin/reform-mcu-tool", line 233, in run
    return args.func(args, device)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/sbin/reform-mcu-tool", line 146, in action_bootsel
    device_reset(handle, reset_interface, RESET_REQUEST_BOOTSEL)
  File "/usr/sbin/reform-mcu-tool", line 74, in device_reset
    handle.controlWrite(
  File "/usr/lib/python3/dist-packages/usb1/__init__.py", line 1330, in controlWrite
    return self._controlTransfer(request_type, request, value, index, data,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3/dist-packages/usb1/__init__.py", line 1307, in _controlTransfer
    mayRaiseUSBError(result)
  File "/usr/lib/python3/dist-packages/usb1/__init__.py", line 127, in mayRaiseUSBError
    __raiseUSBError(value)
  File "/usr/lib/python3/dist-packages/usb1/__init__.py", line 119, in raiseUSBError
    raise __STATUS_TO_EXCEPTION_DICT.get(value, __USBError)(value)
usb1.USBErrorPipe: LIBUSB_ERROR_PIPE [-9]
Edited by Chris Hofstaedtler

Merge request reports

Pipeline #3290 passed

Pipeline passed for f64db25f on zeha:zeha-kbd-fwupd

Approval is optional

Merged by minuteminute 2 months ago (Jan 2, 2025 8:39pm UTC)

Loading

Pipeline #3291 passed

Pipeline passed for b7520c17 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading