Attention is currently required from: Tim Wawrzynczak, Karthik Ramasubramanian. Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54098 )
Change subject: ec/google/chromeec: OEM variables support for DPTF ......................................................................
Patch Set 1:
(4 comments)
File src/ec/google/chromeec/ec_dptf_helpers.c:
https://review.coreboot.org/c/coreboot/+/54098/comment/ae16839b_d8fe68e1 PS1, Line 135: ODVP
Same comment as below.
Done
https://review.coreboot.org/c/coreboot/+/54098/comment/8f7d86ad_03ca90b8 PS1, Line 136: acpigen_pop_len();
Actually I recently noticed there are some new `acpigen_*_end` inline functions... […]
Done
https://review.coreboot.org/c/coreboot/+/54098/comment/5b993530_230ce8f6 PS1, Line 300: ODVE
Besides Tim's comment below, the method name is mentioned as ODVP in BWG.
Done
https://review.coreboot.org/c/coreboot/+/54098/comment/e6ae68dc_e9737a7b PS1, Line 301: ODVP
shouldn't this be IETM?
Done