Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46404 )
Change subject: ec/google/chromeec: Add wrappers to get/set the voltage ......................................................................
Patch Set 3:
Please don't rebase the entire patch train at once. From the gerrit guidelines:
Don’t submit patch trains longer than around 20 patches unless you understand how to manage long patch trains. Long patch trains can become difficult to handle and tie up the build servers for long periods of time if not managed well. Rebasing a patch train over and over as you fix earlier patches in the train can hide comments, and make people review the code multiple times to see if anything has changed between revisions. When pushing long patch trains, it is recommended to only push the full patch train once - the initial time, and only to rebase three or four patches at a time.
https://doc.coreboot.org/getting_started/gerrit_guidelines.html
Thanks.