From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2F03CC6FD18 for ; Tue, 18 Apr 2023 13:25:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zkO6/F2KbWQeFqxcShZjcKkZoThXQlsgZRunpzA12ng=; b=NrIP/JVyj58zss 3FleU7jtAmDtlEmUWwaYBZ8EliC9p0nvFwDwc5gdyUX8P/BLyw7QcPnVVKD1KHJOwWhgxm9LZcwcY fogreh6D/hVzHpVOjMjfFJGdHgRkBBco9GzrisXEAa19GiimktedkXteVl2lNYJuRfacECtDPqbuH 5oZwxxkGqOio6THuyvpxjfHhE0fH4KmRMY+Kh1T/3LKVjGdKqr7mi6+mAR4Sned7StU0U8Kql5zo0 wVrHHVoEkkHgcGmjLju1wIHAI5wrVy6wv34SFDeLGxZan+rgst0ctCLJZavZWTyi/qVfUWKMCFXqd VqYHpdNhyE7S4qvW+w5A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1polKj-002GUf-0e; Tue, 18 Apr 2023 13:25:21 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1polKf-002GTP-0f; Tue, 18 Apr 2023 13:25:19 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 0EAEF240002; Tue, 18 Apr 2023 13:25:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1681824311; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=a4ugRgZNu/Y+baSK+9UqHRS8pZ9Nhe2pwx9QcHGO7qQ=; b=oLiGo4PRzihRAJLpguz+PlMJwr7om8taIquydk8GT6iIDMZI0eemdwaItHjQju6h9yYRQd 75TGkfQ8FlW6TunA7ncve2pBpkxeh68RNtsyTsEMakpYGhuz9QQncJPNnBOx++2x/35XZ1 YfYmIV25hOM8VU3hNyOJ5clsnDrwKF1x96n56h6yc+W2TQtmS+0TTr6pA8TQTOPpXjsCnA zoqCCNCrtjKuPxquL55ja8iNzy/LyBUZj0GqExbz00XaqkjOEp8Z8gzZ3cL7HakbKpfwQ8 OoX2fR9wc0DFvtNY3NIjUaSsybzkgTBGfJQG4BQlE7ZTAo/GQECnlYk4Mr95SA== Date: Tue, 18 Apr 2023 15:25:05 +0200 From: Miquel Raynal To: Arseniy Krasnov Cc: Liang Yang , Richard Weinberger , Vignesh Raghavendra , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Jianxin Pan , Yixun Lan , , , , , , , "yonghui.yu" Subject: Re: [PATCH v1 4/5] mtd: rawnand: meson: clear OOB buffer before read Message-ID: <20230418152505.72fc16da@xps-13> In-Reply-To: References: <20230412061700.1492474-1-AVKrasnov@sberdevices.ru> <20230412094400.3c82f631@xps-13> <20230412113654.183350d0@xps-13> <4eace0a0-f6af-7d99-a52f-7913a2139330@sberdevices.ru> <20230412141824.755b2bca@xps-13> <20230412145715.58c2be4a@xps-13> <7c996832-258f-001c-56bd-87bbdf23eeaa@amlogic.com> <20230412163214.0c764bb3@xps-13> <0c61eaae-053e-5768-a533-70b2ff0cc95d@amlogic.com> <7520e512-8a19-ff04-364c-b5be0a579ef0@sberdevices.ru> <20230413102200.309fbe9c@xps-13> <20230413122252.0a8efcd8@xps-13> <569a948e-654a-b21f-8a4f-55dc4b295387@sberdevices.ru> <60fa656e-bda1-1de6-a79e-3e3041cd69a8@sberdevices.ru> <780c0cae-18b6-2652-1c2c-6d398ea60451@amlogic.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230418_062517_718003_63473675 X-CRM114-Status: GOOD ( 23.92 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org SGkgQXJzZW5peSwKCj4gPj4gSGVsbG8gYWdhaW4gQExpYW5nIEBNaXF1ZWwhCj4gPj4KPiA+PiBP bmUgbW9yZSBxdWVzdGlvbiBhYm91dCBPT0IgYWNjZXNzLCBhcyBJIGNhbiBzZWUgY3VycmVudCBk cml2ZXIgdXNlcyB0aGUgZm9sbG93aW5nCj4gPj4gY2FsbGJhY2tzOgo+ID4+Cj4gPj4gwqDCoMKg wqBuYW5kLT5lY2Mud3JpdGVfb29iX3JhdyA9IG5hbmRfd3JpdGVfb29iX3N0ZDsKPiA+PiDCoMKg wqDCoG5hbmQtPmVjYy53cml0ZV9vb2IgPSBuYW5kX3dyaXRlX29vYl9zdGQ7Cj4gPj4KPiA+Pgo+ ID4+IEZ1bmN0aW9uICduYW5kX3dyaXRlX29vYl9zdGQoKScgd3JpdGVzIGRhdGEgdG8gdGhlIGVu ZCBvZiB0aGUgcGFnZS4gQnV0IGFzIEkKPiA+PiBjYW4gc2VlIGJ5IGR1bXBpbmcgJ2RhdGFfYnVm JyBkdXJpbmcgcmVhZCwgcGh5c2ljYWwgbGF5b3V0IG9mIGVhY2ggcGFnZSBpcyB0aGUKPiA+PiBm b2xsb3dpbmcgKDFLQiBFQ0MpOgo+ID4+Cj4gPj4gMHgwMDA6IFvCoMKgwqDCoMKgwqDCoMKgIDEg S0Igb2YgZGF0YcKgwqDCoMKgwqDCoMKgIF0KPiA+PiAweDQwMDogWyAyQiB1c2VyIGRhdGFdIFsg MTRCIEVDQyBjb2RlXQo+ID4+IDB4NDEwOiBbwqDCoMKgwqDCoMKgwqDCoCAxIEtCIG9mIGRhdGHC oMKgwqDCoMKgwqDCoCBdwqDCoMKgIChBKQo+ID4+IDB4ODEwOiBbIDJCIHVzZXIgZGF0YV0gWyAx NEIgRUNDIGNvZGVdCj4gPj4gMHg4MjA6IFvCoMKgwqDCoMKgwqDCoCAzMkIgdW51c2VkwqDCoMKg wqDCoMKgwqDCoMKgwqAgXQo+ID4+Cj4gPj4KPiA+Pgo+ID4+IFNvLCBhZnRlciAnbmFuZF93cml0 ZV9vb2Jfc3RkKCknIChsZXQgZGF0YSBiZSBzZXF1ZW5jZSBmcm9tIFsweDAgLi4uIDB4M2ZdKSwK PiA+PiBwYWdlIHdpbGwgbG9vayBsaWtlIHRoaXM6Cj4gPj4KPiA+PiAweDAwMDogW8KgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoCAweEZGwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4gMHg0MDA6 IFvCoMKgwqDCoMKgwqDCoMKgwqDCoCAuLi4uLi4uLsKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4g MHg3ZjA6IFvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgMHhGRsKgwqDCoMKgwqDCoMKgwqDCoMKg wqAgXQo+ID4+IDB4ODAwOiBbIDAwIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uwqAgXQo+ID4+IDB4 ODMwOiBbIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLiAzZiBdCj4gPj4KPiA+PiBIZXJlIHdlIGhh dmUgdHdvIHByb2JsZW1zOgo+ID4+IDEpIEF0dGVtcHQgdG8gZGlzcGxheSByYXcgZGF0YSBieSAn bmFuZGR1bXAnIHV0aWxpdHkgcHJvZHVjZXMgYSBsaXR0bGUgYml0Cj4gPj4gwqDCoMKgIGludmFs aWQgb3V0cHV0LCBhcyBkcml2ZXIgcmVsaWVzIG9uIGxheW91dCAoQSkgZnJvbSBhYm92ZS4gRS5n LiBPT0IgZGF0YQo+ID4+IMKgwqDCoCBpcyBhdCAweDQwMCBhbmQgMHg4MTAuIEhlcmUgaXMgYW4g ZXhhbXBsZSAoYXR0ZW1wdCB0byB3cml0ZSAweDExIDB4MjIgMHgzMyAweDQ0KToKPiA+Pgo+ID4+ IDB4MDAwMDA3ZjA6IDExIDIyIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZm IGZmwqAgfC4iLi4uLi4uLi4uLi4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwK PiA+PiDCoMKgIE9PQiBEYXRhOiAzMyA0NCBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBm ZiBmZiBmZiBmZsKgIHwzRC4uLi4uLi4uLi4uLi4ufAo+ID4+IMKgwqAgT09CIERhdGE6IGZmIGZm IGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmwqAgfC4uLi4uLi4uLi4u Li4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwKPiA+PiAgCj4gPiBIaSBBcnNl bml5LAo+ID4gCj4gPiBJIHJlYWxpemVkIHRoZSB3cml0ZV9vb2JfcmF3KCkgYW5kIHdyaXRlX29v YigpIGFyZSB3cm9uZyBpbiBtZXNvbl9uYW5kLmMuIEkgc3VnZ2VzdCBib3RoIG9mIHRoZW0gc2hv dWxkIGJlIHJld29ya2VkIGFuZCBmb2xsb3cgdGhlIGZvcm1hdCBvZiBtZXNvbiBuYW5kIGNvbnRy b2xsZXIuIGkuZS4gZmlyc3RseSBmb3JtYXQgdGhlIGRhdGEgaW4gTGF5b3V0IChBKSBhbmQgdGhl biB3cml0ZS4gcmVhZGluZyBpcyBmaXJzdGx5IHJlYWRpbmcgdGhlIGRhdGEgb2YgbGF5b3V0IChB KSBhbmQgdGhlbiBjb21wb3N0IHRoZSBsYXlvdXQgKEIpLiAgCj4gCj4gSUlVQyBhZnRlciBzdWNo IHdyaXRpbmcgb25seSBPT0IgKGUuZy4gdXNlciBieXRlcykgYWNjb3JkaW5nIGxheW91dCAoQSks IGh3IHdpbGwgYWxzbyB3cml0ZSBFQ0MgY29kZXMsIHNvCj4gaXQgd2lsbCBiZSBpbXBvc3NpYmxl IHRvIHdyaXRlIGRhdGEgdG8gdGhpcyBwYWdlIGxhdGVyLCBiZWNhdXNlIHdlIGNhbm5vdCB1cGRh dGUgRUNDIGNvZGVzIHByb3Blcmx5IGZvciB0aGUgbmV3bHkKPiB3cml0dGVuIGRhdGEgKHdlIGNh bid0IHVwZGF0ZSBiaXRzIGZyb20gMCB0byAxKS4KPiAKPiA+IAo+ID4gICAKPiA+Pgo+ID4+IDIp IEF0dGVtcHQgdG8gcmVhZCBkYXRhIGluIEVDQyBtb2RlIHdpbGwgZmFpbCwgYmVjYXVzZSBJSVVD IHBhZ2UgaXMgaW4gZGlydHkKPiA+PiDCoMKgwqAgc3RhdGUgKEkgbWVhbiB3YXMgd3JpdHRlbiBh dCBsZWFzdCBvbmNlKSBhbmQgTkFORCBjb250cm9sbGVyIHRyaWVzIHRvIHVzZQo+ID4+IMKgwqDC oCBFQ0MgY29kZXMgYXQgMHg0MDAgYW5kIDB4ODEwLCB3aGljaCBhcmUgb2J2aW91c2x5IGJyb2tl biBpbiB0aGlzIGNhc2UuIFRodXMgIAo+ID4gCj4gPiBBcyBpIHNhaWQgYWJvdmUsIHdyaXRlX29v Yl9yYXcoKSBhbmQgd3JpdGVfb29iKCkgc2hvdWxkIGJlIHJld29ya2VkLgo+ID4gaSBkb24ndCBr bm93IHdoYXQgZG8geW91IG1lYW4gcGFnZSB3YXMgd3JpdHRlbiBhdCBsZWFzdCBvbmNlLiBhbnl3 YXkgdGhlIHBhZ2Ugc2hvdWxkIGJlIHdyaXR0ZW4gb25jZSwgZXZlbiBqdXN0IHdyaXRlX29vYl9y YXcoKS4gIAo+IAo+IFNvcnJ5LCBZb3UgbWVhbiB0aGF0IGFmdGVyIE9PQiB3cml0ZSwgd2UgY2Fu bm90IHdyaXRlIHRvIHRoZSBkYXRhIGFyZWEgKGUuZy4gMHgwIC4uIDB4ODEwKSB1bnRpbCBwYWdl IHdpbGwgYmUgZXJhc2VkPyBGb3IgZXhhbXBsZQo+IEpGRlMyIHdyaXRlcyB0byBPT0Igb3duIG1h cmtlcnMsIHRoZW4gaXQgdHJpZXMgdG8gd3JpdGUgdG8gdGhlIGRhdGEgYXJlYSBvZiBzdWNoIHBh Z2UuCgpBIHBhZ2UgaXMgd3JpdHRlbiBhZnRlciB0d28gc3RlcHM6Ci0gbG9hZGluZyB0aGUgZGF0 YSBpbnRvIHRoZSBOQU5EIGNoaXAgY2FjaGUgKHRoYXQncyB3aGVuIHlvdSB1c2UgdGhlCiAgYnVz KQotIHByb2dyYW1taW5nIHRoZSBOQU5EIGFycmF5IHdpdGggdGhlIGRhdGEgbG9hZGVkIGluIGNh Y2hlICh0aGF0J3Mgd2hlbgogIHlvdSB3YWl0KQoKSW4gdGhlb3J5IGl0IGRvZXMgbm90IG1hdHRl ciB3aGVyZSB5b3Ugd3JpdGUgaW4gdGhlIGNhY2hlLCBpdCdzIHJlZ3VsYXIKRFJBTSwgeW91IGNh biBtYWtlIHJhbmRvbSB3cml0ZXMgdGhlcmUgd2l0aCB0aGUgYXBwcm9wcmlhdGUgTkFORApjb21t YW5kcy4gT2YgY291cnNlIHdoZW4gdXNpbmcgZW1iZWRkZWQgaGFyZHdhcmUgRUNDIGVuZ2luZXMs IHRoZQpjb250cm9sbGVycyB1c3VhbGx5IGV4cGVjdCB0byBiZSBmZWQgaW4gYSBjZXJ0YWluIHdh eSBpbiBvcmRlciB0bwpwcm9kdWNlIHRoZSBFQ0MgYnl0ZXMgYW5kIHB1dCB0aGVtIGF0IHRoZSBy aWdodCBsb2NhdGlvbiBpbiBjYWNoZS4KCkFuZCB0aGVuLCB3aGVuIHlvdSBhY3R1YWxseSBzZW5k IHRoZSAicHJvZ3JhbSIgY29tbWFuZCwgdGhlIE5BTkQgY2VsbHMKYWN0dWFsbHkgZ2V0IHByb2dy YW1tZWQgYmFzZWQgb24gd2hhdCBoYXMgYmVlbiBsb2FkZWQgaW4gY2FjaGUuCgpUaGFua3MsCk1p cXXDqGwKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxp bnV4LWFtbG9naWMgbWFpbGluZyBsaXN0CmxpbnV4LWFtbG9naWNAbGlzdHMuaW5mcmFkZWFkLm9y ZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFtbG9n aWMK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 856D9C77B75 for ; Tue, 18 Apr 2023 13:25:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=CDfWKKzipCXy4rN15siy6CB7+d8qiQECIz5EIFFtBN0=; b=gsOHout3lHlk1r nR3IgqQr127NPT81MsrzFzbjR8G5J1auO4xTRzNNhioTZ7UKMrL0m4ApqV7N+NgY+Kd/bmncWDYbD Vc2JbmbzU2yr5YKQM/mjJUQPc6m1scDfdOQyZ3veA2mO3dzvoXgv6YnPpd6jn339kAjqFZg7lG4d8 UjdyUDRGtxPtMYYsbqCGeJh67D1j2c7H+FyLZz2djhMhhYmPvRrPOcDmGDmHfeT9cFsa9spjznjyM AwYjyHMZqDeDdZrqup7P4cvjitUUUclb03tzdtvumsE1HODEu/4hbowexn2xv5rN5xh7/+dqj2A7i hFqPjsSMXecSL3ZW5mMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1polKk-002GVF-1x; Tue, 18 Apr 2023 13:25:22 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1polKf-002GTP-0f; Tue, 18 Apr 2023 13:25:19 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 0EAEF240002; Tue, 18 Apr 2023 13:25:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1681824311; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=a4ugRgZNu/Y+baSK+9UqHRS8pZ9Nhe2pwx9QcHGO7qQ=; b=oLiGo4PRzihRAJLpguz+PlMJwr7om8taIquydk8GT6iIDMZI0eemdwaItHjQju6h9yYRQd 75TGkfQ8FlW6TunA7ncve2pBpkxeh68RNtsyTsEMakpYGhuz9QQncJPNnBOx++2x/35XZ1 YfYmIV25hOM8VU3hNyOJ5clsnDrwKF1x96n56h6yc+W2TQtmS+0TTr6pA8TQTOPpXjsCnA zoqCCNCrtjKuPxquL55ja8iNzy/LyBUZj0GqExbz00XaqkjOEp8Z8gzZ3cL7HakbKpfwQ8 OoX2fR9wc0DFvtNY3NIjUaSsybzkgTBGfJQG4BQlE7ZTAo/GQECnlYk4Mr95SA== Date: Tue, 18 Apr 2023 15:25:05 +0200 From: Miquel Raynal To: Arseniy Krasnov Cc: Liang Yang , Richard Weinberger , Vignesh Raghavendra , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Jianxin Pan , Yixun Lan , , , , , , , "yonghui.yu" Subject: Re: [PATCH v1 4/5] mtd: rawnand: meson: clear OOB buffer before read Message-ID: <20230418152505.72fc16da@xps-13> In-Reply-To: References: <20230412061700.1492474-1-AVKrasnov@sberdevices.ru> <20230412094400.3c82f631@xps-13> <20230412113654.183350d0@xps-13> <4eace0a0-f6af-7d99-a52f-7913a2139330@sberdevices.ru> <20230412141824.755b2bca@xps-13> <20230412145715.58c2be4a@xps-13> <7c996832-258f-001c-56bd-87bbdf23eeaa@amlogic.com> <20230412163214.0c764bb3@xps-13> <0c61eaae-053e-5768-a533-70b2ff0cc95d@amlogic.com> <7520e512-8a19-ff04-364c-b5be0a579ef0@sberdevices.ru> <20230413102200.309fbe9c@xps-13> <20230413122252.0a8efcd8@xps-13> <569a948e-654a-b21f-8a4f-55dc4b295387@sberdevices.ru> <60fa656e-bda1-1de6-a79e-3e3041cd69a8@sberdevices.ru> <780c0cae-18b6-2652-1c2c-6d398ea60451@amlogic.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230418_062517_718003_63473675 X-CRM114-Status: GOOD ( 23.92 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org SGkgQXJzZW5peSwKCj4gPj4gSGVsbG8gYWdhaW4gQExpYW5nIEBNaXF1ZWwhCj4gPj4KPiA+PiBP bmUgbW9yZSBxdWVzdGlvbiBhYm91dCBPT0IgYWNjZXNzLCBhcyBJIGNhbiBzZWUgY3VycmVudCBk cml2ZXIgdXNlcyB0aGUgZm9sbG93aW5nCj4gPj4gY2FsbGJhY2tzOgo+ID4+Cj4gPj4gwqDCoMKg wqBuYW5kLT5lY2Mud3JpdGVfb29iX3JhdyA9IG5hbmRfd3JpdGVfb29iX3N0ZDsKPiA+PiDCoMKg wqDCoG5hbmQtPmVjYy53cml0ZV9vb2IgPSBuYW5kX3dyaXRlX29vYl9zdGQ7Cj4gPj4KPiA+Pgo+ ID4+IEZ1bmN0aW9uICduYW5kX3dyaXRlX29vYl9zdGQoKScgd3JpdGVzIGRhdGEgdG8gdGhlIGVu ZCBvZiB0aGUgcGFnZS4gQnV0IGFzIEkKPiA+PiBjYW4gc2VlIGJ5IGR1bXBpbmcgJ2RhdGFfYnVm JyBkdXJpbmcgcmVhZCwgcGh5c2ljYWwgbGF5b3V0IG9mIGVhY2ggcGFnZSBpcyB0aGUKPiA+PiBm b2xsb3dpbmcgKDFLQiBFQ0MpOgo+ID4+Cj4gPj4gMHgwMDA6IFvCoMKgwqDCoMKgwqDCoMKgIDEg S0Igb2YgZGF0YcKgwqDCoMKgwqDCoMKgIF0KPiA+PiAweDQwMDogWyAyQiB1c2VyIGRhdGFdIFsg MTRCIEVDQyBjb2RlXQo+ID4+IDB4NDEwOiBbwqDCoMKgwqDCoMKgwqDCoCAxIEtCIG9mIGRhdGHC oMKgwqDCoMKgwqDCoCBdwqDCoMKgIChBKQo+ID4+IDB4ODEwOiBbIDJCIHVzZXIgZGF0YV0gWyAx NEIgRUNDIGNvZGVdCj4gPj4gMHg4MjA6IFvCoMKgwqDCoMKgwqDCoCAzMkIgdW51c2VkwqDCoMKg wqDCoMKgwqDCoMKgwqAgXQo+ID4+Cj4gPj4KPiA+Pgo+ID4+IFNvLCBhZnRlciAnbmFuZF93cml0 ZV9vb2Jfc3RkKCknIChsZXQgZGF0YSBiZSBzZXF1ZW5jZSBmcm9tIFsweDAgLi4uIDB4M2ZdKSwK PiA+PiBwYWdlIHdpbGwgbG9vayBsaWtlIHRoaXM6Cj4gPj4KPiA+PiAweDAwMDogW8KgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoCAweEZGwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4gMHg0MDA6 IFvCoMKgwqDCoMKgwqDCoMKgwqDCoCAuLi4uLi4uLsKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4g MHg3ZjA6IFvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgMHhGRsKgwqDCoMKgwqDCoMKgwqDCoMKg wqAgXQo+ID4+IDB4ODAwOiBbIDAwIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uwqAgXQo+ID4+IDB4 ODMwOiBbIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLiAzZiBdCj4gPj4KPiA+PiBIZXJlIHdlIGhh dmUgdHdvIHByb2JsZW1zOgo+ID4+IDEpIEF0dGVtcHQgdG8gZGlzcGxheSByYXcgZGF0YSBieSAn bmFuZGR1bXAnIHV0aWxpdHkgcHJvZHVjZXMgYSBsaXR0bGUgYml0Cj4gPj4gwqDCoMKgIGludmFs aWQgb3V0cHV0LCBhcyBkcml2ZXIgcmVsaWVzIG9uIGxheW91dCAoQSkgZnJvbSBhYm92ZS4gRS5n LiBPT0IgZGF0YQo+ID4+IMKgwqDCoCBpcyBhdCAweDQwMCBhbmQgMHg4MTAuIEhlcmUgaXMgYW4g ZXhhbXBsZSAoYXR0ZW1wdCB0byB3cml0ZSAweDExIDB4MjIgMHgzMyAweDQ0KToKPiA+Pgo+ID4+ IDB4MDAwMDA3ZjA6IDExIDIyIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZm IGZmwqAgfC4iLi4uLi4uLi4uLi4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwK PiA+PiDCoMKgIE9PQiBEYXRhOiAzMyA0NCBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBm ZiBmZiBmZiBmZsKgIHwzRC4uLi4uLi4uLi4uLi4ufAo+ID4+IMKgwqAgT09CIERhdGE6IGZmIGZm IGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmwqAgfC4uLi4uLi4uLi4u Li4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwKPiA+PiAgCj4gPiBIaSBBcnNl bml5LAo+ID4gCj4gPiBJIHJlYWxpemVkIHRoZSB3cml0ZV9vb2JfcmF3KCkgYW5kIHdyaXRlX29v YigpIGFyZSB3cm9uZyBpbiBtZXNvbl9uYW5kLmMuIEkgc3VnZ2VzdCBib3RoIG9mIHRoZW0gc2hv dWxkIGJlIHJld29ya2VkIGFuZCBmb2xsb3cgdGhlIGZvcm1hdCBvZiBtZXNvbiBuYW5kIGNvbnRy b2xsZXIuIGkuZS4gZmlyc3RseSBmb3JtYXQgdGhlIGRhdGEgaW4gTGF5b3V0IChBKSBhbmQgdGhl biB3cml0ZS4gcmVhZGluZyBpcyBmaXJzdGx5IHJlYWRpbmcgdGhlIGRhdGEgb2YgbGF5b3V0IChB KSBhbmQgdGhlbiBjb21wb3N0IHRoZSBsYXlvdXQgKEIpLiAgCj4gCj4gSUlVQyBhZnRlciBzdWNo IHdyaXRpbmcgb25seSBPT0IgKGUuZy4gdXNlciBieXRlcykgYWNjb3JkaW5nIGxheW91dCAoQSks IGh3IHdpbGwgYWxzbyB3cml0ZSBFQ0MgY29kZXMsIHNvCj4gaXQgd2lsbCBiZSBpbXBvc3NpYmxl IHRvIHdyaXRlIGRhdGEgdG8gdGhpcyBwYWdlIGxhdGVyLCBiZWNhdXNlIHdlIGNhbm5vdCB1cGRh dGUgRUNDIGNvZGVzIHByb3Blcmx5IGZvciB0aGUgbmV3bHkKPiB3cml0dGVuIGRhdGEgKHdlIGNh bid0IHVwZGF0ZSBiaXRzIGZyb20gMCB0byAxKS4KPiAKPiA+IAo+ID4gICAKPiA+Pgo+ID4+IDIp IEF0dGVtcHQgdG8gcmVhZCBkYXRhIGluIEVDQyBtb2RlIHdpbGwgZmFpbCwgYmVjYXVzZSBJSVVD IHBhZ2UgaXMgaW4gZGlydHkKPiA+PiDCoMKgwqAgc3RhdGUgKEkgbWVhbiB3YXMgd3JpdHRlbiBh dCBsZWFzdCBvbmNlKSBhbmQgTkFORCBjb250cm9sbGVyIHRyaWVzIHRvIHVzZQo+ID4+IMKgwqDC oCBFQ0MgY29kZXMgYXQgMHg0MDAgYW5kIDB4ODEwLCB3aGljaCBhcmUgb2J2aW91c2x5IGJyb2tl biBpbiB0aGlzIGNhc2UuIFRodXMgIAo+ID4gCj4gPiBBcyBpIHNhaWQgYWJvdmUsIHdyaXRlX29v Yl9yYXcoKSBhbmQgd3JpdGVfb29iKCkgc2hvdWxkIGJlIHJld29ya2VkLgo+ID4gaSBkb24ndCBr bm93IHdoYXQgZG8geW91IG1lYW4gcGFnZSB3YXMgd3JpdHRlbiBhdCBsZWFzdCBvbmNlLiBhbnl3 YXkgdGhlIHBhZ2Ugc2hvdWxkIGJlIHdyaXR0ZW4gb25jZSwgZXZlbiBqdXN0IHdyaXRlX29vYl9y YXcoKS4gIAo+IAo+IFNvcnJ5LCBZb3UgbWVhbiB0aGF0IGFmdGVyIE9PQiB3cml0ZSwgd2UgY2Fu bm90IHdyaXRlIHRvIHRoZSBkYXRhIGFyZWEgKGUuZy4gMHgwIC4uIDB4ODEwKSB1bnRpbCBwYWdl IHdpbGwgYmUgZXJhc2VkPyBGb3IgZXhhbXBsZQo+IEpGRlMyIHdyaXRlcyB0byBPT0Igb3duIG1h cmtlcnMsIHRoZW4gaXQgdHJpZXMgdG8gd3JpdGUgdG8gdGhlIGRhdGEgYXJlYSBvZiBzdWNoIHBh Z2UuCgpBIHBhZ2UgaXMgd3JpdHRlbiBhZnRlciB0d28gc3RlcHM6Ci0gbG9hZGluZyB0aGUgZGF0 YSBpbnRvIHRoZSBOQU5EIGNoaXAgY2FjaGUgKHRoYXQncyB3aGVuIHlvdSB1c2UgdGhlCiAgYnVz KQotIHByb2dyYW1taW5nIHRoZSBOQU5EIGFycmF5IHdpdGggdGhlIGRhdGEgbG9hZGVkIGluIGNh Y2hlICh0aGF0J3Mgd2hlbgogIHlvdSB3YWl0KQoKSW4gdGhlb3J5IGl0IGRvZXMgbm90IG1hdHRl ciB3aGVyZSB5b3Ugd3JpdGUgaW4gdGhlIGNhY2hlLCBpdCdzIHJlZ3VsYXIKRFJBTSwgeW91IGNh biBtYWtlIHJhbmRvbSB3cml0ZXMgdGhlcmUgd2l0aCB0aGUgYXBwcm9wcmlhdGUgTkFORApjb21t YW5kcy4gT2YgY291cnNlIHdoZW4gdXNpbmcgZW1iZWRkZWQgaGFyZHdhcmUgRUNDIGVuZ2luZXMs IHRoZQpjb250cm9sbGVycyB1c3VhbGx5IGV4cGVjdCB0byBiZSBmZWQgaW4gYSBjZXJ0YWluIHdh eSBpbiBvcmRlciB0bwpwcm9kdWNlIHRoZSBFQ0MgYnl0ZXMgYW5kIHB1dCB0aGVtIGF0IHRoZSBy aWdodCBsb2NhdGlvbiBpbiBjYWNoZS4KCkFuZCB0aGVuLCB3aGVuIHlvdSBhY3R1YWxseSBzZW5k IHRoZSAicHJvZ3JhbSIgY29tbWFuZCwgdGhlIE5BTkQgY2VsbHMKYWN0dWFsbHkgZ2V0IHByb2dy YW1tZWQgYmFzZWQgb24gd2hhdCBoYXMgYmVlbiBsb2FkZWQgaW4gY2FjaGUuCgpUaGFua3MsCk1p cXXDqGwKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwpMaW51eCBNVEQgZGlzY3Vzc2lvbiBtYWlsaW5nIGxpc3QKaHR0cDovL2xpc3RzLmluZnJh ZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1tdGQvCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1E334C77B75 for ; Tue, 18 Apr 2023 13:26:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tyre0O7o6sxSdOizv5xyV44CtQjfozMDh78LNrg1eXY=; b=R6yDHxC1FOGr24 dt8HXtPvXCoAtF9sVXjC2PpZEIrPRjsnRnEok0l9R0dZh0DmF/PWjLfM3AXG6T3Q8zWScMK7uKZXg whSUBLtjt1KMUA+TcDc/i6mNOsWQYcDvMySfIeotK913yTuvpBhHPdMcmmTyHAn0F6f0rR48Tra6w Bw41bICGMsSmoaw3643DahbRnwX65ChWefbpqPJOB05FuWLlbCPp/z9iW0AzPyUy0vSrlRosv08dN zc8xikRFkAw0uriTDTjLelPRYMNu4z+cGieDQcaWphmO/UB3kES2Ezunzd2oa6+PVMoO5sGnukKWQ KOO++z5ehT6/iRHHP8XA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1polKj-002GV4-2B; Tue, 18 Apr 2023 13:25:21 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1polKf-002GTP-0f; Tue, 18 Apr 2023 13:25:19 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 0EAEF240002; Tue, 18 Apr 2023 13:25:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1681824311; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=a4ugRgZNu/Y+baSK+9UqHRS8pZ9Nhe2pwx9QcHGO7qQ=; b=oLiGo4PRzihRAJLpguz+PlMJwr7om8taIquydk8GT6iIDMZI0eemdwaItHjQju6h9yYRQd 75TGkfQ8FlW6TunA7ncve2pBpkxeh68RNtsyTsEMakpYGhuz9QQncJPNnBOx++2x/35XZ1 YfYmIV25hOM8VU3hNyOJ5clsnDrwKF1x96n56h6yc+W2TQtmS+0TTr6pA8TQTOPpXjsCnA zoqCCNCrtjKuPxquL55ja8iNzy/LyBUZj0GqExbz00XaqkjOEp8Z8gzZ3cL7HakbKpfwQ8 OoX2fR9wc0DFvtNY3NIjUaSsybzkgTBGfJQG4BQlE7ZTAo/GQECnlYk4Mr95SA== Date: Tue, 18 Apr 2023 15:25:05 +0200 From: Miquel Raynal To: Arseniy Krasnov Cc: Liang Yang , Richard Weinberger , Vignesh Raghavendra , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Jianxin Pan , Yixun Lan , , , , , , , "yonghui.yu" Subject: Re: [PATCH v1 4/5] mtd: rawnand: meson: clear OOB buffer before read Message-ID: <20230418152505.72fc16da@xps-13> In-Reply-To: References: <20230412061700.1492474-1-AVKrasnov@sberdevices.ru> <20230412094400.3c82f631@xps-13> <20230412113654.183350d0@xps-13> <4eace0a0-f6af-7d99-a52f-7913a2139330@sberdevices.ru> <20230412141824.755b2bca@xps-13> <20230412145715.58c2be4a@xps-13> <7c996832-258f-001c-56bd-87bbdf23eeaa@amlogic.com> <20230412163214.0c764bb3@xps-13> <0c61eaae-053e-5768-a533-70b2ff0cc95d@amlogic.com> <7520e512-8a19-ff04-364c-b5be0a579ef0@sberdevices.ru> <20230413102200.309fbe9c@xps-13> <20230413122252.0a8efcd8@xps-13> <569a948e-654a-b21f-8a4f-55dc4b295387@sberdevices.ru> <60fa656e-bda1-1de6-a79e-3e3041cd69a8@sberdevices.ru> <780c0cae-18b6-2652-1c2c-6d398ea60451@amlogic.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230418_062517_718003_63473675 X-CRM114-Status: GOOD ( 23.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGkgQXJzZW5peSwKCj4gPj4gSGVsbG8gYWdhaW4gQExpYW5nIEBNaXF1ZWwhCj4gPj4KPiA+PiBP bmUgbW9yZSBxdWVzdGlvbiBhYm91dCBPT0IgYWNjZXNzLCBhcyBJIGNhbiBzZWUgY3VycmVudCBk cml2ZXIgdXNlcyB0aGUgZm9sbG93aW5nCj4gPj4gY2FsbGJhY2tzOgo+ID4+Cj4gPj4gwqDCoMKg wqBuYW5kLT5lY2Mud3JpdGVfb29iX3JhdyA9IG5hbmRfd3JpdGVfb29iX3N0ZDsKPiA+PiDCoMKg wqDCoG5hbmQtPmVjYy53cml0ZV9vb2IgPSBuYW5kX3dyaXRlX29vYl9zdGQ7Cj4gPj4KPiA+Pgo+ ID4+IEZ1bmN0aW9uICduYW5kX3dyaXRlX29vYl9zdGQoKScgd3JpdGVzIGRhdGEgdG8gdGhlIGVu ZCBvZiB0aGUgcGFnZS4gQnV0IGFzIEkKPiA+PiBjYW4gc2VlIGJ5IGR1bXBpbmcgJ2RhdGFfYnVm JyBkdXJpbmcgcmVhZCwgcGh5c2ljYWwgbGF5b3V0IG9mIGVhY2ggcGFnZSBpcyB0aGUKPiA+PiBm b2xsb3dpbmcgKDFLQiBFQ0MpOgo+ID4+Cj4gPj4gMHgwMDA6IFvCoMKgwqDCoMKgwqDCoMKgIDEg S0Igb2YgZGF0YcKgwqDCoMKgwqDCoMKgIF0KPiA+PiAweDQwMDogWyAyQiB1c2VyIGRhdGFdIFsg MTRCIEVDQyBjb2RlXQo+ID4+IDB4NDEwOiBbwqDCoMKgwqDCoMKgwqDCoCAxIEtCIG9mIGRhdGHC oMKgwqDCoMKgwqDCoCBdwqDCoMKgIChBKQo+ID4+IDB4ODEwOiBbIDJCIHVzZXIgZGF0YV0gWyAx NEIgRUNDIGNvZGVdCj4gPj4gMHg4MjA6IFvCoMKgwqDCoMKgwqDCoCAzMkIgdW51c2VkwqDCoMKg wqDCoMKgwqDCoMKgwqAgXQo+ID4+Cj4gPj4KPiA+Pgo+ID4+IFNvLCBhZnRlciAnbmFuZF93cml0 ZV9vb2Jfc3RkKCknIChsZXQgZGF0YSBiZSBzZXF1ZW5jZSBmcm9tIFsweDAgLi4uIDB4M2ZdKSwK PiA+PiBwYWdlIHdpbGwgbG9vayBsaWtlIHRoaXM6Cj4gPj4KPiA+PiAweDAwMDogW8KgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoCAweEZGwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4gMHg0MDA6 IFvCoMKgwqDCoMKgwqDCoMKgwqDCoCAuLi4uLi4uLsKgwqDCoMKgwqDCoMKgwqDCoCBdCj4gPj4g MHg3ZjA6IFvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgMHhGRsKgwqDCoMKgwqDCoMKgwqDCoMKg wqAgXQo+ID4+IDB4ODAwOiBbIDAwIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uwqAgXQo+ID4+IDB4 ODMwOiBbIC4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLiAzZiBdCj4gPj4KPiA+PiBIZXJlIHdlIGhh dmUgdHdvIHByb2JsZW1zOgo+ID4+IDEpIEF0dGVtcHQgdG8gZGlzcGxheSByYXcgZGF0YSBieSAn bmFuZGR1bXAnIHV0aWxpdHkgcHJvZHVjZXMgYSBsaXR0bGUgYml0Cj4gPj4gwqDCoMKgIGludmFs aWQgb3V0cHV0LCBhcyBkcml2ZXIgcmVsaWVzIG9uIGxheW91dCAoQSkgZnJvbSBhYm92ZS4gRS5n LiBPT0IgZGF0YQo+ID4+IMKgwqDCoCBpcyBhdCAweDQwMCBhbmQgMHg4MTAuIEhlcmUgaXMgYW4g ZXhhbXBsZSAoYXR0ZW1wdCB0byB3cml0ZSAweDExIDB4MjIgMHgzMyAweDQ0KToKPiA+Pgo+ID4+ IDB4MDAwMDA3ZjA6IDExIDIyIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZm IGZmwqAgfC4iLi4uLi4uLi4uLi4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwK PiA+PiDCoMKgIE9PQiBEYXRhOiAzMyA0NCBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBmZiBm ZiBmZiBmZiBmZsKgIHwzRC4uLi4uLi4uLi4uLi4ufAo+ID4+IMKgwqAgT09CIERhdGE6IGZmIGZm IGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmIGZmwqAgfC4uLi4uLi4uLi4u Li4uLi58Cj4gPj4gwqDCoCBPT0IgRGF0YTogZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYgZmYg ZmYgZmYgZmYgZmYgZmYgZmbCoCB8Li4uLi4uLi4uLi4uLi4uLnwKPiA+PiAgCj4gPiBIaSBBcnNl bml5LAo+ID4gCj4gPiBJIHJlYWxpemVkIHRoZSB3cml0ZV9vb2JfcmF3KCkgYW5kIHdyaXRlX29v YigpIGFyZSB3cm9uZyBpbiBtZXNvbl9uYW5kLmMuIEkgc3VnZ2VzdCBib3RoIG9mIHRoZW0gc2hv dWxkIGJlIHJld29ya2VkIGFuZCBmb2xsb3cgdGhlIGZvcm1hdCBvZiBtZXNvbiBuYW5kIGNvbnRy b2xsZXIuIGkuZS4gZmlyc3RseSBmb3JtYXQgdGhlIGRhdGEgaW4gTGF5b3V0IChBKSBhbmQgdGhl biB3cml0ZS4gcmVhZGluZyBpcyBmaXJzdGx5IHJlYWRpbmcgdGhlIGRhdGEgb2YgbGF5b3V0IChB KSBhbmQgdGhlbiBjb21wb3N0IHRoZSBsYXlvdXQgKEIpLiAgCj4gCj4gSUlVQyBhZnRlciBzdWNo IHdyaXRpbmcgb25seSBPT0IgKGUuZy4gdXNlciBieXRlcykgYWNjb3JkaW5nIGxheW91dCAoQSks IGh3IHdpbGwgYWxzbyB3cml0ZSBFQ0MgY29kZXMsIHNvCj4gaXQgd2lsbCBiZSBpbXBvc3NpYmxl IHRvIHdyaXRlIGRhdGEgdG8gdGhpcyBwYWdlIGxhdGVyLCBiZWNhdXNlIHdlIGNhbm5vdCB1cGRh dGUgRUNDIGNvZGVzIHByb3Blcmx5IGZvciB0aGUgbmV3bHkKPiB3cml0dGVuIGRhdGEgKHdlIGNh bid0IHVwZGF0ZSBiaXRzIGZyb20gMCB0byAxKS4KPiAKPiA+IAo+ID4gICAKPiA+Pgo+ID4+IDIp IEF0dGVtcHQgdG8gcmVhZCBkYXRhIGluIEVDQyBtb2RlIHdpbGwgZmFpbCwgYmVjYXVzZSBJSVVD IHBhZ2UgaXMgaW4gZGlydHkKPiA+PiDCoMKgwqAgc3RhdGUgKEkgbWVhbiB3YXMgd3JpdHRlbiBh dCBsZWFzdCBvbmNlKSBhbmQgTkFORCBjb250cm9sbGVyIHRyaWVzIHRvIHVzZQo+ID4+IMKgwqDC oCBFQ0MgY29kZXMgYXQgMHg0MDAgYW5kIDB4ODEwLCB3aGljaCBhcmUgb2J2aW91c2x5IGJyb2tl biBpbiB0aGlzIGNhc2UuIFRodXMgIAo+ID4gCj4gPiBBcyBpIHNhaWQgYWJvdmUsIHdyaXRlX29v Yl9yYXcoKSBhbmQgd3JpdGVfb29iKCkgc2hvdWxkIGJlIHJld29ya2VkLgo+ID4gaSBkb24ndCBr bm93IHdoYXQgZG8geW91IG1lYW4gcGFnZSB3YXMgd3JpdHRlbiBhdCBsZWFzdCBvbmNlLiBhbnl3 YXkgdGhlIHBhZ2Ugc2hvdWxkIGJlIHdyaXR0ZW4gb25jZSwgZXZlbiBqdXN0IHdyaXRlX29vYl9y YXcoKS4gIAo+IAo+IFNvcnJ5LCBZb3UgbWVhbiB0aGF0IGFmdGVyIE9PQiB3cml0ZSwgd2UgY2Fu bm90IHdyaXRlIHRvIHRoZSBkYXRhIGFyZWEgKGUuZy4gMHgwIC4uIDB4ODEwKSB1bnRpbCBwYWdl IHdpbGwgYmUgZXJhc2VkPyBGb3IgZXhhbXBsZQo+IEpGRlMyIHdyaXRlcyB0byBPT0Igb3duIG1h cmtlcnMsIHRoZW4gaXQgdHJpZXMgdG8gd3JpdGUgdG8gdGhlIGRhdGEgYXJlYSBvZiBzdWNoIHBh Z2UuCgpBIHBhZ2UgaXMgd3JpdHRlbiBhZnRlciB0d28gc3RlcHM6Ci0gbG9hZGluZyB0aGUgZGF0 YSBpbnRvIHRoZSBOQU5EIGNoaXAgY2FjaGUgKHRoYXQncyB3aGVuIHlvdSB1c2UgdGhlCiAgYnVz KQotIHByb2dyYW1taW5nIHRoZSBOQU5EIGFycmF5IHdpdGggdGhlIGRhdGEgbG9hZGVkIGluIGNh Y2hlICh0aGF0J3Mgd2hlbgogIHlvdSB3YWl0KQoKSW4gdGhlb3J5IGl0IGRvZXMgbm90IG1hdHRl ciB3aGVyZSB5b3Ugd3JpdGUgaW4gdGhlIGNhY2hlLCBpdCdzIHJlZ3VsYXIKRFJBTSwgeW91IGNh biBtYWtlIHJhbmRvbSB3cml0ZXMgdGhlcmUgd2l0aCB0aGUgYXBwcm9wcmlhdGUgTkFORApjb21t YW5kcy4gT2YgY291cnNlIHdoZW4gdXNpbmcgZW1iZWRkZWQgaGFyZHdhcmUgRUNDIGVuZ2luZXMs IHRoZQpjb250cm9sbGVycyB1c3VhbGx5IGV4cGVjdCB0byBiZSBmZWQgaW4gYSBjZXJ0YWluIHdh eSBpbiBvcmRlciB0bwpwcm9kdWNlIHRoZSBFQ0MgYnl0ZXMgYW5kIHB1dCB0aGVtIGF0IHRoZSBy aWdodCBsb2NhdGlvbiBpbiBjYWNoZS4KCkFuZCB0aGVuLCB3aGVuIHlvdSBhY3R1YWxseSBzZW5k IHRoZSAicHJvZ3JhbSIgY29tbWFuZCwgdGhlIE5BTkQgY2VsbHMKYWN0dWFsbHkgZ2V0IHByb2dy YW1tZWQgYmFzZWQgb24gd2hhdCBoYXMgYmVlbiBsb2FkZWQgaW4gY2FjaGUuCgpUaGFua3MsCk1p cXXDqGwKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxp bnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFk ZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4 LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7E449C77B75 for ; Tue, 18 Apr 2023 13:25:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232272AbjDRNZQ (ORCPT ); Tue, 18 Apr 2023 09:25:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46850 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229838AbjDRNZO (ORCPT ); Tue, 18 Apr 2023 09:25:14 -0400 Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B7149CC29 for ; Tue, 18 Apr 2023 06:25:12 -0700 (PDT) Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 0EAEF240002; Tue, 18 Apr 2023 13:25:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1681824311; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=a4ugRgZNu/Y+baSK+9UqHRS8pZ9Nhe2pwx9QcHGO7qQ=; b=oLiGo4PRzihRAJLpguz+PlMJwr7om8taIquydk8GT6iIDMZI0eemdwaItHjQju6h9yYRQd 75TGkfQ8FlW6TunA7ncve2pBpkxeh68RNtsyTsEMakpYGhuz9QQncJPNnBOx++2x/35XZ1 YfYmIV25hOM8VU3hNyOJ5clsnDrwKF1x96n56h6yc+W2TQtmS+0TTr6pA8TQTOPpXjsCnA zoqCCNCrtjKuPxquL55ja8iNzy/LyBUZj0GqExbz00XaqkjOEp8Z8gzZ3cL7HakbKpfwQ8 OoX2fR9wc0DFvtNY3NIjUaSsybzkgTBGfJQG4BQlE7ZTAo/GQECnlYk4Mr95SA== Date: Tue, 18 Apr 2023 15:25:05 +0200 From: Miquel Raynal To: Arseniy Krasnov Cc: Liang Yang , Richard Weinberger , Vignesh Raghavendra , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Jianxin Pan , Yixun Lan , , , , , , , "yonghui.yu" Subject: Re: [PATCH v1 4/5] mtd: rawnand: meson: clear OOB buffer before read Message-ID: <20230418152505.72fc16da@xps-13> In-Reply-To: References: <20230412061700.1492474-1-AVKrasnov@sberdevices.ru> <20230412094400.3c82f631@xps-13> <20230412113654.183350d0@xps-13> <4eace0a0-f6af-7d99-a52f-7913a2139330@sberdevices.ru> <20230412141824.755b2bca@xps-13> <20230412145715.58c2be4a@xps-13> <7c996832-258f-001c-56bd-87bbdf23eeaa@amlogic.com> <20230412163214.0c764bb3@xps-13> <0c61eaae-053e-5768-a533-70b2ff0cc95d@amlogic.com> <7520e512-8a19-ff04-364c-b5be0a579ef0@sberdevices.ru> <20230413102200.309fbe9c@xps-13> <20230413122252.0a8efcd8@xps-13> <569a948e-654a-b21f-8a4f-55dc4b295387@sberdevices.ru> <60fa656e-bda1-1de6-a79e-3e3041cd69a8@sberdevices.ru> <780c0cae-18b6-2652-1c2c-6d398ea60451@amlogic.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arseniy, > >> Hello again @Liang @Miquel! > >> > >> One more question about OOB access, as I can see current driver uses t= he following > >> callbacks: > >> > >> =C2=A0=C2=A0=C2=A0=C2=A0nand->ecc.write_oob_raw =3D nand_write_oob_std; > >> =C2=A0=C2=A0=C2=A0=C2=A0nand->ecc.write_oob =3D nand_write_oob_std; > >> > >> > >> Function 'nand_write_oob_std()' writes data to the end of the page. Bu= t as I > >> can see by dumping 'data_buf' during read, physical layout of each pag= e is the > >> following (1KB ECC): > >> > >> 0x000: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 1 KB of data= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ] > >> 0x400: [ 2B user data] [ 14B ECC code] > >> 0x410: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 1 KB of data= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ]=C2=A0=C2=A0=C2=A0 (A) > >> 0x810: [ 2B user data] [ 14B ECC code] > >> 0x820: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 32B unused=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ] > >> > >> > >> > >> So, after 'nand_write_oob_std()' (let data be sequence from [0x0 ... 0= x3f]), > >> page will look like this: > >> > >> 0x000: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0 0xFF=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0 ] > >> 0x400: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 .= .......=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ] > >> 0x7f0: [=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0 0xFF=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0 ] > >> 0x800: [ 00 .......................=C2=A0 ] > >> 0x830: [ ........................ 3f ] > >> > >> Here we have two problems: > >> 1) Attempt to display raw data by 'nanddump' utility produces a little= bit > >> =C2=A0=C2=A0=C2=A0 invalid output, as driver relies on layout (A) from= above. E.g. OOB data > >> =C2=A0=C2=A0=C2=A0 is at 0x400 and 0x810. Here is an example (attempt = to write 0x11 0x22 0x33 0x44): > >> > >> 0x000007f0: 11 22 ff ff ff ff ff ff ff ff ff ff ff ff ff ff=C2=A0 |.".= .............| > >> =C2=A0=C2=A0 OOB Data: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff= =C2=A0 |................| > >> =C2=A0=C2=A0 OOB Data: 33 44 ff ff ff ff ff ff ff ff ff ff ff ff ff ff= =C2=A0 |3D..............| > >> =C2=A0=C2=A0 OOB Data: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff= =C2=A0 |................| > >> =C2=A0=C2=A0 OOB Data: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff= =C2=A0 |................| > >> =20 > > Hi Arseniy, > >=20 > > I realized the write_oob_raw() and write_oob() are wrong in meson_nand.= c. I suggest both of them should be reworked and follow the format of meson= nand controller. i.e. firstly format the data in Layout (A) and then write= . reading is firstly reading the data of layout (A) and then compost the la= yout (B). =20 >=20 > IIUC after such writing only OOB (e.g. user bytes) according layout (A), = hw will also write ECC codes, so > it will be impossible to write data to this page later, because we cannot= update ECC codes properly for the newly > written data (we can't update bits from 0 to 1). >=20 > >=20 > > =20 > >> > >> 2) Attempt to read data in ECC mode will fail, because IIUC page is in= dirty > >> =C2=A0=C2=A0=C2=A0 state (I mean was written at least once) and NAND c= ontroller tries to use > >> =C2=A0=C2=A0=C2=A0 ECC codes at 0x400 and 0x810, which are obviously b= roken in this case. Thus =20 > >=20 > > As i said above, write_oob_raw() and write_oob() should be reworked. > > i don't know what do you mean page was written at least once. anyway th= e page should be written once, even just write_oob_raw(). =20 >=20 > Sorry, You mean that after OOB write, we cannot write to the data area (e= .g. 0x0 .. 0x810) until page will be erased? For example > JFFS2 writes to OOB own markers, then it tries to write to the data area = of such page. A page is written after two steps: - loading the data into the NAND chip cache (that's when you use the bus) - programming the NAND array with the data loaded in cache (that's when you wait) In theory it does not matter where you write in the cache, it's regular DRAM, you can make random writes there with the appropriate NAND commands. Of course when using embedded hardware ECC engines, the controllers usually expect to be fed in a certain way in order to produce the ECC bytes and put them at the right location in cache. And then, when you actually send the "program" command, the NAND cells actually get programmed based on what has been loaded in cache. Thanks, Miqu=C3=A8l