Re: [PATCH net-next v2] net/usb/r8153_ecm: support ECM mode for RTL8153
From: Jakub Kicinski <kuba@kernel.org>
Date: 2020-10-31 23:08:43
Also in:
linux-usb, lkml
On Fri, 30 Oct 2020 11:23:08 +0800 Hayes Wang wrote:
Support ECM mode based on cdc_ether with relative mii functions, when CONFIG_USB_RTL8152 is not set, or the device is not supported by r8152 driver. Signed-off-by: Hayes Wang <redacted>
Can you describe the use case in more detail? AFAICT r8152 defines a match for the exact same device. Does it not mean that which driver is used will be somewhat random if both are built?
+/* 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_LINKSYS 0x13b1 +#define VENDOR_ID_NVIDIA 0x0955 +#define VENDOR_ID_TPLINK 0x2357
$ git grep 0x2357 | grep -i tplink drivers/net/usb/cdc_ether.c:#define TPLINK_VENDOR_ID 0x2357 drivers/net/usb/r8152.c:#define VENDOR_ID_TPLINK 0x2357 drivers/usb/serial/option.c:#define TPLINK_VENDOR_ID 0x2357 $ git grep 0x17ef | grep -i lenovo drivers/hid/hid-ids.h:#define USB_VENDOR_ID_LENOVO 0x17ef drivers/hid/wacom.h:#define USB_VENDOR_ID_LENOVO 0x17ef drivers/net/usb/cdc_ether.c:#define LENOVO_VENDOR_ID 0x17ef drivers/net/usb/r8152.c:#define VENDOR_ID_LENOVO 0x17ef Time to consolidate those vendor id defines perhaps?