[Buildroot] [PATCH] linux: Introduce DTB's bootargs fixup feature

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Mar 29 22:11:29 UTC 2016


Hello,

On Mon, 21 Mar 2016 13:53:50 +0100, Maxime Coquelin wrote:

> > However, I am not entirely sure this is something we want to take in
> > Buildroot. Indeed, your patch only allows to customize
> > the /chosen/bootargs property, and not any other property of the Device
> > Tree. If we want to do that via Config.in options, it suddenly becomes
> > a lot more complicated.
> Indeed, it will be difficult to extend it further than bootargs.
> It works well for bootargs for two reasons:
>  1- It is a generic property
>  2- It is a string
> 
> There could be other uses for DTB patching, such as MAC address assignment.
> But I agree than doing this in Config.in makes no sense.

Right.

> > It is IMO even more important to do it this way since as of today, only
> > one platform/board needs to adjust its bootargs this way. All other
> > platforms have an ATAGs or DT compatible bootloader, where the bootargs
> > customization is done via the bootloader, not via patching the DTB.
> 
> Ok, For sure it can be done, but I wanted to propose a generic way of
> doing this.
> Indeed, this is something other platforms could potentially use, even
> if not the case today.
> 
> > Of course, let's see what others think.
> 
> Agree.
> If it is decided to do this as post-image script, I will be happy to
> implement it.

Nobody spoke up, so let's say we'll do it in a post-image script for
now. host-dtc is already visible in menuconfig, so it can be enabled in
a defconfig, and therefore it's just a matter of enabling it and write
the appropriate post-image script.

Do you want to write a patch on top of my series adding Cortex-M
support?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list