[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