Commit e753f331 authored by David Gibson's avatar David Gibson
Browse files

sam460ex: Don't check for errors from qemu_fdt_*()



The qemu_fdt_*() helper functions already exit with a message instead of
returning errors, so we don't need to check for errors in the caller.

Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
parent ad633de6
Loading
Loading
Loading
Loading
+6 −20
Original line number Diff line number Diff line
@@ -276,32 +276,18 @@ static int sam460ex_load_device_tree(hwaddr addr,

    /* Manipulate device tree in memory. */

    ret = qemu_fdt_setprop(fdt, "/memory", "reg", mem_reg_property,
    qemu_fdt_setprop(fdt, "/memory", "reg", mem_reg_property,
                     sizeof(mem_reg_property));
    if (ret < 0) {
        error_report("couldn't set /memory/reg");
    }

    /* default FDT doesn't have a /chosen node... */
    qemu_fdt_add_subnode(fdt, "/chosen");

    ret = qemu_fdt_setprop_cell(fdt, "/chosen", "linux,initrd-start",
                                    initrd_base);
    if (ret < 0) {
        error_report("couldn't set /chosen/linux,initrd-start");
    }
    qemu_fdt_setprop_cell(fdt, "/chosen", "linux,initrd-start", initrd_base);

    ret = qemu_fdt_setprop_cell(fdt, "/chosen", "linux,initrd-end",
    qemu_fdt_setprop_cell(fdt, "/chosen", "linux,initrd-end",
                          (initrd_base + initrd_size));
    if (ret < 0) {
        error_report("couldn't set /chosen/linux,initrd-end");
    }

    ret = qemu_fdt_setprop_string(fdt, "/chosen", "bootargs",
                                      kernel_cmdline);
    if (ret < 0) {
        error_report("couldn't set /chosen/bootargs");
    }
    qemu_fdt_setprop_string(fdt, "/chosen", "bootargs", kernel_cmdline);

    /* Copy data from the host device tree into the guest. Since the guest can
     * directly access the timebase without host involvement, we must expose