greatscottgadgets / cynthion

USB test instrument
https://greatscottgadgets.com/cynthion/
BSD 3-Clause "New" or "Revised" License
78 stars 19 forks source link

Update analyzer descriptors and request handling #87

Closed martinling closed 4 months ago

martinling commented 4 months ago

This PR updates the analyzer gateware's descriptors and request handling as per recent discussions.

The changes are as follows:

Device descriptor:

Interface descriptor:

Request handler:

Corresponding Packetry support is in https://github.com/greatscottgadgets/packetry/pull/93.

martinling commented 4 months ago

Just realised that the bcdVersion usage here isn't consistent with Apollo. Cynthion r1.4 should be encoded as 1.04, not 1.40. I'll fix that.

martinling commented 4 months ago

Fixed. Now stores the version as a tuple on the platforms, and converts to the appropriate form when setting bcdDevice.

martinling commented 4 months ago

Rebased now that #86 is merged, so that bInterfaceProtocol version starts at 0x00.