On Tue, 2014-04-15 at 08:02 +0300, Fathi Boudra wrote: > On 14 April 2014 22:55, Ben Hutchings <ben@xxxxxxxxxxxxxxx> wrote: > > On Sat, 2014-04-12 at 20:15 +0300, Fathi Boudra wrote: > >> Signed-off-by: Fathi Boudra <fathi.boudra@xxxxxxxxxx> > >> --- > >> scripts/package/builddeb | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/scripts/package/builddeb b/scripts/package/builddeb > >> index f46e4dd..24cb3b1 100644 > >> --- a/scripts/package/builddeb > >> +++ b/scripts/package/builddeb > >> @@ -165,6 +165,11 @@ if grep -q '^CONFIG_MODULES=y' $KCONFIG_CONFIG ; then > >> fi > >> fi > >> > >> +if grep -q '^CONFIG_OF=y' $KCONFIG_CONFIG ; then > >> + mkdir -p "$tmpdir/boot/dtb/$version" > > > > The boot loader only needs to read at most one of the installed DTB > > files on any given system, and you can't in general expect it to read a > > DTB from the filesystem. So why not install them in > > /usr/lib/linux-image-$version where flash-kernel expects? > > From various discussions, I think it's agreed that /usr/lib/... isn't > the right path to use (and we use it since years in Linaro images > unfortunately). > The consensus seems to follow bootloader spec: > http://www.freedesktop.org/wiki/Specifications/BootLoaderSpec/ > and install the DTBs under /boot. I wasn't aware of such consensus. I have seen an earlier version of this spec and it seemed to be only supported by one distribution. > Get it versioned is preferable. > It should address the various bootloaders (U-boot, UEFI, GRUB, > barebox, etc...) requirements, except the VFAT file system. It seems to me that: - It might work with U-boot, except for the many exising versions that it doesn't work with. - It might work with GRUB, though that doesn't seem to be widely used yet on ARM. - It won't work with UEFI because /boot/dtb will not be on the ESP. (And dpkg can't upgrade on a filesystem that doesn't support hard links so you should never expect to install files directly on the ESP.) > IMO we should support the distribution wide consensus in this case, > instead of flash-kernel (Debian/Ubuntu specific tool) expectations. > We'll probably end up by adding a symlink to keep compatibility with > older systems or fix flash-kernel. > > Any other feedback on this topic is appreciated. [...] flash-kernel works today and you are proposing to do something that might eventually work some time in the future. Ben. -- Ben Hutchings The obvious mathematical breakthrough [to break modern encryption] would be development of an easy way to factor large prime numbers. - Bill Gates
Attachment:
signature.asc
Description: This is a digitally signed message part