[U-Boot] [PATCH 01/60] ARM: tegra: remove unused definitions in headers

Stephen Warren
2016-04-25T19:34:13Z

On 04/24/2016 04:20 AM, Wolfgang Denk wrote:
> Dear Stephen,
>
> In message <1461099580-3866-2-git-send-email-swarren@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.