diff options
author | Alexander Sverdlin <alexander.sverdlin@gmail.com> | 2022-12-12 00:58:17 +0100 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2022-12-16 11:22:58 -0600 |
commit | a7d550f82b445cf218b47a2c1a9c56e97ecb8c7a (patch) | |
tree | 8d11e5f28727196d36f32f188c74d971bb573d46 /io_uring/io_uring.c | |
parent | 71a7507afbc3f27c346898f13ab9bfd918613c34 (diff) |
of: fdt: Honor CONFIG_CMDLINE* even without /chosen node
I do not read a strict requirement on /chosen node in either ePAPR or in
Documentation/devicetree. Help text for CONFIG_CMDLINE and
CONFIG_CMDLINE_EXTEND doesn't make their behavior explicitly dependent on
the presence of /chosen or the presense of /chosen/bootargs.
However the early check for /chosen and bailing out in
early_init_dt_scan_chosen() skips CONFIG_CMDLINE handling which is not
really related to /chosen node or the particular method of passing cmdline
from bootloader.
This leads to counterintuitive combinations (assuming
CONFIG_CMDLINE_EXTEND=y):
a) bootargs="foo", CONFIG_CMDLINE="bar" => cmdline=="foo bar"
b) /chosen missing, CONFIG_CMDLINE="bar" => cmdline==""
c) bootargs="", CONFIG_CMDLINE="bar" => cmdline==" bar"
Move CONFIG_CMDLINE handling outside of early_init_dt_scan_chosen() so that
cases b and c above result in the same cmdline.
Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/11af73e05bad75e4ef49067515e3214f6d944b3d.camel@gmail.com
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'io_uring/io_uring.c')
0 files changed, 0 insertions, 0 deletions