Hi Magnus, On Thu, Nov 17, 2016 at 9:51 AM, Magnus Damm <magnus.damm@xxxxxxxxx> wrote: > On Thu, Nov 17, 2016 at 5:31 PM, Geert Uytterhoeven > <geert@xxxxxxxxxxxxxx> wrote: >> On Thu, Nov 17, 2016 at 3:28 AM, Magnus Damm <magnus.damm@xxxxxxxxx> wrote: >>> First of all, we might have slightly different view of the hardware, >>> so this might need some further discussions. Also, this topic in my >>> mind is mainly about DT integration code merge ordering for r8a7796 to >>> enable >4GB memory access early on without introducing any potential >>> issues. >> >> Note that >4GB memory is already enabled on r8a7796 by U-Boot. > > Right, can you remind me - did we get any conclusion about how to > handle the memory ranges in DTS and the ones from u-boot? > > It would be good with a consistent plan how to handle such. I think we should just apply "arm64: dts: r8a7796: salvator-x: Update memory node to 4 GiB map". IMHO it doesn't make much sense to pretend the memory is not present nor enabled. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds