Attention is currently required from: Angel Pons.
Nico Huber has posted comments on this change by Nico Huber. ( https://review.coreboot.org/c/coreboot/+/82765?usp=email )
Change subject: via: Start template for VIA C7 w/ CX700 northbridge ......................................................................
Patch Set 2:
(2 comments)
File src/northbridge/via/cx700/clock.c:
https://review.coreboot.org/c/coreboot/+/82765/comment/3412bf3e_0c354848?usp... : PS1, Line 20: default:
I see a few more cases in `PM_CX700_202.pdf` (System Programming Manual): https://i.imgur. […]
They are not officially supported. I guess it won't hurt much to add them, though. Originally I was in a do-only-exactly-what-is-needed mood, to see how a less bloated coreboot would look like. I've a better case in the queue for that though :D
File src/northbridge/via/cx700/clock.c:
https://review.coreboot.org/c/coreboot/+/82765/comment/d691b32b_bdb3ea9a?usp... : PS2, Line 26: return 200; Changed this too, because assuming too high makes delays too long, which is usually better than too short.