laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified tnt: Looks good to me, but someone else must approve laforge: Looks good to me, approved
fw/e1-tracer: Fix bNumInterfaces in new configuration

This fixes the following Linux kernel error message on the host:
usb 1-1: config 2 has 2 interfaces, different from the descriptor's value: 3

Change-Id: Ideb816169a1e995907901c018e7bd2f963c1a831
---
M firmware/ice40-riscv/e1-tracer/usb_desc_app.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/firmware/ice40-riscv/e1-tracer/usb_desc_app.c b/firmware/ice40-riscv/e1-tracer/usb_desc_app.c
index e9aa9d6..a87025c 100644
--- a/firmware/ice40-riscv/e1-tracer/usb_desc_app.c
+++ b/firmware/ice40-riscv/e1-tracer/usb_desc_app.c
@@ -156,7 +156,7 @@
.bLength = sizeof(struct usb_conf_desc),
.bDescriptorType = USB_DT_CONF,
.wTotalLength = sizeof(_app_conf_desc_e1d),
- .bNumInterfaces = 3,
+ .bNumInterfaces = 2,
.bConfigurationValue = 2,
.iConfiguration = 7,
.bmAttributes = 0x80,

To view, visit change 29971. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: Ideb816169a1e995907901c018e7bd2f963c1a831
Gerrit-Change-Number: 29971
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: tnt <tnt@246tNt.com>
Gerrit-MessageType: merged