[PATCH v2 7/8] cmd: update smbios cmd

Raymond Mao raymond.mao at linaro.org
Mon Oct 28 21:02:20 CET 2024


Hi Simon,

On Mon, 28 Oct 2024 at 13:05, Simon Glass <sjg at chromium.org> wrote:

> On Tue, 22 Oct 2024 at 22:07, Raymond Mao <raymond.mao at linaro.org> wrote:
> >
> > Update the cmd according to the changes of the smbios library:
> > 1. Refactor smbios cmd print functions to match the content defined
> >    by the specification.
> > 2. Add new print functions for Type 0, 3, 4 and 7.
> > 3. Update the expected pytest result to align with the smbios
> >    library changes.
> >
> > Signed-off-by: Raymond Mao <raymond.mao at linaro.org>
> > ---
> > Changes in v2
> > - Initial patch. Move all changes related to cmd/smbios into this patch.
> > - Combine with #9(v1) patch.
> >
> >  cmd/smbios.c                 | 342 ++++++++++++++++++++++++++++++++---
> >  include/smbios.h             |   5 +
> >  test/py/tests/test_smbios.py |   2 +-
> >  3 files changed, 319 insertions(+), 30 deletions(-)
> >
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> OK, so it should be possible to expand the test for sandbox to check each
> field? Something like test/dm/acpi.c ?
>

Yes, I can add some more field checkers in the next version.

Regards,
Raymond


More information about the U-Boot mailing list