Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48304 )
Change subject: soc/intel/skl: set PEG port state to auto
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/48304
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3933a219b77d7234af273217df031cf627b4071f
Gerrit-Change-Number: 48304
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 04 Dec 2020 19:26:13 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48304 )
Change subject: soc/intel/skl: set PEG port state to auto
......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48304/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/48304/1//COMMIT_MSG@9
PS1, Line 9: them
> Nit: Does that mean the PEG ports? Maybe write it out.
Done
https://review.coreboot.org/c/coreboot/+/48304/1//COMMIT_MSG@13
PS1, Line 13: Test: powertop on X11SSM-F shows SoC in PC8 instead of just PC3
> Wow. Very nice! Maybe mention if the vendor firmware does that too.
I'm not sure, iirc I got to PC5 only or so on vendor; if you insist, I can test it, but not sure if that helps anything
--
To view, visit https://review.coreboot.org/c/coreboot/+/48304
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3933a219b77d7234af273217df031cf627b4071f
Gerrit-Change-Number: 48304
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 04 Dec 2020 17:49:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment
Hello Felix Singer, build bot (Jenkins), Nico Huber, Furquan Shaikh, Paul Menzel, Tim Wawrzynczak, Angel Pons, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/48304
to look at the new patch set (#2).
Change subject: soc/intel/skl: set PEG port state to auto
......................................................................
soc/intel/skl: set PEG port state to auto
Setting PegXEnable to 1, statically enables the PEG ports, which blocks
the SoC from going to deeper PC states. Instead, set the state to "auto"
(2), so the port gets disabled, when no device was detected.
Test: powertop on X11SSM-F shows SoC in PC8 instead of just PC3
Signed-off-by: Michael Niewöhner <foss(a)mniewoehner.de>
Change-Id: I3933a219b77d7234af273217df031cf627b4071f
---
M src/soc/intel/skylake/romstage/romstage.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/48304/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/48304
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3933a219b77d7234af273217df031cf627b4071f
Gerrit-Change-Number: 48304
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Tim Crawford has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47892 )
Change subject: mb/system76/oryp5: Add System76 Oryx Pro 5
......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47892/3/Documentation/mainboard/sy…
File Documentation/mainboard/system76/oryp5.md:
https://review.coreboot.org/c/coreboot/+/47892/3/Documentation/mainboard/sy…
PS3, Line 72:
> Please remove blank lines at the end of the file.
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/47892
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae6e530dcd52df3642cdfe74b65bfff5aa0dd402
Gerrit-Change-Number: 47892
Gerrit-PatchSet: 4
Gerrit-Owner: Tim Crawford <tcrawford(a)system76.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Jeremy Soller <jeremy(a)system76.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <felixsinger(a)posteo.net>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Fri, 04 Dec 2020 17:35:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Jeremy Soller, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/47892
to look at the new patch set (#4).
Change subject: mb/system76/oryp5: Add System76 Oryx Pro 5
......................................................................
mb/system76/oryp5: Add System76 Oryx Pro 5
Tested with TianoCore payload.
Not working:
- Discrete/Hybrid graphics
- Internal speakers
Change-Id: Iae6e530dcd52df3642cdfe74b65bfff5aa0dd402
Signed-off-by: Jeremy Soller <jeremy(a)system76.com>
Signed-off-by: Tim Crawford <tcrawford(a)system76.com>
---
M Documentation/mainboard/index.md
A Documentation/mainboard/system76/oryp5.md
A configs/config.system76_oryp5
A src/mainboard/system76/oryp5/Kconfig
A src/mainboard/system76/oryp5/Kconfig.name
A src/mainboard/system76/oryp5/Makefile.inc
A src/mainboard/system76/oryp5/acpi/gpe.asl
A src/mainboard/system76/oryp5/acpi/mainboard.asl
A src/mainboard/system76/oryp5/acpi/sleep.asl
A src/mainboard/system76/oryp5/board_info.txt
A src/mainboard/system76/oryp5/bootblock.c
A src/mainboard/system76/oryp5/data.vbt
A src/mainboard/system76/oryp5/devicetree.cb
A src/mainboard/system76/oryp5/dsdt.asl
A src/mainboard/system76/oryp5/gpio.c
A src/mainboard/system76/oryp5/gpio_early.c
A src/mainboard/system76/oryp5/hda_verb.c
A src/mainboard/system76/oryp5/include/mainboard/gpio.h
A src/mainboard/system76/oryp5/ramstage.c
A src/mainboard/system76/oryp5/romstage.c
20 files changed, 834 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/92/47892/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/47892
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae6e530dcd52df3642cdfe74b65bfff5aa0dd402
Gerrit-Change-Number: 47892
Gerrit-PatchSet: 4
Gerrit-Owner: Tim Crawford <tcrawford(a)system76.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Jeremy Soller <jeremy(a)system76.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <felixsinger(a)posteo.net>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48304 )
Change subject: soc/intel/skl: set PEG port state to auto
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/48304
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3933a219b77d7234af273217df031cf627b4071f
Gerrit-Change-Number: 48304
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 04 Dec 2020 17:20:25 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment