From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.bootlin.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gH7VC-0002Ta-Mc for linux-mtd@lists.infradead.org; Mon, 29 Oct 2018 13:22:54 +0000 Date: Mon, 29 Oct 2018 14:22:30 +0100 From: Miquel Raynal To: Naga Sureshkumar Relli Cc: "boris.brezillon@bootlin.com" , "richard@nod.at" , "dwmw2@infradead.org" , "computersforpeace@gmail.com" , "marek.vasut@gmail.com" , "linux-mtd@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Michal Simek , "nagasuresh12@gmail.com" Subject: Re: [LINUX PATCH v11 0/3] Add support for Arasan NAND Flash controller Message-ID: <20181029142230.2b3635d6@xps13> In-Reply-To: References: <1537878031-22253-1-git-send-email-naga.sureshkumar.relli@xilinx.com> <20181029105108.562ddd21@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Naga, Naga Sureshkumar Relli wrote on Mon, 29 Oct 2018 13:05:47 +0000: > Hi Miquel, >=20 > > -----Original Message----- > > From: Miquel Raynal [mailto:miquel.raynal@bootlin.com] > > Sent: Monday, October 29, 2018 3:21 PM > > To: Naga Sureshkumar Relli > > Cc: boris.brezillon@bootlin.com; richard@nod.at; dwmw2@infradead.org; > > computersforpeace@gmail.com; marek.vasut@gmail.com; linux-mtd@lists.inf= radead.org; > > linux-kernel@vger.kernel.org; Michal Simek ; > > nagasuresh12@gmail.com > > Subject: Re: [LINUX PATCH v11 0/3] Add support for Arasan NAND Flash co= ntroller > >=20 > > Hi Naga, > >=20 > > Naga Sureshkumar Relli wrote on Tue= , 25 Sep 2018 > > 17:50:28 +0530: > > =20 > > > This patch series adds the basic driver support for Arasan NAND Flash= controller. > > > We are reinitiating the patch series by fixing the comments given by = Miquel and Boris. > > > Major changes are exec_op() implementation related. > > > Rebased to 4.19 nand tree. > > > Tested MT29F32G08ABCDBJ4. > > > > > > As suggested by Boris, it also adds a new varaible called mode in > > > struct nand_sdr_timings, which will give directly the sdr operating > > > mode. and it is useful for some controllers, where we can configure > > > direclty the operating mode instead of timings. > > > =20 > >=20 > > Unless I missed something there are still comments from Boris pending? > > I will wait for your v12 to give this driver another shot. =20 > shall I send the v12 on top of nand/for-4.20? Yes please, or wait for -rc1 to be out. There is no rush. Thanks, Miqu=C3=A8l