> On Jan 14, 2020, at 15:51, Prashant Malani <pmalani@xxxxxxxxxxxx> wrote: > > On Mon, Jan 13, 2020 at 8:41 PM Kai-Heng Feng > <kai.heng.feng@xxxxxxxxxxxxx> wrote: >> >> Device 0xa387 also supports MAC passthrough, therefore add it to the >> whitelst. >> >> BugLink: https://bugs.launchpad.net/bugs/1827961/comments/30 >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@xxxxxxxxxxxxx> >> --- >> drivers/net/usb/r8152.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c >> index c5ebf35d2488..42dcf1442cc0 100644 >> --- a/drivers/net/usb/r8152.c >> +++ b/drivers/net/usb/r8152.c >> @@ -6657,7 +6657,8 @@ static int rtl8152_probe(struct usb_interface *intf, >> } >> >> if (le16_to_cpu(udev->descriptor.idVendor) == VENDOR_ID_LENOVO && >> - le16_to_cpu(udev->descriptor.idProduct) == 0x3082) >> + (le16_to_cpu(udev->descriptor.idProduct) == 0x3082 || >> + le16_to_cpu(udev->descriptor.idProduct) == 0xa387)) > Perhaps we can try to use #define's for these vendor IDs (like > https://github.com/torvalds/linux/blob/master/drivers/net/usb/r8152.c#L680) > ? We can, but it'll bring some inconsistencies inside of rtl8152_table[], since we don't know idProduct for other devices. Kai-Heng > >> set_bit(LENOVO_MACPASSTHRU, &tp->flags); >> >> if (le16_to_cpu(udev->descriptor.bcdDevice) == 0x3011 && udev->serial && >> -- >> 2.17.1 >>
