[coreboot-gerrit] [M] Change in coreboot[master]: acpi: Call acpi_fill_ssdt() only for enabled devices

[Felix Held (Code Review)]

Attention is currently required from: Karthik Ramasubramanian.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47148?usp=email )

Change subject: acpi: Call acpi_fill_ssdt() only for enabled devices
......................................................................

Patch Set 3:

(1 comment)

File src/drivers/intel/dptf/dptf.c:

https://review.coreboot.org/c/coreboot/+/47148/comment/c2f2a111_6d78757d :
PS1, Line 235: if (!dev->enabled)
             : 		return;
             :
> IIUC, this is because the AMD UART device is using a hybrid approach of using raw ASL as well as acp […]

i had some out of band discussion with Matt and i'd say that CB:77092 is the best feasible solution that doesn't have significant side effects