summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2022-10-20 18:23:12 -0600
committerTom Rini <trini@konsulko.com>2022-10-31 11:03:36 -0400
commite45d22655aed0c81fa5890f47c1647c6e95bedb6 (patch)
tree3a53749d09f992266d063a774a0e235a563a5a37
parentc263e21bcb01f19e6ccb2452fdcc601ff84942db (diff)
downloadu-boot-e45d22655aed0c81fa5890f47c1647c6e95bedb6.tar.gz
u-boot-e45d22655aed0c81fa5890f47c1647c6e95bedb6.tar.bz2
u-boot-e45d22655aed0c81fa5890f47c1647c6e95bedb6.zip
vbe: Drop the U-Boot prefix from the version
We don't need the U-Boot prefix on the version and in fact it is harmful since pytest gets confused seeing the U-Boot banner bring displayed when the version is printed. Drop the prefix from the string. We could produce an entirely new string from the component parts, but this adds to the rodata size and would break the use of version_string as the only thing which holds this information. Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r--boot/vbe_simple_os.c9
-rw-r--r--test/boot/vbe_simple.c2
2 files changed, 8 insertions, 3 deletions
diff --git a/boot/vbe_simple_os.c b/boot/vbe_simple_os.c
index 7761b9ef65..058db6154b 100644
--- a/boot/vbe_simple_os.c
+++ b/boot/vbe_simple_os.c
@@ -18,7 +18,7 @@
int vbe_simple_fixup_node(ofnode node, struct simple_state *state)
{
- char *version;
+ const char *version, *str;
int ret;
version = strdup(state->fw_version);
@@ -31,7 +31,12 @@ int vbe_simple_fixup_node(ofnode node, struct simple_state *state)
ret = ofnode_write_u32(node, "cur-vernum", state->fw_vernum);
if (ret)
return log_msg_ret("num", ret);
- ret = ofnode_write_string(node, "bootloader-version", version_string);
+
+ /* Drop the 'U-Boot ' at the start */
+ str = version_string;
+ if (!strncmp("U-Boot ", str, 7))
+ str += 7;
+ ret = ofnode_write_string(node, "bootloader-version", str);
if (ret)
return log_msg_ret("bl", ret);
diff --git a/test/boot/vbe_simple.c b/test/boot/vbe_simple.c
index faba9e8f90..a50785dbbf 100644
--- a/test/boot/vbe_simple.c
+++ b/test/boot/vbe_simple.c
@@ -77,7 +77,7 @@ static int vbe_simple_test_base(struct unit_test_state *uts)
bl_version = ofnode_read_string(node, "bootloader-version");
ut_assertnonnull(bl_version);
- ut_asserteq_str(version_string, bl_version);
+ ut_asserteq_str(version_string + 7, bl_version);
return 0;
}