From: York Sun <york.sun@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] armv8/ls2080ardb: Update DDR timing to support more UDIMMs
Date: Thu, 5 May 2016 12:29:18 -0700 [thread overview]
Message-ID: <572B9F0E.7050302@nxp.com> (raw)
In-Reply-To: <1460011290-28704-1-git-send-email-Shengzhou.Liu@nxp.com>
On 04/06/2016 11:50 PM, Shengzhou Liu wrote:
> Optimize DDR timing for good margins to support new Transcend
> and Apacer DDR4 UDIMM besides current Micron UDIMM.
>
> Verified 1333MT/s, 1600MT/s, 1866MT/s, 2133MT/s rate with
> following UDIMM on LS2080ARDB.
> - Micron UDIMM: MTA18ASF1G72AZ-2G1A1Z
> - Apacer UDIMM: 78.C1GM4.AF10B
> - Transcend UDIMM: TS1GLH72V1H
>
> Signed-off-by: Shengzhou Liu <Shengzhou.Liu@nxp.com>
> ---
> v2: verified lower rate, for 1333MT/s no changes are necessary.
>
> board/freescale/ls2080ardb/ddr.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/board/freescale/ls2080ardb/ddr.h b/board/freescale/ls2080ardb/ddr.h
> index bda9d4a..b3c6306 100644
> --- a/board/freescale/ls2080ardb/ddr.h
> +++ b/board/freescale/ls2080ardb/ddr.h
> @@ -29,9 +29,9 @@ static const struct board_specific_parameters udimm0[] = {
> * ranks| mhz| GB |adjst| start | ctl2 | ctl3
> */
> {2, 1350, 0, 4, 6, 0x0708090B, 0x0C0D0E09,},
> - {2, 1666, 0, 4, 8, 0x08090B0D, 0x0E10100C,},
> - {2, 1900, 0, 4, 8, 0x090A0C0E, 0x1012120D,},
> - {2, 2300, 0, 4, 9, 0x0A0B0C10, 0x1114140E,},
> + {2, 1666, 0, 5, 9, 0x090A0B0E, 0x0F11110C,},
> + {2, 1900, 0, 6, 0xA, 0x0B0C0E11, 0x1214140F,},
> + {2, 2300, 0, 6, 0xB, 0x0C0D0F12, 0x14161610,},
> {}
> };
>
>
Shengzhou,
I see a problem with this patch on one LS2085ARDB board, at 1866MT/s with
MT18ASF1G72AZ-2G1A1. Please try it yourself on ls2085rdb-3 (ATX boardfarm). My
order of patches are
http://patchwork.ozlabs.org/patch/607290/
http://patchwork.ozlabs.org/patch/598135/
http://patchwork.ozlabs.org/patch/598136/
http://patchwork.ozlabs.org/patch/607274/
My test code base is master branch bbca7108db79076d3a9a9c112792d7c4608a665c.
York
next prev parent reply other threads:[~2016-05-05 19:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-07 6:41 [U-Boot] [PATCH v2] armv8/ls2080ardb: Update DDR timing to support more UDIMMs Shengzhou Liu
2016-05-05 19:29 ` York Sun [this message]
2016-05-17 16:25 ` York Sun
2016-05-18 20:27 ` York Sun
2016-05-27 11:28 ` Shengzhou Liu
2016-05-24 17:06 ` York Sun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=572B9F0E.7050302@nxp.com \
--to=york.sun@nxp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox