linux-kernel - Re: [PATCH v1 1/1] media: atomisp: Remove License information boilerplate

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <028717bf-e10c-4876-9be4-1b41661c447c@redhat.com>
Date: Thu, 7 Nov 2024 12:45:49 +0100
From: Hans de Goede <hdegoede@...hat.com>
To: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>,
 Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Hans Verkuil <hverkuil-cisco@...all.nl>,
 Laurent Pinchart <laurent.pinchart@...asonboard.com>,
 Jeff Johnson <quic_jjohnson@...cinc.com>,
 Mauro Carvalho Chehab <mchehab@...nel.org>,
 Jonathan Bergh <bergh.jonathan@...il.com>,
 Roshan Khatri <topofeverest8848@...il.com>,
 Dipendra Khadka <kdipendra88@...il.com>,
 Sergio de Almeida Cipriano Junior <sergiosacj@...eup.net>,
 Kent Overstreet <kent.overstreet@...ux.dev>,
 Kartik Kulkarni <kartik.koolks@...il.com>,
 Kathara Sasikumar <katharasasikumar007@...il.com>,
 Tchadel Icard <hello@...adelicard.fr>, Kate Hsuan <hpa@...hat.com>,
 Colin Ian King <colin.i.king@...il.com>,
 Lorenzo Stoakes <lorenzo.stoakes@...cle.com>, linux-media@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-staging@...ts.linux.dev,
 Sakari Ailus <sakari.ailus@...ux.intel.com>,
 Andy Shevchenko <andy@...nel.org>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: Re: [PATCH v1 1/1] media: atomisp: Remove License information
 boilerplate

Hi,

On 7-Nov-24 12:42 PM, Mauro Carvalho Chehab wrote:
> Em Thu, 7 Nov 2024 12:32:33 +0100
> Mauro Carvalho Chehab <mchehab+huawei@...nel.org> escreveu:
> 
>> Em Mon, 23 Sep 2024 13:53:06 +0300
>> Andy Shevchenko <andriy.shevchenko@...ux.intel.com> escreveu:
>>
>>> We have the respective SPDX identifiers that are already being applied
>>> to the files in question. Remove the License information boilerplate.
>>>
>>> Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
>>> ---
>>>
>>> Here to fix one of the missed TODO item before moving from staging.
>>> This depends on 20240923085652.3457117-1-andriy.shevchenko@...ux.intel.com.
>>> But can be applied before that (it's trivial to adjust).
>>>
>>>  drivers/staging/media/atomisp/i2c/atomisp-gc0310.c  | 10 ----------
>>>  drivers/staging/media/atomisp/i2c/atomisp-gc2235.c  | 10 ----------
>>>  .../media/atomisp/i2c/atomisp-libmsrlisthelper.c    | 11 -----------
>>>  drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 11 -----------
>>>  drivers/staging/media/atomisp/i2c/atomisp-ov2722.c  | 10 ----------
>>>  drivers/staging/media/atomisp/i2c/gc2235.h          | 13 -------------
>>>  drivers/staging/media/atomisp/i2c/mt9m114.h         | 11 -----------
>>>  drivers/staging/media/atomisp/i2c/ov2722.h          | 11 -----------
>>>  drivers/staging/media/atomisp/include/hmm/hmm.h     | 11 -----------  
>>
>> Hmm... there's no SPDX for hmm.h header. We need to ensure that all files
>> have SPDX license before removing the boilerplate.
>>
>>>  drivers/staging/media/atomisp/include/hmm/hmm_bo.h  | 11 -----------
>>>  .../staging/media/atomisp/include/hmm/hmm_common.h  | 11 -----------  
>>
>> Same for the above and for other header files.
> 
> Nevermind. I didn't show enough context while checking it. All files
> have SPDX. Patch LGTM.

That is good to hear.

Note I've included this + a bunch of other atomisp patches in an
atomisp pull-request for 6.13, see:

https://lore.kernel.org/linux-media/51e5e1cf-8aff-44fc-b40c-a0074a770a69@redhat.com/

(in case you have not seen that yet)

Regards,

Hans