[OpenWrt-Devel] [PATCH] adb: fix for SuperSpeed devices

Bjørn Mork bjorn at mork.no
Tue Nov 19 08:37:24 EST 2019


The USB descriptor parsing in adb fails to detect SuperSpeed devices
because of the SuperSpeed Endpoint Companion Descriptor.  This
cherry-picks the upstream fix for the problem.

Unfortunately there never were a release with this fix before the
conversion to C++, so upgrading to a newer version isn't an option.

This makes adb work with SuperSpeed devices like the Sierra Wireless
EM7565.  Tested and verified.

Signed-off-by: Bjørn Mork <bjorn at mork.no>
---
 package/utils/adb/Makefile                    |  2 +-
 .../020-cherry-picked-superspeed-fix.patch    | 44 +++++++++++++++++++
 2 files changed, 45 insertions(+), 1 deletion(-)
 create mode 100644 package/utils/adb/patches/020-cherry-picked-superspeed-fix.patch

diff --git a/package/utils/adb/Makefile b/package/utils/adb/Makefile
index 7be8ee9a1cee..0482a40da15b 100644
--- a/package/utils/adb/Makefile
+++ b/package/utils/adb/Makefile
@@ -3,7 +3,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=adb
 PKG_VERSION:=android.5.0.2_r1
-PKG_RELEASE:=2
+PKG_RELEASE:=3
 
 PKG_SOURCE_PROTO:=git
 PKG_SOURCE_URL:=https://android.googlesource.com/platform/system/core
diff --git a/package/utils/adb/patches/020-cherry-picked-superspeed-fix.patch b/package/utils/adb/patches/020-cherry-picked-superspeed-fix.patch
new file mode 100644
index 000000000000..5192dfb93706
--- /dev/null
+++ b/package/utils/adb/patches/020-cherry-picked-superspeed-fix.patch
@@ -0,0 +1,44 @@
+From 58b01e01875e2f6ae593ded197430bc23713dd0a Mon Sep 17 00:00:00 2001
+From: Ingo Rohloff <lundril at gmx.de>
+Date: Fri, 16 May 2014 21:51:41 +0200
+Subject: [PATCH] ADB on linux: Handle USB SuperSpeed extra Descriptors
+
+Under Linux, ADB manually parses USB Descriptors to check for
+possible ADB USB Interfaces. USB Devices connected with SuperSpeed
+will exhibit extra USB SuperSpeed Endpoint Companion Descriptors.
+This patch handles these USB SuperSpeed specific USB Descriptors.
+
+Change-Id: Icd1e5fdde0b324c7df4f933583499f2c52a922f3
+Signed-off-by: Ingo Rohloff <lundril at gmx.de>
+---
+ adb/usb_linux.c | 12 ++++++++++++
+ 1 file changed, 12 insertions(+)
+
+diff --git a/adb/usb_linux.c b/adb/usb_linux.c
+index 8ff753e43383..7973cb06c7e9 100644
+--- a/adb/usb_linux.c
++++ b/adb/usb_linux.c
+@@ -237,8 +237,20 @@ static void find_usb_device(const char *base,
+                             // looks like ADB...
+                         ep1 = (struct usb_endpoint_descriptor *)bufptr;
+                         bufptr += USB_DT_ENDPOINT_SIZE;
++                            // For USB 3.0 SuperSpeed devices, skip potential
++                            // USB 3.0 SuperSpeed Endpoint Companion descriptor
++                        if (bufptr+2 <= devdesc + desclength &&
++                            bufptr[0] == USB_DT_SS_EP_COMP_SIZE &&
++                            bufptr[1] == USB_DT_SS_ENDPOINT_COMP) {
++                            bufptr += USB_DT_SS_EP_COMP_SIZE;
++                        }
+                         ep2 = (struct usb_endpoint_descriptor *)bufptr;
+                         bufptr += USB_DT_ENDPOINT_SIZE;
++                        if (bufptr+2 <= devdesc + desclength &&
++                            bufptr[0] == USB_DT_SS_EP_COMP_SIZE &&
++                            bufptr[1] == USB_DT_SS_ENDPOINT_COMP) {
++                            bufptr += USB_DT_SS_EP_COMP_SIZE;
++                        }
+ 
+                         if (bufptr > devdesc + desclength ||
+                             ep1->bLength != USB_DT_ENDPOINT_SIZE ||
+-- 
+2.20.1
+
-- 
2.20.1


_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list