On Tue, 2017-11-14 at 11:18 -0600, Rob Herring wrote: > I understand you can't apply random strings in any order you like. I'm > questioning the need to do that and what are the concrete example(s) > where you need that ability. Concrete: Pre-DT bootloader provides "console=/dev/someunusualdevice" and "root=/dev/mtdblock1" in argv. The dtb arranges flash differently and supplies "root=/dev/mtd2" in bootargs. Kernel uses /dev/mtdblock1 and panics, or alternatively uses /dev/mtd2 with no console. > I'd generally expect that only board specific options are in the dtb, > and a distro will add it's own options common for all and the specific > arch into the bootloader config files. And generally, the last thing > loaded gets the last say in what is set. Bootloaders and their configs are not distro-dependent. > > What h/w specific options would be needed for recovery kernel? That's > getting into putting not just Linux specifics into the dtb, but distro > specifics there. While yes, the dts files often already have bootargs > filled with Linux options, the intention is really that the bootloader > fills in bootargs. And if you have multiple kernels or OSs, then the > bootloader provides the mechanism to choose and boot with the right > options. Kernels _are_ distro-specific. A recovery kernel might also have options that are not hardware specific in addition to options that are. > I think the kernel (being last) should fully decide what to do: > append, prepend, and/or override. There was some work a while back to > support more flexible command line handling and be arch neutral, but > it never got merged. The kernel would not always have all the information needed for this decision now like it would before DT. The dtb should decide what precedence its bootargs have, and the kernel should decide what precedence compiled-in bootargs have. Such logic is provided by this patch, but not completely, because there is still no "bootargs-prepend" property. The only use case I can think of for "bootargs-prepend" would be to provide a default command line that is specific to a board that would sometimes have one bootloader, and sometimes another. Currently, such a situation can be worked around using a board-specific kernel, but this mechanism is a relic from pre-DT days. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
