Hi Simon, On Mon, 28 Oct 2024 at 13:05, Simon Glass <s...@chromium.org> wrote:
> 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 ? > Yes, I can add some more field checkers in the next version. Regards, Raymond
