Sender: =?UTF-8?Q?Timo_Ter=C3=A4s?= <timo.teras@gmail.com>
Date: Tue, 19 Apr 2016 16:16:35 +0300
From: Timo Teras <timo.teras@iki.fi>
To: Marian Buschsieweke <m.buschsieweke@uni-muenster.de>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] linux-rpi: Installing dtb-files into
 /boot
Message-ID: <20160419161635.57dd6cfd@vostro.util.wtbts.net>
In-Reply-To: <1460733346-1466-1-git-send-email-m.buschsieweke@uni-muenster.de>
References: <1460733346-1466-1-git-send-email-m.buschsieweke@uni-muenster.de>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Fri, 15 Apr 2016 17:15:46 +0200
Marian Buschsieweke <m.buschsieweke@uni-muenster.de> wrote:

> Instead of "/usr/lib/linux-${_abi_release}/" the dtb-files will be
> installed into "/boot/", as these are required to be stored on the
> boot partition along with the kernel, which is installed into
> "/boot/". With this change a kernel update on a Pi will work fully
> automatically, as the user no longer needs to take care to copy
> the .dtb-Files manually into her boot partition. ---
>  main/linux-rpi/APKBUILD | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

If this changes, we need to fix also the alpine-iso image building
scripts, and update-kernel script.

But I don't fully like this. The problem is that you can't then have
several kernels installed as the dtb's would overlap. It's not
currently a problem, but might become one.

I do understand the issue at hand. I wonder if we could symlink things,
or just ship config.txt so the dtbs are looked up from other place. Or
potentially just ship script that copies the latest dtbs to /boot.

But generally the dtbs and the kernel should match.

> 
> diff --git a/main/linux-rpi/APKBUILD b/main/linux-rpi/APKBUILD
> index 80183d8b..bc94310 100644
> --- a/main/linux-rpi/APKBUILD
> +++ b/main/linux-rpi/APKBUILD
> @@ -108,7 +108,7 @@ _package() {
>  	local _install
>  	case "$CARCH" in
>  	arm*)
> -		local
> _dtbdir="$_outdir"/usr/lib/linux-${_abi_release}
> +		local _dtbdir="$_outdir"/boot
>  		cd "$srcdir"/build-$_buildflavor/arch/arm/boot/dts
> || return 1 mkdir -p "$_dtbdir"
>  		find . -name "*.dtb" -type f | cpio -pdm "$_dtbdir"



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---