Patrick Rudolph has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/30114 )
Change subject: arch/x86/assembly_entry: Enable long mode on x86_64
......................................................................
Abandoned
replaced by new patch series
--
To view, visit https://review.coreboot.org/c/coreboot/+/30114
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I57974a55f3b778c90b3587f39e86e4eb8692ad48
Gerrit-Change-Number: 30114
Gerrit-PatchSet: 14
Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Huang Jin <huang.jin(a)intel.com>
Gerrit-Reviewer: Krystian Hebel <krystian.hebel(a)3mdeb.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: York Yang <yyang024(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: abandon
Stefan Reinauer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42330 )
Change subject: util/cavium: port devicetree_convert.py to python3
......................................................................
util/cavium: port devicetree_convert.py to python3
converted with 2to3 and manually changed shebang to reflect python3.
Tested by calling the script on the command line.
Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Change-Id: Iecc29729d01c0096f8b3fc62acb9f306c8fb2958
---
M util/cavium/devicetree_convert.py
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/42330/1
diff --git a/util/cavium/devicetree_convert.py b/util/cavium/devicetree_convert.py
index 05f85aa..e4eb33b5 100755
--- a/util/cavium/devicetree_convert.py
+++ b/util/cavium/devicetree_convert.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python3
# SPDX-License-Identifier: GPL-3.0-or-later
# devicetree_convert Tool to convert a DTB to a static C file
@@ -27,15 +27,15 @@
path = path.replace("/", "")
if len(node) == 1:
for i in node:
- if type(i) is not unicode:
- print "%s: Type is not string" % path
+ if not isinstance(i, str):
+ print("%s: Type is not string" % path)
continue
if args.verbose:
- print "%s = %s" % (path, i)
+ print("%s = %s" % (path, i))
if outfile is not None:
outfile.write("{\"%s\", \"%s\"},\n" % (path, i))
else:
- print "%s: Arrays aren't supported" % path
+ print("%s: Arrays aren't supported" % path)
if outfile is not None:
outfile.write("{0, 0},\n")
--
To view, visit https://review.coreboot.org/c/coreboot/+/42330
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iecc29729d01c0096f8b3fc62acb9f306c8fb2958
Gerrit-Change-Number: 42330
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-MessageType: newchange
Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44369 )
Change subject: soc/intel/skylake: Factor out unnecessary if-else-block
......................................................................
soc/intel/skylake: Factor out unnecessary if-else-block
Move InternalGfx config option out of the if-else-block and replace the
left over config option IgdDvmt50PreAlloc by a ternary expression and
adjust related code comments.
This changes the logic of the code, since InternalGfx is configured
first and IgdDvmt50PreAlloc depends on its value. The negation in the
ternary expression is removed to improve the readability.
Change-Id: I89ff17f4574a7ade228c1791f17ea072fb731775
Signed-off-by: Felix Singer <felixsinger(a)posteo.net>
---
M src/soc/intel/skylake/romstage/romstage.c
1 file changed, 14 insertions(+), 19 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/44369/1
diff --git a/src/soc/intel/skylake/romstage/romstage.c b/src/soc/intel/skylake/romstage/romstage.c
index da5ec9e..cefe742 100644
--- a/src/soc/intel/skylake/romstage/romstage.c
+++ b/src/soc/intel/skylake/romstage/romstage.c
@@ -241,25 +241,20 @@
const struct device *dev;
dev = pcidev_path_on_root(SA_DEVFN_IGD);
- if (!dev || !dev->enabled) {
- /*
- * If iGPU is disabled or not defined in the devicetree.cb,
- * the FSP does not initialize this device
- */
- m_cfg->InternalGfx = 0;
- m_cfg->IgdDvmt50PreAlloc = 0;
- } else {
- m_cfg->InternalGfx = 1;
- /*
- * Set IGD stolen size to 64MB. The FBC hardware for skylake
- * does not have access to the bios_reserved range so it always
- * assumes 8MB is used and so the kernel will avoid the last
- * 8MB of the stolen window. With the default stolen size of
- * 32MB(-8MB) there is not enough space for FBC to work with
- * a high resolution panel
- */
- m_cfg->IgdDvmt50PreAlloc = 2;
- }
+ m_cfg->InternalGfx = dev && dev->enabled;
+
+ /*
+ * If iGPU is enabled, set IGD stolen size to 64MB. The FBC
+ * hardware for skylake does not have access to the bios
+ * reserved range so it always assumes 8MB is used and so the
+ * kernel will avoid the last 8MB of the stolen window. With
+ * the default stolen size of 32MB(-8MB) there is not enough
+ * space for FBC to work with a high resolution panel.
+ *
+ * If disabled, don't reserve memory for it.
+ */
+ m_cfg->IgdDvmt50PreAlloc = m_cfg->InternalGfx ? 2 : 0;
+
m_cfg->PrimaryDisplay = config->PrimaryDisplay;
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/44369
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I89ff17f4574a7ade228c1791f17ea072fb731775
Gerrit-Change-Number: 44369
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-MessageType: newchange
Hello Felix Singer, build bot (Jenkins), Patrick Georgi, Angel Pons, Andrey Petrov, Patrick Rudolph, Aaron Durbin, Lance Zhao, Nico Huber, Martin Roth, Werner Zeh, Felix Held,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39133
to look at the new patch set (#37).
Change subject: mb/kontron/mal10: Add COMe-mAL10 minimal support
......................................................................
mb/kontron/mal10: Add COMe-mAL10 minimal support
This patch adds minimal support for the Kontron COMe-mAL10 module with
the Apollo Lake processor together with Kontron T10-TNI carrierboard.
Working:
- console (Kontron CPLD)
- USB2/3
- Ethernet network
- eMMC
- SATA
- PCIe ports
- Video/DP (*)
Not tested:
- Video/LVDS
TODO:
- HDA (codec IDT 92HD73C1X5, currently disabled)
- SDIO
- Documentation
As the payload was used:
- SeaBIOS
- Tianocore (UEFIPayload - video works only in linux) (*)
(*) Booting with the "CorebootPayload" crashes:
https://pastebin.com/cpCfrPCL
Tested on COMe module with Intel Atom x5-E3940 processor (4 Core,
1.6/1.8GHz, 9.5W TDP). xubuntu 18.04.2 was used as a bootable OS
(5.0.0-32-generic linux kernel)
Change-Id: Ib8432e10396f77eb05a71af1ccaaa4437a2e43ea
Signed-off-by: Maxim Polyakov <max.senia.poliak(a)gmail.com>
---
A src/mainboard/kontron/mal10/Kconfig
A src/mainboard/kontron/mal10/Kconfig.name
A src/mainboard/kontron/mal10/Makefile.inc
A src/mainboard/kontron/mal10/acpi/cpld.asl
A src/mainboard/kontron/mal10/acpi/dptf.asl
A src/mainboard/kontron/mal10/acpi/smbus.asl
A src/mainboard/kontron/mal10/board_info.txt
A src/mainboard/kontron/mal10/bootblock.c
A src/mainboard/kontron/mal10/carriers/t10-tni/Makefile.inc
A src/mainboard/kontron/mal10/carriers/t10-tni/board_info.txt
A src/mainboard/kontron/mal10/carriers/t10-tni/gpio.c
A src/mainboard/kontron/mal10/carriers/t10-tni/include/carrier/gpio.h
A src/mainboard/kontron/mal10/carriers/t10-tni/overridetree.cb
A src/mainboard/kontron/mal10/cmos.default
A src/mainboard/kontron/mal10/cmos.layout
A src/mainboard/kontron/mal10/data.vbt
A src/mainboard/kontron/mal10/dsdt.asl
A src/mainboard/kontron/mal10/mal10.fmd
A src/mainboard/kontron/mal10/ramstage.c
A src/mainboard/kontron/mal10/romstage.c
A src/mainboard/kontron/mal10/variants/mal10/Makefile.inc
A src/mainboard/kontron/mal10/variants/mal10/board_info.txt
A src/mainboard/kontron/mal10/variants/mal10/devicetree.cb
A src/mainboard/kontron/mal10/variants/mal10/gma-mainboard.ads
A src/mainboard/kontron/mal10/variants/mal10/gpio.c
A src/mainboard/kontron/mal10/variants/mal10/include/variant/gpio.h
26 files changed, 2,260 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/39133/37
--
To view, visit https://review.coreboot.org/c/coreboot/+/39133
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib8432e10396f77eb05a71af1ccaaa4437a2e43ea
Gerrit-Change-Number: 39133
Gerrit-PatchSet: 37
Gerrit-Owner: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Lance Zhao
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Andrey Petrov <anpetrov(a)fb.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset