Attention is currently required from: Daniel Maslowski, Nicholas Sudsgaard.
Felix Singer has posted comments on this change by Daniel Maslowski. ( https://review.coreboot.org/c/coreboot/+/84815?usp=email )
Change subject: Doc/soc/amd/family15h: Fix URLs to AMD documents
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/84815?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ibb478b56d02842dc05475235b0fe80ab6c4e7d04
Gerrit-Change-Number: 84815
Gerrit-PatchSet: 3
Gerrit-Owner: Daniel Maslowski <info(a)orangecms.org>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Maxim <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Daniel Maslowski <info(a)orangecms.org>
Gerrit-Attention: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-Comment-Date: Wed, 25 Dec 2024 13:32:56 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Angel Pons, Máté Kukri, Nicholas Chin.
Jan Philipp Groß has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/85767?usp=email )
Change subject: mb/asus: Add Maximus VI Gene (Haswell)
......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/85767/comment/04158020_3bdde0bb?us… :
PS2, Line 14: not working on vendor firmware, assuming hardware defect)
> Can be a single bent pin in the socket
Indeed! You can actually see it with the bare eye, a single bent pin.
--
To view, visit https://review.coreboot.org/c/coreboot/+/85767?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31029c78cba65cad96718132235c140c3997c815
Gerrit-Change-Number: 85767
Gerrit-PatchSet: 3
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Wed, 25 Dec 2024 13:30:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Attention is currently required from: Jan Philipp Groß, Máté Kukri, Nicholas Chin.
Angel Pons has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/85767?usp=email )
Change subject: mb/asus: Add Maximus VI Gene (Haswell)
......................................................................
Patch Set 2: Code-Review+1
Copied votes on follow-up patch sets have been updated:
* Code-Review+1 has been copied to patch set 3 (copy condition: "changekind:NO_CHANGE OR changekind:NO_CODE_CHANGE OR changekind:TRIVIAL_REBASE OR is:MIN").
(2 comments)
Patchset:
PS2:
Just a quick look
Commit Message:
https://review.coreboot.org/c/coreboot/+/85767/comment/00cee5d1_bda7a486?us… :
PS2, Line 14: not working on vendor firmware, assuming hardware defect)
Can be a single bent pin in the socket
--
To view, visit https://review.coreboot.org/c/coreboot/+/85767?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31029c78cba65cad96718132235c140c3997c815
Gerrit-Change-Number: 85767
Gerrit-PatchSet: 2
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Attention: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Wed, 25 Dec 2024 13:27:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: Angel Pons, Máté Kukri, Nicholas Chin.
Hello Angel Pons, Máté Kukri, Nicholas Chin,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85767?usp=email
to look at the new patch set (#3).
Change subject: mb/asus: Add Maximus VI Gene (Haswell)
......................................................................
mb/asus: Add Maximus VI Gene (Haswell)
This port was done via autoport and subsequent manual tweaking.
Working:
- Haswell MRC.bin
- DIMM_B1 and DIMM_B2 DDR3/DDR3L DIMM slots (DIMM_A1 and
DIMM_A2 also not working on vendor firmware, assuming
hardware defect)
- HDMI-Out Port
- RJ-45 Gigabit LAN Port
- All four back panel USB 2.0 Ports
- All four back panel ASMedia USB 3.0 Ports
- Both back panel USB 3.0 Ports
- USB 3.0 Header
- All six SATA3 6.0 Gb/s connectors by Intel
- PCI Express 3.0 x16 slot (tested with AMD RX 550 dGPU)
- PCI Express 2.0 x16 slot (tested with AMD RX 550 dGPU)
- PCI Express 2.0 x4 slots (tested with TL-WDN4800 WiFi adapter)
- HD Audio Jack (Audio output tested only)
- Front Audio Jack (Audio output tested only)
not working:
- Both SATA3 6.0 GB/s connectors by ASMedia
not (yet) tested:
- ASUS Extension Board
- ASUS mPCIe Combo II connector
- USB 2.0 headers
- Optical S/PDIF out
not (yet) working:
- Software fan control: While the Nuvoton chip is correctly
discovered, the numbering of the fan connectors is faulty,
resulting in the wrong fan being controlled.
Change-Id: I31029c78cba65cad96718132235c140c3997c815
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
---
A src/mainboard/asus/maximus_vi_gene/Kconfig
A src/mainboard/asus/maximus_vi_gene/Kconfig.name
A src/mainboard/asus/maximus_vi_gene/Makefile.mk
A src/mainboard/asus/maximus_vi_gene/acpi/ec.asl
A src/mainboard/asus/maximus_vi_gene/acpi/platform.asl
A src/mainboard/asus/maximus_vi_gene/acpi/superio.asl
A src/mainboard/asus/maximus_vi_gene/board_info.txt
A src/mainboard/asus/maximus_vi_gene/bootblock.c
A src/mainboard/asus/maximus_vi_gene/data.vbt
A src/mainboard/asus/maximus_vi_gene/devicetree.cb
A src/mainboard/asus/maximus_vi_gene/dsdt.asl
A src/mainboard/asus/maximus_vi_gene/gma-mainboard.ads
A src/mainboard/asus/maximus_vi_gene/gpio.c
A src/mainboard/asus/maximus_vi_gene/hda_verb.c
A src/mainboard/asus/maximus_vi_gene/romstage.c
15 files changed, 526 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/85767/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/85767?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31029c78cba65cad96718132235c140c3997c815
Gerrit-Change-Number: 85767
Gerrit-PatchSet: 3
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Angel Pons, Máté Kukri, Nicholas Chin.
Hello Angel Pons, Máté Kukri, Nicholas Chin,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85767?usp=email
to look at the new patch set (#2).
Change subject: mb/asus: Add Maximus VI Gene (Haswell)
......................................................................
mb/asus: Add Maximus VI Gene (Haswell)
This port was done via autoport and subsequent manual tweaking.
Working:
- Haswell MRC.bin
- DIMM_B1 and DIMM_B2 DDR3/DDR3L DIMM slots (DIMM_A1 and DIMM_A2 also
not working on vendor firmware, assuming hardware defect)
- HDMI-Out Port
- RJ-45 Gigabit LAN Port
- All four back panel USB 2.0 Ports
- All four back panel ASMedia USB 3.0 Ports
- Both back panel USB 3.0 Ports
- USB 3.0 Header
- All six SATA3 6.0 Gb/s connectors by Intel
- PCI Express 3.0 x16 slot (tested with AMD RX 550 dGPU)
- PCI Express 2.0 x16 slot (tested with AMD RX 550 dGPU)
- PCI Express 2.0 x4 slots (tested with TL-WDN4800 WiFi adapter)
- HD Audio Jack (Audio output tested only)
- Front Audio Jack (Audio output tested only)
not working:
- Both SATA3 6.0 GB/s connectors by ASMedia
not (yet) tested:
- ASUS Extension Board
- ASUS mPCIe Combo II connector
- USB 2.0 headers
- Optical S/PDIF out
not (yet) working:
- Software fan control: While the Nuvoton chip is correctly discovered,
the numbering of the fan connectors is faulty, resulting in the wrong
fan being controlled.
Change-Id: I31029c78cba65cad96718132235c140c3997c815
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
---
A src/mainboard/asus/maximus_vi_gene/Kconfig
A src/mainboard/asus/maximus_vi_gene/Kconfig.name
A src/mainboard/asus/maximus_vi_gene/Makefile.mk
A src/mainboard/asus/maximus_vi_gene/acpi/ec.asl
A src/mainboard/asus/maximus_vi_gene/acpi/platform.asl
A src/mainboard/asus/maximus_vi_gene/acpi/superio.asl
A src/mainboard/asus/maximus_vi_gene/board_info.txt
A src/mainboard/asus/maximus_vi_gene/bootblock.c
A src/mainboard/asus/maximus_vi_gene/data.vbt
A src/mainboard/asus/maximus_vi_gene/devicetree.cb
A src/mainboard/asus/maximus_vi_gene/dsdt.asl
A src/mainboard/asus/maximus_vi_gene/gma-mainboard.ads
A src/mainboard/asus/maximus_vi_gene/gpio.c
A src/mainboard/asus/maximus_vi_gene/hda_verb.c
A src/mainboard/asus/maximus_vi_gene/romstage.c
15 files changed, 526 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/85767/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/85767?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31029c78cba65cad96718132235c140c3997c815
Gerrit-Change-Number: 85767
Gerrit-PatchSet: 2
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Angel Pons, Máté Kukri, Nicholas Chin.
Jan Philipp Groß has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/85767?usp=email )
Change subject: mb/asus: Add Maximus VI Gene (Haswell)
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Hey Guys,
got tired of ASRock boards so I tried an ASUS board 😊
Hope you all enjoy your holidays!
Best regards
--
To view, visit https://review.coreboot.org/c/coreboot/+/85767?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31029c78cba65cad96718132235c140c3997c815
Gerrit-Change-Number: 85767
Gerrit-PatchSet: 1
Gerrit-Owner: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Máté Kukri <km(a)mkukri.xyz>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Wed, 25 Dec 2024 13:08:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Hung-Te Lin, Xi Chen, Yidi Lin.
Yu-Ping Wu has posted comments on this change by Yidi Lin. ( https://review.coreboot.org/c/coreboot/+/85764?usp=email )
Change subject: soc/mediatek: Distinguish pmic_init_setting function name
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/85764?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Id591bf3089aaa2148d34c77d021c17403494a776
Gerrit-Change-Number: 85764
Gerrit-PatchSet: 1
Gerrit-Owner: Yidi Lin <yidilin(a)google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Xi Chen <xixi.chen(a)mediatek.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Xi Chen <xixi.chen(a)mediatek.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Yidi Lin <yidilin(a)google.com>
Gerrit-Comment-Date: Wed, 25 Dec 2024 13:06:02 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes