All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: i.mitsyanko@samsung.com
Cc: peter.maydell@linaro.org, kyungmin.park@samsung.com,
	d.solodkiy@samsung.com, aliguori@us.ibm.com,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH RESEND 1.1] exynos4210: drop "All rights reserved" line from files license
Date: Fri, 18 May 2012 17:21:56 +0200	[thread overview]
Message-ID: <4FB66914.1080700@suse.de> (raw)
In-Reply-To: <4FB65051.1080006@samsung.com>

Am 18.05.2012 15:36, schrieb Igor Mitsyanko:
> On 05/18/2012 05:26 PM, Andreas Färber wrote:
>> Am 18.05.2012 14:58, schrieb Igor Mitsyanko:
>>> It has been noted that "All rights reserved" statement conflicts with
>>> GPL,
>>> remove it.
>>>
>>> Signed-off-by: Igor Mitsyanko<i.mitsyanko@samsung.com>
>>> ---
>> [...]
>>> diff --git a/hw/exynos4210_combiner.c b/hw/exynos4210_combiner.c
>>> index 80af22c..a4a159a 100644
>>> --- a/hw/exynos4210_combiner.c
>>> +++ b/hw/exynos4210_combiner.c
>>> @@ -1,8 +1,7 @@
>>>   /*
>>>    * Samsung exynos4210 Interrupt Combiner
>>>    *
>>> - * Copyright (c) 2000 - 2011 Samsung Electronics Co., Ltd.
>>> - * All rights reserved.
>>> + * Copyright (c) 2011 Samsung Electronics Co., Ltd.
>>>    *
>>>    * Evgeny Voevodin<e.voevodin@samsung.com>
>>>    *
>> [snip]
>>
>> In multiple files you change the copyright from 2000-2011 to just 2011.
>> Intentional?
>>
> Yes, all these files were created in 2011 so this time period makes no
> sense. I hope you wouldn't tell me that its an unrelated changes and
> should be in a separate patch :)

No, no, that's a Samsung decision. But if you had explained it in the
commit message it would've avoided my question. :) No need to respin.

I guess it's more a question of through who's queue this goes.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  reply	other threads:[~2012-05-18 15:22 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-18 12:58 [Qemu-devel] [PATCH RESEND 1.1] Exynos4210: drop "All rights reserved" line Igor Mitsyanko
2012-05-18 12:58 ` [Qemu-devel] [PATCH RESEND 1.1] exynos4210: drop "All rights reserved" line from files license Igor Mitsyanko
2012-05-18 13:26   ` Andreas Färber
2012-05-18 13:36     ` Igor Mitsyanko
2012-05-18 15:21       ` Andreas Färber [this message]
2012-05-18 13:36     ` Dmitry Solodkiy

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=4FB66914.1080700@suse.de \
    --to=afaerber@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=d.solodkiy@samsung.com \
    --cc=i.mitsyanko@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.