[coreboot-gerrit] Change in ...coreboot[master]: smbios: Add memory type 9 system slot support

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32293 )

Change subject: smbios: Add memory type 9 system slot support
......................................................................

Patch Set 7: Code-Review+1

(1 comment)

Tested using the follow on patches.
Works fine on Lenovo T520 using the devicetree provided values (except current_usage due to a bug in coreboot).

https://review.coreboot.org/#/c/32293/4/src/arch/x86/smbios.c 
File src/arch/x86/smbios.c:

https://review.coreboot.org/#/c/32293/4/src/arch/x86/smbios.c@847 
PS4, Line 847: 	t->slot_designation = smbios_add_string(t->eos, name);
> Done. […]

yes, slot_id is still open.
I'm not sure how to integrate it, yet.
Best would be to mark it as TODO.