[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/23] RISC-V: Make virt create_fdt interface
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/23] RISC-V: Make virt create_fdt interface consistent |
Date: |
Sat, 10 Mar 2018 21:33:43 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 03/09/2018 05:12 AM, Michael Clark wrote:
> create_fdt sets the fdt variable on RISCVVirtState and this is
> used to access the fdt. This reverts a change introduced in
> https://github.com/riscv/riscv-qemu/pull/109 which introduced
> a redundant return value, overlooking the RISCVVirtState
> structure member that made create_fdt inconsistent with the
> other RISC-V machines. The other alternative is to change
> the other boards to return the fdt. Note: the RISCVVirtState
> also contains fdt_size.
>
> Cc: Sagar Karandikar <address@hidden>
> Cc: Bastian Koppelmann <address@hidden>
> Signed-off-by: Michael Clark <address@hidden>
> Signed-off-by: Palmer Dabbelt <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/riscv/virt.c | 13 +++++--------
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> index e2c214e..37968d2 100644
> --- a/hw/riscv/virt.c
> +++ b/hw/riscv/virt.c
> @@ -108,7 +108,7 @@ static hwaddr load_initrd(const char *filename, uint64_t
> mem_size,
> return *start + size;
> }
>
> -static void *create_fdt(RISCVVirtState *s, const struct MemmapEntry *memmap,
> +static void create_fdt(RISCVVirtState *s, const struct MemmapEntry *memmap,
> uint64_t mem_size, const char *cmdline)
> {
> void *fdt;
> @@ -264,8 +264,6 @@ static void *create_fdt(RISCVVirtState *s, const struct
> MemmapEntry *memmap,
> qemu_fdt_setprop_string(fdt, "/chosen", "stdout-path", nodename);
> qemu_fdt_setprop_string(fdt, "/chosen", "bootargs", cmdline);
> g_free(nodename);
> -
> - return fdt;
> }
>
> static void riscv_virt_board_init(MachineState *machine)
> @@ -279,7 +277,6 @@ static void riscv_virt_board_init(MachineState *machine)
> char *plic_hart_config;
> size_t plic_hart_config_len;
> int i;
> - void *fdt;
>
> /* Initialize SOC */
> object_initialize(&s->soc, sizeof(s->soc), TYPE_RISCV_HART_ARRAY);
> @@ -299,7 +296,7 @@ static void riscv_virt_board_init(MachineState *machine)
> main_mem);
>
> /* create device tree */
> - fdt = create_fdt(s, memmap, machine->ram_size, machine->kernel_cmdline);
> + create_fdt(s, memmap, machine->ram_size, machine->kernel_cmdline);
>
> /* boot rom */
> memory_region_init_ram(boot_rom, NULL, "riscv_virt_board.bootrom",
> @@ -314,9 +311,9 @@ static void riscv_virt_board_init(MachineState *machine)
> hwaddr end = load_initrd(machine->initrd_filename,
> machine->ram_size, kernel_entry,
> &start);
> - qemu_fdt_setprop_cell(fdt, "/chosen",
> - "linux,initrd-start", start);
> - qemu_fdt_setprop_cell(fdt, "/chosen", "linux,initrd-end",
> + qemu_fdt_setprop_cell(s->fdt, "/chosen", "linux,initrd-start",
> + start);
> + qemu_fdt_setprop_cell(s->fdt, "/chosen", "linux,initrd-end",
> end);
> }
> }
>
- [Qemu-devel] [PATCH v2 00/23] RISC-V Post-merge spec conformance and cleanup, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 01/23] RISC-V: Make virt create_fdt interface consistent, Michael Clark, 2018/03/08
- Re: [Qemu-devel] [PATCH v2 01/23] RISC-V: Make virt create_fdt interface consistent,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v2 02/23] RISC-V: Replace hardcoded constants with enum values, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 04/23] RISC-V: Use ROM base address and size from memmap, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 05/23] RISC-V: Remove identity_translate from load_elf, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 03/23] RISC-V: Make virt board description match spike, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 06/23] RISC-V: Mark ROM read-only after copying in code, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 07/23] RISC-V: Remove unused class definitions, Michael Clark, 2018/03/08
- [Qemu-devel] [PATCH v2 08/23] RISC-V: Make sure rom has space for fdt, Michael Clark, 2018/03/08