[PATCH 0/3] lib: smbios: Extend driver with using sysinfo driver

[Simon Glass]

Hi Michal,

On Fri, 7 Jun 2024 at 01:53, Michal Simek <michal.simek at amd.com> wrote:
>
>
>
> On 5/24/24 12:07, Ilias Apalodimas wrote:
> > Hi Michal
> >
> >
> > On Fri, 24 May 2024 at 12:45, Michal Simek <michal.simek at amd.com> wrote:
> >>
> >> Hi Ilias,
> >>
> >> On 4/26/24 15:38, Michal Simek wrote:
> >>> Hi,
> >>>
> >>> currently only DT way is supported and it is added directly to lib/smbios.c
> >>> but I think DT and env is only one way how information can be found that's
> >>> why this series is improving handling with using sysinfo driver which can
> >>> be platform specific.
> >>> At the end of day DT should be taken from smbios.c and put to sysinfo DT
> >>> driver instead of implementing it directly in this generic file.
> >>>
> >>> Thanks,
> >>> Michal
> >>>
> >>>
> >>> Michal Simek (3):
> >>>     xilinx: Enable SMBIOS command
> >>>     lib: smbios: Let detect the system via sysinfo
> >>>     lib: smbios: Detect system properties via SYSINFO IDs
> >>>
> >>>    configs/xilinx_versal_net_virt_defconfig |  1 +
> >>>    configs/xilinx_versal_virt_defconfig     |  1 +
> >>>    configs/xilinx_zynqmp_kria_defconfig     |  1 +
> >>>    configs/xilinx_zynqmp_virt_defconfig     |  1 +
> >>>    include/sysinfo.h                        |  9 +++++
> >>>    lib/smbios.c                             | 42 +++++++++++++++++++-----
> >>>    6 files changed, 46 insertions(+), 9 deletions(-)
> >>>
> >>
> >>
> >> Any comment on this series?
> >
> > It's mostly Simon that created that sysinfo stuff, so I was expecting
> > him to have a look.
> > I can put in on my bucket list, but that's going to take some time
>
> Simon: Any comment?

It looks OK to me.

Regards,
Simon