-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 04/30/2013 04:49 PM, Doug Anderson wrote: > 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.
Yes please, lets hold. Thanks! - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRgDQoAAoJENk4IS6UOR1WztkP+QEs7IvExh9Dq0AHrj81wQ9Q Ml29BZGsdJ5mLIt6jhJ7HSr310cu3FODgbVuNt01Aj0Q2X+C1mCRYqhoIDwfcSUJ EWVhUaphlmiBd2OrMH+3HPUwQ+kFfjt5LNFuXwRei0tgz+sy6NTQ+QZFuZ9FiBJD UKtavOsvd3XipdklU5UEGoBj6OJxU6hBOyehZ3Cckwgfeg0L/1uV07Vd8kSFFc5e xoWXN7O+QkdlNkWeruxPF7uq1MeM2VusCuvGWK4srrED+WSAKFhqsi7t3N66iNny lXDhYPtuSr5HF5xua4kwWdbM/GneVd5m0p979TvIwvwhM1bMr00mfIoH9HEjzNF6 Bvq0wcCwIEZLwBFNNpn9X9zIzwXIgUKbMqjHQXiuizY8LROdXXnkg53k9o2pDO5+ uGO8cKZMXJYEU4zW+wbSlI/Cz7WoylsXhSBPfF5gkRSIxKtYmcS/iQn/nKMgebVO TaGx76/r8xOvA5WY+wCs7HMEJip5UU00rG7MvjokwxOSUf/2rVHiDWl0MEAlh7M4 4KAMzb61P/fUiXrZv5K9Z6sgPmGynjItKnw0UigTWKG6DvRy0HuOlF//O8qAuWKH +eyjg2F24pS9cGRMni3M9cUBH1W6secIpZkqs3goxeNVZyfb29kswolymfbcU4GC zXmnz8gBTLDKGtTzLlXC =s42z -----END PGP SIGNATURE----- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot