Hi Michal, On Fri, 7 Jun 2024 at 01:53, Michal Simek <michal.si...@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.si...@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