On Tuesday, December 01, 2015 at 05:31:30 PM, Stephen Warren wrote: > On 12/01/2015 08:10 AM, Marek Vasut wrote: > > On Tuesday, December 01, 2015 at 12:24:41 PM, Ted Chen wrote: > >> This patch adds driver support for the Realtek RTL8152B/RTL8153 USB > >> network adapters. > > > > Hi! > > > > looks pretty good, just a few final nitpicks below. > > > >> Signed-off-by: Ted Chen <tedchen at realtek.com> > >> [swarren, fixed a few compiler warnings] > >> [swarren, with permission, converted license header to SPDX] > >> [swarren, removed printf() spew during probe()] > > > > This changelog should be removed, really. It doesn't have to be in the > > commit message, right ? > > In general it's good practice when people edit someone else's patch to > note that in the patch description so that it's obvious who to blame for > what. > > However in this particular case an exception could likely be made since > the original patch author has taken over the patch again and made > significantly more changes than listed above.
OK, whichever way you prefer then. Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot