All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chin Liang See <clsee@altera.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 1/2] nand/denali: Adding Denali NAND driver support
Date: Tue, 4 Mar 2014 17:57:35 -0600	[thread overview]
Message-ID: <1393977455.2475.7.camel@clsee-VirtualBox.altera.com> (raw)
In-Reply-To: <53106707.3060107@monstr.eu>

On Fri, 2014-02-28 at 11:37 +0100, Michal Simek wrote:
> >>> +/* lld_nand.h */
> >>> +/*
> >>> + * NAND Flash Controller Device Driver
> >>> + * Copyright (c) 2009, Intel Corporation and its suppliers.
> >>> + *
> >>> + * This program is free software; you can redistribute it and/or modify it
> >>> + * under the terms and conditions of the GNU General Public License,
> >>> + * version 2, as published by the Free Software Foundation.
> >>> + *
> >>> + * This program is distributed in the hope 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.
> >>
> >>
> >> Isn't this pretty weird if we are using SPDX?
> > 
> > I believe the Linux driver owner wish to maintain this header when
> > he/she copy into NAND driver. Anyhow, the SPDX license already stated at
> > top of the header file.
> 
> 1. Using address in license is just wrong and it should be removed
> because they can move to different location.
> 
> 2. u-boot adopted SPDX and all these fragments were removed.
> 
> Wolfgang/Tom: IMHO this should be also changed/fixed.


Just check the latest denali.h in Linux and this license header still
there. But removing the license header put by original author might not
a good thing to do. Unless we can get some approval from Wolfgang or
Jason (who is submitter of this file to Linux).

Thanks
Chin Liang


> 
> Thanks,
> Michal
> 
> 

  reply	other threads:[~2014-03-04 23:57 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-21 20:51 [U-Boot] [PATCH v2 1/2] nand/denali: Adding Denali NAND driver support Chin Liang See
2014-02-24  7:48 ` Michal Simek
2014-02-24  8:06   ` Masahiro Yamada
2014-02-24  8:16     ` Michal Simek
2014-02-27 17:04   ` Chin Liang See
2014-02-28 10:37     ` Michal Simek
2014-03-04 23:57       ` Chin Liang See [this message]
2014-03-05  6:22         ` Michal Simek
2014-02-27 14:35 ` Masahiro Yamada
2014-02-27 21:02   ` Chin Liang See
2014-02-27 22:32     ` Scott Wood
2014-02-27 23:03       ` Chin Liang See
2014-02-28 12:57         ` Masahiro Yamada
2014-03-05  0:24           ` Chin Liang See
2014-03-04  0:03 ` Scott Wood
2014-03-04 10:31   ` Masahiro Yamada
2014-03-04 18:44     ` Scott Wood
2014-03-05 17:40     ` Chin Liang See
2014-03-05 17:34   ` Chin Liang See
2014-03-05 18:23     ` Scott Wood
2014-03-05 23:01       ` Chin Liang See
2014-03-05 23:04         ` Scott Wood
2014-03-05 23:09           ` Chin Liang See
2014-03-05 23:11             ` Scott Wood
2014-03-05 23:21               ` Chin Liang See
2014-05-30 10:50 ` Rik Smith
2014-05-30 11:36   ` Masahiro Yamada

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=1393977455.2475.7.camel@clsee-VirtualBox.altera.com \
    --to=clsee@altera.com \
    --cc=u-boot@lists.denx.de \
    /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.