Skip to content

Add Cadyce CA-U3HE as compatible device #626

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Raymo111
Copy link

No description provided.

@mvp
Copy link
Owner

mvp commented Jun 16, 2025

We can't just add devices without any evidence of vbus off testing.
Please tell how you tested vbus off support.

@Raymo111
Copy link
Author

Raymo111 commented Jun 16, 2025

@mvp I followed the instructions in the readme for new devices, checked lsusb -v:

Bus 003 Device 025: ID 2109:2812 VIA Labs, Inc. VL812 Hub
...
Hub Descriptor:
  bLength               9
  bDescriptorType      41
  nNbrPorts             4 # 1-3 are USB-A, 4 is ethernet
  wHubCharacteristic 0x01e9
    Per-port power switching
    Per-port overcurrent protection
    TT think time 32 FS bits
    Port indicator

Then checked that sudo uhubctl -a {on,off,cycle} -l 3-1 -p [1-3] -f (where 3-1* was this hub) worked (Actually turned off the devices plugged in - mouse, keyboard and charger).

*found this by sudo rg 2812 /sys/bus/usb/devices/*/idProduct, you may want to add a note on this in the readme because lsusb doesn't show the part after the bus number, only the bus number.

Lmk if you want me to record a video proving this.

@mvp
Copy link
Owner

mvp commented Jun 16, 2025

Thanks. Of course I don't need video. But best test is voltage dropping to zero or phone stopping charging when port is off

Reason why I want this confirmation - all too often, when hubs advertise per port power switching, uhubctl is usually able to turn port off which always stops data communication. But some hubs don't remove power (aka VBUS) upon port turn off. We don't want to add such devices.

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