Thread (10 messages) 10 messages, 4 authors, 2021-06-12

Re: [PATCH -fixes] riscv: Fix BUILTIN_DTB for sifive and microchip soc

From: Alex Ghiti <alex@ghiti.fr>
Date: 2021-06-05 06:34:00
Also in: linux-riscv, lkml

Hi Arnd,

Le 4/06/2021 à 15:08, Arnd Bergmann a écrit :
On Fri, Jun 4, 2021 at 2:06 PM Alexandre Ghiti [off-list ref] wrote:
quoted
Fix BUILTIN_DTB config which resulted in a dtb that was actually not built
into the Linux image: in the same manner as Canaan soc does, create an object
file from the dtb file that will get linked into the Linux image.

Signed-off-by: Alexandre Ghiti <alex@ghiti.fr>
Along the same lines as the comment that Jisheng Zhang made on the fixed
address, building a dtb into the kernel itself fundamentally breaks generic
kernel images.

I can understand using it on K210, which is extremely limited and wouldn't
run a generic kernel anyway, but for normal platforms like microchip and
sifive, it would be better to disallow CONFIG_BUILTIN_DTB in Kconfig
and require a non-broken boot loader.
I kind of disagree because if I want to build a custom kernel for those 
platforms with a builtin dtb for some reasons (debug, development..Etc), 
I think I should be able to do so.
       Arnd

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help