netdev - Re: [PATCH] Make Surface Dock 3 work again

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <75b12b2a-ac98-20fb-1d93-c5ec105a0071@uib.de>
Date:   Wed, 25 Jul 2018 10:39:20 +0200
From:   Mathias Radtke <m.radtke@....de>
To:     David Miller <davem@...emloft.net>
Cc:     oliver@...kum.org, hayeswang@...ltek.com, netdev@...r.kernel.org
Subject: Re: [PATCH] Make Surface Dock 3 work again

Hi,
thanks for the feedback on my submission.
Here's a corrected version of the submission

> since Kernel 4.10 the usage of the Surface Dock 3 is broken.
> The network driver r8152 reports Tx Error -71.
> The patch removes the dock from the r8152 driver and also removed it 
> from the cdc_ether drivers blacklist. Therefore the cdc_driver takes 
> over and the dock works again.

 From 8170ebce17a5c6e590c42123495031e9fb83836c Mon Sep 17 00:00:00 2001
From: Mathias Radtke <m.radtke@....de>
Date: Wed, 25 Jul 2018 10:20:02 +0200
Subject: [PATCH] drivers/net/usb/cdc_ether: removed Surface Dock 3 from
  blacklisted devices drivers/net/usb/r8152: removed Surface Dock 3 from
  supported devices

Signed-off-by: Mathias Radtke <m.radtke@....de>
---
  drivers/net/usb/cdc_ether.c | 7 -------
  drivers/net/usb/r8152.c     | 1 -
  2 files changed, 8 deletions(-)

diff --git a/drivers/net/usb/cdc_ether.c b/drivers/net/usb/cdc_ether.c
index 5c42cf8..5eb92c3 100644
--- a/drivers/net/usb/cdc_ether.c
+++ b/drivers/net/usb/cdc_ether.c
@@ -807,13 +807,6 @@ static const struct usb_device_id  products[] = {
      .driver_info = 0,
  },

-/* Microsoft Surface 3 dock (based on Realtek RTL8153) */
-{
-    USB_DEVICE_AND_INTERFACE_INFO(MICROSOFT_VENDOR_ID, 0x07c6, 
USB_CLASS_COMM,
-            USB_CDC_SUBCLASS_ETHERNET, USB_CDC_PROTO_NONE),
-    .driver_info = 0,
-},
-
      /* TP-LINK UE300 USB 3.0 Ethernet Adapters (based on Realtek 
RTL8153) */
  {
      USB_DEVICE_AND_INTERFACE_INFO(TPLINK_VENDOR_ID, 0x0601, 
USB_CLASS_COMM,
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
index 2a58607..9542204 100644
--- a/drivers/net/usb/r8152.c
+++ b/drivers/net/usb/r8152.c
@@ -5323,7 +5323,6 @@ static const struct usb_device_id rtl8152_table[] = {
      {REALTEK_USB_DEVICE(VENDOR_ID_REALTEK, 0x8152)},
      {REALTEK_USB_DEVICE(VENDOR_ID_REALTEK, 0x8153)},
      {REALTEK_USB_DEVICE(VENDOR_ID_MICROSOFT, 0x07ab)},
-    {REALTEK_USB_DEVICE(VENDOR_ID_MICROSOFT, 0x07c6)},
      {REALTEK_USB_DEVICE(VENDOR_ID_SAMSUNG, 0xa101)},
      {REALTEK_USB_DEVICE(VENDOR_ID_LENOVO,  0x304f)},
      {REALTEK_USB_DEVICE(VENDOR_ID_LENOVO,  0x3062)},
-- 
2.7.4

Cheers

-- 
Mathias Radtke


---

http://uib.de/de/support-schulung/schulung/

uib gmbh
Bonifaziusplatz 1B
55118 Mainz


E-Mail: m.radtke@....de
Internet: www.uib.de

Geschäftsführer: dr. detlef oertel & erol ueluekmen
Handelsregister: Amtsgericht Mainz HRB 6942