James has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37146 )
Change subject: mb/lenovo/x131e: remove non-applicable options from CMOS
......................................................................
mb/lenovo/x131e: remove non-applicable options from CMOS
These options do not work in the same way as on the Lenovo H8 EC.
Since they don't work correctly, drop them from the CMOS settings.
Change-Id: Iebc55afd5cc621df562fd7aa9557207147a8199b
Signed-off-by: James Ye <jye836(a)gmail.com>
---
M src/mainboard/lenovo/x131e/cmos.default
M src/mainboard/lenovo/x131e/cmos.layout
2 files changed, 2 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/37146/1
diff --git a/src/mainboard/lenovo/x131e/cmos.default b/src/mainboard/lenovo/x131e/cmos.default
index b01978a..ca10a85 100644
--- a/src/mainboard/lenovo/x131e/cmos.default
+++ b/src/mainboard/lenovo/x131e/cmos.default
@@ -3,12 +3,7 @@
power_on_after_fail=Disable
nmi=Enable
volume=0x3
-bluetooth=Enable
wwan=Enable
-wlan=Enable
-touchpad=Enable
sata_mode=AHCI
-fn_ctrl_swap=Disable
sticky_fn=Disable
-trackpoint=Enable
usb_always_on=Disable
diff --git a/src/mainboard/lenovo/x131e/cmos.layout b/src/mainboard/lenovo/x131e/cmos.layout
index 93c74fb..7696046 100644
--- a/src/mainboard/lenovo/x131e/cmos.layout
+++ b/src/mainboard/lenovo/x131e/cmos.layout
@@ -60,12 +60,9 @@
411 1 e 8 sata_mode
# coreboot config options: EC
-412 1 e 1 bluetooth
+#412 1 r 0 unused
413 1 e 1 wwan
-414 1 e 1 touchpad
-415 1 e 1 wlan
-416 1 e 1 trackpoint
-417 1 e 1 fn_ctrl_swap
+#414 4 r 0 unused
418 1 e 1 sticky_fn
419 2 e 12 usb_always_on
#421 3 r 0 unused
--
To view, visit https://review.coreboot.org/c/coreboot/+/37146
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iebc55afd5cc621df562fd7aa9557207147a8199b
Gerrit-Change-Number: 37146
Gerrit-PatchSet: 1
Gerrit-Owner: James <jye836(a)gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: James <jye836(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newchange
Vani Ganji has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35961 )
Change subject: Mic_Bias voltage changed to 2.9V for Earpods Mic
......................................................................
Mic_Bias voltage changed to 2.9V for Earpods Mic
Earpods Mic is not working with 2.6Volts and it requires
a volatge greater than 2.7Volts.
BUG=b:134361881
Change-Id: Iceb62d4940a1d93404dc3f69884c1289f462f587
Signed-off-by: Vani Ganji <vani.ganji(a)intel.com>
---
M src/mainboard/google/poppy/variants/atlas/devicetree.cb
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/35961/1
diff --git a/src/mainboard/google/poppy/variants/atlas/devicetree.cb b/src/mainboard/google/poppy/variants/atlas/devicetree.cb
index ac86e79..b42313f 100644
--- a/src/mainboard/google/poppy/variants/atlas/devicetree.cb
+++ b/src/mainboard/google/poppy/variants/atlas/devicetree.cb
@@ -347,7 +347,7 @@
register "c_mic_btn_thr" = "0x3e"
register "btn_avg" = "4"
register "adc_1bit_rpt" = "1"
- register "micbias_lvl" = "2600"
+ register "micbias_lvl" = "2900"
register "mic_amp_in_sel" = ""diff""
device i2c 1a on end
end
--
To view, visit https://review.coreboot.org/c/coreboot/+/35961
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iceb62d4940a1d93404dc3f69884c1289f462f587
Gerrit-Change-Number: 35961
Gerrit-PatchSet: 1
Gerrit-Owner: Vani Ganji <vani.ganji(a)intel.com>
Gerrit-MessageType: newchange
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37197 )
Change subject: drivers/amd/agesa/romstage: Only mark cbmem as UC if needed
......................................................................
drivers/amd/agesa/romstage: Only mark cbmem as UC if needed
Now cbmem is flushed to dram before calling postcar stage.
UNTESTED.
Change-Id: Iaa0d154e2c5b2052027d07ad26e31f3ff63ae9f3
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/drivers/amd/agesa/mtrr_fixme.c
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/37197/1
diff --git a/src/drivers/amd/agesa/mtrr_fixme.c b/src/drivers/amd/agesa/mtrr_fixme.c
index bbb9eb0..9e63425 100644
--- a/src/drivers/amd/agesa/mtrr_fixme.c
+++ b/src/drivers/amd/agesa/mtrr_fixme.c
@@ -51,6 +51,10 @@
if (s3resume)
return;
+ /* We worry about cbmem hitting dram later */
+ if (clflush_supported())
+ return;
+
/* For normal path, INIT_POST has returned with all
* memory set WB cacheable. But we need CBMEM as UC
* to make CAR teardown with invalidation without
--
To view, visit https://review.coreboot.org/c/coreboot/+/37197
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaa0d154e2c5b2052027d07ad26e31f3ff63ae9f3
Gerrit-Change-Number: 37197
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange
Michał Żygowski has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32989
Change subject: mb/pcengines/apu2: describe serial ports in ACPI
......................................................................
mb/pcengines/apu2: describe serial ports in ACPI
FreeBSD users had to manually configure serial ports on their
installations. Defining serial ports in ACPI save that effort.
Signed-off-by: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Change-Id: I0de4172a1884abbe9d625060a9045c9d71469e27
---
A src/mainboard/pcengines/apu2/acpi/superio.asl
M src/mainboard/pcengines/apu2/dsdt.asl
2 files changed, 68 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/32989/1
diff --git a/src/mainboard/pcengines/apu2/acpi/superio.asl b/src/mainboard/pcengines/apu2/acpi/superio.asl
new file mode 100644
index 0000000..c61e3d7
--- /dev/null
+++ b/src/mainboard/pcengines/apu2/acpi/superio.asl
@@ -0,0 +1,65 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2019 PC Engines Gmbh
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+Scope (\_SB.PCI0.LIBR) {
+
+ Device (COM1) {
+ Name (_HID, EISAID ("PNP0501"))
+ Name (_UID, 1)
+
+ Method (_STA, 0, NotSerialized) {
+ Return (0x0F)
+ }
+
+ Name (_CRS, ResourceTemplate ()
+ {
+ IO (Decode16, 0x03F8, 0x3F8, 0x08, 0x08)
+ IRQNoFlags () {4}
+ })
+
+ Name (_PRS, ResourceTemplate ()
+ {
+ StartDependentFn (0, 0) {
+ IO (Decode16, 0x03F8, 0x3F8, 0x08, 0x08)
+ IRQNoFlags () {4}
+ }
+ EndDependentFn ()
+ })
+ }
+
+ Device (COM2) {
+ Name (_HID, EISAID ("PNP0501"))
+ Name (_UID, 2)
+
+ Method (_STA, 0, NotSerialized) {
+ Return (0x0F)
+ }
+
+ Name (_CRS, ResourceTemplate ()
+ {
+ IO (Decode16, 0x02F8, 0x2F8, 0x08, 0x08)
+ IRQNoFlags () {3}
+ })
+
+ Name (_PRS, ResourceTemplate ()
+ {
+ StartDependentFn (0, 0) {
+ IO (Decode16, 0x02F8, 0x2F8, 0x08, 0x08)
+ IRQNoFlags () {3}
+ }
+ EndDependentFn ()
+ })
+ }
+}
\ No newline at end of file
diff --git a/src/mainboard/pcengines/apu2/dsdt.asl b/src/mainboard/pcengines/apu2/dsdt.asl
index 3bf0ed6..409f03d 100644
--- a/src/mainboard/pcengines/apu2/dsdt.asl
+++ b/src/mainboard/pcengines/apu2/dsdt.asl
@@ -81,5 +81,8 @@
/* Define the System Indicators for the platform */
#include "acpi/si.asl"
+
+ /* Super IO devices (COM ports) */
+ #include "acpi/superio.asl"
}
/* End of ASL file */
--
To view, visit https://review.coreboot.org/c/coreboot/+/32989
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0de4172a1884abbe9d625060a9045c9d71469e27
Gerrit-Change-Number: 32989
Gerrit-PatchSet: 1
Gerrit-Owner: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-MessageType: newchange