Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36340 )
Change subject: ec/google/chromeec: Clean up ec.[ch] ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36340/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36340/3//COMMIT_MSG@11 PS3, Line 11: Use static struct initialization to ensure all fields are initialized. I think something unintended happened here, you didn't just do the clean-up you also squashed CB:36207 into here. Please separate them back out, and please reuse the CL number 36207 for the patch adding EC functions so that we can diff it with the older version more easily (as long as you reuse the Change-Id from that, Gerrit will treat it as a new version for the existing CL).