From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: Bitan Biswas <bbiswas@nvidia.com>,
Rob Herring <robh+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Steven Rostedt <rostedt@goodmis.org>,
Ingo Molnar <mingo@redhat.com>,
Jassi Brar <jassisinghbrar@gmail.com>,
Philipp Zabel <p.zabel@pengutronix.de>
Cc: Sowjanya Komatineni <skomatineni@nvidia.com>,
Mark Brown <broonie@kernel.org>,
Linus Walleij <linus.walleij@linaro.org>,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-tegra@vger.kernel.org
Subject: Re: [PATCH V1 3/3] dt-bindings: tegra: clock,memory,thermal: add header Copyright
Date: Tue, 21 Sep 2021 11:47:24 +0200 [thread overview]
Message-ID: <de4ad32b-ab60-a4dd-793f-9bd520ed5a87@canonical.com> (raw)
In-Reply-To: <20210921094206.2632-4-bbiswas@nvidia.com>
On 21/09/2021 11:42, Bitan Biswas wrote:
> Add Copyright for below Tegra dt-bindings headers:
> 1. clock
> 2. gpio
> 3. mailbox
> 4. memory
> 5. thermal
> 6. reset
> 7. pinctrl
Why? Please explain.
These are trivial constants, so I doubt there is here any copyright value.
>
> Signed-off-by: Bitan Biswas <bbiswas@nvidia.com>
> ---
> include/dt-bindings/clock/tegra114-car.h | 4 ++++
> include/dt-bindings/clock/tegra124-car-common.h | 4 ++++
> include/dt-bindings/clock/tegra124-car.h | 4 ++++
> include/dt-bindings/clock/tegra186-clock.h | 6 ++++++
> include/dt-bindings/clock/tegra20-car.h | 4 ++++
> include/dt-bindings/clock/tegra210-car.h | 4 ++++
> include/dt-bindings/clock/tegra30-car.h | 4 ++++
> include/dt-bindings/gpio/tegra-gpio.h | 4 ++++
> include/dt-bindings/gpio/tegra186-gpio.h | 4 ++++
> include/dt-bindings/mailbox/tegra186-hsp.h | 4 ++++
> include/dt-bindings/memory/tegra114-mc.h | 6 ++++++
> include/dt-bindings/memory/tegra124-mc.h | 6 ++++++
> include/dt-bindings/memory/tegra186-mc.h | 6 ++++++
> include/dt-bindings/memory/tegra194-mc.h | 6 ++++++
> include/dt-bindings/memory/tegra210-mc.h | 6 ++++++
> include/dt-bindings/memory/tegra30-mc.h | 6 ++++++
> include/dt-bindings/pinctrl/pinctrl-tegra-xusb.h | 6 ++++++
> include/dt-bindings/reset/tegra124-car.h | 4 ++++
> include/dt-bindings/reset/tegra210-car.h | 4 ++++
> include/dt-bindings/thermal/tegra124-soctherm.h | 4 ++++
> include/dt-bindings/thermal/tegra186-bpmp-thermal.h | 4 ++++
> include/dt-bindings/thermal/tegra194-bpmp-thermal.h | 4 ++++
> 22 files changed, 104 insertions(+)
>
(...)
> @@ -1,5 +1,9 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> /*
> + * Copyright (c) 2016-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Joseph Lo <josephl@nvidia.com>
> + *
> * This header provides constants for binding nvidia,tegra186-hsp.
> */
>
> diff --git a/include/dt-bindings/memory/tegra114-mc.h b/include/dt-bindings/memory/tegra114-mc.h
> index dfe99c8a5ba5..1a2d16853847 100644
> --- a/include/dt-bindings/memory/tegra114-mc.h
> +++ b/include/dt-bindings/memory/tegra114-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2014-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
> + */
> #ifndef DT_BINDINGS_MEMORY_TEGRA114_MC_H
> #define DT_BINDINGS_MEMORY_TEGRA114_MC_H
>
> diff --git a/include/dt-bindings/memory/tegra124-mc.h b/include/dt-bindings/memory/tegra124-mc.h
> index 7e73bb400eca..8a7abf2325b6 100644
> --- a/include/dt-bindings/memory/tegra124-mc.h
> +++ b/include/dt-bindings/memory/tegra124-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2014-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
> + */
> #ifndef DT_BINDINGS_MEMORY_TEGRA124_MC_H
> #define DT_BINDINGS_MEMORY_TEGRA124_MC_H
>
> diff --git a/include/dt-bindings/memory/tegra186-mc.h b/include/dt-bindings/memory/tegra186-mc.h
> index be313d3790ae..283ea29bf426 100644
> --- a/include/dt-bindings/memory/tegra186-mc.h
> +++ b/include/dt-bindings/memory/tegra186-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Copyright (c) 2017-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
> + */
> #ifndef DT_BINDINGS_MEMORY_TEGRA186_MC_H
> #define DT_BINDINGS_MEMORY_TEGRA186_MC_H
>
> diff --git a/include/dt-bindings/memory/tegra194-mc.h b/include/dt-bindings/memory/tegra194-mc.h
> index 16bb62bf8166..30182fd94f0f 100644
> --- a/include/dt-bindings/memory/tegra194-mc.h
> +++ b/include/dt-bindings/memory/tegra194-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Copyright (c) 2020-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
> + */
> #ifndef DT_BINDINGS_MEMORY_TEGRA194_MC_H
> #define DT_BINDINGS_MEMORY_TEGRA194_MC_H
>
> diff --git a/include/dt-bindings/memory/tegra210-mc.h b/include/dt-bindings/memory/tegra210-mc.h
> index 5e082547f179..c4e3ba88f77b 100644
> --- a/include/dt-bindings/memory/tegra210-mc.h
> +++ b/include/dt-bindings/memory/tegra210-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2015-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
> + */
> #ifndef DT_BINDINGS_MEMORY_TEGRA210_MC_H
> #define DT_BINDINGS_MEMORY_TEGRA210_MC_H
>
> diff --git a/include/dt-bindings/memory/tegra30-mc.h b/include/dt-bindings/memory/tegra30-mc.h
> index 930f708aca17..4972cf7e3cf2 100644
> --- a/include/dt-bindings/memory/tegra30-mc.h
> +++ b/include/dt-bindings/memory/tegra30-mc.h
> @@ -1,4 +1,10 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2014-2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> + *
> + * Author: Thierry Reding <treding@nvidia.com>
> + *
I am sorry, but that's not entirely how it works. You cannot add only
selected (stripped) copyrights backwards. If adding, please include all
copyright holders.
Best regards,
Krzysztof
next prev parent reply other threads:[~2021-09-21 9:47 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-21 9:42 [PATCH V1 0/3] Add tegra header license and dt-bindings Copyright Bitan Biswas
2021-09-21 9:42 ` [PATCH V1 1/3] dt-bindings: tegra: memory,bpmp-thermal: add SPDX license Bitan Biswas
2021-09-21 9:49 ` Krzysztof Kozlowski
2021-09-24 21:58 ` Bitan Biswas
2021-09-21 9:42 ` [PATCH V1 2/3] trace: events: tegra_apb_dma: add SPDX license identifier Bitan Biswas
2021-09-21 9:51 ` Krzysztof Kozlowski
2021-09-24 22:18 ` Bitan Biswas
2021-09-21 9:42 ` [PATCH V1 3/3] dt-bindings: tegra: clock,memory,thermal: add header Copyright Bitan Biswas
2021-09-21 9:47 ` Krzysztof Kozlowski [this message]
2021-09-24 21:43 ` Bitan Biswas
2021-09-21 11:27 [PATCH V1 0/3] Add tegra header license and dt-bindings Copyright Bitan Biswas
2021-09-21 11:27 ` [PATCH V1 3/3] dt-bindings: tegra: clock,memory,thermal: add header Copyright Bitan Biswas
2021-09-21 17:44 [PATCH V1 0/3] Add tegra header license and dt-bindings Copyright Bitan Biswas
2021-09-21 17:44 ` [PATCH V1 3/3] dt-bindings: tegra: clock,memory,thermal: add header Copyright Bitan Biswas
2021-09-24 7:42 ` Krzysztof Kozlowski
2021-09-24 21:51 ` Bitan Biswas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=de4ad32b-ab60-a4dd-793f-9bd520ed5a87@canonical.com \
--to=krzysztof.kozlowski@canonical.com \
--cc=bbiswas@nvidia.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jassisinghbrar@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=rostedt@goodmis.org \
--cc=skomatineni@nvidia.com \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®