[coreboot-gerrit] [M] Change in coreboot[master]: mb/lenovo/x200: Configure ck505 clockgen

[Swift Geek (Sebastian Grzywna) (Code Review)]

Attention is currently required from: Stefan Ott, Bill XIE, Arthur Heymans, Alexander Couzens.

Swift Geek (Sebastian Grzywna) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74418 )

Change subject: mb/lenovo/x200: Configure ck505 clockgen
......................................................................

Patch Set 5:

(1 comment)

File src/mainboard/lenovo/x200/mainboard.c:

https://review.coreboot.org/c/coreboot/+/74418/comment/599ff14f_a45b0d4f 
PS5, Line 16: 	set_gpio(42, GPIO_LEVEL_LOW);
Would it be feasible to restore here state captured in start of ck505_set_mux_enable(), instead of hardcoding it ?
That i2c bus switch in thinkpad is already causing enough issues for memtest86+ and other users, so I think it would be better if code wouldn't depend on particular expectation there for default state of the switch, as it would make eventual switch a bit be easier perhaps.