Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37560 )
Change subject: src: Conditionally include TEVT ......................................................................
Patch Set 3:
Patch Set 2:
On Sat, Dec 7, 2019 at 18:40 Matt DeVillier wrote: perhaps I am misunderstanding the purpose of the patch. Are you wanting to decouple DPTF >support from TEVT?
The existing code utilized HAVE_THERM_EVENT_HANDLER define to enable calling TEVT from the chromeec dptf asl code. The HAVE_THERM_EVENT_HANDLER define is created in chipset or board asl code at the location where the TEVT event is defined. This is fine for all boards that are using chromeec. None of the other EC code implements calling the TEVT event and you end up with a method that will never be called. To prevent this situation (and the warning generated because of it) we added this dependency.
Additionally we changed the code for the google wilco ec as this called the TEVT method unconditionally. Now this is inline with the chromeec asl code.
if