Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37807 )
Change subject: util/lint: Provide some exemption to common parts of a vendor's MB
......................................................................
util/lint: Provide some exemption to common parts of a vendor's MB
Adding common definitions for a vendor's mainboard(eg. CB:37705) requires
creating dummy board_info.txt and Kconfig.name files due to lint checks.
Update the lint scripts to provide exemption to common definitions for a
vendor's mainboard.
BUG=None
TEST=Ensure that no lint errors are observed when common definitions are
added for a vendor's mainboard.
Signed-off-by: Karthikeyan Ramasubramanian <kramasub(a)google.com>
Change-Id: I55c3474c17eacd008fee5f4089da33d4f7d6002a
---
M util/lint/lint-stable-005-board-status
M util/lint/lint-stable-006-board-name
2 files changed, 7 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/37807/1
diff --git a/util/lint/lint-stable-005-board-status b/util/lint/lint-stable-005-board-status
index 952a9b5..e8313f2 100755
--- a/util/lint/lint-stable-005-board-status
+++ b/util/lint/lint-stable-005-board-status
@@ -17,6 +17,10 @@
LC_ALL=C export LC_ALL
for mobodir in $(git ls-files src/mainboard | sed -n 's,^\(src/mainboard/[^/]*/[^/]*\)/.*$,\1,p'|sort|uniq); do
+ if [[ ${mobodir} =~ ^src\/mainboard\/.*\/common$ ]]; then
+ continue
+ fi
+
board_info="$mobodir/board_info.txt"
if ! [ -f "$board_info" ]; then
echo "No $board_info found"
diff --git a/util/lint/lint-stable-006-board-name b/util/lint/lint-stable-006-board-name
index b2418a1..cf3b0e2 100755
--- a/util/lint/lint-stable-006-board-name
+++ b/util/lint/lint-stable-006-board-name
@@ -29,6 +29,9 @@
done
for i in src/mainboard/*/*/; do
+ if [[ ${i} =~ ^src\/mainboard\/.*\/common\/$ ]]; then
+ continue
+ fi
if [ -r $i/Kconfig ]; then
if [ ! -r $i/Kconfig.name ]; then
BOARD="$(grep -A2 MAINBOARD_PART_NUMBER $i/Kconfig | tail -1 | cut -f2 -d\")"
--
To view, visit https://review.coreboot.org/c/coreboot/+/37807
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I55c3474c17eacd008fee5f4089da33d4f7d6002a
Gerrit-Change-Number: 37807
Gerrit-PatchSet: 1
Gerrit-Owner: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-MessageType: newchange
Tim Wawrzynczak has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48512 )
Change subject: mb/google/volteer: Clean up romstage.c
......................................................................
mb/google/volteer: Clean up romstage.c
Move the manual calls to fw_config_probe() into the devicetree; the
AUDIO probe is trivial, and the TCSS devices (DMA0, iTBT RP0 & RP1) are
already guarded with probe statements in the baseboard devicetree, so
the code in romstage.c was redundant.
Change-Id: I1d067ff3d181b152c784634ff99202bb2b9202f7
Signed-off-by: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
---
M src/mainboard/google/volteer/romstage.c
M src/mainboard/google/volteer/variants/baseboard/devicetree.cb
2 files changed, 7 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/48512/1
diff --git a/src/mainboard/google/volteer/romstage.c b/src/mainboard/google/volteer/romstage.c
index 315ec20..67d3489 100644
--- a/src/mainboard/google/volteer/romstage.c
+++ b/src/mainboard/google/volteer/romstage.c
@@ -22,17 +22,5 @@
};
bool half_populated = gpio_get(GPIO_MEM_CH_SEL);
- /* Disable HDA device if no audio board is present. */
- if (fw_config_probe(FW_CONFIG(AUDIO, NONE)))
- mem_cfg->PchHdaEnable = 0;
-
meminit_ddr(mem_cfg, board_cfg, &spd_info, half_populated);
-
- /* Disable TBT if no USB4 hardware */
- if (!(fw_config_probe(FW_CONFIG(DB_USB, USB4_GEN2)) ||
- fw_config_probe(FW_CONFIG(DB_USB, USB4_GEN3)))) {
- mem_cfg->TcssDma0En = 0;
- mem_cfg->TcssItbtPcie0En = 0;
- mem_cfg->TcssItbtPcie1En = 0;
- }
}
diff --git a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb
index 7964885..c61b003 100644
--- a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb
+++ b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb
@@ -501,6 +501,12 @@
device pnp 0c09.0 on end
end
end
- device ref hda on end
+ device ref hda on
+ probe AUDIO MAX98357_ALC5682I_I2S
+ probe AUDIO MAX98373_ALC5682I_I2S
+ probe AUDIO MAX98373_ALC5682_SNDW
+ probe AUDIO MAX98373_ALC5682I_I2S_UP4
+ probe AUDIO MAX98360_ALC5682I_I2S
+ end
end
end
--
To view, visit https://review.coreboot.org/c/coreboot/+/48512
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1d067ff3d181b152c784634ff99202bb2b9202f7
Gerrit-Change-Number: 48512
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-MessageType: newchange
Mike Banon has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44637 )
Change subject: mb/asus/a88xm-e: Disable SeaBIOS options unsupported by hardware
......................................................................
mb/asus/a88xm-e: Disable SeaBIOS options unsupported by hardware
A88XM-E doesn't have any SAS or NVMe controllers and couldn't have a TPM
so it makes sense to disable the related SeaBIOS options for this board.
This reduces the size of compiled SeaBIOS by 129344-110048=19296 bytes.
Signed-off-by: Mike Banon <mikebdp2(a)gmail.com>
Change-Id: Ie9760449734499628637e015cacc0541ec0002a2
---
M src/mainboard/asus/a88xm-e/Kconfig
A src/mainboard/asus/a88xm-e/config_seabios
2 files changed, 11 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/44637/1
diff --git a/src/mainboard/asus/a88xm-e/Kconfig b/src/mainboard/asus/a88xm-e/Kconfig
index 6874e9e..139f989 100644
--- a/src/mainboard/asus/a88xm-e/Kconfig
+++ b/src/mainboard/asus/a88xm-e/Kconfig
@@ -56,4 +56,8 @@
bool
default n
+config PAYLOAD_CONFIGFILE
+ string
+ default "$(top)/src/mainboard/$(MAINBOARDDIR)/config_seabios" if PAYLOAD_SEABIOS
+
endif # BOARD_ASUS_A88XM_E
diff --git a/src/mainboard/asus/a88xm-e/config_seabios b/src/mainboard/asus/a88xm-e/config_seabios
new file mode 100644
index 0000000..03bf0e8
--- /dev/null
+++ b/src/mainboard/asus/a88xm-e/config_seabios
@@ -0,0 +1,7 @@
+###
+### SeaBIOS custom configuration for ASUS A88XM-E
+###
+# CONFIG_MEGASAS is not set
+# CONFIG_NVME is not set
+# CONFIG_TCGBIOS is not set
+#
--
To view, visit https://review.coreboot.org/c/coreboot/+/44637
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie9760449734499628637e015cacc0541ec0002a2
Gerrit-Change-Number: 44637
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Banon <mikebdp2(a)gmail.com>
Gerrit-MessageType: newchange