Attention is currently required from: Nico Huber.
Angel Pons 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 1: Code-Review+1
(2 comments)
File src/northbridge/via/cx700/Makefile.mk:
https://review.coreboot.org/c/coreboot/+/82765/comment/068fd9ba_73cd0258?usp... : PS1, Line 2: ifeq ($(CONFIG_NORTHBRIDGE_VIA_CX700),y) nit: blank line before?
File src/northbridge/via/cx700/clock.c:
https://review.coreboot.org/c/coreboot/+/82765/comment/c5deb17e_bfc73381?usp... : PS1, Line 20: default: I see a few more cases in `PM_CX700_202.pdf` (System Programming Manual): https://i.imgur.com/8d0Oknq.png