Ziyuan, I retested the kernel loading. I'm using 3.14 kernel. I also tested with linux-4.7-rc6. I still require both the cache reverts and disabling efi_loader on u-boot/master and u-boot-rockchip/master. (see below for testing parameters)
Hans, I have been unable to boot an RK3288 rock2 square board since v2016.03 release. The kernel fails to start. Before reverting I get this on my console: Starting kernel ... ### ERROR ### Please RESET the board ### Another possibly related problem which has similar effect without the ERROR potion above is a call to invalidate_icache_all() from efi_runtime.c. Commenting out this line or disabled CONFIG_EFI_LOADER seems to allow the kernel to boot. testing: board: radxa rock2 square kernel: 3.14 custom, and linux-4.7-rc6 - multi_v7_defconfig zImage rk3288-radxa-rock.dtb toolchain: gcc-linaro-5.3-2016.02-x86_64_arm-linux-gnueabi loading from sd card - the actual loading seems fine though. I'm not sure how emmc would change anything. booting command for 4.7 (we use and care about 3.14, but i thought it would be easier to show with 4.7-rc6. I get the same behavior with 3.14): setenv bootargs root=/dev/mmcblk0p4 ext4load mmc 0:3 0x02000000 /zImage; ext4load mmc 0:3 0x01f00000 /rk3288-rock2-square.dtb bootz 0x02000000 - 0x01f00000 u-boot/master: 95d52733036af7438a5285d729d53844ec48c63e CONFIG_EFI_LOADER=n and cache reverts required to load kernel *_high changes don't seem to be required anymore u-boot-rockchip: 1781acd31b01225402167fbdb8fa97157cc269eb CONFIG_EFI_LOADER=n and cache reverts required to load kernel *_high changes don't seem to be required anymore Sandy Patterson On Thu, Jul 21, 2016 at 11:21 PM, Simon Glass <s...@chromium.org> wrote: > +Hans, Tom > > Hi Sandy, > > On 15 July 2016 at 07:40, Sandy Patterson <apatter...@sightlogix.com> > wrote: > > Hi Simon, > > > > I think I screwed up submitting, and didn't cc the maintainers for the > > reverts. I can resubmit and get patman to behave. What do you suggest? > > > > I still think this is the best patch to getting rock2 to load kernels > again. > > > > I just retested with a patch specific to my kernel. I think 3.14 linux > has > > some quirks in how it wants the dram setup, and I get instability without > > some changes to the dram init. I don't have enough of a system to test > for > > this instability using the latest mainline kernel, so I left that patch > out. > > > > Ziyuan had an alternative fix for efi_loader, but it looks like it may > break > > efi_loader for others, and I don't know how to test the efi > functionality, > > so I think disabling for RK3288 is best. I think I should move the > > CONFIG_EFI_LOADER patch to somehow patching rk3288_common.h instead (can > I > > #undef config vars in that file?) > > Yes please can you resend and cc the maintainers and original author. > Patman -t should do this automatically. > > Also with the revert of Han's ARM patches, please can you describe the > problem so it is clear why we need to revert? > > Regards, > Simon > > [...] > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot