> Sent: Monday, August 19, 2019 2:25 PM
> To: Troy Kisky <troy.ki...@boundarydevices.com>; Fabio Estevam
> <fabio.este...@nxp.com>; Stefano Babic <sba...@denx.de>;
> u-boot@lists.denx.de; Jacky Bai <ping....@nxp.com>
> Subject: RE: imx: add i.MX8MQ EVK support
> 
> Loop Jacky who has more knowledge in the ddr stuff.
> 
> > Subject: re: imx: add i.MX8MQ EVK support
> >
> > Hi Peng
> >
> > In spl.c you have
> >
> > _________
> > static void spl_dram_init(void)
> > {
> >     /* ddr init */
> >     if ((get_cpu_rev() & 0xfff) == CHIP_REV_2_1)
> >             ddr_init(&dram_timing);
> >     else
> >             ddr_init(&dram_timing_b0);
> > }
> >
> > _________
> >
> > Could you explain why this is dependent on chip rev ?

It is due an errata in DDR PHY, for chip Rev lower than 2.1, set point < 667mts 
can NOT be supported

> >
> > It it just the extra frequency in lpddr4_timing.c ?
> >
> > __________
> >     {
> >             /* P1 100mts 1D */
> >             .drate = 100,
> >             .fw_type = FW_1D_IMAGE,
> >             .fsp_cfg = lpddr4_fsp2_cfg,
> >             .fsp_cfg_num = ARRAY_SIZE(lpddr4_fsp2_cfg),
> >     },
> > _________
> >
> >
> > Will other i.MX8MQ boards also need this check ?
> >

For most of the customer board, I think the chip Rev is 2.1, so no need such 
check.

> >
> > Thanks
> > Troy
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to