[PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier

Bence Csókás posted 8 patches 1 month, 3 weeks ago
There is a newer version of this series
[PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Bence Csókás 1 month, 3 weeks ago
Replace verbatim license text with a `SPDX-License-Identifier`.

The comment header mis-attributes this license to be "X11", but the
license text does not include the last line "Except as contained in this
notice, the name of the X Consortium shall not be used in advertising or
otherwise to promote the sale, use or other dealings in this Software
without prior written authorization from the X Consortium.". Therefore,
this license is actually equivalent to the SPDX "MIT" license (confirmed
by text diffing).

Cc: Andrej Rosano <andrej@inversepath.com>
Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>
---
 arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts | 39 +--------------------------
 1 file changed, 1 insertion(+), 38 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
index acc44010d510695b28dc8e6599ba006856a89efb..3ad9db4b144254f5400ce894f99fb038e51f64f4 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
@@ -1,47 +1,10 @@
+// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
 /*
  * USB armory MkI device tree file
  * https://inversepath.com/usbarmory
  *
  * Copyright (C) 2015, Inverse Path
  * Andrej Rosano <andrej@inversepath.com>
- *
- * This file is dual-licensed: you can use it either under the terms
- * of the GPL or the X11 license, at your option. Note that this dual
- * licensing only applies to this file, and not this project as a
- * whole.
- *
- *  a) This file is free software; 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.
- *
- *     This file is distributed in the hope that it will be useful,
- *     but WITHOUT ANY WARRANTY; without even the implied warranty of
- *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *     GNU General Public License for more details.
- *
- * Or, alternatively,
- *
- *  b) Permission is hereby granted, free of charge, to any person
- *     obtaining a copy of this software and associated documentation
- *     files (the "Software"), to deal in the Software without
- *     restriction, including without limitation the rights to use,
- *     copy, modify, merge, publish, distribute, sublicense, and/or
- *     sell copies of the Software, and to permit persons to whom the
- *     Software is furnished to do so, subject to the following
- *     conditions:
- *
- *     The above copyright notice and this permission notice shall be
- *     included in all copies or substantial portions of the Software.
- *
- *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
- *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
- *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
- *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
- *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
- *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- *     OTHER DEALINGS IN THE SOFTWARE.
  */
 
 /dts-v1/;

-- 
2.43.0


Re: [PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Andrej Rosano 3 weeks, 2 days ago
On 2025-08-14 Thu, Bence Csókás wrote:
> Replace verbatim license text with a `SPDX-License-Identifier`.
> 
> The comment header mis-attributes this license to be "X11", but the
> license text does not include the last line "Except as contained in this
> notice, the name of the X Consortium shall not be used in advertising or
> otherwise to promote the sale, use or other dealings in this Software
> without prior written authorization from the X Consortium.". Therefore,
> this license is actually equivalent to the SPDX "MIT" license (confirmed
> by text diffing).
> 
> Cc: Andrej Rosano <andrej@inversepath.com>
> Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>
> ---
>  arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts | 39 +--------------------------
>  1 file changed, 1 insertion(+), 38 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
> index acc44010d510695b28dc8e6599ba006856a89efb..3ad9db4b144254f5400ce894f99fb038e51f64f4 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
> +++ b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
> @@ -1,47 +1,10 @@
> +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
>  /*
>   * USB armory MkI device tree file
>   * https://inversepath.com/usbarmory
>   *
>   * Copyright (C) 2015, Inverse Path
>   * Andrej Rosano <andrej@inversepath.com>
> - *
> - * This file is dual-licensed: you can use it either under the terms
> - * of the GPL or the X11 license, at your option. Note that this dual
> - * licensing only applies to this file, and not this project as a
> - * whole.
> - *
> - *  a) This file is free software; 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.
> - *
> - *     This file is distributed in the hope that it will be useful,
> - *     but WITHOUT ANY WARRANTY; without even the implied warranty of
> - *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - *     GNU General Public License for more details.
> - *
> - * Or, alternatively,
> - *
> - *  b) Permission is hereby granted, free of charge, to any person
> - *     obtaining a copy of this software and associated documentation
> - *     files (the "Software"), to deal in the Software without
> - *     restriction, including without limitation the rights to use,
> - *     copy, modify, merge, publish, distribute, sublicense, and/or
> - *     sell copies of the Software, and to permit persons to whom the
> - *     Software is furnished to do so, subject to the following
> - *     conditions:
> - *
> - *     The above copyright notice and this permission notice shall be
> - *     included in all copies or substantial portions of the Software.
> - *
> - *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> - *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
> - *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> - *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> - *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> - *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> - *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> - *     OTHER DEALINGS IN THE SOFTWARE.
>   */
>  
>  /dts-v1/;
> 
> -- 
> 2.43.0
> 
> 

Acked-by: Andrej Rosano <andrej.rosano@reversec.com>

-- 
Andrej Rosano | Hardware Security | Reversec Foundry
www.reversec.com/foundry
BDE1 62F4 7020 1588 8046 AE02 EA17 8C32 AB56 54CE
Re: [PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Bence Csókás 1 week, 2 days ago
Dear maintainers,

On 2025. 09. 11. 10:11, Andrej Rosano wrote:
> On 2025-08-14 Thu, Bence Csókás wrote:
>> Replace verbatim license text with a `SPDX-License-Identifier`.
>>
>> The comment header mis-attributes this license to be "X11", but the
>> license text does not include the last line "Except as contained in this
>> notice, the name of the X Consortium shall not be used in advertising or
>> otherwise to promote the sale, use or other dealings in this Software
>> without prior written authorization from the X Consortium.". Therefore,
>> this license is actually equivalent to the SPDX "MIT" license (confirmed
>> by text diffing).
>>
>> Cc: Andrej Rosano <andrej@inversepath.com>
>> Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>
>> ---
>>   arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts | 39 +--------------------------
>>   1 file changed, 1 insertion(+), 38 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
>> index acc44010d510695b28dc8e6599ba006856a89efb..3ad9db4b144254f5400ce894f99fb038e51f64f4 100644
>> --- a/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
>> +++ b/arch/arm/boot/dts/nxp/imx/imx53-usbarmory.dts
>> @@ -1,47 +1,10 @@
>> +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
>>   /*
>>    * USB armory MkI device tree file
>>    * https://inversepath.com/usbarmory
>>    *
>>    * Copyright (C) 2015, Inverse Path
>>    * Andrej Rosano <andrej@inversepath.com>
>> - *
>> - * This file is dual-licensed: you can use it either under the terms
>> - * of the GPL or the X11 license, at your option. Note that this dual
>> - * licensing only applies to this file, and not this project as a
>> - * whole.
>> - *
>> - *  a) This file is free software; 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.
>> - *
>> - *     This file is distributed in the hope that it will be useful,
>> - *     but WITHOUT ANY WARRANTY; without even the implied warranty of
>> - *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> - *     GNU General Public License for more details.
>> - *
>> - * Or, alternatively,
>> - *
>> - *  b) Permission is hereby granted, free of charge, to any person
>> - *     obtaining a copy of this software and associated documentation
>> - *     files (the "Software"), to deal in the Software without
>> - *     restriction, including without limitation the rights to use,
>> - *     copy, modify, merge, publish, distribute, sublicense, and/or
>> - *     sell copies of the Software, and to permit persons to whom the
>> - *     Software is furnished to do so, subject to the following
>> - *     conditions:
>> - *
>> - *     The above copyright notice and this permission notice shall be
>> - *     included in all copies or substantial portions of the Software.
>> - *
>> - *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> - *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>> - *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>> - *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>> - *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>> - *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> - *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> - *     OTHER DEALINGS IN THE SOFTWARE.
>>    */
>>   
>>   /dts-v1/;
>>
>> -- 
>> 2.43.0
>>
>>
> 
> Acked-by: Andrej Rosano <andrej.rosano@reversec.com>

Would it be possible to partial-apply just this patch? Or should I maybe 
resubmit it alone? The patches in this "series" are not really related 
after all.

I'm trying to pick this up again, now having left Prolan, and it would 
be great if I could grep through linux-next to see which patches did not 
receive an Ack yet.

Bence
Re: [PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Csókás Bence 3 weeks, 4 days ago
Hi,

On 2025. 08. 14. 9:47, Bence Csókás wrote:
> Replace verbatim license text with a `SPDX-License-Identifier`.
> 
> The comment header mis-attributes this license to be "X11", but the
> license text does not include the last line "Except as contained in this
> notice, the name of the X Consortium shall not be used in advertising or
> otherwise to promote the sale, use or other dealings in this Software
> without prior written authorization from the X Consortium.". Therefore,
> this license is actually equivalent to the SPDX "MIT" license (confirmed
> by text diffing).
> 
> Cc: Andrej Rosano <andrej@inversepath.com>
> Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>

+To: Andrea Barisani <andrea@inversepath.com>

Andrej, Andrea, do you agree with this?

Thanks,
Bence

Re: [PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Andrej Rosano 3 weeks, 3 days ago
Hi Bence,

yes, we agree.

Thanks
Andrej

On 2025-09-09 Tue, Csókás Bence wrote:
> [You don't often get email from csokas.bence@prolan.hu. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> 
> Hi,
> 
> On 2025. 08. 14. 9:47, Bence Csókás wrote:
> > Replace verbatim license text with a `SPDX-License-Identifier`.
> > 
> > The comment header mis-attributes this license to be "X11", but the
> > license text does not include the last line "Except as contained in this
> > notice, the name of the X Consortium shall not be used in advertising or
> > otherwise to promote the sale, use or other dealings in this Software
> > without prior written authorization from the X Consortium.". Therefore,
> > this license is actually equivalent to the SPDX "MIT" license (confirmed
> > by text diffing).
> > 
> > Cc: Andrej Rosano <andrej@inversepath.com>
> > Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>
> 
> +To: Andrea Barisani <andrea@inversepath.com>
> 
> Andrej, Andrea, do you agree with this?
> 
> Thanks,
> Bence
> 

-- 
Andrej Rosano | Hardware Security | Reversec Foundry
www.reversec.com/foundry
BDE1 62F4 7020 1588 8046 AE02 EA17 8C32 AB56 54CE
Re: [PATCH v2 2/8] ARM: dts: imx53-usbarmory: Replace license text comment with SPDX identifier
Posted by Shawn Guo 3 weeks, 3 days ago
On Wed, Sep 10, 2025 at 12:01:02PM +0000, Andrej Rosano wrote:
> Hi Bence,
> 
> yes, we agree.

Andrej, could you give an explicit Acked-by tag for this patch?

Shawn

> On 2025-09-09 Tue, Csókás Bence wrote:
> > [You don't often get email from csokas.bence@prolan.hu. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> > 
> > Hi,
> > 
> > On 2025. 08. 14. 9:47, Bence Csókás wrote:
> > > Replace verbatim license text with a `SPDX-License-Identifier`.
> > > 
> > > The comment header mis-attributes this license to be "X11", but the
> > > license text does not include the last line "Except as contained in this
> > > notice, the name of the X Consortium shall not be used in advertising or
> > > otherwise to promote the sale, use or other dealings in this Software
> > > without prior written authorization from the X Consortium.". Therefore,
> > > this license is actually equivalent to the SPDX "MIT" license (confirmed
> > > by text diffing).
> > > 
> > > Cc: Andrej Rosano <andrej@inversepath.com>
> > > Signed-off-by: Bence Csókás <csokas.bence@prolan.hu>
> > 
> > +To: Andrea Barisani <andrea@inversepath.com>
> > 
> > Andrej, Andrea, do you agree with this?
> > 
> > Thanks,
> > Bence
> > 
> 
> -- 
> Andrej Rosano | Hardware Security | Reversec Foundry
> www.reversec.com/foundry
> BDE1 62F4 7020 1588 8046 AE02 EA17 8C32 AB56 54CE