Attention is currently required from: Tarun Tuli, Subrata Banik, Kapil Porwal.
Jay Patel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74877 )
Change subject: src/mb/googe/rex/variants/rex0_ISH/overridetree: Enable ISH
......................................................................
src/mb/googe/rex/variants/rex0_ISH/overridetree: Enable ISH
This commit enables ISH and assigns ISH firmware file name.
BUG=
TEST="lspci" renders PCI ID 12.0 for Serial Controller: Device 7e45.
It is also required that the ISH has been enabled using mFIT.
Signed-off-by: Jay Patel <jay2.patel(a)intel.com>
Change-Id: Idc8adaf62727d5bc00a36408e2850b56d83fe5f2
---
M src/mainboard/google/rex/variants/rex0_ISH/overridetree.cb
1 file changed, 22 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/74877/1
diff --git a/src/mainboard/google/rex/variants/rex0_ISH/overridetree.cb b/src/mainboard/google/rex/variants/rex0_ISH/overridetree.cb
index ef517df..8e43bde 100644
--- a/src/mainboard/google/rex/variants/rex0_ISH/overridetree.cb
+++ b/src/mainboard/google/rex/variants/rex0_ISH/overridetree.cb
@@ -269,6 +269,12 @@
.flags = PCIE_RP_LTR | PCIE_RP_AER,
}"
end #PCIE9 SSD card
+ device ref ish on
+ chip drivers/intel/ish
+ register "firmware_name" = ""rex_ish.bin""
+ device generic 0 on end
+ end
+ end
device ref tbt_pcie_rp0 on end
device ref tbt_pcie_rp2 on end
device ref tcss_xhci on
--
To view, visit https://review.coreboot.org/c/coreboot/+/74877
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Idc8adaf62727d5bc00a36408e2850b56d83fe5f2
Gerrit-Change-Number: 74877
Gerrit-PatchSet: 1
Gerrit-Owner: Jay Patel <jay2.patel(a)intel.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-MessageType: newchange
Attention is currently required from: Tarun Tuli, Cliff Huang.
Hello build bot (Jenkins), Cliff Huang, Tarun Tuli,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/74842
to look at the new patch set (#4).
Change subject: mb/google/brya: configure GPP_D1 using PAD_CFG_OD_GPO
......................................................................
mb/google/brya: configure GPP_D1 using PAD_CFG_OD_GPO
Use the new PAD_CFG_OD() macro to configure the GPP_D1 gpio for
the open-drain FP_RST_ODL signal for both brya0 and skolas.
BUG=b:278732600
BRANCH=firmware-brya-14505.B
TEST='emerge-brya coreboot chromeos-bootimage', flash and boot skolas to
kernel, sleep and wake device and use fingerprint for wake
authentication.
Change-Id: I66ff974c6222e38a7584a756dba0e4714d1e5797
Signed-off-by: Nick Vaccaro <nvaccaro(a)google.com>
---
M src/mainboard/google/brya/variants/baseboard/brya/gpio.c
M src/mainboard/google/brya/variants/brya0/gpio.c
M src/mainboard/google/brya/variants/brya0/variant.c
M src/mainboard/google/brya/variants/skolas/gpio.c
4 files changed, 40 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/74842/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/74842
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I66ff974c6222e38a7584a756dba0e4714d1e5797
Gerrit-Change-Number: 74842
Gerrit-PatchSet: 4
Gerrit-Owner: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Sean Rhodes, Benjamin Doron, Christian Walter, Maximilian Brune, Lean Sheng Tan.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74122 )
Change subject: [WIP] mb/prodrive/atlas: Put options in CFR cbtable
......................................................................
Patch Set 3:
(4 comments)
File src/mainboard/prodrive/atlas/cfr.c:
https://review.coreboot.org/c/coreboot/+/74122/comment/64aca694_d4e27d00
PS2, Line 123: .ui_helptext = "Specify what to do when power is re-applied "
: "after a power loss. This option has no effect "
: "on systems without a RTC battery.", /* TODO: check */
:
> The help text is correct, to my knowledge. […]
Will try
File src/mainboard/prodrive/atlas/cfr.c:
https://review.coreboot.org/c/coreboot/+/74122/comment/fea72a0d_e8b0309b
PS3, Line 77: .opt_name = "DONOTSTOREME_sn", /* TODO: Make optional? */
Name still needed so that the value can be read.
https://review.coreboot.org/c/coreboot/+/74122/comment/ca703e39_8a112188
PS3, Line 79: .flags = CFR_OPTFLAG_READONLY,
Add another flag to indicate an option is volatile.
https://review.coreboot.org/c/coreboot/+/74122/comment/13d4729c_43e8d98c
PS3, Line 92: .ui_name = "", /* Empty UI name */
Define new tag for text fields (i.e. "comments"), these do not have an underlying variable but they should still have an option ID for dependencies
--
To view, visit https://review.coreboot.org/c/coreboot/+/74122
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I47585a9a6f94ab5005f2ab63a0df267c0caef231
Gerrit-Change-Number: 74122
Gerrit-PatchSet: 3
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Mon, 01 May 2023 20:11:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-MessageType: comment
Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74905 )
Change subject: soc/amd/phoenix: Add default vBIOS ID and location
......................................................................
soc/amd/phoenix: Add default vBIOS ID and location
Signed-off-by: Martin Roth <gaumless(a)gmail.com>
Change-Id: Iadc32f4dbf8bd48d8666a213d7b5f3ba42175a90
---
M src/soc/amd/phoenix/Kconfig
1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/74905/1
diff --git a/src/soc/amd/phoenix/Kconfig b/src/soc/amd/phoenix/Kconfig
index 6df8fe7..f1b9e63 100644
--- a/src/soc/amd/phoenix/Kconfig
+++ b/src/soc/amd/phoenix/Kconfig
@@ -247,6 +247,17 @@
help
Maximum number of threads the platform can have.
+config VGA_BIOS_ID
+ string
+ default "1002,15BF"
+ help
+ The default VGA BIOS PCI vendor/device ID should be set to the
+ result of the map_oprom_vendev() function in graphics.c.
+
+config VGA_BIOS_FILE
+ string
+ default "3rdparty/amd_blobs/phoenix/Vbios.bin"
+
config CONSOLE_UART_BASE_ADDRESS
depends on CONSOLE_SERIAL && AMD_SOC_CONSOLE_UART
hex
--
To view, visit https://review.coreboot.org/c/coreboot/+/74905
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iadc32f4dbf8bd48d8666a213d7b5f3ba42175a90
Gerrit-Change-Number: 74905
Gerrit-PatchSet: 1
Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com>
Gerrit-MessageType: newchange
Attention is currently required from: Tarun Tuli, Cliff Huang.
Hello build bot (Jenkins), Cliff Huang, Tarun Tuli,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/74842
to look at the new patch set (#3).
Change subject: mb/google/brya: configure GPP_D1 using PAD_CFG_OD_GPO
......................................................................
mb/google/brya: configure GPP_D1 using PAD_CFG_OD_GPO
Use the new PAD_CFG_OD() macro to configure the GPP_D1 gpio for
the open-drain FP_RST_ODL signal.
BUG=b:278732600
BRANCH=firmware-brya-14505.B
TEST='emerge-brya coreboot chromeos-bootimage', flash and boot skolas to
kernel, sleep and wake device and use fingerprint for wake
authentication.
Change-Id: I66ff974c6222e38a7584a756dba0e4714d1e5797
Signed-off-by: Nick Vaccaro <nvaccaro(a)google.com>
---
M src/mainboard/google/brya/variants/baseboard/brya/gpio.c
M src/mainboard/google/brya/variants/brya0/gpio.c
M src/mainboard/google/brya/variants/brya0/variant.c
3 files changed, 36 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/74842/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/74842
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I66ff974c6222e38a7584a756dba0e4714d1e5797
Gerrit-Change-Number: 74842
Gerrit-PatchSet: 3
Gerrit-Owner: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-MessageType: newpatchset