[coreboot-gerrit] Change in coreboot[master]: payloads/LinuxBoot: Fix remaining issues

[Philipp Deppenwiese (Code Review)]

Philipp Deppenwiese has uploaded this change for review. ( https://review.coreboot.org/28401


Change subject: payloads/LinuxBoot: Fix remaining issues
......................................................................

payloads/LinuxBoot: Fix remaining issues

* Update kernel versions
* Fix initrd path handling
* Use build/fit.dtb as default

Change-Id: Ia0641fd0c0db1e47ee4829b73871af662d4a6370
Signed-off-by: Philipp Deppenwiese <zaolin at das-labor.org>
---
M payloads/external/LinuxBoot/Kconfig
M payloads/external/LinuxBoot/Makefile
M payloads/external/LinuxBoot/targets/u-root.mk
M payloads/external/Makefile.inc
4 files changed, 14 insertions(+), 13 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/01/28401/1

diff --git a/payloads/external/LinuxBoot/Kconfig b/payloads/external/LinuxBoot/Kconfig
index 7a40645..2193ec7 100644
--- a/payloads/external/LinuxBoot/Kconfig
+++ b/payloads/external/LinuxBoot/Kconfig
@@ -50,12 +50,12 @@
 	default LINUXBOOT_KERNEL_STABLE
 
 config LINUXBOOT_KERNEL_STABLE
-	bool "4.15.3"
+	bool "4.14.67"
 	help
 	  Stable kernel version
 
 config LINUXBOOT_KERNEL_LATEST
-	bool "4.17.11"
+	bool "4.18.5"
 	help
 	  Latest kernel version
 
@@ -63,8 +63,8 @@
 
 config LINUXBOOT_KERNEL_VERSION
 	string
-	default "4.17.11" if LINUXBOOT_KERNEL_LATEST
-	default "4.15.3" if LINUXBOOT_KERNEL_STABLE
+	default "4.18.5" if LINUXBOOT_KERNEL_LATEST
+	default "4.14.67" if LINUXBOOT_KERNEL_STABLE
 
 config LINUXBOOT_KERNEL_CONFIGFILE
 	string "Kernel config file"
@@ -82,7 +82,7 @@
 config LINUXBOOT_DTB_FILE
 	string "Compiled devicetree file"
 	depends on LINUXBOOT_ARM64
-	default ""
+	default "build/fit.dtb"
 
 config PAYLOAD_FILE
 	default "payloads/external/LinuxBoot/linuxboot/kernel-image"
diff --git a/payloads/external/LinuxBoot/Makefile b/payloads/external/LinuxBoot/Makefile
index f259373..c051389 100644
--- a/payloads/external/LinuxBoot/Makefile
+++ b/payloads/external/LinuxBoot/Makefile
@@ -77,26 +77,26 @@
 $(project_dir)/vmlinux.bin: $(kernel_dir)/vmlinux
 	$(OBJCOPY) -O binary $< $@
 
-$(project_dir)/target.dtb: $(CONFIG_LINUXBOOT_DTB_FILE)
+$(project_dir)/target.dtb: $(PWD)/$(CONFIG_LINUXBOOT_DTB_FILE)
 	cp $< $@
 
 $(project_dir)/vmlinux.bin.lzma: $(project_dir)/vmlinux.bin
 	xz -c -k -f --format=lzma --lzma1=dict=1MiB,lc=3,lp=0,pb=3 $< > $@
 
-$(project_dir)/kernel-image: $(project_dir)/vmlinux.bin.lzma $(project_dir)/../arm64/kernel_fdt_lzma.its $(project_dir)/target.dtb $(project_dir)/initramfs.cpio.xz
+$(project_dir)/kernel-image: $(project_dir)/vmlinux.bin.lzma $(project_dir)/../arm64/kernel_fdt_lzma.its $(project_dir)/target.dtb $(PWD)/$(CONFIG_PAYLOAD_USERSPACE)
 	cp $(project_dir)/../arm64/kernel_fdt_lzma.its $(project_dir)
 	mkimage -f $(project_dir)/kernel_fdt_lzma.its $@
 endif
 
 ifeq ($(CONFIG_LINUXBOOT_UROOT),y)
-$(project_dir)/initramfs.cpio.xz:
+$(PWD)/$(CONFIG_PAYLOAD_USERSPACE):
 	$(MAKE) -f targets/u-root.mk
 else
-$(project_dir)/initramfs.cpio.xz:
+$(PWD)/$(CONFIG_PAYLOAD_USERSPACE):
 	echo "Building without u-root support"
 endif
 
-payload: $(project_dir)/kernel-image $(project_dir)/initramfs.cpio.xz
+payload: $(project_dir)/kernel-image $(PWD)/$(CONFIG_PAYLOAD_USERSPACE)
 
 clean:
 	if [ -d "$(kernel_dir)" ]; then rm -rf $(kernel_dir); fi
diff --git a/payloads/external/LinuxBoot/targets/u-root.mk b/payloads/external/LinuxBoot/targets/u-root.mk
index f3c70a6..001f7c4 100644
--- a/payloads/external/LinuxBoot/targets/u-root.mk
+++ b/payloads/external/LinuxBoot/targets/u-root.mk
@@ -71,10 +71,10 @@
 else
 ifneq ($(CONFIG_LINUXBOOT_UROOT_FILES),)
 	cd $(uroot_dir); GOARCH=$(CONFIG_LINUXBOOT_ARCH) GOPATH=$(go_path_dir) ./u-root \
-	-build=bb -files $(CONFIG_LINUXBOOT_UROOT_FILES) -o $(project_dir)/initramfs.cpio
+	-build=bb -files $(CONFIG_LINUXBOOT_UROOT_FILES) -o $(project_dir)/initramfs.cpio coreboot-app
 else
 	cd $(uroot_dir); GOARCH=$(CONFIG_LINUXBOOT_ARCH) GOPATH=$(go_path_dir) ./u-root \
-	-build=bb -o $(project_dir)/initramfs.cpio
+	-build=bb -o $(project_dir)/initramfs.cpio coreboot-app
 endif
 endif
 	xz -f --check=crc32 -9 --lzma2=dict=1MiB --threads=$(CPUS) $(project_dir)/initramfs.cpio
diff --git a/payloads/external/Makefile.inc b/payloads/external/Makefile.inc
index f350622..8bd0dc3 100644
--- a/payloads/external/Makefile.inc
+++ b/payloads/external/Makefile.inc
@@ -274,7 +274,8 @@
 		CONFIG_LINUXBOOT_ARCH=$(CONFIG_LINUXBOOT_ARCH) \
 		CONFIG_LINUXBOOT_UROOT=$(CONFIG_LINUXBOOT_UROOT) \
 		CONFIG_LINUXBOOT_UROOT_FILES=$(CONFIG_LINUXBOOT_UROOT_FILES) \
-		CONFIG_LINUXBOOT_DTB_FILE=$(CONFIG_LINUXBOOT_DTB_FILE)
+		CONFIG_LINUXBOOT_DTB_FILE=$(CONFIG_LINUXBOOT_DTB_FILE) \
+		CONFIG_PAYLOAD_USERSPACE=$(CONFIG_PAYLOAD_USERSPACE)
 
 
 payloads/external/LinuxBoot/linuxboot/kernel-image: linuxboot

-- 
To view, visit https://review.coreboot.org/28401
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia0641fd0c0db1e47ee4829b73871af662d4a6370
Gerrit-Change-Number: 28401
Gerrit-PatchSet: 1
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180830/98975296/attachment-0001.html>