Skip to content

Add PANORAMA 391a:1011 printer protocol support - #17

Merged
DXVSI merged 1 commit into
DXVSI:feature/panorama-1011-supportfrom
bolidbake:production
Aug 7, 2026
Merged

Add PANORAMA 391a:1011 printer protocol support#17
DXVSI merged 1 commit into
DXVSI:feature/panorama-1011-supportfrom
bolidbake:production

Conversation

@bolidbake

Copy link
Copy Markdown
Contributor

Warning

I used ChatGPT as a guide while working through the development process. The
code changes, testing, and hardware verification were performed by me.

No GUI changes were made. This PR only adds support for detecting the original
TRYX PANORAMA ARGB 360 hardware (391a:1011). After this change, the existing
runtime works with the device correctly.

Summary

Adds USB printer-class support for TRYX PANORAMA devices exposing
VID:PID 391a:1011.

Changes

  • Add kPanoProductId = 0x1011
  • Allow 391a:1011 devices through libusb discovery and interface selection
  • Include 391a:1011 in udev discovery handling
  • Add udev access rule for direct libusb access

Testing

Tested on physical TRYX PANORAMA ARGB 360 hardware.

USB identity:

  • VID:PID: 391a:1011
  • USB product string: RK PANO

Verified:

  • USB device discovery
  • Printer interface claim
  • DeviceInfo bootstrap
  • Post-bootstrap ping
  • Active PASE display session

Runtime logs reached:

device_info_confirmed
bootstrap_completed
display_session_active

@DXVSI

DXVSI commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Thanks a lot for the PR! I’ll test it on my side, and if everything works as expected, I’ll approve and merge it.

@DXVSI
DXVSI self-requested a review August 7, 2026 02:14
@DXVSI
DXVSI changed the base branch from production to feature/panorama-1011-support August 7, 2026 02:22

@DXVSI DXVSI left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One blocking packaging issue found. The implementation and local test suite otherwise look good.

Comment thread packaging/70-tryx-pase-access.rules

@DXVSI DXVSI left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The requested printer-suppression rule is present, and all eight package and clean-runtime jobs pass for head 1d5a96e. Approved for the feature branch.

@DXVSI
DXVSI merged commit 2ee85b3 into DXVSI:feature/panorama-1011-support Aug 7, 2026
8 checks passed
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.

2 participants