Re: [U-Boot] [U-Boot,7/8] ARM: highbank: remove DRAM bank setup
On Sat, Jun 13, 2015 at 12:01:37PM +0100, Andre Przywara wrote: > On Fri, 12 Jun 2015 22:10:07 -0400 > Tom Rini wrote: > > > On Fri, Jun 05, 2015 at 12:58:48AM +0100, Andre Przywara wrote: > > > > > From: Rob Herring > > > > > > On the highbank platform the SoC's management controller firmware > > > will probe the DRAM modules and populates the initial device tree > > > with the correct values. Therefore the memory sizes in are already > > > correct, so remove U-Boot's DRAM bank setup so the memory node is > > > not "fixed up" by u-boot. > > > > > > Signed-off-by: Rob Herring > > > Signed-off-by: Andre Przywara > > > > Applied to u-boot/master, thanks! > > Hi Tom, > > thanks for applying the patches! > I sent out a reworked version of patch 6/8, which saw a comment from > Simon. Please consider applying the new one _or_ revert this very patch > here (7/8), as it alone breaks Calxeda machines (which now end up with > an empty memory/reg property instead of an untouched one). > Sorry for the mess! "Good" news! I somehow failed to push things when I said I did, so this was all still just local. I've dropped this patch for now as I want Simon to chime in on your updated patch. -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [U-Boot,7/8] ARM: highbank: remove DRAM bank setup
On Fri, 12 Jun 2015 22:10:07 -0400 Tom Rini wrote: > On Fri, Jun 05, 2015 at 12:58:48AM +0100, Andre Przywara wrote: > > > From: Rob Herring > > > > On the highbank platform the SoC's management controller firmware > > will probe the DRAM modules and populates the initial device tree > > with the correct values. Therefore the memory sizes in are already > > correct, so remove U-Boot's DRAM bank setup so the memory node is > > not "fixed up" by u-boot. > > > > Signed-off-by: Rob Herring > > Signed-off-by: Andre Przywara > > Applied to u-boot/master, thanks! Hi Tom, thanks for applying the patches! I sent out a reworked version of patch 6/8, which saw a comment from Simon. Please consider applying the new one _or_ revert this very patch here (7/8), as it alone breaks Calxeda machines (which now end up with an empty memory/reg property instead of an untouched one). Sorry for the mess! Cheers, Andre. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [U-Boot,7/8] ARM: highbank: remove DRAM bank setup
On Fri, Jun 05, 2015 at 12:58:48AM +0100, Andre Przywara wrote: > From: Rob Herring > > On the highbank platform the SoC's management controller firmware > will probe the DRAM modules and populates the initial device tree with > the correct values. Therefore the memory sizes in are already correct, > so remove U-Boot's DRAM bank setup so the memory node is not "fixed > up" by u-boot. > > Signed-off-by: Rob Herring > Signed-off-by: Andre Przywara Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot