[coreboot-gerrit] Change in coreboot[master]: [WIP] mb/gigabyte: Add Gigabyte Z170X-Gaming 7

[Nico Huber (Code Review)]

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

Change subject: [WIP] mb/gigabyte: Add Gigabyte Z170X-Gaming 7
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38920/9/src/mainboard/gigabyte/ga-z... 
File src/mainboard/gigabyte/ga-z170x-gaming7/romstage.c:

https://review.coreboot.org/c/coreboot/+/38920/9/src/mainboard/gigabyte/ga-z... 
PS9, Line 24: 	const u16 RcompResistor[3] = { 200, 81, 162 };
> 121, 75, 100

Just take these numbers. Intel secret documents list them for SKL-H
and CML-S, plus Angel confirmed that the resistors on the bottom of
a SKL-S CPU match.

https://review.coreboot.org/c/coreboot/+/38920/9/src/mainboard/gigabyte/ga-z... 
PS9, Line 33: 		100, 40, 40, 23, 40 };
> What about this one?

Same here.