On Fri, Mar 22, 2019 at 03:53:07PM +0800, Simon Glass wrote: > Hi Thierry, > > On Fri, 22 Mar 2019 at 02:10, Thierry Reding <thierry.red...@gmail.com> wrote: > > > > From: Thierry Reding <tred...@nvidia.com> > > > > Implement carveout tests for 32-bit and 64-bit builds. > > > > Signed-off-by: Thierry Reding <tred...@nvidia.com> > > --- > > Changes in v2: > > - new patch > > > > lib/fdtdec_test.c | 152 ++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 152 insertions(+) > > Reviewed-by: Simon Glass <s...@chromium.org> > > Just an idea - as an alternative you could use the built-in device > tree as your base rather than creating a new one. But perhaps that > would only be safe on sandbox?
Yeah, running that test on a live system would mess up the reserved memory regions. If U-Boot does something based on the reserved-memory node, or passes it on to the kernel, that's going to cause issues. The test also uses rather arbitrary values for the carveout, which may not point at system memory at all. Thierry Applied to u-boot-dm, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
