Attention is currently required from: Kapil Porwal, Pranava Y N.
Hello Kapil Porwal, Pranava Y N,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85827?usp=email
to look at the new patch set (#2).
Change subject: soc/intel/pantherlake: Refactor FSP-M params for debug message control
......................................................................
soc/intel/pantherlake: Refactor FSP-M params for debug message control
The fsp_params.c file is refactored to move the debug message
control logic to a separate function, fsp_control_log_level().
This function takes an FSPM_UPD pointer and a boolean value
indicating whether debug messages should be enabled or disabled.
The fill_fsp_event_handler() function is updated to call
fsp_control_log_level() with the appropriate boolean value based on
the CONFIG(CONSOLE_SERIAL) and CONFIG(FSP_ENABLE_SERIAL_DEBUG)
Kconfig options.
BUG=b:227151510
TEST=Able to build and boot google/fatcat.
Change-Id: Ie2916ce82133058464d20eed327de7c7288e78a4
Signed-off-by: Subrata Banik <subratabanik(a)google.com>
---
M src/soc/intel/pantherlake/romstage/fsp_params.c
1 file changed, 24 insertions(+), 15 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/85827/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/85827?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: Ie2916ce82133058464d20eed327de7c7288e78a4
Gerrit-Change-Number: 85827
Gerrit-PatchSet: 2
Gerrit-Owner: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Pranava Y N <pranavayn(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Pranava Y N <pranavayn(a)google.com>
Attention is currently required from: Dinesh Gehlot, Jayvik Desai, Kun Liu, Nick Vaccaro.
Kapil Porwal has posted comments on this change by Kun Liu. ( https://review.coreboot.org/c/coreboot/+/85826?usp=email )
Change subject: mb/google/nissa/var/telith: Configure Acoustic noise mitigation
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/85826?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: I11e1fae6d0b8508760090956ca6d77b012aa4bad
Gerrit-Change-Number: 85826
Gerrit-PatchSet: 1
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jayvik Desai <jayvik(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Rui Zhou <zhourui(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jayvik Desai <jayvik(a)google.com>
Gerrit-Attention: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 02 Jan 2025 08:59:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Sean Rhodes.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85715?usp=email
to look at the new patch set (#6).
Change subject: mb/starlabs/starbook: Add Meteor Lake (165H) variant
......................................................................
mb/starlabs/starbook: Add Meteor Lake (165H) variant
Change-Id: I6621585086c58d19574841314796ed9db779036e
Signed-off-by: Sean Rhodes <sean(a)starlabs.systems>
---
M src/mainboard/starlabs/starbook/Kconfig
M src/mainboard/starlabs/starbook/Kconfig.name
M src/mainboard/starlabs/starbook/dsdt.asl
A src/mainboard/starlabs/starbook/variants/mtl/Makefile.mk
A src/mainboard/starlabs/starbook/variants/mtl/board.fmd
A src/mainboard/starlabs/starbook/variants/mtl/data.vbt
A src/mainboard/starlabs/starbook/variants/mtl/devicetree.cb
A src/mainboard/starlabs/starbook/variants/mtl/devtree.c
A src/mainboard/starlabs/starbook/variants/mtl/gpio.c
A src/mainboard/starlabs/starbook/variants/mtl/hda_verb.c
A src/mainboard/starlabs/starbook/variants/mtl/romstage.c
11 files changed, 725 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/85715/6
--
To view, visit https://review.coreboot.org/c/coreboot/+/85715?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: I6621585086c58d19574841314796ed9db779036e
Gerrit-Change-Number: 85715
Gerrit-PatchSet: 6
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Attention is currently required from: Sean Rhodes.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85715?usp=email
to look at the new patch set (#5).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: mb/starlabs/starbook: Add Meteor Lake (165H) variant
......................................................................
mb/starlabs/starbook: Add Meteor Lake (165H) variant
Change-Id: I6621585086c58d19574841314796ed9db779036e
Signed-off-by: Sean Rhodes <sean(a)starlabs.systems>
---
M src/mainboard/starlabs/starbook/Kconfig
M src/mainboard/starlabs/starbook/Kconfig.name
M src/mainboard/starlabs/starbook/dsdt.asl
A src/mainboard/starlabs/starbook/variants/mtl/Makefile.mk
A src/mainboard/starlabs/starbook/variants/mtl/board.fmd
A src/mainboard/starlabs/starbook/variants/mtl/data.vbt
A src/mainboard/starlabs/starbook/variants/mtl/devicetree.cb
A src/mainboard/starlabs/starbook/variants/mtl/devtree.c
A src/mainboard/starlabs/starbook/variants/mtl/gpio.c
A src/mainboard/starlabs/starbook/variants/mtl/hda_verb.c
A src/mainboard/starlabs/starbook/variants/mtl/romstage.c
11 files changed, 742 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/85715/5
--
To view, visit https://review.coreboot.org/c/coreboot/+/85715?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: I6621585086c58d19574841314796ed9db779036e
Gerrit-Change-Number: 85715
Gerrit-PatchSet: 5
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Sean Rhodes has posted comments on this change by Sean Rhodes. ( https://review.coreboot.org/c/coreboot/+/85711?usp=email )
Change subject: mb/starlabs/starlite_adl: Put options in CFR cbtable
......................................................................
Set Ready For Review
--
To view, visit https://review.coreboot.org/c/coreboot/+/85711?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: If92f61dece7e67bfd2e29927198c9ebb81c4d363
Gerrit-Change-Number: 85711
Gerrit-PatchSet: 6
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 02 Jan 2025 08:37:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No