[PATCH v8 2/2] mtd: nand: Add support for Arasan NAND Flash Controller

Naga Sureshkumar Relli nagasure at xilinx.com
Thu Dec 14 03:31:33 PST 2017


Hi Philippe,

Thanks for the review.
I will update it in next version.

Thanks,
Naga Sureshkumar Relli.

> -----Original Message-----
> From: Philippe Ombredanne [mailto:pombredanne at nexb.com]
> Sent: Thursday, December 14, 2017 2:42 PM
> To: Naga Sureshkumar Relli <nagasure at xilinx.com>
> Cc: boris.brezillon at free-electrons.com; richard at nod.at;
> computersforpeace at gmail.com; marek.vasut at gmail.com;
> cyrille.pitchen at wedev4u.fr; dwmw2 at infradead.org; linux-
> mtd at lists.infradead.org; LKML <linux-kernel at vger.kernel.org>; Michal Simek
> <michals at xilinx.com>; Punnaiah Choudary Kalluri <punnaia at xilinx.com>;
> nagasuresh12 at gmail.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 at xilinx.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 at xilinx.com>
> > Signed-off-by: Punnaiah Choudary Kalluri <punnaia at xilinx.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


More information about the linux-mtd mailing list