Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36207 )
Change subject: ec/google/chromeec: Add EC driver support for software sync ......................................................................
Patch Set 13:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36207/13/src/ec/google/chromeec/ec.... File src/ec/google/chromeec/ec.c:
https://review.coreboot.org/c/coreboot/+/36207/13/src/ec/google/chromeec/ec.... PS13, Line 682: EC_VER_FLASH_WRITE
That said, if you really want this to support older boards for some reason <snip>
I've been using a ported version of the depthcharge/vboot code for ~3yrs now to update devices running my upstream coreboot builds, so it would be nice if the newly upstreamed code wasn't less functional. I'll do a diff and push some patches for review once the entire set is merged. If it's usable great, otherwise I'll just keep it in my tree as I have been doing.
I'm not sure why anyone building "unofficial" images would really want EC software sync though. If you're building your own images you'd presumably just flash the EC once to what you want it to be. Software sync is really just needed when you want automatic updates from somewhere.
well, EC SWS provides an easy way to perform that update from coreboot though; otherwise one would have to reinvent the wheel, so to speak.