Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35427 )
Change subject: mb/supermicro/x11: add x11ssm-f board ......................................................................
Patch Set 13: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/35427/13/Documentation/mainboard/su... File Documentation/mainboard/supermicro/x11-lga1151-series/x11ssm-f/x11ssh-tf.md:
https://review.coreboot.org/c/coreboot/+/35427/13/Documentation/mainboard/su... PS13, Line 38: - After S5 resume coreboot detects more DIMMs than installed, causing FSP-M to fail. that's already fixed on SSH-TF
https://review.coreboot.org/c/coreboot/+/35427/13/src/mainboard/supermicro/x... File src/mainboard/supermicro/x11-lga1151-series/variants/x11ssm-f/Kconfig:
https://review.coreboot.org/c/coreboot/+/35427/13/src/mainboard/supermicro/x... PS13, Line 3: config VARIANT_DIR should be part of the baseboard Kconfig