On Thu, Mar 28, 2019 at 4:55 PM Jagan Teki <ja...@amarulasolutions.com> wrote: > > On Mon, Mar 18, 2019 at 3:20 PM Shyam Saini > <shyam.sa...@amarulasolutions.com> wrote: > > > > From: Michael Trimarchi <mich...@amarulasolutions.com> > > > > While the exact problem is not known, based on discussion between > > Philipp Tomsich and André Przywara it is guessed that exit self-refresh > > timing is not set with correct value. There may be implicit enter or > > exit Self-Refresh anywhere as part of some training phase. > > > > In ZynqMP register guide [1], which is close to the various > > Allwinner DRAM controllers, tXSDLL is bits [14:8], while the non-DLL > > tXS is bits [6:0]: Self refresh exit delay. So it could be safely > > increased and it only affects the time after the self-refresh “exit”, > > which happens only after (re-)initialisation. > > > > There was no document for cpu in question so based on oscilloscope > > readings [2][3] and observed result by comparing allwinner architecture. > > So set it same as Allwinner H5 silicon. > > > > Before this patch, failure rate of was 7%. > > > > This was tested on A33 allwinner cpu, dual rank connection connected > > with two MT41K512M16HA-125:A memory model. Memory is configured as DDR3 > > 1.5V > > > > [1] https://www.xilinx.com/html_docs/registers/ug1087/ddrc___dramtmg8.html > > [2] https://ibb.co/R70zmyS > > [3] https://ibb.co/HVVCGQ8 > > > > Signed-off-by: Michael Trimarchi <mich...@amarulasolutions.com> > > Signed-off-by: Shyam Saini <shyam.sa...@amarulasolutions.com> > > --- > > Changelogs: > > V1->V2: adjust commit message > > V2->V3: Adjust commit message based on V2 Discussion > > --- > > Acked-by: Jagan Teki <ja...@openedev.com>
Applied to u-boot-sunxi/master _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot