[U-Boot] [PATCH 13/60] ARM: tegra: sort some board file include directives

Tom Rini trini at konsulko.com
Tue Apr 26 20:15:29 CEST 2016


On Tue, Apr 26, 2016 at 10:23:36AM -0600, Stephen Warren wrote:
> On 04/25/2016 05:22 PM, Tom Rini wrote:
> 
> >Lawyers can argue, but projects have guidelines.  I mean heck, I've see
> >you remind people to fix the include order in new patches.  Do they need
> >to add an NVIDIA copyright notice too?  No, of course not.
> 
> Well honestly all patch review does contribute to the creative
> process and affect the final result, so I don't actually think it'd
> be that much of a stretch that a reviewer's copyright applies to the
> final patch; they were involved in the creation of it. Perhaps
> Reviewed-by tags or the mailing list history would be enough
> evidence of this though, if such were needed later.

And this is why I love patchwork and hope that as it starts to handle
series more directly it will also handle "For the series, ..." messages
in addition to collecting up Acked/Reviewed/Tested by lines.  These are
important for history of changes to the codebase.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160426/c9b450fc/attachment.sig>


More information about the U-Boot mailing list