[PATCH v2 4/8] smbios: Refactor smbios library

[Simon Glass]

Hi Raymond,

On Tue, 22 Oct 2024 at 22:07, Raymond Mao <raymond.mao at linaro.org> wrote:
>
> Current smbios library does not fully match to the specification.
> It hardcodes values instead of exposing values from the device.
> It does not support dynamic length for contained object handles
> or elements and misses the handling of a few of fields.
>
> The refactoring of this patch includes:
> 1. Expose values from device via sysinfo interface.
> 2. Replace smbios_add_prop with smbios_add_prop_si to allow getting
>    string values from sysinfo.
> 3. Add smbios_get_val_si to get int values from sysinfo.
> 4. Use sysinfo_get_data to get data area.
> 5. Miscellaneous fixes in smbios.
>
> Moreover, this patch adds SMBIOS type 7 (cache information) write
> functions.
> Link cache handles from type 7 to type 4.
>
> Signed-off-by: Raymond Mao <raymond.mao at linaro.org>
> ---
> Changes in v2
> - Combine with #7(v1) patch.
> - Move the changes of cmd/smbios.c to a separated patch.
> - Refactor smbios_get_val_si to get values from dt when values are
>    not available from sysinfo driver.
> - Miscellaneous refactoring.
>
>  include/smbios.h |  14 +-
>  lib/smbios.c     | 420 +++++++++++++++++++++++++++++++++++++++++------
>  2 files changed, 383 insertions(+), 51 deletions(-)
>

There's an awful lot in this patch.

Can it be split up?

Regards,
Simon