[U-Boot] [PATCH 0/2] SPDX: Convert tags to Linux Kernel style

Tom Rini trini at konsulko.com
Mon May 7 14:07:23 UTC 2018


As per the emails I had sent before on this subject, I've converted our
SPDX tags to Linux Kernel style.  What this means, to quote from the
kernel's Documentation/process/license-rules.rst file is:
2. Style:

   The SPDX license identifier is added in form of a comment.  The comment
   style depends on the file type::

      C source:	// SPDX-License-Identifier: <SPDX License Expression>
      C header:	/* SPDX-License-Identifier: <SPDX License Expression> */
      ASM:	/* SPDX-License-Identifier: <SPDX License Expression> */
      scripts:	# SPDX-License-Identifier: <SPDX License Expression>
      .rst:	.. SPDX-License-Identifier: <SPDX License Expression>
      .dts{i}:	// SPDX-License-Identifier: <SPDX License Expression>

Where, yes, the C source comments are not our usual style.  But I believe
that cross-project consistency is important and will further help us when
we fixup some of our from Linux but un-tagged-currently files.  I'm posting
these patches currently for posterity as I will be pushing them both
shortly (they have build tested the world successfully).  I'm also not
going to approve 1/2 to the ML as it's huge and I frankly expect Google to
tell me I can't send it.  The commit log is:

commit 83d290c56fab2d38cd1ab4c4cc7099559c1d5046
Author: Tom Rini <trini at konsulko.com>
Date:   Sun May 6 17:58:06 2018 -0400

    SPDX: Convert all of our single license tags to Linux Kernel style
    
    When U-Boot started using SPDX tags we were among the early adopters and
    there weren't a lot of other examples to borrow from.  So we picked the
    area of the file that usually had a full license text and replaced it
    with an appropriate SPDX-License-Identifier: entry.  Since then, the
    Linux Kernel has adopted SPDX tags and they place it as the very first
    line in a file (except where shebangs are used, then it's second line)
    and with slightly different comment styles than us.
    
    In part due to community overlap, in part due to better tag visibility
    and in part for other minor reasons, switch over to that style.
    
    This commit changes all instances where we have a single declared
    license in the tag as both the before and after are identical in tag
    contents.  There's also a few places where I found we did not have a tag
    and have introduced one.
    
    Signed-off-by: Tom Rini <trini at konsulko.com>

And I think the above summarizes things well.  I am posting 2/2 for
posterity as it contains no functional changes but to repeat what the
commit log says, we go from "LICENSE-A LICENSE-B" to "LICENSE-A OR
LICENSE-B" which allows for useful expressions to be made.

-- 
Tom


More information about the U-Boot mailing list