Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37972 )
Change subject: src/Kconfig: enable USE_BLOBS by default
......................................................................
src/Kconfig: enable USE_BLOBS by default
To provide sane defaults for most of the user base, this patch switches
on the USE_BLOBS option by default. Since it only changes the default,
this behaviour can stil be easily disabled.
Change-Id: Ia0632b9ae7a1f212a8640b3faec2695d17d238c5
Signed-off-by: Felix Held <felix-coreboot(a)felixheld.de>
---
M src/Kconfig
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/37972/1
diff --git a/src/Kconfig b/src/Kconfig
index 762cf89..c89e49f 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -228,6 +228,7 @@
config USE_BLOBS
bool "Allow use of binary-only repository"
+ default y
help
This draws in the blobs repository, which contains binary files that
might be required for some chipsets or boards.
--
To view, visit https://review.coreboot.org/c/coreboot/+/37972
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia0632b9ae7a1f212a8640b3faec2695d17d238c5
Gerrit-Change-Number: 37972
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: newchange
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37064 )
Change subject: 4.12 release notes: Add some explanation behind depreciations
......................................................................
4.12 release notes: Add some explanation behind depreciations
Some features are made mandatory, meaning that some platforms have
been dropped from master. This also explains that further development
on these popular platforms can happen on the 4.11 branch.
TODO is this really the right place or is it too technical for release
notes?
Change-Id: I95e01c301e7db6f81ef88a89d709ebab35c9ccfb
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M Documentation/releases/coreboot-4.12-relnotes.md
1 file changed, 61 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/37064/1
diff --git a/Documentation/releases/coreboot-4.12-relnotes.md b/Documentation/releases/coreboot-4.12-relnotes.md
index f9c5f7e..67bef34 100644
--- a/Documentation/releases/coreboot-4.12-relnotes.md
+++ b/Documentation/releases/coreboot-4.12-relnotes.md
@@ -10,6 +10,67 @@
* The chip and board additions and removals will be updated right
before the release, so those do not need to be added.
+Deprecations
+------------
+
+For the 4.12 release a few features on x86 became mandatory. These are
+relocatable ramstage, postcar stage and C_ENVIRONMENT_BOOTBLOCK.
+
+### Relocatable ramstage
+
+Relocatable stages are a feature only implemented only on x86, where
+stages can be relocated at runtime. This is used to place ramstage in
+a better location that does not collide with memory the OS tends to
+use. The rationale behind making this mandatory is that you always
+want cbmem to be cached so it's a good location to run ramstage from.
+It avoids using lower memory altogether so the OS can make use
+of it and no backing up needs to happen on S3 resume.
+
+### Postcar stage
+
+With Postcar stage tearing down Cache-as-Ram is done in a separate
+stage. This means that romstage has a clean program boundary and
+that all variables in romstage can be accessed via their initial
+reference at all times. There is no need to link global and static
+variables via the CAR\_GLOBAL macro and no need to access them with
+car\_set/get\_var/ptr functions.
+
+### C\_ENVIRONMENT\_BOOTBLOCK
+
+Historically the bootblock on x86 platforms has been compiled with
+romcc. This means that the generated code only uses CPU registers
+and therefore no stack. This 20K+ LOC compiler is limited and hard
+to maintain. A different solution is to set up Cache-as-Ram in the
+bootblock and run GCC compiled code in the bootblock. The advantages
+are increased flexibility: e.g. printing to console is possible and
+VBOOT can run before romstage, making romstage updatable via RW FMAP
+regions.
+
+### Platforms dropped from master
+
+The following platforms did not implement those feature are dropped
+from master to allow the master branch to move on:
+- AMDFAM10
+- all FSP1.0 platforms: BROADWELL_DE, FSP_BAYTRAIL, RANGELEY
+- VIA VX900
+- TODO (AMD?)
+
+In particular on FSP1.0 it is impossible to implement POSTCAR stage.
+The reason is that FSP1.0 relocates the CAR region to the HOB before
+returning to coreboot. This means that after FSP returns to coreboot
+accessing variables via their original address is not possible. One
+way of obtaining that behavior would be to set up Cache-as-Ram again
+(but with open source code) and copy the relocated data from the HOB
+there. This solution is deemed too hacky. Maybe a lesson can be
+learned from this: blobs should not interfere with the execution
+environment, as this makes proper integration much harder.
+
+### 4.11_branch
+
+Given that some platforms supported by FSP1.0 are being produced and
+popular, the 4.11 release was made into a branch in which further
+development can happen.
+
Significant changes
-------------------
--
To view, visit https://review.coreboot.org/c/coreboot/+/37064
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I95e01c301e7db6f81ef88a89d709ebab35c9ccfb
Gerrit-Change-Number: 37064
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange
DaLao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36371 )
Change subject: Signed-off-by: dalao <dalao(a)tutanota.com>
......................................................................
Signed-off-by: dalao <dalao(a)tutanota.com>
src/mainboard/lenovo: Use the original lenovo hardware id for touchpad
Currently the coreboot's touchpad hardware id is different from the original lenovo ones, and this causes some problems. On Windows, the touchpad driver can't automatically be matched and installed through Windows Update. On Linux, there also maybe some strange issues (https://bugzilla.kernel.org/show_bug.cgi?id=91541). So it's better to use the original lenovo hardware id for each model than the PNP-id. By searching for dmesg logs, I found the following hardware ids for these thinkpad models:
LEN0015
T510: https://bbs.archlinux.org/viewtopic.php?id=120287
T410: ?
T420: ?
T420s: https://bbs.archlinux.org/viewtopic.php?id=191510
T520: ?
T430: https://github.com/farjump/fwtr/blob/master/lenovo/thinkpad-t430/2347ds2/le…
T430s: ?
T530: ?
LEN0020
X220: https://bbs.archlinux.org/viewtopic.php?id=237669
X230: https://forums.bunsenlabs.org/viewtopic.php?id=2460
LEN0036
T540: https://bugzilla.kernel.org/show_bug.cgi?id=91541
Then I made several copies of the ps2_controller.asl changed the superio.asl to include their correct ones. Then reboot to Windows and the touchpad driver is automatically installed. Done. But I still have some querstions:
1, Is making copies of the ps2_controller.asl a good way? Are there any better methods for doing this?
2, Is there any website that collect dmesg logs so I can search the hardware ids?
3, The keyboard's hardware id maybe different like the touchpad as well. But they are working ok, so I haven't looked into it.
Change-Id: Ifb635da99c5e05f987aaf4f172108d788dcc2932
---
A src/drivers/pc80/pc/ps2_controller_len0015.asl
A src/drivers/pc80/pc/ps2_controller_len0020.asl
A src/drivers/pc80/pc/ps2_controller_len0036.asl
M src/mainboard/lenovo/t410/acpi/superio.asl
M src/mainboard/lenovo/t420/acpi/superio.asl
M src/mainboard/lenovo/t420s/acpi/superio.asl
M src/mainboard/lenovo/t430/acpi/superio.asl
M src/mainboard/lenovo/t430s/acpi/superio.asl
M src/mainboard/lenovo/t520/acpi/superio.asl
M src/mainboard/lenovo/t530/acpi/superio.asl
M src/mainboard/lenovo/x220/acpi/superio.asl
M src/mainboard/lenovo/x230/acpi/superio.asl
12 files changed, 145 insertions(+), 9 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/36371/1
diff --git a/src/drivers/pc80/pc/ps2_controller_len0015.asl b/src/drivers/pc80/pc/ps2_controller_len0015.asl
new file mode 100644
index 0000000..750b24f
--- /dev/null
+++ b/src/drivers/pc80/pc/ps2_controller_len0015.asl
@@ -0,0 +1,46 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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.
+ */
+ Device (PS2K) // Keyboard
+ {
+ Name(_HID, EISAID("PNP0303"))
+ Name(_CID, EISAID("PNP030B"))
+
+ Name(_CRS, ResourceTemplate()
+ {
+ IO (Decode16, 0x60, 0x60, 0x01, 0x01)
+ IO (Decode16, 0x64, 0x64, 0x01, 0x01)
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x01 } // IRQ 1
+ })
+
+ Method (_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
+
+ Device (PS2M) // Mouse
+ {
+ Name(_HID, EISAID("LEN0015"))
+ Name(_CID, EISAID("PNP0F13"))
+
+ Name(_CRS, ResourceTemplate()
+ {
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x0c } // IRQ 12
+ })
+
+ Method(_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
diff --git a/src/drivers/pc80/pc/ps2_controller_len0020.asl b/src/drivers/pc80/pc/ps2_controller_len0020.asl
new file mode 100644
index 0000000..ba476a6
--- /dev/null
+++ b/src/drivers/pc80/pc/ps2_controller_len0020.asl
@@ -0,0 +1,45 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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.
+ */
+ Device (PS2K) // Keyboard
+ {
+ Name(_HID, EISAID("PNP0303"))
+ Name(_CID, EISAID("PNP030B"))
+
+ Name(_CRS, ResourceTemplate()
+ {
+ IO (Decode16, 0x60, 0x60, 0x01, 0x01)
+ IO (Decode16, 0x64, 0x64, 0x01, 0x01)
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x01 } // IRQ 1
+ })
+
+ Method (_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
+
+ Device (PS2M) // Mouse
+ {
+ Name(_HID, EISAID("LEN0020"))
+ Name(_CID, EISAID("PNP0F13"))
+ Name(_CRS, ResourceTemplate()
+ {
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x0c } // IRQ 12
+ })
+
+ Method(_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
diff --git a/src/drivers/pc80/pc/ps2_controller_len0036.asl b/src/drivers/pc80/pc/ps2_controller_len0036.asl
new file mode 100644
index 0000000..4b10381
--- /dev/null
+++ b/src/drivers/pc80/pc/ps2_controller_len0036.asl
@@ -0,0 +1,45 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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.
+ */
+ Device (PS2K) // Keyboard
+ {
+ Name(_HID, EISAID("PNP0303"))
+ Name(_CID, EISAID("PNP030B"))
+
+ Name(_CRS, ResourceTemplate()
+ {
+ IO (Decode16, 0x60, 0x60, 0x01, 0x01)
+ IO (Decode16, 0x64, 0x64, 0x01, 0x01)
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x01 } // IRQ 1
+ })
+
+ Method (_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
+
+ Device (PS2M) // Mouse
+ {
+ Name(_HID, EISAID("LEN0036"))
+ Name(_CID, EISAID("PNP0F13"))
+ Name(_CRS, ResourceTemplate()
+ {
+ IRQ (Edge, ActiveHigh, Exclusive) { 0x0c } // IRQ 12
+ })
+
+ Method(_STA, 0)
+ {
+ Return (0xf)
+ }
+ }
diff --git a/src/mainboard/lenovo/t410/acpi/superio.asl b/src/mainboard/lenovo/t410/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t410/acpi/superio.asl
+++ b/src/mainboard/lenovo/t410/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t420/acpi/superio.asl b/src/mainboard/lenovo/t420/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t420/acpi/superio.asl
+++ b/src/mainboard/lenovo/t420/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t420s/acpi/superio.asl b/src/mainboard/lenovo/t420s/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t420s/acpi/superio.asl
+++ b/src/mainboard/lenovo/t420s/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t430/acpi/superio.asl b/src/mainboard/lenovo/t430/acpi/superio.asl
index 8964c36..4ba8af3 100644
--- a/src/mainboard/lenovo/t430/acpi/superio.asl
+++ b/src/mainboard/lenovo/t430/acpi/superio.asl
@@ -13,4 +13,4 @@
* GNU General Public License for more details.
*/
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t430s/acpi/superio.asl b/src/mainboard/lenovo/t430s/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t430s/acpi/superio.asl
+++ b/src/mainboard/lenovo/t430s/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t520/acpi/superio.asl b/src/mainboard/lenovo/t520/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t520/acpi/superio.asl
+++ b/src/mainboard/lenovo/t520/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/t530/acpi/superio.asl b/src/mainboard/lenovo/t530/acpi/superio.asl
index f2b35ba..24afccc 100644
--- a/src/mainboard/lenovo/t530/acpi/superio.asl
+++ b/src/mainboard/lenovo/t530/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0015.asl>
diff --git a/src/mainboard/lenovo/x220/acpi/superio.asl b/src/mainboard/lenovo/x220/acpi/superio.asl
index f2b35ba..069cdea 100644
--- a/src/mainboard/lenovo/x220/acpi/superio.asl
+++ b/src/mainboard/lenovo/x220/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0020.asl>
diff --git a/src/mainboard/lenovo/x230/acpi/superio.asl b/src/mainboard/lenovo/x230/acpi/superio.asl
index f2b35ba..069cdea 100644
--- a/src/mainboard/lenovo/x230/acpi/superio.asl
+++ b/src/mainboard/lenovo/x230/acpi/superio.asl
@@ -1 +1 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
+#include <drivers/pc80/pc/ps2_controller_len0020.asl>
--
To view, visit https://review.coreboot.org/c/coreboot/+/36371
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifb635da99c5e05f987aaf4f172108d788dcc2932
Gerrit-Change-Number: 36371
Gerrit-PatchSet: 1
Gerrit-Owner: DaLao <dalao(a)tutanota.com>
Gerrit-MessageType: newchange