On 6/12/20 12:30 AM, Havard Skinnemoen wrote:
> If a -bios option is specified on the command line, load the image into
> the internal ROM memory region, which contains the first instructions
> run by the CPU after reset.
>
> A minimal Apache-2.0-licensed boot ROM can be found at
>
> https://github.com/google/vbootrom
you might want to consider adding this rom to QEMU also.
>
> It is by no means feature complete, but it is enough to launch the
> Nuvoton bootblock[1] from offset 0 in the flash, which in turn will
> launch u-boot and finally the Linux kernel.
>
> [1] https://github.com/Nuvoton-Israel/bootblock
>
> Change-Id: I41eae4fc7786d33f532c14087e930bf1b5012562
> Reviewed-by: Tyrone Ting <kfting@nuvoton.com>
> Signed-off-by: Havard Skinnemoen <hskinnemoen@google.com>
> ---
> hw/arm/npcm7xx.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c
> index a5dbf08c00..417bc16b5c 100644
> --- a/hw/arm/npcm7xx.c
> +++ b/hw/arm/npcm7xx.c
> @@ -22,6 +22,7 @@
> #include "hw/misc/unimp.h"
> #include "hw/qdev-properties.h"
> #include "qapi/error.h"
> +#include "qemu-common.h"
> #include "qemu/units.h"
> #include "sysemu/sysemu.h"
>
> @@ -270,6 +271,22 @@ static void npcm7xx_realize(DeviceState *dev, Error **errp)
> }
> memory_region_add_subregion(get_system_memory(), NPCM7XX_ROM_BA, &s->irom);
>
> + if (bios_name) {
> + g_autofree char *filename = NULL;
> + int ret;
> +
> + filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
> + if (!filename) {
> + error_setg(errp, "Could not find ROM image '%s'", bios_name);
> + return;
> + }
> + ret = load_image_mr(filename, &s->irom);
> + if (ret < 0) {
> + error_setg(errp, "Failed to load ROM image '%s'", filename);
> + return;
> + }
> + }
> +
I think this belongs to the machine init routine and not the soc.
> /* External DDR4 SDRAM */
> memory_region_add_subregion(get_system_memory(), NPCM7XX_DRAM_BA, s->dram);
> }
>