Re: [PATCH] cmd: fwu: Also print information about size

[Michal Simek]

On 6/5/24 17:09, Ilias Apalodimas wrote:
On Wed, 5 Jun 2024 at 17:58, Michal Simek <michal.si...@amd.com> wrote:

It is useful when structure is also used for saving vendor data covered
by CRC32.

Signed-off-by: Michal Simek <michal.si...@amd.com>
---

  cmd/fwu_mdata.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/cmd/fwu_mdata.c b/cmd/fwu_mdata.c
index 3c8be576ac7a..9c048d69a131 100644
--- a/cmd/fwu_mdata.c
+++ b/cmd/fwu_mdata.c
@@ -22,6 +22,7 @@ static void print_mdata(struct fwu_data *data)
         printf("\tFWU Metadata\n");
         printf("crc32: %#x\n", data->crc32);
         printf("version: %#x\n", data->version);
+       printf("size: %#x\n", data->metadata_size);

That's only available in v2
IS_ENABLED(CONFIG_FWU_MDATA_V1) etc?

This field should be present in v1 and v2 case because it is the part of 
fwu_data (u-boot private structure) not fwu_mdata (which match the spec).
But I have sent v2 and only printing it in v2 case.

Thanks,
Michal