Message ID | 20160114.184626.1327276960383833164.rene@exactcode.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Thu, Jan 14, 2016 at 06:46:26PM +0100, Rene Rebe wrote: > Hi David, > (my appologies for wasting your time with this noise) > > rebased against your netdev git, and sent with Mew: Hi Rene So, slowly getting better. However, go read Documentation/SubmittingPatches Read the bit about the "---" marker. netdev also has a few additional rules, which are not documented in SubmittingPatches. In the subject line, you should include which tree this patch is supposed to be applied to. In this case, it is net-next. So you subject line should be Subject: [PATCH net-next] r8152 add Microsoft Dock IDs and obviously, your patch needs to cleanly apply to net-next, not some other random tree. The other rule which is relevant at the moment, is that net-next is closed, because the merge window is open. So please wait about 12 days for net-next to reopen. David will probably send an email saying he is ready to accept new patches. Andrew
--- linux-git/drivers/net/usb/r8152.c.orig 2016-01-14 15:17:54.786662456 +0100 +++ linux-git/drivers/net/usb/r8152.c 2016-01-14 15:17:21.050661823 +0100 @@ -504,6 +504,7 @@ /* Define these values to match your device */ #define VENDOR_ID_REALTEK 0x0bda +#define VENDOR_ID_MICROSOFT 0x045e #define VENDOR_ID_SAMSUNG 0x04e8 #define VENDOR_ID_LENOVO 0x17ef #define VENDOR_ID_NVIDIA 0x0955 @@ -4294,6 +4295,8 @@ static 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, 0x7205)}, {REALTEK_USB_DEVICE(VENDOR_ID_LENOVO, 0x304f)},
Hi David, (my appologies for wasting your time with this noise) rebased against your netdev git, and sent with Mew: The Microsoft Surface Docks uses a R8152, too. Signed-off-by: René Rebe <rene@exactcode.com>