[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <BY2PR02MB14116E2BD437F1CBEB38BA32AF0A0@BY2PR02MB1411.namprd02.prod.outlook.com>
Date: Thu, 14 Dec 2017 11:31:33 +0000
From: Naga Sureshkumar Relli <nagasure@...inx.com>
To: Philippe Ombredanne <pombredanne@...b.com>
CC: "boris.brezillon@...e-electrons.com"
<boris.brezillon@...e-electrons.com>,
"richard@....at" <richard@....at>,
"computersforpeace@...il.com" <computersforpeace@...il.com>,
"marek.vasut@...il.com" <marek.vasut@...il.com>,
"cyrille.pitchen@...ev4u.fr" <cyrille.pitchen@...ev4u.fr>,
"dwmw2@...radead.org" <dwmw2@...radead.org>,
"linux-mtd@...ts.infradead.org" <linux-mtd@...ts.infradead.org>,
LKML <linux-kernel@...r.kernel.org>,
Michal Simek <michals@...inx.com>,
"Punnaiah Choudary Kalluri" <punnaia@...inx.com>,
"nagasuresh12@...il.com" <nagasuresh12@...il.com>
Subject: RE: [PATCH v8 2/2] mtd: nand: Add support for Arasan NAND Flash
Controller
Hi Philippe,
Thanks for the review.
I will update it in next version.
Thanks,
Naga Sureshkumar Relli.
> -----Original Message-----
> From: Philippe Ombredanne [mailto:pombredanne@...b.com]
> Sent: Thursday, December 14, 2017 2:42 PM
> To: Naga Sureshkumar Relli <nagasure@...inx.com>
> Cc: boris.brezillon@...e-electrons.com; richard@....at;
> computersforpeace@...il.com; marek.vasut@...il.com;
> cyrille.pitchen@...ev4u.fr; dwmw2@...radead.org; linux-
> mtd@...ts.infradead.org; LKML <linux-kernel@...r.kernel.org>; Michal Simek
> <michals@...inx.com>; Punnaiah Choudary Kalluri <punnaia@...inx.com>;
> nagasuresh12@...il.com
> Subject: Re: [PATCH v8 2/2] mtd: nand: Add support for Arasan NAND Flash
> Controller
>
> Dear Naga,
>
> On Thu, Dec 14, 2017 at 8:49 AM, Naga Sureshkumar Relli
> <naga.sureshkumar.relli@...inx.com> wrote:
> > Added the basic driver for Arasan NAND Flash Controller used in Zynq
> > UltraScale+ MPSoC. It supports only Hw ECC and upto 24bit correction.
> >
> > Signed-off-by: Naga Sureshkumar Relli <nagasure@...inx.com>
> > Signed-off-by: Punnaiah Choudary Kalluri <punnaia@...inx.com>
>
> <snip>
>
> > --- /dev/null
> > +++ b/drivers/mtd/nand/arasan_nand.c
> > @@ -0,0 +1,1020 @@
> > +/*
> > + * Arasan NAND Flash Controller Driver
> > + *
> > + * Copyright (C) 2014 - 2017 Xilinx, Inc.
> > + *
> > + * 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; either version 2 of the License, or (at
> > +your
> > + * option) any later version.
> > + */
>
> Could you use the new SPDX tags instead of this fine and long boilerplate? See
> Thomas doc for details [1]
>
> [1] https://lkml.org/lkml/2017/12/4/934
>
> Thanks!
>
> --
> Cordially
> Philippe Ombredanne
Powered by blists - more mailing lists