Attention is currently required from: Michał Żygowski, Michał Kopeć, Angel Pons.
Hello build bot (Jenkins), Michał Żygowski, Maxim Polyakov, Paul Menzel,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/63403
to look at the new patch set (#4).
Change subject: util/intelp2m: Add support for Alder Lake macro generation
......................................................................
util/intelp2m: Add support for Alder Lake macro generation
Add support for Alder Lake as a separate parsing profile, copying the
existing 'Cannon' profile and adjusting for differences in reset mapping
and GPIO macro generation.
TEST=Generate GPIO macros for MSI PRO Z690-A
Change-Id: I5871394bcb0636c2c803607ffb129441aa934417
Signed-off-by: Michał Kopeć <michal.kopec(a)3mdeb.com>
---
M util/intelp2m/config/config.go
M util/intelp2m/description.md
M util/intelp2m/main.go
M util/intelp2m/parser/parser.go
A util/intelp2m/platforms/adl/macro.go
A util/intelp2m/platforms/adl/template.go
6 files changed, 153 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/03/63403/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/63403
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5871394bcb0636c2c803607ffb129441aa934417
Gerrit-Change-Number: 63403
Gerrit-PatchSet: 4
Gerrit-Owner: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Attention: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Arthur Heymans.
Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63486 )
Change subject: soc/intel/common: Enable rom cache on all CPU threads
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63486/comment/22051300_a404346b
PS2, Line 11: However the BSP doesn't wait all threads to finish _x86_setup_mtrrs
> I think it's better to fix this than to program something temporary on APs too. […]
appreciate your suggestion:)
let me do some experiments .
Patchset:
PS2:
> Is restoring the temp mtrr also done APs. I thought it didn't.
ah yes, thanks for capturing this.
i miss this.
it only restores BSP MTRR.
--
To view, visit https://review.coreboot.org/c/coreboot/+/63486
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1d7ffc6e5f5ec49abf848d3cd9f0435c93f834dc
Gerrit-Change-Number: 63486
Gerrit-PatchSet: 2
Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Comment-Date: Fri, 08 Apr 2022 07:25:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-MessageType: comment
Attention is currently required from: Michał Żygowski, Tim Wawrzynczak.
Hello Michał Żygowski, Tim Wawrzynczak,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/63467
to look at the new patch set (#2).
Change subject: soc/intel/alderlake: add GPIO definitions for PCH-S
......................................................................
soc/intel/alderlake: add GPIO definitions for PCH-S
Add GPIO definitions for ADL-S, similarly to how TGL/TGL-H handles
the split.
Based on:
- Intel PCH-S EDS Vol1 (#621483)
- Alderlake-S FSP
- slimbootloader sources
- Linux alderlake-pinctrl driver
Change-Id: I0fd1dc645c19c33bf14424703f966271e884ed3d
Signed-off-by: Michał Kopeć <michal.kopec(a)3mdeb.com>
---
M src/soc/intel/alderlake/Makefile.inc
M src/soc/intel/alderlake/acpi/gpio.asl
M src/soc/intel/alderlake/fsp_params.c
A src/soc/intel/alderlake/gpio_pch_s.c
M src/soc/intel/alderlake/include/soc/gpio.h
A src/soc/intel/alderlake/include/soc/gpio_defs_pch_s.h
A src/soc/intel/alderlake/include/soc/gpio_soc_defs_pch_s.h
M src/soc/intel/alderlake/include/soc/pmc.h
M src/soc/intel/alderlake/romstage/fsp_params.c
9 files changed, 1,149 insertions(+), 12 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/63467/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/63467
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0fd1dc645c19c33bf14424703f966271e884ed3d
Gerrit-Change-Number: 63467
Gerrit-PatchSet: 2
Gerrit-Owner: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Taniya Das, Paul Menzel, Julius Werner.
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63289 )
Change subject: soc/qualcomm/common: Make clock_configure() check for exact matches
......................................................................
Patch Set 7:
(1 comment)
File src/soc/qualcomm/sc7280/clock.c:
https://review.coreboot.org/c/coreboot/+/63289/comment/cc859ca9_66e781ab
PS6, Line 425: return clock_configure((struct clock_rcg *)
> Ok this is a good point. […]
Assigned mdss_clk_cfg.hz.
--
To view, visit https://review.coreboot.org/c/coreboot/+/63289
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9cfad7236241f4d03ff1a56683654649658b68fc
Gerrit-Change-Number: 63289
Gerrit-PatchSet: 7
Gerrit-Owner: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Taniya Das <quic_tdas(a)quicinc.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Ravi Kumar Bokka <rbokka(a)codeaurora.org>
Gerrit-CC: mturney mturney <quic_mturney(a)quicinc.com>
Gerrit-Attention: Taniya Das <quic_tdas(a)quicinc.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Comment-Date: Fri, 08 Apr 2022 06:53:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Shelley Chen <shchen(a)google.com>
Comment-In-Reply-To: Julius Werner <jwerner(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Kane Chen.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63486 )
Change subject: soc/intel/common: Enable rom cache on all CPU threads
......................................................................
Patch Set 2: Code-Review-1
--
To view, visit https://review.coreboot.org/c/coreboot/+/63486
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1d7ffc6e5f5ec49abf848d3cd9f0435c93f834dc
Gerrit-Change-Number: 63486
Gerrit-PatchSet: 2
Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Kane Chen <kane.chen(a)intel.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 08 Apr 2022 06:52:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Kane Chen.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63486 )
Change subject: soc/intel/common: Enable rom cache on all CPU threads
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63486/comment/ca1fba5b_dc07e566
PS2, Line 11: However the BSP doesn't wait all threads to finish _x86_setup_mtrrs
I think it's better to fix this than to program something temporary on APs too. It's already hard enough to keep track of what is going on, on the BSP ;-)
Maybe you can add another AP call with a NOOP function to make sure that they all completed the previous call?
Patchset:
PS2:
Is restoring the temp mtrr also done APs. I thought it didn't.
--
To view, visit https://review.coreboot.org/c/coreboot/+/63486
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1d7ffc6e5f5ec49abf848d3cd9f0435c93f834dc
Gerrit-Change-Number: 63486
Gerrit-PatchSet: 2
Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Kane Chen <kane.chen(a)intel.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 08 Apr 2022 06:52:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment