[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