Hello Sam, On 2/4/19 20:01, Sam Ravnborg wrote: > Hi Ahmad. > > On Mon, Apr 01, 2019 at 12:18:22PM +0200, Ahmad Fatoum wrote: >> The newly added device tree based first stage fails to load the second >> stage from MMC, which might be in relation to a preceding atmel_mci >> "command/data timeout" message. > > I had similar troubles with MMC on at91sam9263. > After some digging the fix was simple: > See 5feabc1e6737742f1cf6a1c41921f68b4dbb5c10 ("arm: at91: fix clock to mci1 for at91sam9263") Might be clock related. But if I need to fit drivers into the PBL as well, device tree in first-stage won't be viable anyway, so I probably won't look further into this. > > Maybe you are hit by something as simple as this? > >> + >> +static const struct devfs_partition sama5d3_xplained_nand0_partitions[] = { >> + { >> + .offset = 0x00000, >> + .size = SZ_256K, >> + .flags = DEVFS_PARTITION_FIXED, >> + .name = "at91bootstrap_raw", >> + .bbname = "at91bootstrap", >> + }, > Strange partition name now we use barebox as first stage. Indeed. > > >> +++ b/arch/arm/configs/microchip_ksz9477_evb_bootstrap_mmc_defconfig > > Naming confuses me. > Do we stick to the name "first stage" or do we use "bootstrap"? > For me they are the same, but I get confused when we use both. For the at91sam9261ek the bootstrap defconfig is the one defining AT91_BOOTSTRAP, so I figured I should follow suit.. > > > Sam > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox