On Tue, 22 Oct 2024 at 22:07, Raymond Mao <raymond....@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....@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 <s...@chromium.org> OK, so it should be possible to expand the test for sandbox to check each field? Something like test/dm/acpi.c ? Regards, Simon