From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Wed, 2 Sep 2015 13:04:33 -0700 Subject: [U-Boot] [PATCH] Revert "fdt: Fix fdtdec_get_addr_size() for 64-bit" In-Reply-To: References: <1438560830-31221-1-git-send-email-sjg@chromium.org> <55BF84E3.7010108@wwwdotorg.org> <55BFB0D7.1040104@wwwdotorg.org> <55C25482.3010409@wwwdotorg.org> <55C3AF8C.2040102@wwwdotorg.org> Message-ID: <55E75651.3040509@nvidia.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 09/02/2015 09:52 AM, Tom Warren wrote: > Simon, et al, > >> Simon Glass wrote at Friday, August 14, 2015 3:05 AM: >> I plan to apply this revert to u-boot-x86 (where SPI is currently >> broken) and (once it has a bit more testing) also this patch which I think makes >> the change in a safer way: >> >> https://patchwork.ozlabs.org/patch/504918/ >> >> At present that patch breaks at least one x86 board and I have not dug into it >> yet. >> >> The revert should not break tegra, according to Stephen. > > Unfortunately, my testing on P2571 with TOT u-boot-tegra (rebased against TOT u-boot/master this morning) shows that that is not true. > > The revert of the disputed 'fdtdec_get_addr_size' patch _does_ break Tegra 64-bit (P2571, at least). Nyan-big is OK. With Simon's revert in place, my board just loops on SPL signon, so I assume it's faulting, etc. in CPU init. Note that this is the current state of TOT u-boot/master. I'm a bit confused. So far, we don't support SPL on T210 since we assume some other bootloader runs on the boot CPU and starts just the main U-Boot on the main CPU. It sounds like you're testing some local-only SPL support?