worproject / Rockchip-Windows-Drivers

Windows on Arm drivers for RK35xx platforms.
123 stars 15 forks source link

Update README.md #8

Closed idigdoug closed 10 months ago

mariobalanica commented 10 months ago

"USB 3 Host port 2" description looks okay, but "USB 3 Host ports 0 and 1" are actually dual-role: Type-C and on most RK3588 boards also a Type-A port. OPi 5+ has a hub connected to one of them, so it's the only supported board so far with two fully working USB 3 ports.

Since dual-role functionality doesn't work and I've seen issues with a few devices (especially connected to Type-C), it would be better for it to remain "Partially working" with the original description + the clarifications above.

PCIe doesn't work with all devices that have drivers: some expect a cache coherent bus, others require MSI (Qualcomm Wi-Fi cards), XHCI wants a root port. We could add a list below with the known working devices.

idigdoug commented 10 months ago

"USB 3 Host port 2" description looks okay, but "USB 3 Host ports 0 and 1" are actually dual-role: Type-C and on most RK3588 boards also a Type-A port. OPi 5+ has a hub connected to one of them, so it's the only supported board so far with two fully working USB 3 ports.

Since dual-role functionality doesn't work and I've seen issues with a few devices (especially connected to Type-C), it would be better for it to remain "Partially working" with the original description + the clarifications above.

Text in main has "USB 3 Host" as "Working", and "USB 3 Dual Role" on a separate line "Partially working". This PR doesn't change that.

And this change has a note "while RK3588s devices have only 1 'full' USB3 port".

So I'm not sure what you're suggesting. Maybe move the "USB 3 Dual Role" line up above the "USB Host port 2"?

PCIe doesn't work with all devices that have drivers: some expect a cache coherent bus, others require MSI (Qualcomm Wi-Fi cards), XHCI wants a root port. We could add a list below with the known working devices.

Good point. I'll fix that.

mariobalanica commented 10 months ago

Looks good now. Thanks!