From: Naga Sureshkumar Relli <nagasure@xilinx.com>
To: Philippe Ombredanne <pombredanne@nexb.com>
CC: "boris.brezillon@free-electrons.com" 
        <boris.brezillon@free-electrons.com>,
        "richard@nod.at" <richard@nod.at>,
        "computersforpeace@gmail.com" <computersforpeace@gmail.com>,
        "marek.vasut@gmail.com" <marek.vasut@gmail.com>,
        "cyrille.pitchen@wedev4u.fr" <cyrille.pitchen@wedev4u.fr>,
        "dwmw2@infradead.org" <dwmw2@infradead.org>,
        "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
        LKML <linux-kernel@vger.kernel.org>, Michal Simek <michals@xilinx.com>,
        "Punnaiah Choudary Kalluri" <punnaia@xilinx.com>,
        "nagasuresh12@gmail.com" <nagasuresh12@gmail.com>
Subject: RE: [PATCH v8 2/2] mtd: nand: Add support for Arasan NAND Flash
 Controller
Thread-Topic: [PATCH v8 2/2] mtd: nand: Add support for Arasan NAND Flash
 Controller
Thread-Index: AQHTdLBOx1jgkvsqs0eZomFArwZpBKNCjdgAgAAlmXA=
Date: Thu, 14 Dec 2017 11:31:33 +0000
Message-ID: <BY2PR02MB14116E2BD437F1CBEB38BA32AF0A0@BY2PR02MB1411.namprd02.prod.outlook.com>
References: <20171214074906.2147-1-nagasure@xilinx.com>
 <20171214074906.2147-3-nagasure@xilinx.com>
 <CAOFm3uFJ9MPpN=2YOsY=kr0j-s=2PvzkKTC51gqJTeaBrSMrrw@mail.gmail.com>
In-Reply-To: <CAOFm3uFJ9MPpN=2YOsY=kr0j-s=2PvzkKTC51gqJTeaBrSMrrw@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 49d0f72d-3356-456a-0ddf-08d542e63aa6
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Dec 2017 11:31:33.5259
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0201MB1008
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by nfs id vBEBVn23019462
Content-Length: 1853
Lines: 60

Hi Philippe,

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

Thanks,
Naga Sureshkumar Relli.

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