On 26-10-15 11:18, Ahmad Fatoum wrote: > linux_bootargs_get() falls back to the legacy bootargs environment variable > whenever the concatenation of all global.linux.bootargs.* variables comes > out empty. That contradicts the documentation, which describes bootargs as > the CONFIG_FLEXIBLE_BOOTARGS=n way of passing a command line, and it only > worked until the first boot entry had run: bootscript_boot() registers > global.linux.bootargs.dyn.ip and .dyn.root, and the separator between the > two empty variables made the result non-empty, so the kernel got a command > line consisting of spaces instead. > > Drop the fallback and return NULL when there is nothing to pass. All > callers already handle a NULL command line, as the fallback could return > NULL as well. > > Assisted-by: Claude:opus-5 > Signed-off-by: Ahmad Fatoum <[email protected]>
Reviewed-by: Marco Felsch <[email protected]>
