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

[Fred Reitberger (Code Review)]

Attention is currently required from: Jason Glenesk, Paul Menzel, Martin Roth, Felix Held.

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

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

Patch Set 8:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/73971/comment/13fadaf0_de3c3c9a 
PS7, Line 7: Update EC configuration for birman
> “for birman” is redundant. Maybe: […]

Good point!  That flows better

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

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

Done

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

Done