[coreboot-gerrit] [L] Change in coreboot[master]: mb/amd/birman/ec.c: Update EC configuration for birman

[Paul Menzel (Code Review)]

Attention is currently required from: Jason Glenesk, Martin Roth, Fred Reitberger, Felix Held.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73971 )

Change subject: mb/amd/birman/ec.c: Update EC configuration for birman
......................................................................

Patch Set 7:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/73971/comment/11ad4356_f3b49cbb 
PS7, Line 7: Update EC configuration for birman
“for birman” is redundant. Maybe:
> Update EC configuration per schematic

File src/mainboard/amd/birman/ec.c:

https://review.coreboot.org/c/coreboot/+/73971/comment/c2806664_32692536 
PS7, Line 219: 	for (i = 0; i < count; i++) {
Add a blank line above?

https://review.coreboot.org/c/coreboot/+/73971/comment/3258e6ad_315727a8 
PS7, Line 222: 			cfg[i].addr, tmp);
Fits on the previous line?