[coreboot-gerrit] Change in coreboot[master]: ec/kontron/kempld: add option to configure I2C frequency

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44476 )

Change subject: ec/kontron/kempld: add option to configure I2C frequency
......................................................................

Patch Set 14:

(2 comments)

https://review.coreboot.org/c/coreboot/+/44476/13/src/ec/kontron/kempld/kemp... 
File src/ec/kontron/kempld/kempld_i2c.c:

https://review.coreboot.org/c/coreboot/+/44476/13/src/ec/kontron/kempld/kemp... 
PS13, Line 254: BIOS_WARNING
> In that case, is BIOS_INFO okay?

The description says INFO is for expected things, I'd use NOTICE.

https://review.coreboot.org/c/coreboot/+/44476/13/src/ec/kontron/kempld/kemp... 
PS13, Line 256: 				config->i2c_frequency);
> Done

Sorry, maybe I should have mentioned. If there are braces on one path, all paths should
get them, c.f. https://doc.coreboot.org/coding_style.html#placing-braces-and-spaces