[coreboot-gerrit] Change in coreboot[master]: cpu/intel/car: Add EC software sync to Intel romstage

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36211 )

Change subject: cpu/intel/car: Add EC software sync to Intel romstage
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36211/1/src/cpu/intel/car/romstage.... 
File src/cpu/intel/car/romstage.c:

https://review.coreboot.org/c/coreboot/+/36211/1/src/cpu/intel/car/romstage.... 
PS1, Line 42: if
> Don't use preprocessor.

Done

https://review.coreboot.org/c/coreboot/+/36211/1/src/cpu/intel/car/romstage.... 
PS1, Line 62: "Failed to sync EC!\n");
> Yes, but that also applies to the printk()s everywhere in this file. […]

Ack