Tom, On Tue, Apr 30, 2013 at 1:35 PM, Tom Rini <tr...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
