Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Naga Sureshkumar Relli <nagasure@xilinx.com>
To:     Helmut Grohne <helmut.grohne@intenta.de>,
        Miquel Raynal <miquel.raynal@bootlin.com>
CC:     "vigneshr@ti.com" <vigneshr@ti.com>,
        "bbrezillon@kernel.org" <bbrezillon@kernel.org>,
        "yamada.masahiro@socionext.com" <yamada.masahiro@socionext.com>,
        "richard@nod.at" <richard@nod.at>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "marek.vasut@gmail.com" <marek.vasut@gmail.com>,
        "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
        "miquel.raynal@bootlin.com" <miquel.raynal@bootlin.com>,
        "computersforpeace@gmail.com" <computersforpeace@gmail.com>,
        "dwmw2@infradead.org" <dwmw2@infradead.org>
Subject: RE: [LINUX PATCH v17 2/2] mtd: rawnand: pl353: Add basic driver for
 arm pl353 smc nand interface
Thread-Topic: [LINUX PATCH v17 2/2] mtd: rawnand: pl353: Add basic driver for
 arm pl353 smc nand interface
Thread-Index: AQHVKxERfpwoj8XL9UqMkCeJJ20MnKasaZ4AgAwLy6A=
Date:   Wed, 3 Jul 2019 06:13:47 +0000
Message-ID: <DM6PR02MB47796F40DB186070031E5113AFFB0@DM6PR02MB4779.namprd02.prod.outlook.com>
References: <20190625044630.31717-1-naga.sureshkumar.relli@xilinx.com>
 <20190625044630.31717-2-naga.sureshkumar.relli@xilinx.com>
 <20190625141126.ggmxjcmdh76lguds@laureti-dev>
In-Reply-To: <20190625141126.ggmxjcmdh76lguds@laureti-dev>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: xilinx.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 3c3423f2-19b9-4b6e-651e-08d6ff7d9bfb
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jul 2019 06:13:47.0853
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: nagasure@xilinx.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR02MB5898
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Miquel,

> -----Original Message-----
> From: linux-mtd <linux-mtd-bounces@lists.infradead.org> On Behalf Of Helm=
ut 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; linu=
x-
> mtd@lists.infradead.org; miquel.raynal@bootlin.com; computersforpeace@gma=
il.com;
> dwmw2@infradead.org
> Subject: Re: [LINUX PATCH v17 2/2] mtd: rawnand: pl353: Add basic driver =
for arm pl353
> smc nand interface
>=20
> Thank you for the quick update.
>=20
> On Tue, Jun 25, 2019 at 06:46:30AM +0200, Naga Sureshkumar Relli wrote:
> > -> Tested Micron MT29F2G08ABAEAWP (On-die capable) and AMD/Spansion
> S34ML01G1.
>=20
> I tested the v17 series with the MT29F2G08ABAEAWP. I can now mount existi=
ng jffs2
> volumes without issues.
>=20
> 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 r=
uns. In comparison, I
> couldn't reproduce these on the older driver on v4.14.
>=20
> When writing random 1MB files on a fresh jffs2 filesystem and reading the=
m back after
> umounting and mounting the filesystem, I got one faulty file in 50 attemp=
ts.
>=20
> So this driver mostly works for me, but I suspect that something (possibl=
y the tested
> hardware) doesn't fully work yet. To say more, I'll need long term testin=
g 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
>=20
> Helmut
>=20
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/