> -----Original Message----- > From: Scott Wood [mailto:o...@buserror.net] > Sent: Tuesday, January 12, 2016 1:47 AM > To: Qianyu Gong <qianyu.g...@nxp.com>; u-boot@lists.denx.de > Cc: b07...@freescale.com; b48...@freescale.com; > wenbin.s...@freescale.com; Mingkai Hu <mingkai...@nxp.com> > Subject: Re: [U-Boot] [Patch V3 3/3] armv8/ls1043aqds: add QSPI boot support > > On Mon, 2016-01-11 at 10:17 +0800, Gong Qianyu wrote: > > diff --git a/board/freescale/ls1043aqds/ls1043aqds.c > > b/board/freescale/ls1043aqds/ls1043aqds.c > > index d6696ca..770b79f 100644 > > --- a/board/freescale/ls1043aqds/ls1043aqds.c > > +++ b/board/freescale/ls1043aqds/ls1043aqds.c > > @@ -43,15 +43,19 @@ enum { > > > > int checkboard(void) > > { > > +#ifndef CONFIG_QSPI_BOOT > > char buf[64]; > > #ifndef CONFIG_SD_BOOT > > u8 sw; > > #endif > > +#endif > > > > puts("Board: LS1043AQDS, boot from "); > > > > #ifdef CONFIG_SD_BOOT > > puts("SD\n"); > > +#elif defined(CONFIG_QSPI_BOOT) > > + puts("QSPI\n"); > > #else > > sw = QIXIS_READ(brdcfg[0]); > > sw = (sw & QIXIS_LBMAP_MASK) >> QIXIS_LBMAP_SHIFT; @@ -68,12 > +72,15 > > @@ int checkboard(void) > > printf("invalid setting of SW%u\n", QIXIS_LBMAP_SWITCH); > #endif > > > > +#ifndef CONFIG_QSPI_BOOT > > + /* For QSPI boot, here I2C is not ready yet. */ > > printf("Sys ID: 0x%02x, Sys Ver: 0x%02x\n", > > QIXIS_READ(id), QIXIS_READ(arch)); > > > > printf("FPGA: v%d (%s), build %d\n", > > (int)QIXIS_READ(scver), qixis_read_tag(buf), > > (int)qixis_read_minor()); > > +#endif > > Why isn't i2c ready? How is DDR inited without it? > > -Scott
Hi Scott, The calling sequence in U-Boot is : checkboard() -> init_func_i2c() -> dram_init() So I2C is not ready in checkboard() but is ready for DDR initialization. Regards, Qianyu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
