Attention is currently required from: Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Subrata Banik.
Hello Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Subrata Banik,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/82796?usp=email
to look at the new patch set (#2).
Change subject: mb/google/brya: Prefer include <soc/gpio.h> via <gpio.h>
......................................................................
mb/google/brya: Prefer include <soc/gpio.h> via <gpio.h>
Change-Id: I8d4342e88fdbac2e326568df9ad03af18a55ce7a
Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr>
---
M src/mainboard/google/brya/mainboard.c
M src/mainboard/google/brya/variants/agah/gpio.c
M src/mainboard/google/brya/variants/anahera/gpio.c
M src/mainboard/google/brya/variants/anahera4es/gpio.c
M src/mainboard/google/brya/variants/anraggar/gpio.c
M src/mainboard/google/brya/variants/aurash/gpio.c
M src/mainboard/google/brya/variants/banshee/gpio.c
M src/mainboard/google/brya/variants/baseboard/brask/gpio.c
M src/mainboard/google/brya/variants/baseboard/brask/include/baseboard/gpio.h
M src/mainboard/google/brya/variants/baseboard/brya/gpio.c
M src/mainboard/google/brya/variants/baseboard/brya/include/baseboard/gpio.h
M src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h
M src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h
M src/mainboard/google/brya/variants/baseboard/nissa/gpio.c
M src/mainboard/google/brya/variants/baseboard/nissa/include/baseboard/gpio.h
M src/mainboard/google/brya/variants/baseboard/trulo/include/baseboard/gpio.h
M src/mainboard/google/brya/variants/brask/gpio.c
M src/mainboard/google/brya/variants/brya0/gpio.c
M src/mainboard/google/brya/variants/bujia/gpio.c
M src/mainboard/google/brya/variants/craask/gpio.c
M src/mainboard/google/brya/variants/craaskov/gpio.c
M src/mainboard/google/brya/variants/crota/gpio.c
M src/mainboard/google/brya/variants/dochi/gpio.c
M src/mainboard/google/brya/variants/felwinter/gpio.c
M src/mainboard/google/brya/variants/gaelin/gpio.c
M src/mainboard/google/brya/variants/gimble/gpio.c
M src/mainboard/google/brya/variants/gimble4es/gpio.c
M src/mainboard/google/brya/variants/gladios/gpio.c
M src/mainboard/google/brya/variants/glassway/gpio.c
M src/mainboard/google/brya/variants/gothrax/gpio.c
M src/mainboard/google/brya/variants/hades/gpio.c
M src/mainboard/google/brya/variants/joxer/gpio.c
M src/mainboard/google/brya/variants/kano/gpio.c
M src/mainboard/google/brya/variants/kinox/gpio.c
M src/mainboard/google/brya/variants/kuldax/gpio.c
M src/mainboard/google/brya/variants/lisbon/gpio.c
M src/mainboard/google/brya/variants/marasov/gpio.c
M src/mainboard/google/brya/variants/mithrax/gpio.c
M src/mainboard/google/brya/variants/nereid/gpio.c
M src/mainboard/google/brya/variants/nivviks/gpio.c
M src/mainboard/google/brya/variants/nova/gpio.c
M src/mainboard/google/brya/variants/omnigul/gpio.c
M src/mainboard/google/brya/variants/orisa/gpio.c
M src/mainboard/google/brya/variants/osiris/gpio.c
M src/mainboard/google/brya/variants/pirrha/gpio.c
M src/mainboard/google/brya/variants/primus/gpio.c
M src/mainboard/google/brya/variants/pujjo/gpio.c
M src/mainboard/google/brya/variants/pujjoga/gpio.c
M src/mainboard/google/brya/variants/quandiso/gpio.c
M src/mainboard/google/brya/variants/redrix/gpio.c
M src/mainboard/google/brya/variants/redrix4es/gpio.c
M src/mainboard/google/brya/variants/skolas/gpio.c
M src/mainboard/google/brya/variants/skolas4es/gpio.c
M src/mainboard/google/brya/variants/sundance/gpio.c
M src/mainboard/google/brya/variants/taeko/gpio.c
M src/mainboard/google/brya/variants/taeko4es/gpio.c
M src/mainboard/google/brya/variants/taniks/gpio.c
M src/mainboard/google/brya/variants/trulo/gpio.c
M src/mainboard/google/brya/variants/uldren/gpio.c
M src/mainboard/google/brya/variants/vell/gpio.c
M src/mainboard/google/brya/variants/volmar/gpio.c
M src/mainboard/google/brya/variants/xivu/gpio.c
M src/mainboard/google/brya/variants/xol/gpio.c
M src/mainboard/google/brya/variants/yaviks/gpio.c
M src/mainboard/google/brya/variants/yavilla/gpio.c
M src/mainboard/google/brya/variants/zydron/gpio.c
66 files changed, 66 insertions(+), 66 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/82796/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/82796?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: I8d4342e88fdbac2e326568df9ad03af18a55ce7a
Gerrit-Change-Number: 82796
Gerrit-PatchSet: 2
Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Reviewer: Eric Lai <ericllai(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-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>
Attention is currently required from: Arthur Heymans, Christian Walter, Jincheng Li, Johnny Lin, Jonathan Zhang, Patrick Rudolph, Shuo Liu, Tim Chu.
Lean Sheng Tan has posted comments on this change by Jincheng Li. ( https://review.coreboot.org/c/coreboot/+/82795?usp=email )
Change subject: soc/intel/xeon_sp: Remove duplicated Kconfig POSTCAR_STAGE
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/82795?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: I3f94e6cc76c8f376119ffa8ec43fa1a43fb40977
Gerrit-Change-Number: 82795
Gerrit-PatchSet: 1
Gerrit-Owner: Jincheng Li <jincheng.li(a)intel.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jon.zhixiong.zhang(a)gmail.com>
Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Shuo Liu <shuo.liu(a)intel.com>
Gerrit-Reviewer: Tim Chu <Tim.Chu(a)quantatw.com>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Jonathan Zhang <jon.zhixiong.zhang(a)gmail.com>
Gerrit-Attention: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Attention: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Attention: Shuo Liu <shuo.liu(a)intel.com>
Gerrit-Attention: Jincheng Li <jincheng.li(a)intel.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Tim Chu <Tim.Chu(a)quantatw.com>
Gerrit-Comment-Date: Tue, 04 Jun 2024 07:38:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Felix Singer.
Elyes Haouas has posted comments on this change by Felix Singer. ( https://review.coreboot.org/c/coreboot/+/82789?usp=email )
Change subject: 3rdparty/intel-microcode: Update submodule to upstream main
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/82789?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: Ia34ba03a9c2f206be760133edbbadcc541ff273b
Gerrit-Change-Number: 82789
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Comment-Date: Tue, 04 Jun 2024 07:34:52 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Forest Mittelberg, Peter Marheine.
Subrata Banik has posted comments on this change by Peter Marheine. ( https://review.coreboot.org/c/coreboot/+/82775?usp=email )
Change subject: chromeec: supporting reading long battery strings
......................................................................
Patch Set 1:
(1 comment)
File src/ec/google/chromeec/acpi/battery.asl:
https://review.coreboot.org/c/coreboot/+/82775/comment/ca43a8e2_56a1a0d5?us… :
PS1, Line 63: If supported by the EC,
may be I'm missing the point here, how do we check if reading the long length Battery string command is supported or not ?
--
To view, visit https://review.coreboot.org/c/coreboot/+/82775?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: Ia29cacb7d86402490f9ac458f0be50e3f2192b04
Gerrit-Change-Number: 82775
Gerrit-PatchSet: 1
Gerrit-Owner: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Forest Mittelberg <bmbm(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Attention: Forest Mittelberg <bmbm(a)google.com>
Gerrit-Comment-Date: Tue, 04 Jun 2024 07:24:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Terry Cheong, Vamshi Krishna Gopal.
Paul Menzel has posted comments on this change by Terry Cheong. ( https://review.coreboot.org/c/coreboot/+/82794?usp=email )
Change subject: mb/google/brox: Update verb table to refine speaker amp DC offset
......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/82794/comment/46496381_00648b3f?us… :
PS2, Line 9: DC offset of class-D amplifier is larger than expectation in Brox.
Please add concrete values.
https://review.coreboot.org/c/coreboot/+/82794/comment/fe165b78_9d70d6f5?us… :
PS2, Line 10: based
: on the updated verb table provided by Realtek
Is that a comment in the bug report?
https://review.coreboot.org/c/coreboot/+/82794/comment/e5eef5eb_b199aafd?us… :
PS2, Line 15: TEST=Verify DC offset of speaker amplier is less than 3mV
How? And what value did you measure?
--
To view, visit https://review.coreboot.org/c/coreboot/+/82794?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: I776f5c24ce3c829cbd64840957c1431608cf2b85
Gerrit-Change-Number: 82794
Gerrit-PatchSet: 2
Gerrit-Owner: Terry Cheong <htcheong(a)chromium.org>
Gerrit-Reviewer: Vamshi Krishna Gopal <vamshi.krishna.gopal(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Vamshi Krishna Gopal <vamshi.krishna.gopal(a)intel.com>
Gerrit-Attention: Terry Cheong <htcheong(a)chromium.org>
Gerrit-Comment-Date: Tue, 04 Jun 2024 07:09:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Derek Huang, Kun Liu, Shelley Chen.
Paul Menzel has posted comments on this change by Kun Liu. ( https://review.coreboot.org/c/coreboot/+/82604?usp=email )
The change is no longer submittable: All-Comments-Resolved is unsatisfied now.
Change subject: spd/lp5: Add SPD for MT62F1G32D2DS-023 WT:C and K3KL8L80DM-MGCU
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/82604/comment/4c419e38_a719d72a?us… :
PS2, Line 7: SPD
SPDs
https://review.coreboot.org/c/coreboot/+/82604/comment/c8596394_ed6878c1?us… :
PS2, Line 9: Add MT62F1G32D2DS-023 WT:C and K3KL8L80DM-MGCU
Please add the vendors.
--
To view, visit https://review.coreboot.org/c/coreboot/+/82604?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: Ic5c3ed46829330f83e144cf8d18be6fa808431aa
Gerrit-Change-Number: 82604
Gerrit-PatchSet: 2
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Derek Huang <derekhuang(a)google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jian Tong <tongjian(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Dengwu Yu <yudengwu(a)huaqin.corp-partner.google.com>
Gerrit-CC: Jinfang Mao <maojinfang(a)huaqin.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Shelley Chen <shchen(a)google.com>
Gerrit-Attention: Derek Huang <derekhuang(a)google.com>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Tue, 04 Jun 2024 07:06:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No