From: Kever Yang <kever.yang@rock-chips.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288
Date: Wed, 21 Sep 2016 10:56:40 +0800 [thread overview]
Message-ID: <57E1F6E8.1040201@rock-chips.com> (raw)
In-Reply-To: <CABCGj3bcb16XP=w85vCtxspQQNoRi2sDOyn2nN0_1JwGMhW=FQ@mail.gmail.com>
Hi Sandy,
On 09/20/2016 11:04 PM, Sandy Patterson wrote:
> I can confirm that this boots on rock2 with current master branch.
> 9b1b6d42256a4c2e59c803afdbf90d39371e61ba
>
Thanks for your test.
> It doesn't boot for me using rock2 on v2016.09 tag. I get:
> U-Boot SPL 2016.09-00002-g690a8a3 (Sep 20 2016 - 10:47:44)
> Trying to boot from MMC1
>
> It continues to detect 2GB of ram. on master.
>
> I get nervous with these memory changes. We found that some memory
> parameters persisted through to linux and had the ability to cause
> instability. I don't run have linux setup right now on my rock2, so I
> can't test that.
>
parameters changes from dts to auto-detect including those I removed
from dts and ddrconfig, stride, they should be the same as without my
patch, which means my patch suppose to not change any parameter for DDR
other than how we get those parameters.
> I also wanted to test that the parameters detected via the new routine
> matched those removed from the dts file. I tried enabling DEBUG but
> that somehow caused no console output anymore (on master). I'm not
> sure what's going on. I also tried doing some printfs but they had the
> same effect. It looks like somehow the console init has moved and
> maybe isn't setup yet.
I think you need to define EARLY_DEBUG in
arch/arm/mach-rockchip/rk3288-board-spl.c and then using printf or debug
with DEBUG macro enable.
Thanks,
- Kever
>
> Sandy
>
>
>
>
next prev parent reply other threads:[~2016-09-21 2:56 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-20 3:28 [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288 Kever Yang
2016-09-20 3:28 ` [U-Boot] [PATCH v2 1/3] rk3288: config change for enable dram capacity auto-detect Kever Yang
2016-09-20 13:21 ` Sandy Patterson
2016-09-21 3:00 ` Kever Yang
2016-09-21 11:53 ` Sandy Patterson
2016-09-22 2:10 ` Kever Yang
2016-09-20 3:28 ` [U-Boot] [PATCH v2 2/3] rk3288: sdram: auto-detect the capacity Kever Yang
2016-09-23 2:53 ` Simon Glass
2016-10-07 9:14 ` Kever Yang
2016-10-06 0:28 ` Vagrant Cascadian
2016-09-20 3:28 ` [U-Boot] [PATCH v2 3/3] dts: rk3288: remove node in dmc which not need anymore Kever Yang
2016-09-23 2:58 ` Simon Glass
2016-10-06 0:28 ` Vagrant Cascadian
2016-09-20 15:04 ` [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288 Sandy Patterson
2016-09-21 2:56 ` Kever Yang [this message]
2016-09-21 12:41 ` Sandy Patterson
2016-09-22 2:15 ` Kever Yang
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=57E1F6E8.1040201@rock-chips.com \
--to=kever.yang@rock-chips.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