Hi, 2014-04-08 2:41 GMT+09:00 Tom Rini <tr...@ti.com>: > On Mon, Apr 07, 2014 at 08:53:39AM +0200, Wolfgang Denk wrote: >> Dear Nobuhiro Iwamatsu, >> >> In message >> <1396846600-15386-1-git-send-email-nobuhiro.iwamatsu...@renesas.com> you >> wrote: >> > Usually, when CONFIG_OF_LIBFDT is enabled, U-Boot is set to >> > the FDT memory information that is set in the U-Boot. This patch >> > disables this behavior. >> > >> > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu...@renesas.com> >> > --- >> > README | 8 ++++++++ >> > arch/arm/lib/bootm-fdt.c | 2 ++ >> > 2 files changed, 10 insertions(+) >> >> Please explain why you would want to do this. To me it makes no >> sense. Either U-Boot knows the correct memory size, then it should >> pass it to Linux. Or it does not, then U-Boot should be fixed. >> >> Also, I object that your implementation is ARM specific. If such a >> feature gets added, it should be architecture independent. > > This has shown up before in the context of platforms with >4GB memory > and we "correct" the node by reducing the system memory, which is > incorrect. I agree this needs to be done for more than just ARM > however.
It is one of the reasons of this patch that you wrote. Thanks, Nobuhiro > > -- > Tom > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > -- Nobuhiro Iwamatsu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
