[U-Boot] [PATCH] fdt_support: Use CONFIG_NR_DRAM_BANKS if defined
Doug Anderson
dianders at chromium.org
Tue Apr 30 22:49:02 CEST 2013
Tom,
On Tue, Apr 30, 2013 at 1:35 PM, Tom Rini <trini at ti.com> wrote:
> And I guess having this knowledge correct for the kernel is useful in
> other contexts like when we want to power down some banks of memory
> but not others? I mean, there's "lots" of platforms that lie and say
> 1 bank since we require contiguous mapping. Thanks!
Thanks for the review!
At the moment I'm _not_ convinced that there's a good reason to
specify 8 banks. We appear to have lied and said 1 bank on
exynos5250-snow (ARM Chromebook) and I don't know of any bad side
effects.
The code I'm looking at right now indicates 8 banks. We need to track
down why someone did that but it doesn't seem totally crazy to allow
specifying the proper number of banks so I figured I'd send this patch
up.
If you prefer, we can leave this patch hanging until we actually track
down if specifying 8 banks was really needed.
-Doug
More information about the U-Boot
mailing list