Skip to content

Expose USB configuration descriptor (#4) in UI#1414

Open
slinkypew wants to merge 3 commits into
jetkvm:devfrom
slinkypew:dev
Open

Expose USB configuration descriptor (#4) in UI#1414
slinkypew wants to merge 3 commits into
jetkvm:devfrom
slinkypew:dev

Conversation

@slinkypew
Copy link
Copy Markdown

@slinkypew slinkypew commented Apr 16, 2026

Addresses #418

Summary

Adds support for configuring USB descriptor #4 (configuration string) via the UI. The previously hardcoded value is now used as a default but can be overridden. This brings it in line with other configurable USB identifiers and improves compatibility across a wider range of host systems.

Checklist

  • Verified functionality on JetKVM device via dev_deploy.sh
  • Confirmed descriptor changes on Windows host using Thesycon USB Descriptor Dumper
  • Added localization entries for all supported languages
  • One problem per PR (no unrelated changes)

Note

Medium Risk
Touches USB gadget descriptor values that affect how hosts enumerate the device; misconfiguration could cause compatibility issues, though the change is additive and keeps the previous value as the default.

Overview
Adds support for editing the USB gadget configuration string (descriptor #4) end-to-end.

Backend usbgadget.Config and defaults now include a Configuration field and propagate it into the gadget’s base_info configfs configuration attribute, while the UI USB identifiers form/store and preset profiles gain a new required configuration input plus localized label/placeholder strings across languages.

Reviewed by Cursor Bugbot for commit c8b9ed0. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 16, 2026

CLA assistant check
All committers have signed the CLA.

@slinkypew
Copy link
Copy Markdown
Author

Configuration (Descriptor 4) setting in JetKVM:
image

Configuration (Descriptor 4) in tdd.exe on Windows:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants