Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86007?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: List another USB debug port
......................................................................
mb/asrock/fatal1ty_z87_professional: List another USB debug port
List an additional USB debug port on one of the USB-2.0-Headers.
Change-Id: Ia2bfb8ff2fbfab426c569198466cc27b83a85bc7
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/86007
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Paul Menzel <paulepanter(a)mailbox.org>
Reviewed-by: Angel Pons <th3fanbus(a)gmail.com>
---
M src/mainboard/asrock/fatal1ty_z87_professional/Kconfig
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Angel Pons: Looks good to me, approved
Paul Menzel: Looks good to me, but someone else must approve
build bot (Jenkins): Verified
diff --git a/src/mainboard/asrock/fatal1ty_z87_professional/Kconfig b/src/mainboard/asrock/fatal1ty_z87_professional/Kconfig
index b1600ef..c83a900 100644
--- a/src/mainboard/asrock/fatal1ty_z87_professional/Kconfig
+++ b/src/mainboard/asrock/fatal1ty_z87_professional/Kconfig
@@ -22,5 +22,5 @@
default "Fatal1ty Z87 Professional"
config USBDEBUG_HCD_INDEX
- default 2 # USB-2.0-Header with the designation USB4_5
+ default 2 # Headers: USB45, USB3_6_7
endif
--
To view, visit https://review.coreboot.org/c/coreboot/+/86007?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia2bfb8ff2fbfab426c569198466cc27b83a85bc7
Gerrit-Change-Number: 86007
Gerrit-PatchSet: 4
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86006?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: Update devicetree
......................................................................
mb/asrock/fatal1ty_z87_professional: Update devicetree
Add various previously missing settings as well as a few devices,
also tidy up the comments and make whitespace consistent.
Tested on hardware, no regressions were observed.
Mainboard boots Arch Linux with EDKII payload, S3 suspend and
resume works, as before.
Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/86006
Reviewed-by: Paul Menzel <paulepanter(a)mailbox.org>
Reviewed-by: Angel Pons <th3fanbus(a)gmail.com>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
1 file changed, 35 insertions(+), 26 deletions(-)
Approvals:
Paul Menzel: Looks good to me, but someone else must approve
Angel Pons: Looks good to me, approved
build bot (Jenkins): Verified
diff --git a/src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb b/src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
index 5b20f3c..c1acd5c 100644
--- a/src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
+++ b/src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
@@ -1,4 +1,8 @@
+# SPDX-License-Identifier: GPL-2.0-or-later
+
chip northbridge/intel/haswell
+ register "gpu_dp_c_hotplug" = "4"
+ register "gpu_dp_d_hotplug" = "4"
register "spd_addresses" = "{0x50, 0x51, 0x52, 0x53}"
chip cpu/intel/haswell
device cpu_cluster 0 on
@@ -8,18 +12,20 @@
device domain 0 on
ops haswell_pci_domain_ops
- device pci 00.0 on # Desktop Host bridge
+ device pci 00.0 on # Desktop Host bridge
subsystemid 0x1849 0x0c00
end
- device pci 01.0 on # PEG
+ device pci 01.0 on # PCIE2
subsystemid 0x1849 0x0c01
end
- device pci 01.1 on end
- device pci 01.2 on end
- device pci 02.0 on # iGPU
+ device pci 01.1 on # PCIE3
+ end
+ device pci 01.2 on # PCIE4
+ end
+ device pci 02.0 on # iGPU
subsystemid 0x1849 0x0412
end
- device pci 03.0 on # Mini-HD audio
+ device pci 03.0 on # Mini-HD audio
subsystemid 0x1849 0x0c0c
end
@@ -27,48 +33,51 @@
register "gen1_dec" = "0x000c0291"
register "gen2_dec" = "0x000c0241"
register "gen3_dec" = "0x000c0251"
+ register "gpe0_en_1" = "0x2246"
+ register "sata_port0_gen3_dtle" = "0x2"
+ register "sata_port1_gen3_dtle" = "0x2"
register "sata_port_map" = "0x3f"
- device pci 14.0 on # xHCI Controller
+ device pci 14.0 on # xHCI Controller
subsystemid 0x1849 0x8c31
end
- device pci 16.0 on # MEI #1
+ device pci 16.0 on # MEI #1
subsystemid 0x1849 0x8c3a
end
- device pci 16.1 off end # MEI #2
- device pci 19.0 on # Intel Gigabit Ethernet
+ device pci 16.1 off end # MEI #2
+ device pci 19.0 on # Intel Gigabit Ethernet
subsystemid 0x1849 0x153b
end
- device pci 1a.0 on # USB2 EHCI #2
+ device pci 1a.0 on # USB2 EHCI #2
subsystemid 0x1849 0x8c2d
end
- device pci 1b.0 on # High Definition Audio
+ device pci 1b.0 on # High Definition Audio
subsystemid 0x1849 0x1020
end
- device pci 1c.0 off end # RP #1
- device pci 1c.1 on # RP #2: mPCIe slot
+ device pci 1c.0 off end # RP #1
+ device pci 1c.1 on # RP #2: mPCIe slot
subsystemid 0x1849 0x8c12
end
- device pci 1c.2 on # RP #3: ASM1061 SATA controller
+ device pci 1c.2 on # RP #3: ASM1061 SATA controller
subsystemid 0x1849 0x8c14
end
- device pci 1c.3 on # RP #4: Intel I211 GbE
+ device pci 1c.3 on # RP #4: Intel I211 GbE
subsystemid 0x1849 0x8c16
device pci 00.0 on end
end
- device pci 1c.4 on # RP #5: ASM1061 SATA controller
+ device pci 1c.4 on # RP #5: ASM1061 SATA controller
subsystemid 0x1849 0x8c18
end
- device pci 1c.5 on # RP #6: PCIe x1 slot
+ device pci 1c.5 on # RP #6: PCIE1
subsystemid 0x1849 0x8c1a
end
- device pci 1c.6 on # RP #7: ASM1083 PCIe-to-PCI bridge
+ device pci 1c.6 on # RP #7: ASM1083 PCIe-to-PCI bridge
subsystemid 0x1849 0x8c1c
end
- device pci 1c.7 off end # RP #8
- device pci 1d.0 on # USB2 EHCI #1
+ device pci 1c.7 off end # RP #8
+ device pci 1d.0 on # USB2 EHCI #1
subsystemid 0x1849 0x8c26
end
- device pci 1f.0 on # LPC bridge
+ device pci 1f.0 on # LPC bridge
subsystemid 0x1849 0x8c44
chip superio/nuvoton/nct6776
device pnp 2e.0 off end # Floppy
@@ -123,12 +132,12 @@
device pnp 2e.17 off end # GPIOA
end
end
- device pci 1f.2 on end # SATA Controller (AHCI)
- device pci 1f.3 on # SMBus
+ device pci 1f.2 on end # SATA Controller (AHCI)
+ device pci 1f.3 on # SMBus
subsystemid 0x1849 0x8c22
end
- device pci 1f.5 off end # SATA Controller (Legacy)
- device pci 1f.6 off end # Thermal
+ device pci 1f.5 off end # SATA Controller (Legacy)
+ device pci 1f.6 off end # Thermal
end
end
end
--
To view, visit https://review.coreboot.org/c/coreboot/+/86006?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
Gerrit-PatchSet: 6
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>