Change in coreboot[master]: ec/google/chromeec: Implement support for DRIVERS_ACPI_THERMAL_ZONE - coreboot-gerrit

12 May 2021


      Attention is currently required from: Raul Rangel, Furquan Shaikh.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54133 )
Change subject: ec/google/chromeec: Implement support for DRIVERS_ACPI_THERMAL_ZONE
......................................................................
Patch Set 1:
(1 comment)
File src/ec/google/chromeec/thermal_zone.c:
https://review.coreboot.org/c/coreboot/+/54133/comment/fe68c58f_c31fb62e 
PS1, Line 12: 	/* TODO: Can we search for the CrosEC device? */
partially... this is so ugly right now 😭 there is no proper `EC0` device in the devicetree hierarchy... see my comments in the next patch
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/54133
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I418b6691a7d00a4c2d89c9c1fe8f9416602be0f1
Gerrit-Change-Number: 54133
Gerrit-PatchSet: 1
Gerrit-Owner: Raul Rangel rrangel@chromium.org
Gerrit-Reviewer: Furquan Shaikh furquan@google.com
Gerrit-Reviewer: Tim Wawrzynczak twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Raul Rangel rrangel@chromium.org
Gerrit-Attention: Furquan Shaikh furquan@google.com
Gerrit-Comment-Date: Thu, 13 May 2021 04:14:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment