> From: John Holland [mailto:jotih...@gmail.com] > Sent: Monday, February 29, 2016 10:56 PM > To: Brown, Aaron F <aaron.f.br...@intel.com> > Cc: intel-wired-...@lists.osuosl.org; netdev@vger.kernel.org > Subject: Re: [Intel-wired-lan] [next] igb: allow setting MAC address on i211 > using a device tree blob V5 > > On Mar 1, 2016, at 03:52, Brown, Aaron F <aaron.f.br...@intel.com> > wrote: > > > This throws a few checkpatch warnings, but I won't withhold my tested by > for these: > > > > total: 0 errors, 2 warnings, 0 checks, 21 lines checked > > > > Your patch has style problems, please review. > > > > NOTE: If any of the errors are false positives, please report > > them to the maintainer, see CHECKPATCH in MAINTAINERS. > > u1463:[0]/usr/src/kernels/next-queue> > > Thanks for testing... > > Do you require me to reformat the patch text? And won't that break the link?
Unless Jeff Kirsher wants you to resubmit it for the space between your name and email address I don't think resubmitting should be necessary. Yes, breaking the URL up would break the URL and for that reason I think it is appropriate to ignore that (Possible unwrapped commit description) warning. Thanks, Aaron > > John