On 26/10/20 20:11, Laurent Vivier wrote: >> /* Load kernel. */ >> if (!kernel_filename) { >> - if (qtest_enabled() || bios_name) { >> + if (qtest_enabled() || machine->firmware) { >> return; >> } >> error_report("Kernel image must be specified"); > Why do you do differently for mcf5208 than the others? > > const char *bios_name = machine->firmware; > > and no other changes?
because in this case you can use machine->firmware, I'm keeping bios_name for the cases where machine->firmware cannot be used directly (e.g. there is a default). Paolo