Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42141 )
Change subject: ec/google/wilco: Fix comment about enclosure type ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42141/1/src/ec/google/wilco/acpi/vb... File src/ec/google/wilco/acpi/vbtn.asl:
https://review.coreboot.org/c/coreboot/+/42141/1/src/ec/google/wilco/acpi/vb... PS1, Line 8: * have SMBIOS_ENCLOSURE_CONVERTIBLE for the SMBIOS enclosure type to
IMHO, this makes it too tempting to manually set the SMBIOS enclosure type for this code to work. […]
My proposal is to drop SYSTEM_TYPE_xx Kconfig.