Attention is currently required from: Felix Singer, Nico Huber, Maxim Polyakov, Angel Pons.
Christoph Pomaska 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 15:
(7 comments)
File src/mainboard/gigabyte/ga-z170x-gaming7/Kconfig:
https://review.coreboot.org/c/coreboot/+/38920/comment/d090dc0f_0877733b PS13, Line 31: config MAX_CPUS : int : default 8
Useless, since 8 is the default.
Done
File src/mainboard/gigabyte/ga-z170x-gaming7/board_info.txt:
https://review.coreboot.org/c/coreboot/+/38920/comment/476970d9_a8763266 PS13, Line 2: Board URL: https://www.gigabyte.com/Motherboard/GA-Z170X-Gaming-7-rev-10
Add these .. […]
Done
File src/mainboard/gigabyte/ga-z170x-gaming7/cmos.layout:
https://review.coreboot.org/c/coreboot/+/38920/comment/cd8db8ff_e48a08c1 PS13, Line 1: ## : ## This file is part of the coreboot project. : ## : ## Copyright (C) 2007-2008 coresystems GmbH : ## Copyright (C) 2016 Intel Corporation. : ## : ## This program is free software; you can redistribute it and/or modify : ## it under the terms of the GNU General Public License as published by : ## the Free Software Foundation; version 2 of the License. : ## : ## This program is distributed in the hope that it will be useful, : ## but WITHOUT ANY WARRANTY; without even the implied warranty of : ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the : ## GNU General Public License for more details. : ##
Replace with .. […]
Done
File src/mainboard/gigabyte/ga-z170x-gaming7/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/38920/comment/c7b687a7_4ce915ff PS13, Line 53: Display_PEG
Uh, then this should indeed be Display_PEG
Added it again.
https://review.coreboot.org/c/coreboot/+/38920/comment/f228f5bc_52a57bd4 PS13, Line 243: Root port 7(x1)
I see much more PCIe slots on the "Z170X-Gaming 7" motherboard (https://thunderbolttechnology. […]
Sure, but I'll try to make the board boot first.
https://review.coreboot.org/c/coreboot/+/38920/comment/9c2b30e8_4fe53242 PS13, Line 267: device domain 0 on
Add the following below this line .. […]
Done
File src/mainboard/gigabyte/ga-z170x-gaming7/gma-mainboard.ads:
https://review.coreboot.org/c/coreboot/+/38920/comment/2028d895_0a7a7863 PS13, Line 29: DP2, -- The board's HDMI port
They are pulled high with two resistors, and that's it.
Unfortunately I'm not sure what to conclude from here. My assumption would be that the approach to list them as DP is correct but that I'm still missing something for the passive DP-to-HDMI converter and the Thunderbolt controller.
What I take from here is that I have to connect a monitor via the USB-C/tbt port to check how it is recognized. Unfortunately I am missing the hardware (a monitor that I can connect to the tbt port directly) to check. The only adapters that I have most likely have their own converter, is that an issue? I'd figure that I'll postpone the thunderbolt controller for later until I have made progress in successfully booting the board.