Attention is currently required from: Dinesh Gehlot, Eric Lai, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Subrata Banik.
Hello Dinesh Gehlot, Eric Lai, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Subrata Banik, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86249?usp=email
to look at the new patch set (#2).
Change subject: mb/google/nissa/var/pujjoga and pujjogatwin: Modify WWAN EM060 power on sequence
......................................................................
mb/google/nissa/var/pujjoga and pujjogatwin: Modify WWAN EM060 power on sequence
Modify GPP_F12 gpio pin to fit em060 power sequence spec.
BUG=b:391066038
TEST=Build and check WWAN EM060 power on sequence.
Change-Id: Ic4e7acf41da004d990529f9ddb0fc5a32ba03b3b
Signed-off-by: Luca Lai <luca.lai(a)lcfc.corp-partner.google.com>
---
M src/mainboard/google/brya/variants/pujjoga/gpio.c
1 file changed, 5 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/86249/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86249?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: Ic4e7acf41da004d990529f9ddb0fc5a32ba03b3b
Gerrit-Change-Number: 86249
Gerrit-PatchSet: 2
Gerrit-Owner: Luca Lai <luca.lai(a)lcfc.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: 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: Eric Lai <ericllai(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(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>
jason-ch chen has uploaded this change for review. ( https://review.coreboot.org/c/blobs/+/86248?usp=email )
Change subject: soc/mediatek/mt8186: Update SSPM firmware from v2.0.1 to v2.0.2
......................................................................
soc/mediatek/mt8186: Update SSPM firmware from v2.0.1 to v2.0.2
- Fix error handler for infra timeout in SSPM. The error handler for
infra timeout in SSPM was not handled properly, causing the ISR to
get stuck. As a result, tasks without ISR would not continue to
operate.
- Enable SSPM debug configs
BUG=b:360756645, b:374015969
TEST=get "MediaTek SSPM firmware: version 2.0.2" by
strings sspm.bin | grep version
Change-Id: Id9491ecf7dde823f5ec034ae92d4b4bcfcdc6905
Signed-off-by: jason-ch chen <jason-ch.chen(a)mediatek.corp-partner.google.com>
---
M soc/mediatek/mt8186/sspm.bin
M soc/mediatek/mt8186/sspm.bin.md5
2 files changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/blobs refs/changes/48/86248/1
diff --git a/soc/mediatek/mt8186/sspm.bin b/soc/mediatek/mt8186/sspm.bin
index f72c1f1..89cad15 100644
--- a/soc/mediatek/mt8186/sspm.bin
+++ b/soc/mediatek/mt8186/sspm.bin
Binary files differ
diff --git a/soc/mediatek/mt8186/sspm.bin.md5 b/soc/mediatek/mt8186/sspm.bin.md5
index 23130ca..534a51b 100644
--- a/soc/mediatek/mt8186/sspm.bin.md5
+++ b/soc/mediatek/mt8186/sspm.bin.md5
@@ -1 +1 @@
-634e532d6acb98e4b1807944233b8f81 *sspm.bin
+c39ba49caff7a4d94c547c391d8ef25b *sspm.bin
--
To view, visit https://review.coreboot.org/c/blobs/+/86248?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: blobs
Gerrit-Branch: main
Gerrit-Change-Id: Id9491ecf7dde823f5ec034ae92d4b4bcfcdc6905
Gerrit-Change-Number: 86248
Gerrit-PatchSet: 1
Gerrit-Owner: jason-ch chen <Jason-ch.Chen(a)mediatek.com>
Attention is currently required from: Intel coreboot Reviewers, Vladimir Serbinenko.
Hello Intel coreboot Reviewers, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86247?usp=email
to look at the new patch set (#3).
Change subject: intel/acpi: Put BSP as the first entry
......................................................................
intel/acpi: Put BSP as the first entry
Linux complains in dmesg as a firmware bug that BSP is not the first entry.
NetBSD hangs and OpenBSD panics early on boot.
With this patch I was able to boot NetBSD and OpenBSD on darp10-b when loaded
in GRUB.
Note: vanilla bootloaders for NetBSD and OpenBSD still result in an apparent
hang for an unknown reason.
Change-Id: I520a2e080c9f07a5866729ae2283990d20c0d691
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
---
M src/soc/intel/common/block/acpi/cpu_hybrid.c
1 file changed, 9 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/86247/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/86247?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: I520a2e080c9f07a5866729ae2283990d20c0d691
Gerrit-Change-Number: 86247
Gerrit-PatchSet: 3
Gerrit-Owner: Vladimir Serbinenko <phcoder(a)gmail.com>
Gerrit-Reviewer: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>
Gerrit-Attention: Vladimir Serbinenko <phcoder(a)gmail.com>
Attention is currently required from: Cliff Huang, Lance Zhao, Tim Wawrzynczak, Vladimir Serbinenko.
Hello Cliff Huang, Lance Zhao, Tim Wawrzynczak, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86246?usp=email
to look at the new patch set (#3).
Change subject: acpi: Zero-out MADT before filling it
......................................................................
acpi: Zero-out MADT before filling it
Otherwise flags field is pre-filled with random garbage.
Change-Id: Ie5dc0720183b8ba07561341003f28a86ffce911e
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
---
M src/acpi/acpi.c
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/86246/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/86246?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: Ie5dc0720183b8ba07561341003f28a86ffce911e
Gerrit-Change-Number: 86246
Gerrit-PatchSet: 3
Gerrit-Owner: Vladimir Serbinenko <phcoder(a)gmail.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Attention: Vladimir Serbinenko <phcoder(a)gmail.com>
Attention is currently required from: Cliff Huang, Lance Zhao, Tim Wawrzynczak, Vladimir Serbinenko.
Hello Cliff Huang, Lance Zhao, Tim Wawrzynczak, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86246?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: acpi: Zero-out MADT before filling it
......................................................................
acpi: Zero-out MADT before filling it
Otherwise flags field is pre-filled with random garbage.
Change-Id: Ie5dc0720183b8ba07561341003f28a86ffce911e
---
M src/acpi/acpi.c
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/86246/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86246?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: Ie5dc0720183b8ba07561341003f28a86ffce911e
Gerrit-Change-Number: 86246
Gerrit-PatchSet: 2
Gerrit-Owner: Vladimir Serbinenko <phcoder(a)gmail.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Attention: Vladimir Serbinenko <phcoder(a)gmail.com>
Attention is currently required from: Intel coreboot Reviewers.
Vladimir Serbinenko has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86247?usp=email )
Change subject: intel/acpi: Put BSP as the first entry
......................................................................
intel/acpi: Put BSP as the first entry
Linux complains in dmesg as a firmware bug that BSP is not the first entry.
NetBSD hangs and OpenBSD panics early on boot.
With this patch I was able to boot NetBSD and OpenBSD on darp10-b when loaded
in GRUB.
Note: vanilla bootloaders for NetBSD and OpenBSD still result in an apparent
hang for an unknown reason.
Change-Id: I520a2e080c9f07a5866729ae2283990d20c0d691
---
M src/soc/intel/common/block/acpi/cpu_hybrid.c
1 file changed, 9 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/86247/1
diff --git a/src/soc/intel/common/block/acpi/cpu_hybrid.c b/src/soc/intel/common/block/acpi/cpu_hybrid.c
index ee4777a..a6b9c21 100644
--- a/src/soc/intel/common/block/acpi/cpu_hybrid.c
+++ b/src/soc/intel/common/block/acpi/cpu_hybrid.c
@@ -48,10 +48,17 @@
int32_t eff_apic_ids[CONFIG_MAX_CPUS] = {0};
extern struct cpu_info cpu_infos[];
uint32_t i, j = 0;
+ u32 bsp_lapicid = lapicid();
+
+ /* As per spec first comes BSP. */
+ cpu_apic_info.apic_ids[0] = bsp_lapicid;
+ perf_core_cnt = 1;
for (i = 0; i < ARRAY_SIZE(cpu_apic_info.apic_ids); i++) {
if (!cpu_infos[i].cpu)
continue;
+ if (cpu_infos[i].cpu->path.apic.apic_id == bsp_lapicid)
+ continue;
if (cpu_infos[i].cpu->path.apic.core_type == CPU_TYPE_PERF)
cpu_apic_info.apic_ids[perf_core_cnt++] =
cpu_infos[i].cpu->path.apic.apic_id;
@@ -60,8 +67,8 @@
cpu_infos[i].cpu->path.apic.apic_id;
}
- if (perf_core_cnt > 1)
- bubblesort(cpu_apic_info.apic_ids, perf_core_cnt, NUM_ASCENDING);
+ if (perf_core_cnt > 2)
+ bubblesort(cpu_apic_info.apic_ids + 1, perf_core_cnt - 1, NUM_ASCENDING);
for (i = perf_core_cnt; j < eff_core_cnt; i++, j++)
cpu_apic_info.apic_ids[i] = eff_apic_ids[j];
--
To view, visit https://review.coreboot.org/c/coreboot/+/86247?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: I520a2e080c9f07a5866729ae2283990d20c0d691
Gerrit-Change-Number: 86247
Gerrit-PatchSet: 1
Gerrit-Owner: Vladimir Serbinenko <phcoder(a)gmail.com>
Gerrit-Reviewer: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>
Gerrit-Attention: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>