Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-18 Thread Shawn Guo
On Tue, Jun 12, 2018 at 01:28:42PM -0500, Li Yang wrote:
> Replace license text with corresponding SPDX identifiers and update the
> format of existing SPDX identifiers to follow the new guideline
> Documentation/process/license-rules.rst.
> 
> Note that some of the files mentioned X11 license previously but the
> license text actually matches MIT license.
> 
> Signed-off-by: Li Yang 

Applied, thanks.


Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-18 Thread Shawn Guo
On Tue, Jun 12, 2018 at 01:28:42PM -0500, Li Yang wrote:
> Replace license text with corresponding SPDX identifiers and update the
> format of existing SPDX identifiers to follow the new guideline
> Documentation/process/license-rules.rst.
> 
> Note that some of the files mentioned X11 license previously but the
> license text actually matches MIT license.
> 
> Signed-off-by: Li Yang 

Applied, thanks.


Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-12 Thread Li Yang
On Tue, Jun 12, 2018 at 1:32 PM, Fabio Estevam  wrote:
> Hi Li,
>
> On Tue, Jun 12, 2018 at 3:28 PM, Li Yang  wrote:
>
>> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts 
>> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> index 7286b1e..f90c040 100644
>> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> @@ -1,45 +1,9 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>
> Here you use GPL-2.0+
>
>>  /*
>>   * Device Tree file for Freescale LS1012A Freedom Board.
>>   *
>>   * Copyright 2016 Freescale Semiconductor, Inc.
>>   *
>> - * This file is dual-licensed: you can use it either under the terms
>> - * of the GPLv2 or the X11 license, at your option. Note that this dual
>
> ,but the original text says  GPL-2.0, not  GPL-2.0+, so there is a mismatch.

The original text has a conflict probably because some people didn't
regard GPLv2+ as a license name.  But the following detailed
explanation mentioned "you can redistribute it and/or modify it under
the terms of the GNU General Public License as published by the Free
Software Foundation;either version 2 of the License, or (at your
option) any later version."  I think we should honor the more verbosed
text than the briefing(same way for X11 vs MIT).

Regards,
Leo


Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-12 Thread Li Yang
On Tue, Jun 12, 2018 at 1:32 PM, Fabio Estevam  wrote:
> Hi Li,
>
> On Tue, Jun 12, 2018 at 3:28 PM, Li Yang  wrote:
>
>> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts 
>> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> index 7286b1e..f90c040 100644
>> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
>> @@ -1,45 +1,9 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>
> Here you use GPL-2.0+
>
>>  /*
>>   * Device Tree file for Freescale LS1012A Freedom Board.
>>   *
>>   * Copyright 2016 Freescale Semiconductor, Inc.
>>   *
>> - * This file is dual-licensed: you can use it either under the terms
>> - * of the GPLv2 or the X11 license, at your option. Note that this dual
>
> ,but the original text says  GPL-2.0, not  GPL-2.0+, so there is a mismatch.

The original text has a conflict probably because some people didn't
regard GPLv2+ as a license name.  But the following detailed
explanation mentioned "you can redistribute it and/or modify it under
the terms of the GNU General Public License as published by the Free
Software Foundation;either version 2 of the License, or (at your
option) any later version."  I think we should honor the more verbosed
text than the briefing(same way for X11 vs MIT).

Regards,
Leo


Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-12 Thread Fabio Estevam
Hi Li,

On Tue, Jun 12, 2018 at 3:28 PM, Li Yang  wrote:

> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts 
> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> index 7286b1e..f90c040 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> @@ -1,45 +1,9 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)

Here you use GPL-2.0+

>  /*
>   * Device Tree file for Freescale LS1012A Freedom Board.
>   *
>   * Copyright 2016 Freescale Semiconductor, Inc.
>   *
> - * This file is dual-licensed: you can use it either under the terms
> - * of the GPLv2 or the X11 license, at your option. Note that this dual

,but the original text says  GPL-2.0, not  GPL-2.0+, so there is a mismatch.


Re: [PATCH] arm64: dts: freescale: Update to use SPDX identifiers

2018-06-12 Thread Fabio Estevam
Hi Li,

On Tue, Jun 12, 2018 at 3:28 PM, Li Yang  wrote:

> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts 
> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> index 7286b1e..f90c040 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> @@ -1,45 +1,9 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)

Here you use GPL-2.0+

>  /*
>   * Device Tree file for Freescale LS1012A Freedom Board.
>   *
>   * Copyright 2016 Freescale Semiconductor, Inc.
>   *
> - * This file is dual-licensed: you can use it either under the terms
> - * of the GPLv2 or the X11 license, at your option. Note that this dual

,but the original text says  GPL-2.0, not  GPL-2.0+, so there is a mismatch.