[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 2/6] hw/core: don't check return on qemu_fdt_setprop_string_ar
From: |
Ben Dooks |
Subject: |
[PATCH v4 2/6] hw/core: don't check return on qemu_fdt_setprop_string_array() |
Date: |
Tue, 9 Aug 2022 19:56:36 +0100 |
The qemu_fdt_setprop_string_array() does not return error codes and
will call exit() if any of the fdt calls fails (and should print an
error with the node being altered). This is done to prepare for the
change for qemu_fdt_setprop_strings() helper which does not return
any error codes (hw/core/guest-loader.c is the only place where an
return is checked).
Signed-off-by: Ben Dooks <qemu@ben.fluff.org>
---
hw/core/guest-loader.c | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/hw/core/guest-loader.c b/hw/core/guest-loader.c
index 391c875a29..c61ebc4144 100644
--- a/hw/core/guest-loader.c
+++ b/hw/core/guest-loader.c
@@ -57,25 +57,17 @@ static void loader_insert_platform_data(GuestLoaderState
*s, int size,
if (s->kernel) {
const char *compat[2] = { "multiboot,module", "multiboot,kernel" };
- if (qemu_fdt_setprop_string_array(fdt, node, "compatible",
- (char **) &compat,
- ARRAY_SIZE(compat)) < 0) {
- error_setg(errp, "couldn't set %s/compatible", node);
- return;
- }
+ qemu_fdt_setprop_string_array(fdt, node, "compatible",
+ (char **) &compat,
+ ARRAY_SIZE(compat));
if (s->args) {
- if (qemu_fdt_setprop_string(fdt, node, "bootargs", s->args) < 0) {
- error_setg(errp, "couldn't set %s/bootargs", node);
- }
+ qemu_fdt_setprop_string(fdt, node, "bootargs", s->args);
}
} else if (s->initrd) {
const char *compat[2] = { "multiboot,module", "multiboot,ramdisk" };
- if (qemu_fdt_setprop_string_array(fdt, node, "compatible",
- (char **) &compat,
- ARRAY_SIZE(compat)) < 0) {
- error_setg(errp, "couldn't set %s/compatible", node);
- return;
- }
+ qemu_fdt_setprop_string_array(fdt, node, "compatible",
+ (char **) &compat,
+ ARRAY_SIZE(compat));
}
}
--
2.35.1
- add qemu_fdt_setprop_strings() and use it in most places, Ben Dooks, 2022/08/09
- [PATCH v4 3/6] hw/riscv: use qemu_fdt_setprop_strings() for string arrays, Ben Dooks, 2022/08/09
- [PATCH v4 6/6] hw/arm: change to use qemu_fdt_setprop_strings(), Ben Dooks, 2022/08/09
- [PATCH v4 2/6] hw/core: don't check return on qemu_fdt_setprop_string_array(),
Ben Dooks <=
- [PATCH v4 1/6] device_tree: add qemu_fdt_setprop_strings() helper, Ben Dooks, 2022/08/09
- [PATCH v4 5/6] hw/mips: use qemu_fdt_setprop_strings(), Ben Dooks, 2022/08/09
- [PATCH v4 4/6] hw/core: use qemu_fdt_setprop_strings(), Ben Dooks, 2022/08/09