mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Péter Ujfalusi" <peter.ujfalusi@linux.intel.com>
To: Sebastian Reichel <sebastian.reichel@collabora.com>,
	Christophe JAILLET <christophe.jaillet@wanadoo.fr>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org,
	Kai Vehmanen <kai.vehmanen@linux.intel.com>
Subject: Re: [PATCH] headers: Remove some left-over license text in include/uapi/linux/hsi/
Date: Tue, 15 Nov 2022 14:59:19 +0200	[thread overview]
Message-ID: <16b18a0d-4921-311e-be24-1103f90a8ac2@linux.intel.com> (raw)
In-Reply-To: <20221114210155.anq5gkggfrvj6nki@mercury.elektranox.org>

Hi,

On 14/11/2022 23:01, Sebastian Reichel wrote:
> Hi,
> 
> On Sun, Sep 11, 2022 at 01:56:59PM +0200, Christophe JAILLET wrote:
>> Remove some left-over from commit e2be04c7f995 ("License cleanup: add SPDX
>> license identifier to uapi header files with a license")
>>
>> When the SPDX-License-Identifier tag has been added, the corresponding
>> license text has not been removed.
>>
>> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
>> ---
> 
> IIRC that was intentional, since the plan was to seek for an
> explicit Ack from the copyright holder when removing the text.
> I've added Greg for clarification and hopefully the latest
> mail address from Kai and Peter.

I'm fine with the removal of the boilerplate text, SPDX is covering that.

I would appreciate if you pick my personal ack for this:

Acked-by: Peter Ujfalusi <peter.ujfalusi@gmail.com>

> 
> -- Sebastian
> 
>>  include/uapi/linux/hsi/cs-protocol.h | 14 --------------
>>  include/uapi/linux/hsi/hsi_char.h    | 14 --------------
>>  2 files changed, 28 deletions(-)
>>
>> diff --git a/include/uapi/linux/hsi/cs-protocol.h b/include/uapi/linux/hsi/cs-protocol.h
>> index c7f6e7672cb5..07c3bfb67463 100644
>> --- a/include/uapi/linux/hsi/cs-protocol.h
>> +++ b/include/uapi/linux/hsi/cs-protocol.h
>> @@ -6,20 +6,6 @@
>>   *
>>   * Contact: Kai Vehmanen <kai.vehmanen@nokia.com>

I don't think Kai's nokia.com mail address is valid as a contact
information...

>>   * Original author: Peter Ujfalusi <peter.ujfalusi@nokia.com>



>> - *
>> - * This program is free software; you can redistribute it and/or
>> - * modify it under the terms of the GNU General Public License
>> - * version 2 as published by the Free Software Foundation.
>> - *
>> - * This program 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.
>> - *
>> - * You should have received a copy of the GNU General Public License
>> - * along with this program; if not, write to the Free Software
>> - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
>> - * 02110-1301 USA
>>   */
>>  
>>  #ifndef _CS_PROTOCOL_H
>> diff --git a/include/uapi/linux/hsi/hsi_char.h b/include/uapi/linux/hsi/hsi_char.h
>> index 91623b0398b1..5ef72f0daf94 100644
>> --- a/include/uapi/linux/hsi/hsi_char.h
>> +++ b/include/uapi/linux/hsi/hsi_char.h
>> @@ -5,20 +5,6 @@
>>   * Copyright (C) 2010 Nokia Corporation. All rights reserved.
>>   *
>>   * Contact: Andras Domokos <andras.domokos at nokia.com>
>> - *
>> - * This program is free software; you can redistribute it and/or
>> - * modify it under the terms of the GNU General Public License
>> - * version 2 as published by the Free Software Foundation.
>> - *
>> - * This program 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.
>> - *
>> - * You should have received a copy of the GNU General Public License
>> - * along with this program; if not, write to the Free Software
>> - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
>> - * 02110-1301 USA
>>   */
>>  
>>  #ifndef __HSI_CHAR_H
>> -- 
>> 2.34.1
>>

-- 
Péter

  parent reply	other threads:[~2022-11-15 12:58 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-11 11:56 Christophe JAILLET
2022-11-14 21:01 ` Sebastian Reichel
2022-11-15  6:48   ` Greg Kroah-Hartman
2022-11-15 12:59   ` Péter Ujfalusi [this message]
2022-11-15 17:21   ` Kai Vehmanen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=16b18a0d-4921-311e-be24-1103f90a8ac2@linux.intel.com \
    --to=peter.ujfalusi@linux.intel.com \
    --cc=christophe.jaillet@wanadoo.fr \
    --cc=gregkh@linuxfoundation.org \
    --cc=kai.vehmanen@linux.intel.com \
    --cc=kernel-janitors@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sebastian.reichel@collabora.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®