Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37639 )
Change subject: superio/common/conf_mode: Add op to write SSDT ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/37639/2/src/include/device/pnp.h File src/include/device/pnp.h:
https://review.coreboot.org/c/coreboot/+/37639/2/src/include/device/pnp.h@73 PS2, Line 73: void (*ssdt_enter_conf_mode)(struct device *dev, const char *idx, const char *data); : void (*ssdt_exit_conf_mode)(struct device *dev, const char *idx, const char *data);
should this be guarded by #if CONFIG(HAVE_ACPI_TABLES) to be consistent with device_operations in de […]
Done
https://review.coreboot.org/c/coreboot/+/37639/2/src/include/device/pnp.h@78 PS2, Line 78: const char *idx, const char *data
maybe add a comment that those two parameters correspond to the index and data register of the sio c […]
Done
https://review.coreboot.org/c/coreboot/+/37639/2/src/superio/common/conf_mod... File src/superio/common/conf_mode.c:
https://review.coreboot.org/c/coreboot/+/37639/2/src/superio/common/conf_mod... PS2, Line 181: 2
0x02 would be more consistent here; same thing a few lines below
Done