Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

Revert "of: fdt: Honor CONFIG_CMDLINE* even without /chosen node"

This reverts commit a7d550f82b445cf218b47a2c1a9c56e97ecb8c7a.

Some arches (PPC at least) don't call early_init_dt_scan_nodes(), so
moving the cmdline processing there breaks them.

Reported-by: Geoff Levand <geoff@infradead.org>
Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Tested-by: Geoff Levand <geoff@infradead.org>
Reviewed-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Link: https://lore.kernel.org/r/20230103-dt-cmdline-fix-v1-1-7038e88b18b6@kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>

+20 -20
+20 -20
drivers/of/fdt.c
··· 1175 1175 if (p != NULL && l > 0) 1176 1176 strscpy(cmdline, p, min(l, COMMAND_LINE_SIZE)); 1177 1177 1178 + /* 1179 + * CONFIG_CMDLINE is meant to be a default in case nothing else 1180 + * managed to set the command line, unless CONFIG_CMDLINE_FORCE 1181 + * is set in which case we override whatever was found earlier. 1182 + */ 1183 + #ifdef CONFIG_CMDLINE 1184 + #if defined(CONFIG_CMDLINE_EXTEND) 1185 + strlcat(cmdline, " ", COMMAND_LINE_SIZE); 1186 + strlcat(cmdline, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1187 + #elif defined(CONFIG_CMDLINE_FORCE) 1188 + strscpy(cmdline, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1189 + #else 1190 + /* No arguments from boot loader, use kernel's cmdl*/ 1191 + if (!((char *)cmdline)[0]) 1192 + strscpy(cmdline, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1193 + #endif 1194 + #endif /* CONFIG_CMDLINE */ 1195 + 1196 + pr_debug("Command line is: %s\n", (char *)cmdline); 1197 + 1178 1198 rng_seed = of_get_flat_dt_prop(node, "rng-seed", &l); 1179 1199 if (rng_seed && l > 0) { 1180 1200 add_bootloader_randomness(rng_seed, l); ··· 1298 1278 rc = early_init_dt_scan_chosen(boot_command_line); 1299 1279 if (rc) 1300 1280 pr_warn("No chosen node found, continuing without\n"); 1301 - 1302 - /* 1303 - * CONFIG_CMDLINE is meant to be a default in case nothing else 1304 - * managed to set the command line, unless CONFIG_CMDLINE_FORCE 1305 - * is set in which case we override whatever was found earlier. 1306 - */ 1307 - #ifdef CONFIG_CMDLINE 1308 - #if defined(CONFIG_CMDLINE_EXTEND) 1309 - strlcat(boot_command_line, " ", COMMAND_LINE_SIZE); 1310 - strlcat(boot_command_line, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1311 - #elif defined(CONFIG_CMDLINE_FORCE) 1312 - strscpy(boot_command_line, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1313 - #else 1314 - /* No arguments from boot loader, use kernel's cmdl */ 1315 - if (!boot_command_line[0]) 1316 - strscpy(boot_command_line, CONFIG_CMDLINE, COMMAND_LINE_SIZE); 1317 - #endif 1318 - #endif /* CONFIG_CMDLINE */ 1319 - 1320 - pr_debug("Command line is: %s\n", boot_command_line); 1321 1281 1322 1282 /* Setup memory, calling early_init_dt_add_memory_arch */ 1323 1283 early_init_dt_scan_memory();