From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp11.smtpout.orange.fr ([80.12.242.133] helo=smtp.smtpout.orange.fr) by bombadil.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1eSTfe-00074Z-DV for linux-mtd@lists.infradead.org; Fri, 22 Dec 2017 20:11:57 +0000 From: Robert Jarzmik To: Boris Brezillon Cc: Miquel RAYNAL , David Woodhouse , Brian Norris , Marek Vasut , Richard Weinberger , Cyrille Pitchen , Rob Herring , Mark Rutland , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Russell King , Daniel Mack , Haojian Zhuang , Eric Miao , Catalin Marinas , Will Deacon , Ezequiel Garcia , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , Antoine Tenart , Nadav Haklai , Ofer Heifetz , Hanna Hawa , Neta Zur Hershkovits Subject: Re: [PATCH 00/12] Marvell NAND controller rework with ->exec_op() References: <20171207201814.30411-1-miquel.raynal@free-electrons.com> <20171214070930.0b885f6d@bbrezillon> <877etkecig.fsf@belgarion.home> <20171218092535.2ca1fe13@xps13> <87y3lxccr7.fsf@belgarion.home> <20171220224121.2cb6f690@bbrezillon> Date: Fri, 22 Dec 2017 21:11:32 +0100 In-Reply-To: <20171220224121.2cb6f690@bbrezillon> (Boris Brezillon's message of "Wed, 20 Dec 2017 22:41:21 +0100") Message-ID: <87lghucykr.fsf@belgarion.home> MIME-Version: 1.0 Content-Type: text/plain List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Boris Brezillon writes: > Looks like there is a mismatch on the nand bus width detected by the > core and the one declared by the driver. Can you try with the following > diff applied? Sure. Now I get a lot of these message which I didn't have before : [ 26.897372] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 126976 bytes from PEB 242:4096, read only 126976 bytes, retry [ 26.928559] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 126976 bytes from PEB 242:4096, read only 126976 bytes, retry [ 26.959623] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 126976 bytes from PEB 242:4096, read only 126976 bytes, retry [ 26.990714] ubi0 error: ubi_io_read: error -74 (ECC error) while reading 126976 bytes from PEB 242:4096, read 126976 bytes [ 27.002084] CPU: 0 PID: 1 Comm: swapper Not tainted 4.15.0-rc1-00041-ge371e97 #737 [ 27.009674] Hardware name: PXA3xx Platform Development Kit (aka Zylonite) [ 27.016750] [] (unwind_backtrace) from [] (show_stack+0x10/0x14) [ 27.024729] [] (show_stack) from [] (ubi_io_read+0x124/0x368) [ 27.032453] [] (ubi_io_read) from [] (ubi_attach+0xbd8/0x1868) [ 27.040148] [] (ubi_attach) from [] (ubi_attach_mtd_dev+0x548/0xe58) [ 27.048473] [] (ubi_attach_mtd_dev) from [] (ubi_init+0x134/0x1d4) [ 27.056623] [] (ubi_init) from [] (do_one_initcall+0x3c/0x17c) [ 27.064466] [] (do_one_initcall) from [] (kernel_init_freeable+0x104/0x1c0) [ 27.073398] [] (kernel_init_freeable) from [] (kernel_init+0x8/0xf8) [ 27.081715] [] (kernel_init) from [] (ret_from_fork+0x14/0x34) [ 27.090115] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 64 bytes from PEB 243:2048, read only 64 bytes, retry [ 27.102363] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 64 bytes from PEB 243:2048, read only 64 bytes, retry [ 27.114586] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading 64 bytes from PEB 243:2048, read only 64 bytes, retry [ 27.126813] ubi0 error: ubi_io_read: error -74 (ECC error) while reading 64 bytes from PEB 243:2048, read 64 bytes [ 27.137416] CPU: 0 PID: 1 Comm: swapper Not tainted 4.15.0-rc1-00041-ge371e97 #737 [ 27.145143] Hardware name: PXA3xx Platform Development Kit (aka Zylonite) [ 27.152207] [] (unwind_backtrace) from [] (show_stack+0x10/0x14) [ 27.160037] [] (show_stack) from [] (ubi_io_read+0x124/0x368) [ 27.167742] [] (ubi_io_read) from [] (ubi_io_read_vid_hdr+0x60/0x3ac) [ 27.176156] [] (ubi_io_read_vid_hdr) from [] (ubi_attach+0x4e0/0x1868) [ 27.184675] [] (ubi_attach) from [] (ubi_attach_mtd_dev+0x548/0xe58) [ 27.193010] [] (ubi_attach_mtd_dev) from [] (ubi_init+0x134/0x1d4) [ 27.201159] [] (ubi_init) from [] (do_one_initcall+0x3c/0x17c) [ 27.208839] [] (do_one_initcall) from [] (kernel_init_freeable+0x104/0x1c0) [ 27.217755] [] (kernel_init_freeable) from [] (kernel_init+0x8/0xf8) [ 27.226074] [] (kernel_init) from [] (ret_from_fork+0x14/0x34) Cheers. -- Robert