[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede/var/magolor: Configure I2C high and low time

[Paul Menzel (Code Review)]

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

Change subject: mb/google/dedede/var/magolor: Configure I2C high and low time
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/c/coreboot/+/46613/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/46613/1//COMMIT_MSG@12 
PS1, Line 12: TEST=Measured the I2C bus frequency reduce to 387 KHz.
What was the value before?

https://review.coreboot.org/c/coreboot/+/46613/1/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/magolor/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/46613/1/src/mainboard/google/dedede... 
PS1, Line 27: .speed = I2C_SPEED_FAST,
            : 			.speed_config[0] = {
            : 				.speed = I2C_SPEED_FAST,
Is `.speed` really specified twice?