[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 16: Code-Review-2

(2 comments)

Regressed since the last time I looked at it.

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 231: dev->chip_info;
> Yes you are right. It would be better if I use config_of () here. […]

config_of() is a shorthand to avoid an explicit null-check which is not
necessary here.

https://review.coreboot.org/c/coreboot/+/44476/13/src/ec/kontron/kempld/kemp... 
PS13, Line 238: 	if (!config)
> Done

That's not true. It dies iff `dev` in NULL.