All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kever Yang <kever.yang@rock-chips.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/2] Add sdram capacity auto detect for rk3288
Date: Tue, 20 Sep 2016 11:34:23 +0800	[thread overview]
Message-ID: <57E0AE3F.2000609@rock-chips.com> (raw)
In-Reply-To: <CABCGj3aZivdYyHei5JzMA53MxWAqWC+0v+JjGe1RWzvBkE67XA@mail.gmail.com>

Hi Sandy,

On 09/19/2016 09:33 PM, Sandy Patterson wrote:
> Hi Kever and Vagrant,
>
> I tested this patch on my rock2 board against 2016.09. (I didn't have 
> my board with me over the weekend)

Thanks for your test and for your log, my source code do have one bug 
and I have fix and upload the V2 patch set.

>
> 1) It seems the BACK_TO_BROM feature is broken. It doesn't boot using 
> the 2016.09. I'm afraid I don't have the time to track it down.
It works fine on my evb, do you flash the image correctly? it's 
different with the way you don't use BACK_TO_BROM.

Regards,
- Kever
> 2) It seems the SPL is still small enough to test this feature. 
> (vagrant you can forget about the back to brom feature unless you need 
> more SPL space).
>
> 3) I get the same behavior as Vagrant on the rock2 board. I'm 
> including DEBUG output in case that helps Kever. I tried to trace it, 
> but it seems a larger undertaking than I can manage right now. 
> (vagrant, you can add DEBUG #define's into the 
> file include/configs/firefly-rk3288.h to get the output below.. my 
> guess is it's similar.)
>
> This trace is almost exactly the same between enabling the SDRAM 
> detection and not. I marked the point where it halts. The fdt is a 
> slightly different size, but that is expected.
>
> Sandy
>
>
> initcall: 0010af55
> U-Boot code: 00100000 -> 0014F8C0  BSS: -> 0018D1B8
> initcall: 0010add1
> initcall: 0010b359
> Model: Radxa Rock 2 Square
> initcall: 0010af95
> DRAM:  initcall: 00100d19
> malloc_simple: size=3c, ptr=7d8, limit=2000: fe79c
> uclass_find_device_by_seq: 0 -1
> uclass_find_device_by_seq: 0 0
>    - -1 -1
>    - not found
> fdtdec_get_int_array: interrupts
> get_prop_check_min_len: interrupts
> malloc_simple: size=4, ptr=7dc, limit=2000: fe7d8
> uclass_find_device_by_seq: 0 -1
> uclass_find_device_by_seq: 0 0
>    - -1 1
>    - -1 -1
>    - -1 0
>    - found
> uclass_find_device_by_seq: 0 1
>    - -1 1
>    - found
> uclass_find_device_by_seq: 0 2
>    - -1 1
>    - -1 -1
>    - -1 0
>    - -1 -1
>    - not found
> fdtdec_get_int_array: interrupts
> get_prop_check_min_len: interrupts
> malloc_simple: size=14, ptr=7f0, limit=2000: fe7dc
> malloc_simple: size=4, ptr=7f4, limit=2000: fe7f0
> uclass_find_device_by_seq: 0 -1
> uclass_find_device_by_seq: 0 0
>    - -1 1
>    - -1 -1
>    - -1 0
>    - found
> uclass_find_device_by_seq: 0 1
>    - -1 1
>    - found
> uclass_find_device_by_seq: 0 2
>    - -1 1
>    - -1 -1
>    - -1 0
>    - -1 2
>    - found
> uclass_find_device_by_seq: 0 3
>    - -1 1
>    - -1 -1
>    - -1 0
>    - -1 2
>    - not found
> fdtdec_get_int_array: interrupts
> get_prop_check_min_len: interrupts
> malloc_simple: size=14, ptr=808, limit=2000: fe7f4
> malloc_simple: size=c, ptr=814, limit=2000: fe808
> uclass_find_device_by_seq: 0 -1
> uclass_find_device_by_seq: 0 0
>    - -1 -1
>    - not found
> fdtdec_get_int_array: interrupts
> get_prop_check_min_len: interrupts
> OF: ** translation for device clock-controller at ff760000 **
> OF: bus is default (na=1, ns=1) on
> OF: translating address: 000076ff
> OF: reached root node
> clk_request(dev=000fe474, clk=000fe7bc)
> SDRAM base=0, size=80000000
> initcall: 0010b129
> Monitor len: 0008D1B8
> Ram size: 80000000
> Ram top: 80000000
> initcall: 0010adf1
> initcall: 0010af0d
> TLB table from 7fff0000 to 7fff4000
> initcall: 0010af75
> video_reserve: Reserving 4f0000 bytes at 7fb00000 for video device 
> 'vop at ff940000'
> Video frame buffers from 7fb00000 to 7fff0000
> initcall: 0010ae05
> initcall: 0010aecd
> Reserving 564k for U-Boot at: 7fa72000
> initcall: 0010aea5
> Reserving 32776k for malloc() at: 7da70000
> initcall: 0010b059
> Reserving 80 Bytes for Board Info at: 7da6ffb0
> initcall: 0010ae09
> initcall: 0010ae7d
> Reserving 200 Bytes for Global Data at: 7da6fee8
> initcall: 0010ae2d
> Reserving 38464 Bytes for FDT at: 7da668a8
> initcall: 0010ae11
> initcall: 0010b18d
> initcall: 0010b113
> initcall: 0010b099
>
> RAM Configuration:
> ====== SDRAM DETECTION FEATURE HALTS HERE ========
> Bank #0: 0 2 GiB
>
> DRAM:  2 GiB
> initcall: 0010ae19
> New Stack Pointer is: 7da66880
> initcall: 0010b02d
> initcall: 0010afd5
> Relocation Offset is: 7f972000
> Relocating to 7fa72000, new gd at 7da6fee8, sp at 7da66880
> initcall: 7fa7d1cd
> initcall: 7fa7d1d1
> initcall: 0010b32d (relocated to 7fa7d32d)
> dram_bank_mmu_setup: bank: 0
> initcall: 0010b305 (relocated to 7fa7d305)
> efi_runtime_relocate: Relocating to offset=7fa72000
>

  reply	other threads:[~2016-09-20  3:34 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-08 13:04 [U-Boot] [PATCH 0/2] Add sdram capacity auto detect for rk3288 Kever Yang
2016-09-08 13:04 ` [U-Boot] [PATCH 1/2] rk3288: sdram: auto-detect the capacity Kever Yang
2016-09-08 13:04 ` [U-Boot] [PATCH 2/2] dts: rk3288: remove node in dmc which not need anymore Kever Yang
2016-09-08 19:28 ` [U-Boot] [PATCH 0/2] Add sdram capacity auto detect for rk3288 Vagrant Cascadian
2016-09-10 13:36   ` Ziyuan Xu
2016-09-10 19:01     ` Vagrant Cascadian
2016-09-12  2:01       ` Ziyuan Xu
2016-09-15 12:00         ` Sandy Patterson
2016-09-18  2:18           ` Kever Yang
2016-09-18 19:34           ` Vagrant Cascadian
2016-09-19 13:33             ` Sandy Patterson
2016-09-20  3:34               ` Kever Yang [this message]
2016-09-20 13:20                 ` Sandy Patterson

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=57E0AE3F.2000609@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.