Attention is currently required from: Angel Pons, Nicholas Chin, Paul Menzel.
Jan Philipp Groß has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/86006?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: Update devicetree
......................................................................
Patch Set 4:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86006/comment/b39394cd_d23ff8c6?us… :
PS3, Line 12: Changes have been tested on the mainboard.
> But was the testing successful? :D […]
Haha sorry my bad, I shouldn't have just implied that. I've changed the commit message accordingly.
--
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: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Thu, 16 Jan 2025 14:18:12 +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ß, Nicholas Chin, Paul Menzel.
Hello Angel Pons, Nicholas Chin, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86006?usp=email
to look at the new patch set (#4).
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.
Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
---
M src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
1 file changed, 35 insertions(+), 26 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/86006/4
--
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: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Jan Philipp Groß, Nicholas Chin, Paul Menzel.
Angel Pons has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/86006?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: Update devicetree
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86006/comment/70b31163_efa1a1ae?us… :
PS3, Line 12: Changes have been tested on the mainboard.
But was the testing successful? :D
I'd suggest testing if the board still boots to OS (I imagine you tested some Linux distro) and suspends/resumes fine. Then you can say something like:
> Tested on hardware, no regressions were observed.
> Mainboard still boots Linux, and S3 resume works.
--
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: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
Gerrit-PatchSet: 3
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Thu, 16 Jan 2025 14:15:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: Nicholas Chin, Paul Menzel.
Jan Philipp Groß has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/86007?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: List another USB debug port
......................................................................
Patch Set 3:
(1 comment)
File src/mainboard/asrock/fatal1ty_z87_professional/Kconfig:
https://review.coreboot.org/c/coreboot/+/86007/comment/50ab1618_2cd6f999?us… :
PS3, Line 25:
> I'll probably change the formatting again when squashing the boards into variants, it'll likely beco […]
Done
--
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: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia2bfb8ff2fbfab426c569198466cc27b83a85bc7
Gerrit-Change-Number: 86007
Gerrit-PatchSet: 3
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>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Thu, 16 Jan 2025 14:13:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Comment-In-Reply-To: Jan Philipp Groß <jeangrande(a)mailbox.org>
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Attention is currently required from: Jan Philipp Groß, Nicholas Chin, Paul Menzel.
Angel Pons has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/86007?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: List another USB debug port
......................................................................
Patch Set 3: Code-Review+2
(1 comment)
File src/mainboard/asrock/fatal1ty_z87_professional/Kconfig:
https://review.coreboot.org/c/coreboot/+/86007/comment/644f7067_3be5ae4c?us… :
PS3, Line 25:
> This was done to be in-line with the formatting on my other ports. […]
I'll probably change the formatting again when squashing the boards into variants, it'll likely become one comment above `config USBDEBUG_HCD_INDEX` describing the debug ports for each board.
--
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: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia2bfb8ff2fbfab426c569198466cc27b83a85bc7
Gerrit-Change-Number: 86007
Gerrit-PatchSet: 3
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>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Thu, 16 Jan 2025 14:11:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Comment-In-Reply-To: Jan Philipp Groß <jeangrande(a)mailbox.org>
Attention is currently required from: Angel Pons, Nicholas Chin, Paul Menzel.
Jan Philipp Groß has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/86006?usp=email )
Change subject: mb/asrock/fatal1ty_z87_professional: Update devicetree
......................................................................
Patch Set 3:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86006/comment/eed1ed1c_2492a5e4?us… :
PS1, Line 9: Add various previously missing settings as well as a few devices,
> The added `register` values (from autoport) make this port more complete but shouldn't have a notice […]
Tested on the mainboard in question on my benchtable, I've added that to the commit message.
File src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/86006/comment/bc0a9d30_d2bccd02?us… :
PS1, Line 4: register "gpu_ddi_e_connected" = "1"
> Hmmm, didn't we say this board doesn't have VGA?
We did indeed. Removed.
--
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: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
Gerrit-PatchSet: 3
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Thu, 16 Jan 2025 14:07:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Attention is currently required from: Jan Philipp Groß, Nicholas Chin.
Hello Angel Pons, Nicholas Chin, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86006?usp=email
to look at the new patch set (#2).
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.
Changes have been tested on the mainboard.
Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Signed-off-by: Jan Philipp Groß <jeangrande(a)mailbox.org>
---
M src/mainboard/asrock/fatal1ty_z87_professional/devicetree.cb
1 file changed, 36 insertions(+), 26 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/86006/2
--
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: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifbbb981cd62a49d112d2bc379f5941819ca70e44
Gerrit-Change-Number: 86006
Gerrit-PatchSet: 2
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jan Philipp Groß <jeangrande(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Ana Carolina Cabral.
Nick Kochlowski has posted comments on this change by Ana Carolina Cabral. ( https://review.coreboot.org/c/coreboot/+/85493?usp=email )
Change subject: mb/amd/birman_plus: Kconfig
......................................................................
Patch Set 12:
(2 comments)
File src/mainboard/amd/birman_plus/port_descriptors_glinda.c:
https://review.coreboot.org/c/coreboot/+/85493/comment/5e184677_2e8233cc?us… :
PS12, Line 42: \
Not sure what the rule is or if it is important, but should the backslash be aligned with the others?
https://review.coreboot.org/c/coreboot/+/85493/comment/b9336f37_b3d4ddc0?us… :
PS12, Line 44: 15
Should this value depend on ENABLE_WLAN02_BIRMANPLUS like in patchset 8? Same for 2-lane WWAN
--
To view, visit https://review.coreboot.org/c/coreboot/+/85493?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: Ia310ea616006479b9a052afb99d08df6a11431f4
Gerrit-Change-Number: 85493
Gerrit-PatchSet: 12
Gerrit-Owner: Ana Carolina Cabral
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-CC: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Ana Carolina Cabral
Gerrit-Comment-Date: Thu, 16 Jan 2025 13:48:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Felix Held, Fred Reitberger, Jason Glenesk, Marshall Dawson, Matt DeVillier, Paul Menzel.
Nick Kochlowski has posted comments on this change by Nick Kochlowski. ( https://review.coreboot.org/c/coreboot/+/85195?usp=email )
Change subject: soc/amd/phoenix/pci_irq_routing.c: Populate PCI IRQ routing table
......................................................................
Patch Set 11:
(2 comments)
File src/soc/amd/phoenix/pci_irq_routing.c:
https://review.coreboot.org/c/coreboot/+/85195/comment/90f39b9c_7e075788?us… :
PS9, Line 35:
> one empty line is sufficient
Done
https://review.coreboot.org/c/coreboot/+/85195/comment/2ded7b2a_18cd211c?us… :
PS9, Line 48:
> i'd mention here that while there is a 1:1 mapping of those two sets of registers on phoenix, it's n […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/85195?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: Id014ff3e675831eec42bc46c0a76271341e0e3e4
Gerrit-Change-Number: 85195
Gerrit-PatchSet: 11
Gerrit-Owner: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Thu, 16 Jan 2025 13:38:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Held <felix-coreboot(a)felixheld.de>