From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailout.micron.com ([137.201.242.129]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fdDV5-0002aw-Ui for linux-mtd@lists.infradead.org; Wed, 11 Jul 2018 11:41:48 +0000 From: "Bean Huo (beanhuo)" To: Boris Brezillon CC: Richard Weinberger , Miquel Raynal , "linux-mtd@lists.infradead.org" , Chris Packham , David Woodhouse , Brian Norris , Marek Vasut Subject: RE: [EXT] Re: [PATCH 2/2] mtd: rawnand: micron: Fix on-die ECC detection logic Date: Wed, 11 Jul 2018 11:41:01 +0000 Message-ID: <63aabb45edfe4d17b00ce8eed5f53f49@SIWEX5A.sing.micron.com> References: <1be1ede4dcf14f24b68b8497b167c1b3@SIWEX5A.sing.micron.com> <20180711123209.5271d411@bbrezillon> In-Reply-To: <20180711123209.5271d411@bbrezillon> Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , >> Here I say for sure, the value of Bit7 in byte 4 of READID is >> permanent. It is a reflection of the default state of the device. it can= not >change by set feature. > >That's not what the board I have on my desk says. I have a >MT29F2G08ABAEAH4, and I can tell you for sure this bit changes when I >enable/disable on-die ECC. Do the test, and you'll see... > This is very interesting. MT29F2G08ABAEAH4 is 60s NAND, unfortunately, I do= n't have 60s sample. I will check, and back you. >See, that's what I'm complaining about. You keep saying things that appear= to >be untrue when when we check on real HW parts. So, either we have NANDs >that are buggy, or you didn't check yourself. > >Regards, > >Boris