Attention is currently required from: Paul Menzel, Sumeet R Pawnikar, Eric Lai. Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62789 )
Change subject: ec/google/chromeec: add fan speed rpm control ......................................................................
Patch Set 4:
(2 comments)
File src/ec/google/chromeec/acpi/ec.asl:
https://review.coreboot.org/c/coreboot/+/62789/comment/3ce6556f_be05bafd PS3, Line 85: CFSP, 16, // Set Fan speed in rpm
If required, we might update EC offset later for the conflict with this above CL (if this will get m […]
Can Active 2 Policy support more than 1 fan?
File src/ec/google/chromeec/ec_dptf_helpers.c:
https://review.coreboot.org/c/coreboot/+/62789/comment/1d8a9a51_58d1e3c7 PS4, Line 112: acpigen_write_store(); : acpigen_emit_namestring(acpi_device_path_join(ec, "CFSP")); : acpigen_emit_byte(INDEX_OP); : acpigen_emit_namestring("TFST"); : acpigen_write_integer(2); : acpigen_emit_byte(ZERO_OP); : acpigen_emit_byte(RETURN_OP); The current version of https://crrev.com/c/3533300 does not support writing to this value