The latest UPL spec uses a compatible string for this node rather than
relying just on the node's name to indicate its purpose. Add one.

Signed-off-by: Simon Glass <[email protected]>
---

 boot/upl_write.c | 4 +++-
 include/upl.h    | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/boot/upl_write.c b/boot/upl_write.c
index c2bb3187803..c29848a9ce3 100644
--- a/boot/upl_write.c
+++ b/boot/upl_write.c
@@ -178,7 +178,9 @@ static int add_upl_params(const struct upl *upl, ofnode 
options)
        if (ret)
                return log_msg_ret("img", ret);
 
-       ret = write_addr(upl, node, UPLP_SMBIOS, upl->smbios);
+       ret = ofnode_write_string(node, "compatible", UPLP_UPL_PARAMS_COMPAT);
+       if (!ret)
+               ret = write_addr(upl, node, UPLP_SMBIOS, upl->smbios);
        if (!ret)
                ret = write_addr(upl, node, UPLP_ACPI, upl->acpi);
        if (!ret && upl->bootmode)
diff --git a/include/upl.h b/include/upl.h
index 5ddcd990de4..a4ed24edbb8 100644
--- a/include/upl.h
+++ b/include/upl.h
@@ -23,6 +23,7 @@ struct unit_test_state;
 
 #define UPLN_OPTIONS           "options"
 #define UPLN_UPL_PARAMS                "upl-params"
+#define UPLP_UPL_PARAMS_COMPAT "upl"
 #define UPLP_SMBIOS            "smbios"
 #define UPLP_ACPI              "acpi"
 #define UPLP_BOOTMODE          "bootmode"
-- 
2.43.0

Reply via email to