On 5/28/24 17:11, Kory Maincent wrote:
On Tue, 28 May 2024 16:53:42 +0200
Michal Simek <michal.si...@amd.com> wrote:
On 5/28/24 16:36, Kory Maincent wrote:
Once the environment was saved, we could not retrieve information about
nit: use imperative mood.
Ah indeed.
the multiboot image used. When dealing with firmware updates, this
information is necessary alongside the saved environment.
what exactly are you trying to do?
Retrieving the multiboot information to know which boot000x.bin image we have
booted. This would allow to check the boot status of a firmware update.
Move the multiboot environment set operation before the saved environment
check to ensure this information is always available.
that make sense.
Signed-off-by: Kory Maincent <kory.mainc...@bootlin.com>
---
board/xilinx/zynqmp/zynqmp.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index f370fb7347a..16292ed1c7e 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -519,6 +519,10 @@ int board_late_init(void)
usb_ether_init();
#endif
+ multiboot = multi_boot();
+ if (multiboot >= 0)
likely make sense to remove this check to because you want to have even
multiboot=0 too.
I don't understand, this check is >= so it supports multiboot=0.
Sorry my fault. You are right. It was there for catching the fault.
Thanks,
Michal
