On Tue, Mar 5, 2019 at 9:00 AM Michael Nazzareno Trimarchi <mich...@amarulasolutions.com> wrote: > > Hi Tim > > On Tue, Mar 5, 2019 at 5:54 PM Tim Harvey <thar...@gateworks.com> wrote: > > > > On Tue, Mar 5, 2019 at 8:41 AM Michael Nazzareno Trimarchi > > <mich...@amarulasolutions.com> wrote: > > > > > > HI all > > > > > > On Mon, Feb 4, 2019 at 7:14 PM Tim Harvey <thar...@gateworks.com> wrote: > > > > > > > > On Sat, Feb 2, 2019 at 5:30 AM Adam Ford <aford...@gmail.com> wrote: > > > > > > > > > > On Sat, Feb 2, 2019 at 12:29 AM Jagan Teki > > > > > <ja...@amarulasolutions.com> wrote: > > > > > > > > > > > > +Adam, Shyam > > > > > > > > > > > > On Sat, 2 Feb, 2019, 8:49 AM Stefan Agner <stefan.ag...@toradex.com > > > > > > wrote: > > > > > > > > > > > > > Hi Tim, > > > > > > > > > > > > > > On 02.02.19 03:32, Tim Harvey wrote: > > > > > > > > Stefan, > > > > > > > > > > > > > > > > I'm trying to track down an IMX6 SPL NAND boot regression that > > > > > > > > started > > > > > > > > in v2018.07 with your patch series to mxs_nand. > > > > > > > > > > > > > > I am sorry about that. Unfortunately I did not had a design at > > > > > > > hand where > > > > > > > I was able to test the NAND driver in SPL... > > > > > > > > > > > > > > > > > > > > > > > I bisected it back to '5346c31e305a37d39f535cc0d5ae87d8b7e81230 > > > > > > > > mtd: > > > > > > > > nand: mxs_nand: use self init'. With this particular patch nand > > > > > > > > bbt > > > > > > > > scanning would crash the board because of nand_chip.scan_btt > > > > > > > > not being > > > > > > > > assigned. This was later fixed in > > > > > > > > '96d0be07e7498e7174daa6f3b56fc807b9feb71d MTD: nand: > > > > > > > > mxs_nand_spl: Fix > > > > > > > > empty function pointer for BBT' but cherry-picking that on top > > > > > > > > of > > > > > > > > 5346c31 fixes the immediate crash while scanning but then I > > > > > > > > find that > > > > > > > > mxs_read_page_ecc() hangs on the 4th page of reading u-boot.img > > > > > > > > from > > > > > > > > the NAND. This gets worse 2 patches later where in > > > > > > > > '28897e8d21f8e197e259a91c693de09cd81f2d5a: mtd: nand: mxs_nand: > > > > > > > > use > > > > > > > > structure for BCH geometry' I find that the first byte of every > > > > > > > > page > > > > > > > > read is wrong because mxs_nand_swap_block_mark() is getting > > > > > > > > called on > > > > > > > > the page which swaps the first bytes with oob. > > > > > > > > > > > > > > > > There are several IMX6 boards out there using both NAND and SPL > > > > > > > > I > > > > > > > > believe that I would assume were broken by this series. Any > > > > > > > > ideas on > > > > > > > > the proper resolution? > > > > > > > > > > > > > > > > > > > Look like 2017.03 can be stable boot from nand as for as my test is > > > > > > concern. > > > > > > > > > > > > We are also trying hard using git bisect, but seems like multiple > > > > > > breakings. > > > > > > > > > > > > Will keep posted if something move further. > > > > > > > > > > > > > > > From a different thread, someone was able to test these patches and > > > > > found they fixed their booting issues: > > > > > > > > > > There was a broken function pointer here that was fixed and applied > > > > > the imx-master, but pending merge with master > > > > > http://patchwork.ozlabs.org/patch/1019440/ > > > > > > > > right, this one was the 96d0be07 merged upstream already. I'm not > > > > clear how any NAND SPL would have worked after Sefan's series back in > > > > June without this one. > > > > > > > > > > >>SPL: board_init_r() > > > spl_init > > > Trying to boot from NAND > > > spl: nand - using hw ecc > > > 0x2c:0xdc erasesize=262144 (>>18) > > > writesize=4096 (>>12) > > > oobsize=224 > > > chipsize=536870912 > > > nand_spl_load_image offset:0x00200000 len:64 page:512 > > > SPL: payload image: U-Boot 2019.04-rc3-dirty for imx� load addr: > > > 0x177fffc0 size: 635435 > > > nand_spl_load_image offset:0x00200000 len:635435 page:512 > > > is_badblock offs=0x00240000 block:9 page:576 > > > is_badblock offs=0x00280000 block:10 page:640 > > > Jumping to U-Boot > > > loaded - jumping to U-Boot... > > > image entry point: 0x17800000 > > > > > > I'm stuck here. Booting from mmc is fine > > > > > > Michael > > > > Michael, > > > > What code are you running (git sha) and what board? > > > > Make sure you have 04568bd0b6 MTD: nand: mxs_nand: Allow driver to > > auto setup ECC in SPL > > > > Top of master and I have that commit included. Can I know one imx6 board now > that can boot from Nand? >
Michael, I just built v2019.04-rc3 for gwventana_nand_defconfig and booted it fine on a board with a 2GiB Cypress NAND flash. This uses NAND and SPL (and would not boot before the mentioned patch). Tim _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
