Package: flash-kernel Tags: patch Control: submitter -1 a.heider@gmail.com X-Debbugs-Cc: Andre Heider a.heider@gmail.com
I'm submitting this to the Debian bug tracking system, since this isn't directly related to u-boot. I've dropped most of the CCs on the thread, presumably should also drop the u-boot CC on follow-ups.
On 2020-09-03, Andre Heider wrote:
On 03/09/2020 20:59, Vagrant Cascadian wrote:
On 2020-09-03, Andre Heider wrote:
On 03/09/2020 18:40, Dennis Gilmore wrote: If so, adding fdtfile like that would break booting debian using its flash-image. It installs dtbs without that subdir into /boot: /boot/dtbs/4.19.0-9-arm64/armada-3720-espressobin.dtb
But it already supports $fdtfile: https://salsa.debian.org/installer-team/flash-kernel/-/blob/master/bootscrip...
If I read that script correctly, it would fail to boot with fdtfile=marvell/armada-3720-espressobin.dtb...
It probably would fall back to the /boot/dtb-$version symlinks...
Support for installing in vendor subdirs was added to flash-kernel 3.91 from 2018, and I believe the vendor subdirs are included in the debian-installer boot media as well.
alright, but if the vendor path is missing from flash-kernel's db, everything silently works with a flattened tree. See attached patch, now it works here too :)
Yes, I believe this was intentional to smooth the transition without breaking booting for some machines...
Before: /boot/dtbs/4.19.0-10-arm64/armada-3720-espressobin.dtb After: /boot/dtbs/4.19.0-10-arm64/marvell/armada-3720-espressobin.dtb
Thanks, Andre From 9565a3a066f59258886ad1216ce957d63fd390cc Mon Sep 17 00:00:00 2001 From: Andre Heider a.heider@gmail.com Date: Thu, 3 Sep 2020 21:33:18 +0200 Subject: [PATCH] Fix missing vendor subdirs for arm64 boards.
The subdir is required for installing the fdt file to the correct path, which is a prerequisite for a boot loader provided $fdtfile.
db/all.db | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/db/all.db b/db/all.db index 16f9606..d7303f7 100644 --- a/db/all.db +++ b/db/all.db @@ -487,7 +487,7 @@ Required-Packages: u-boot-tools Bootloader-Sets-Incorrect-Root: yes
Machine: Globalscale Marvell ESPRESSOBin Board -DTB-Id: armada-3720-espressobin.dtb +DTB-Id: marvell/armada-3720-espressobin.dtb Boot-Script-Path: /boot/boot.scr U-Boot-Script-Name: bootscr.uboot-generic Required-Packages: u-boot-tools @@ -874,7 +874,7 @@ Required-Packages: u-boot-tools
Machine: Marvell Armada 8040 DB board Kernel-Flavors: arm64 -DTB-Id: armada-8040-db.dtb +DTB-Id: marvell/armada-8040-db.dtb Boot-Script-Path: /boot/boot.scr U-Boot-Script-Name: bootscr.uboot-generic Required-Packages: u-boot-tools @@ -1186,7 +1186,7 @@ Required-Packages: u-boot-tools
Machine: Olimex A64 Teres-I Kernel-Flavors: arm64 -DTB-Id: sun50i-a64-teres-i.dtb +DTB-Id: allwinner/sun50i-a64-teres-i.dtb Boot-Script-Path: /boot/boot.scr U-Boot-Script-Name: bootscr.uboot-generic Required-Packages: u-boot-tools @@ -1345,7 +1345,7 @@ Required-Packages: u-boot-tools
Machine: Purism Librem 5 devkit Kernel-Flavors: arm64 -DTB-Id: imx8mq-librem5-devkit.dtb +DTB-Id: freescale/imx8mq-librem5-devkit.dtb Boot-Script-Path: /boot/boot.scr U-Boot-Script-Name: bootscr.uboot-generic Required-Packages: u-boot-tools -- 2.20.1
live well, vagrant