Messages in this thread | | | From | Naga Sureshkumar Relli <> | Subject | RE: [LINUX PATCH v17 2/2] mtd: rawnand: pl353: Add basic driver for arm pl353 smc nand interface | Date | Wed, 3 Jul 2019 06:13:47 +0000 |
| |
Hi Miquel,
> -----Original Message----- > From: linux-mtd <linux-mtd-bounces@lists.infradead.org> On Behalf Of Helmut Grohne > Sent: Tuesday, June 25, 2019 7:41 PM > To: Naga Sureshkumar Relli <nagasure@xilinx.com> > Cc: vigneshr@ti.com; bbrezillon@kernel.org; yamada.masahiro@socionext.com; > richard@nod.at; linux-kernel@vger.kernel.org; marek.vasut@gmail.com; linux- > mtd@lists.infradead.org; miquel.raynal@bootlin.com; computersforpeace@gmail.com; > dwmw2@infradead.org > Subject: Re: [LINUX PATCH v17 2/2] mtd: rawnand: pl353: Add basic driver for arm pl353 > smc nand interface > > Thank you for the quick update. > > On Tue, Jun 25, 2019 at 06:46:30AM +0200, Naga Sureshkumar Relli wrote: > > -> Tested Micron MT29F2G08ABAEAWP (On-die capable) and AMD/Spansion > S34ML01G1. > > I tested the v17 series with the MT29F2G08ABAEAWP. I can now mount existing jffs2 > volumes without issues. > > When running nandtest on a 64MB area, I no longer see lots of consecutive errors. However I > see few (4-8) single byte errors for random locations on about half the runs. In comparison, I > couldn't reproduce these on the older driver on v4.14. > > When writing random 1MB files on a fresh jffs2 filesystem and reading them back after > umounting and mounting the filesystem, I got one faulty file in 50 attempts. > > So this driver mostly works for me, but I suspect that something (possibly the tested > hardware) doesn't fully work yet. To say more, I'll need long term testing results. In the mean > time, I'm in favour of merging the driver. What is your take on this? Can we consider to merge this driver? Could you please let me know?
Thanks, Naga Sureshkumar Relli > > Helmut > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/
| |