Attention is currently required from: Dinesh Gehlot, Eric Lai, Jayvik Desai, Kapil Porwal, Lawrence Chang, Nick Vaccaro.
SH Kim has posted comments on this change by SH Kim. ( https://review.coreboot.org/c/coreboot/+/86376?usp=email )
Change subject: mb/google/nissa/var/meliks: Update memory DQ/DQS map
......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/google/brya/variants/meliks/memory.c:
https://review.coreboot.org/c/coreboot/+/86376/comment/5b979cd3_31d5f5c9?us… :
PS1, Line 71: .UserBd = BOARD_TYPE_MOBILE,
> @Lawrence, is it correct for meliks?
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/86376?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: Iae3f2c65b4d1004d1d9ebf76b099fc7f50e8365f
Gerrit-Change-Number: 86376
Gerrit-PatchSet: 2
Gerrit-Owner: SH Kim <sh_.kim(a)samsung.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: Lawrence Chang <lawrence.chang(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jayvik Desai <jayvik(a)google.com>
Gerrit-Attention: Eric Lai <ericllai(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Lawrence Chang <lawrence.chang(a)intel.corp-partner.google.com>
Gerrit-Comment-Date: Wed, 19 Feb 2025 04:55:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: SH Kim <sh_.kim(a)samsung.corp-partner.google.com>
Attention is currently required from: Dinesh Gehlot, Jayvik Desai, Kapil Porwal, Nick Vaccaro.
Subrata Banik has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/86422?usp=email )
Change subject: mb/google/brya: Do not select HAVE_ACPI_RESUME
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86422/comment/7ba0e34d_05f6eba1?us… :
PS1, Line 10: Also trigger a fail-safe board
: reset if the system resumes from S3.
> Sorry for not being clear in my comment. How can Chromebook with ChromeOS go into ACPI S3 if it’s not advertised by firmware?
Paul, I agree with your concern. To me, the change in bootblock.c doesn't make sense. S3 is the firmware's published state, and if the firmware doesn't provide any hooks to allow the system to enter S3, why do we need an additional reset at resume path? It just seems like dead code to me. I haven't adopted it for MTL, if you noticed.
--
To view, visit https://review.coreboot.org/c/coreboot/+/86422?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: Ic0dce9c7779333ca079001e3763e843a4aad9a81
Gerrit-Change-Number: 86422
Gerrit-PatchSet: 2
Gerrit-Owner: 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: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jayvik Desai <jayvik(a)google.com>
Gerrit-Attention: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Comment-Date: Wed, 19 Feb 2025 03:12:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Comment-In-Reply-To: Dinesh Gehlot <digehlot(a)google.com>
SH Kim has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86508?usp=email )
Change subject: mb/google/nissa/var/meliks: Add VBT data file
......................................................................
mb/google/nissa/var/meliks: Add VBT data file
Add data.vbt file for meliks recovery image.
Select INTEL_GMA_HAVE_VBT for meliks as it has a VBT file now. The VBT
file is copied from chrome-internal:8030469 which is the initial VBT
file for meliks.
BUG=b:394359785
TEST=emerge-nissa coreboot
Change-Id: Ifd21fbce6fdcc77ebb3d372fd7eebd7c4a2d789d
Signed-off-by: Seunghwan Kim <sh_.kim(a)samsung.corp-partner.google.com>
---
M src/mainboard/google/brya/Kconfig
A src/mainboard/google/brya/variants/meliks/data.vbt
2 files changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/86508/1
diff --git a/src/mainboard/google/brya/Kconfig b/src/mainboard/google/brya/Kconfig
index 2fda738..85c6aec 100644
--- a/src/mainboard/google/brya/Kconfig
+++ b/src/mainboard/google/brya/Kconfig
@@ -351,6 +351,7 @@
select BOARD_GOOGLE_BASEBOARD_NISSA
select DRIVERS_I2C_DA7219
select DRIVERS_INTEL_MIPI_CAMERA
+ select INTEL_GMA_HAVE_VBT
select SOC_INTEL_TWINLAKE
config BOARD_GOOGLE_MITHRAX
diff --git a/src/mainboard/google/brya/variants/meliks/data.vbt b/src/mainboard/google/brya/variants/meliks/data.vbt
new file mode 100644
index 0000000..5040d65
--- /dev/null
+++ b/src/mainboard/google/brya/variants/meliks/data.vbt
Binary files differ
--
To view, visit https://review.coreboot.org/c/coreboot/+/86508?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ifd21fbce6fdcc77ebb3d372fd7eebd7c4a2d789d
Gerrit-Change-Number: 86508
Gerrit-PatchSet: 1
Gerrit-Owner: SH Kim <sh_.kim(a)samsung.corp-partner.google.com>