Tryanks/gousb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Set alternate setting for the interface only if required. (#38)

Browse Source 
Spec says devices must support "set interface" if they have more than
one alternate setting.
This commit is contained in:
Sebastian Zagrodzki
2018-05-23 10:08:10 +02:00
committed by GitHub
parent 815369492f
commit 21b0ed46ae
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
config.go
View File

@@ -124,9 +124,12 @@ func (c *Config) Interface(num, alt int) (*Interface, error) {
return nil, fmt.Errorf("failed to claim interface %d on %s: %v", num, c, err)
}
if err := c.dev.ctx.libusb.setAlt(c.dev.handle, uint8(num), uint8(alt)); err != nil {
c.dev.ctx.libusb.release(c.dev.handle, uint8(num))
return nil, fmt.Errorf("failed to set alternate config %d on interface %d of %s: %v", alt, num, c, err)
// Select an alternate setting if needed (device has multiple alternate settings).
if len(c.Desc.Interfaces[num].AltSettings) > 1 {
if err := c.dev.ctx.libusb.setAlt(c.dev.handle, uint8(num), uint8(alt)); err != nil {
c.dev.ctx.libusb.release(c.dev.handle, uint8(num))
return nil, fmt.Errorf("failed to set alternate config %d on interface %d of %s: %v", alt, num, c, err)
}
}
c.claimed[num] = true