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

[Nico Huber (Code Review)]

Attention is currently required from: Felix Singer, Maxim Polyakov, Angel Pons, Christoph Pomaska.

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 17:

(1 comment)

File src/mainboard/gigabyte/ga-z170x-gaming7/romstage.c:

https://review.coreboot.org/c/coreboot/+/38920/comment/7f4d2962_ec86b3ad 
PS17, Line 7: static void mainboard_fill_rcomp_res_data(void *rcomp_ptr)
            : {
            : 	const u16 RcompResistor[3] = { 121, 75, 100 };
            : 	memcpy(rcomp_ptr, RcompResistor, sizeof(RcompResistor));
            : }
            : 
            : static void mainboard_fill_rcomp_strength_data(void *rcomp_strength_ptr)
            : {
            : 	static const u16 RcompTarget[5] = { 60, 26, 20, 20, 26 };
            : 	memcpy(rcomp_strength_ptr, RcompTarget, sizeof(RcompTarget));
            : }
> See if any of these configurations work: […]

Anything wrong with the current numbers (I suggested them based on Intel docs).
Also, why guess?