From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nobuhiro Iwamatsu Date: Tue, 8 Apr 2014 10:50:01 +0900 Subject: [U-Boot] [PATCH] lib: Add CONFIG_FDT_IGNORE_FIXUP_MEMORY_NODE In-Reply-To: <20140407065339.499CC3804B4@gemini.denx.de> References: <1396846600-15386-1-git-send-email-nobuhiro.iwamatsu.yj@renesas.com> <20140407065339.499CC3804B4@gemini.denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, Thanks for your comment. 2014-04-07 15:53 GMT+09:00 Wolfgang Denk : > Dear Nobuhiro Iwamatsu, > > In message <1396846600-15386-1-git-send-email-nobuhiro.iwamatsu.yj@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 >> --- >> 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. For example, I can access the memory of all in the U-Boot, but I may want to control the highmem on Linux,I do not want to show a specific area from kernel and userland. > > Also, I object that your implementation is ARM specific. If such a > feature gets added, it should be architecture independent. I see. But arch_fixup_memory_node() is used by ARM only. So, we see to be dependent on the ARM is only this. > > Thanks. > > Wolfgang Denk > > -- > DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de > To be a winner, all you need to give is all you have. > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot -- Nobuhiro Iwamatsu