Hi Stephen, On 25 April 2016 at 13:34, Stephen Warren <swar...@wwwdotorg.org> wrote: > > On 04/24/2016 04:20 AM, Wolfgang Denk wrote: >> >> Dear Stephen, >> >> In message <1461099580-3866-2-git-send-email-swar...@wwwdotorg.org> you >> wrote: >>> >>> >> ... >>> >>> /* >>> * Copyright (c) 2011 The Chromium OS Authors. >>> + * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved. >> >> >> Can you please get rid of this "All rights reserved." ? > > > Sorry, that is the format I'm required to use by NVIDIA legal. > > To address your next question, "all rights reserved" is 100% compatible with > OSS licenses. Equally, the phrase already exists throughout both the U-Boot > and Linux kernel code-base (just a couple of prominent examples) for both > NVIDIA's copyright notices, and those of many other prominent entities such > as The Linux Foundation, Red Hat, Intel, The Chromium OS Authors, etc. > >> Please fix this globally in the whole patch set. >> >> I assume it cannot be a problem to do that, as in other places you did >> not add this either, for example here: >> >>> --- a/arch/arm/include/asm/arch-tegra/gp_padctrl.h >>> +++ b/arch/arm/include/asm/arch-tegra/gp_padctrl.h >>> @@ -1,6 +1,6 @@ >>> /* >>> - * (C) Copyright 2010-2015 >>> - * NVIDIA Corporation <www.nvidia.com> >>> + * (C) Copyright 2010-2016 >>> + * NVIDIA Corporation <www.nvidia.com> > > > The primary purpose of this patch-set isn't to clean up copyright notices. As > such, where I added new copyright notices I followed the format that NVIDIA > legal requests me to. However, where I simply edited the date in existing > notices I didn't do any other cleanup.
Perhaps you could consider just not updating the notices? It isn't required by U-Boot. I tend not to do it. Also the all rights reserved thing is apparently pointless. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot