Attention is currently required from: Kun Liu, Shelley Chen.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81817?usp=email )
Change subject: mb/google/brox: Create lotso variant
......................................................................
Patch Set 3:
(2 comments)
File src/mainboard/google/brox/Kconfig.name:
https://review.coreboot.org/c/coreboot/+/81817/comment/f0c7019c_bdf1d02f :
PS3, Line 13: select BOARD_GOOGLE_BA
Move this out of this file and add as:
```
config BOARD_GOOGLE_LOTSO
select BOARD_GOOGLE_BASEBOARD_BROX
```
in Kconfig file.
File src/mainboard/google/brox/variants/lotso/Makefile.mk:
https://review.coreboot.org/c/coreboot/+/81817/comment/1549d6a8_5952ffa8 :
PS3, Line 1: bootblock-y += gpio.c
Please add SPDX License Identifier for the makefile.
```
## SPDX-License-Identifier: GPL-2.0-only
```
--
To view, visit https://review.coreboot.org/c/coreboot/+/81817?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I5939127f9e6abe5b792c0627d9d67e739b27083b
Gerrit-Change-Number: 81817
Gerrit-PatchSet: 3
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Attention: Shelley Chen <shchen(a)google.com>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Mon, 15 Apr 2024 15:24:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Felix Held, Maxim.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81915?usp=email )
Change subject: superio/fintek/f81866d: Fix UART numbers
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/81915?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I996b8e56d943e26ab426f1802ada07cde805286d
Gerrit-Change-Number: 81915
Gerrit-PatchSet: 1
Gerrit-Owner: Maxim <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Maxim <max.senia.poliak(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Mon, 15 Apr 2024 14:49:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Alexander Couzens, Angel Pons, Evgeny Zinoviev, Keith Hui, Kevin Keijzer.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81882?usp=email )
Change subject: SNB/BD82x6x boards: Remove old USB configurations
......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/81882/comment/b6e704b8_63af6248 :
PS2, Line 7: SNB/BD82x6x boards:
Maybe (to start with lowercase letter):
> mb: SNB/BD82x6x: …
https://review.coreboot.org/c/coreboot/+/81882/comment/306eb605_4914a61d :
PS2, Line 13:
It’d be great if you documented the boards, the tested boards.
Patchset:
PS2:
Great work. Thank you.
--
To view, visit https://review.coreboot.org/c/coreboot/+/81882?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ie1cd21077136998a6e90050c95263f2efed68a67
Gerrit-Change-Number: 81882
Gerrit-PatchSet: 2
Gerrit-Owner: Keith Hui <buurin(a)gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Evgeny Zinoviev <me(a)ch1p.io>
Gerrit-Reviewer: Kevin Keijzer <kevin(a)quietlife.nl>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Kevin Keijzer <kevin(a)quietlife.nl>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Attention: Evgeny Zinoviev <me(a)ch1p.io>
Gerrit-Attention: Keith Hui <buurin(a)gmail.com>
Gerrit-Comment-Date: Mon, 15 Apr 2024 14:18:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Aseda Aboagye.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81863?usp=email )
Change subject: acpigen_ps2_keybd: Add support for dictation key
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/81863/comment/d4ccee89_c108e16f :
PS2, Line 14: Flash DUT that emits a scancode for a dictation key, verify that it
: is mapped to KEY_DICTATE in the Linux kernel.
Next time, it’d be great if you also documented your comments, so it’s easier for people to reproduce. (I guess `evtest` or `libinput` was used.)
--
To view, visit https://review.coreboot.org/c/coreboot/+/81863?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iabc56662a9d6b29e84ab81ed93cb46d2e8372de9
Gerrit-Change-Number: 81863
Gerrit-PatchSet: 2
Gerrit-Owner: Aseda Aboagye <aaboagye(a)google.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Aseda Aboagye <aaboagye(a)google.com>
Gerrit-Comment-Date: Mon, 15 Apr 2024 14:16:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Paul Menzel, Subrata Banik.
Saurabh Mishra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81846?usp=email )
Change subject: soc/intel/common: Add Lunar Lake CNVI device IDs
......................................................................
Patch Set 3:
(1 comment)
File src/soc/intel/common/block/cnvi/cnvi.c:
https://review.coreboot.org/c/coreboot/+/81846/comment/dbac843f_8291cf3f :
PS2, Line 27: PCI_DID_INTEL_LNL_CNVI_WIFI_3,
> > `please, no spaces at the start of a line` […]
ACK.
--
To view, visit https://review.coreboot.org/c/coreboot/+/81846?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I5a0a3fbc9f43a6a573e33fcf3901055e10faaed1
Gerrit-Change-Number: 81846
Gerrit-PatchSet: 3
Gerrit-Owner: Saurabh Mishra <mishra.saurabh(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Appukuttan V K <appukuttan.vk(a)intel.com>
Gerrit-CC: Ashish Kumar Mishra <ashish.k.mishra(a)intel.com>
Gerrit-CC: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-CC: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-CC: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Sanju Jose Thottan <sanjujose.thottan(a)intel.com>
Gerrit-CC: Wonkyu Kim <wonkyu.kim(a)intel.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Mon, 15 Apr 2024 13:54:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment