2023-01-22 21:37:02

by SeongJae Park

[permalink] [raw]
Subject: [PATCH v2 0/8] Docs: Add missing SPDX license identifiers of

Some subsystem documents are missing SPDX license identifiers on index
files. This patchset adds those.

Changes from v1
(https://lore.kernel.org/lkml/[email protected]/)
- Separate from index file content changes
- Separate patch for each subsystem doc (Alex Deucher)
- Use MIT license for gpu (Alex Deucher)

SeongJae Park (8):
Docs/crypto/index: Add missing SPDX License Identifier
Docs/driver-api/index: Add missing SPDX License Identifier
Docs/gpu/index: Add missing SPDX License Identifier
Docs/hwmon/index: Add missing SPDX License Identifier
Docs/input/index: Add missing SPDX License Identifier
Docs/mm/index: Add missing SPDX License Identifier
Docs/scheduler/index: Add missing SPDX License Identifier
Docs/sound/index: Add missing SPDX License Identifier

Documentation/crypto/index.rst | 2 ++
Documentation/driver-api/index.rst | 2 ++
Documentation/gpu/index.rst | 2 ++
Documentation/hwmon/index.rst | 2 ++
Documentation/input/index.rst | 2 ++
Documentation/mm/index.rst | 2 ++
Documentation/scheduler/index.rst | 2 ++
Documentation/sound/index.rst | 2 ++
8 files changed, 16 insertions(+)

--
2.25.1



2023-01-22 21:37:03

by SeongJae Park

[permalink] [raw]
Subject: [PATCH v2 1/8] Docs/crypto/index: Add missing SPDX License Identifier

Add missing SPDX License Identifier for crypto documentation index file.

Signed-off-by: SeongJae Park <[email protected]>
---
Documentation/crypto/index.rst | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/crypto/index.rst b/Documentation/crypto/index.rst
index da5d5ad2bdf3..95b0870e09b8 100644
--- a/Documentation/crypto/index.rst
+++ b/Documentation/crypto/index.rst
@@ -1,3 +1,5 @@
+.. SPDX-License-Identifier: GPL-2.0
+
==========
Crypto API
==========
--
2.25.1


2023-01-23 15:02:41

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH v2 0/8] Docs: Add missing SPDX license identifiers of

SeongJae Park <[email protected]> writes:

> Some subsystem documents are missing SPDX license identifiers on index
> files. This patchset adds those.

Documentation/ is far behind on the SPDX front, and I'm all in favor of
fixing that. I do worry, though, about sticking license declarations
onto files that did not previously have them; do we know what the
original contributor's intent was?

> Changes from v1
> (https://lore.kernel.org/lkml/[email protected]/)
> - Separate from index file content changes
> - Separate patch for each subsystem doc (Alex Deucher)
> - Use MIT license for gpu (Alex Deucher)
>
> SeongJae Park (8):
> Docs/crypto/index: Add missing SPDX License Identifier
> Docs/driver-api/index: Add missing SPDX License Identifier
> Docs/gpu/index: Add missing SPDX License Identifier
> Docs/hwmon/index: Add missing SPDX License Identifier
> Docs/input/index: Add missing SPDX License Identifier
> Docs/mm/index: Add missing SPDX License Identifier
> Docs/scheduler/index: Add missing SPDX License Identifier
> Docs/sound/index: Add missing SPDX License Identifier
>
> Documentation/crypto/index.rst | 2 ++
> Documentation/driver-api/index.rst | 2 ++

I added this one, and GPLv2 is fine there.

> Documentation/gpu/index.rst | 2 ++
> Documentation/hwmon/index.rst | 2 ++
> Documentation/input/index.rst | 2 ++
> Documentation/mm/index.rst | 2 ++
> Documentation/scheduler/index.rst | 2 ++
> Documentation/sound/index.rst | 2 ++
> 8 files changed, 16 insertions(+)

Thanks,

jon

2023-01-24 22:31:22

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH v2 0/8] Docs: Add missing SPDX license identifiers of

SeongJae Park <[email protected]> writes:

> Some subsystem documents are missing SPDX license identifiers on index
> files. This patchset adds those.
>
> Changes from v1
> (https://lore.kernel.org/lkml/[email protected]/)
> - Separate from index file content changes
> - Separate patch for each subsystem doc (Alex Deucher)
> - Use MIT license for gpu (Alex Deucher)
>
> SeongJae Park (8):
> Docs/crypto/index: Add missing SPDX License Identifier
> Docs/driver-api/index: Add missing SPDX License Identifier
> Docs/gpu/index: Add missing SPDX License Identifier
> Docs/hwmon/index: Add missing SPDX License Identifier
> Docs/input/index: Add missing SPDX License Identifier
> Docs/mm/index: Add missing SPDX License Identifier
> Docs/scheduler/index: Add missing SPDX License Identifier
> Docs/sound/index: Add missing SPDX License Identifier

So I've applied patches 2 (since I wrote the initial file) and 8 (with
Takashi's ack). The others are also fine, I think, but I hesitate to
apply license texts to files without knowing that they match the
author's intent. I hate to say it, but I think the best approach is to
send each of the remaining patches to the appropriate maintainers for
the subsystem involved.

Thanks,

jon

2023-01-24 23:03:51

by SeongJae Park

[permalink] [raw]
Subject: Re: [PATCH v2 0/8] Docs: Add missing SPDX license identifiers of

On Tue, 24 Jan 2023 15:31:04 -0700 Jonathan Corbet <[email protected]> wrote:

> SeongJae Park <[email protected]> writes:
>
> > Some subsystem documents are missing SPDX license identifiers on index
> > files. This patchset adds those.
> >
> > Changes from v1
> > (https://lore.kernel.org/lkml/[email protected]/)
> > - Separate from index file content changes
> > - Separate patch for each subsystem doc (Alex Deucher)
> > - Use MIT license for gpu (Alex Deucher)
> >
> > SeongJae Park (8):
> > Docs/crypto/index: Add missing SPDX License Identifier
> > Docs/driver-api/index: Add missing SPDX License Identifier
> > Docs/gpu/index: Add missing SPDX License Identifier
> > Docs/hwmon/index: Add missing SPDX License Identifier
> > Docs/input/index: Add missing SPDX License Identifier
> > Docs/mm/index: Add missing SPDX License Identifier
> > Docs/scheduler/index: Add missing SPDX License Identifier
> > Docs/sound/index: Add missing SPDX License Identifier
>
> So I've applied patches 2 (since I wrote the initial file) and 8 (with
> Takashi's ack). The others are also fine, I think, but I hesitate to
> apply license texts to files without knowing that they match the
> author's intent. I hate to say it, but I think the best approach is to
> send each of the remaining patches to the appropriate maintainers for
> the subsystem involved.

That makes 100% sense, thank you, Jon!


Thanks,
SJ

>
> Thanks,
>
> jon