Skip to content

Commit e26f0b3

Browse files
deadprogramaykevl
authored andcommitted
machine/atsamd21: correct order of params for USB CDC descriptor
Signed-off-by: Ron Evans <[email protected]>
1 parent 4ee26fd commit e26f0b3

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed

src/machine/machine_atsamd21.go

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1408,12 +1408,8 @@ func handleUSB() {
14081408
handleEndpoint(i)
14091409
}
14101410
setEPINTFLAG(i, epFlags)
1411-
case usb_CDC_ENDPOINT_IN, usb_CDC_ENDPOINT_ACM:
1412-
// set bank ready
1413-
setEPSTATUSCLR(i, sam.USB_DEVICE_EPSTATUSCLR_BK1RDY)
1414-
1415-
// ack transfer complete
1416-
setEPINTFLAG(i, sam.USB_DEVICE_EPINTFLAG_TRCPT1)
1411+
case usb_CDC_ENDPOINT_ACM:
1412+
setEPINTFLAG(i, epFlags)
14171413
}
14181414
}
14191415
}
@@ -1808,9 +1804,9 @@ func sendConfiguration(setup usbSetup) {
18081804

18091805
dif := NewInterfaceDescriptor(usb_CDC_DATA_INTERFACE, 2, usb_CDC_DATA_INTERFACE_CLASS, 0, 0)
18101806

1811-
in := NewEndpointDescriptor((usb_CDC_ENDPOINT_OUT | usbEndpointOut), usb_ENDPOINT_TYPE_BULK, usbEndpointPacketSize, 0)
1807+
out := NewEndpointDescriptor((usb_CDC_ENDPOINT_OUT | usbEndpointOut), usb_ENDPOINT_TYPE_BULK, usbEndpointPacketSize, 0)
18121808

1813-
out := NewEndpointDescriptor((usb_CDC_ENDPOINT_IN | usbEndpointIn), usb_ENDPOINT_TYPE_BULK, usbEndpointPacketSize, 0)
1809+
in := NewEndpointDescriptor((usb_CDC_ENDPOINT_IN | usbEndpointIn), usb_ENDPOINT_TYPE_BULK, usbEndpointPacketSize, 0)
18141810

18151811
cdc := NewCDCDescriptor(iad,
18161812
cif,
@@ -1820,8 +1816,8 @@ func sendConfiguration(setup usbSetup) {
18201816
callManagement,
18211817
cifin,
18221818
dif,
1823-
in,
1824-
out)
1819+
out,
1820+
in)
18251821

18261822
sz := uint16(configDescriptorSize + cdcSize)
18271823
config := NewConfigDescriptor(sz, 2)

src/machine/usb.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -316,8 +316,8 @@ func NewCDCDescriptor(i IADDescriptor, c InterfaceDescriptor,
316316
callm CMFunctionalDescriptor,
317317
ci EndpointDescriptor,
318318
di InterfaceDescriptor,
319-
inp EndpointDescriptor,
320-
outp EndpointDescriptor) CDCDescriptor {
319+
outp EndpointDescriptor,
320+
inp EndpointDescriptor) CDCDescriptor {
321321
return CDCDescriptor{iad: i,
322322
cif: c,
323323
header: h,
@@ -352,8 +352,8 @@ func (d CDCDescriptor) Bytes() []byte {
352352
buf.Write(d.callManagement.Bytes())
353353
buf.Write(d.cifin.Bytes())
354354
buf.Write(d.dif.Bytes())
355-
buf.Write(d.in.Bytes())
356355
buf.Write(d.out.Bytes())
356+
buf.Write(d.in.Bytes())
357357
return buf.Bytes()
358358
}
359359

0 commit comments

Comments
 (0)