On Mon, Jun 06, 2016 at 02:43:37PM +0000, Mario_Limonciello@xxxxxxxx wrote: > > -----Original Message----- > > From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx] > > Sent: Monday, June 6, 2016 9:40 AM > > To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx> > > Cc: hayeswang@xxxxxxxxxxx; LKML <linux-kernel@xxxxxxxxxxxxxxx>; Netdev > > <netdev@xxxxxxxxxxxxxxx>; Linux USB <linux-usb@xxxxxxxxxxxxxxx>; > > pali.rohar@xxxxxxxxx; anthony.wong@xxxxxxxxxxxxx > > Subject: Re: [PATCH v3] r8152: Add support for setting pass through MAC > > address on RTL8153-AD > > > > On Mon, Jun 06, 2016 at 09:15:20AM -0500, Mario Limonciello wrote: > > > Since this is a Realtek feature, I feel this shouldn't be moved into a platform > > > MAC address lookup. The code should only be run when the correct > > Realtek device > > > is plugged in. > > > > > > Changes from v2: > > > * Only apply to RTL8153-AD w/ eFuse pass through mac address pass thru > > > bit set. > > > * Drop matching DMI information on Dell. Although this is implemented > > on > > > Dell, this is a Realtek feature that may may be implemented on other > > > OEMs as well. > > > * Test that pass through MAC address is valid, fall back to HW address if > > > invalid. > > > * Don't track status of which device has MAC pass through activated. > > > - Expected experience is that if two docks (RTL8153-AD's w/ mac pass thru > > > bit set) were plugged in both should have MAC pass through activated. > > > > cover letters for single-patch submissions is overkill and confusing, > > please don't. > > I was trying to convey differences between versions of this patch, I'll avoid > that in the future and let the audience find them themselves. No, put them in the patch itself, under the --- line, like is documented to do so. Don't make people "find them themselves", if you do that, your patch will just be ignored. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
