Angel Pons has uploaded this change for review. ( https://review.coreboot.org/29447
Change subject: src/mb/asus/p5qc/gpio.c: Use a per-variant gpio.c
......................................................................
src/mb/asus/p5qc/gpio.c: Use a per-variant gpio.c
Even though these two mainboards use the same gpio.c file, other boards
such as the p5ql_pro do not.*
Change-Id: I2f7c8c12cb1bdcf47f3b4d4cef0b11e44a5b8863
Signed-off-by: Angel Pons <th3fanbus(a)gmail.com>
---
M src/…
[View More]mainboard/asus/p5qc/Makefile.inc
C src/mainboard/asus/p5qc/variants/p5q_pro/gpio.c
R src/mainboard/asus/p5qc/variants/p5qc/gpio.c
3 files changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/29447/1
diff --git a/src/mainboard/asus/p5qc/Makefile.inc b/src/mainboard/asus/p5qc/Makefile.inc
index 630abda..9fcbdf1 100644
--- a/src/mainboard/asus/p5qc/Makefile.inc
+++ b/src/mainboard/asus/p5qc/Makefile.inc
@@ -12,4 +12,4 @@
#
ramstage-y += cstates.c
-romstage-y += gpio.c
+romstage-y += variants/$(VARIANT_DIR)/gpio.c
diff --git a/src/mainboard/asus/p5qc/gpio.c b/src/mainboard/asus/p5qc/variants/p5q_pro/gpio.c
similarity index 100%
copy from src/mainboard/asus/p5qc/gpio.c
copy to src/mainboard/asus/p5qc/variants/p5q_pro/gpio.c
diff --git a/src/mainboard/asus/p5qc/gpio.c b/src/mainboard/asus/p5qc/variants/p5qc/gpio.c
similarity index 100%
rename from src/mainboard/asus/p5qc/gpio.c
rename to src/mainboard/asus/p5qc/variants/p5qc/gpio.c
--
To view, visit https://review.coreboot.org/29447
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2f7c8c12cb1bdcf47f3b4d4cef0b11e44a5b8863
Gerrit-Change-Number: 29447
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
[View Less]
Hello Patrick Rudolph, Angel Pons, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/29434
to look at the new patch set (#4).
Change subject: mb/lenovo/x220: Add x1 as a variant
......................................................................
mb/lenovo/x220: Add x1 as a variant
ThinkPad X1 ( https://www.thinkwiki.org/wiki/Category:X1 ) is nearly a
clone of X220, with additional USB3 controller on pci-e (as i7 variant
of x220), and …
[View More]a powered ESATA port wired to ata4 (Linux' annotation).
Documentation added.
Tested:
- CPU i5-2520M
- Slotted DIMM 8GiB
- Camera
- Mini pci-e on wlan slot
- Msata on wwan slot
- On board SDHCI connected to pci-e
- USB3 controller connected to pci-e
- NVRAM options for North and South bridges
- S3
- TPM1 on LPC
- Linux 4.9.110-3 within Debian GNU/Linux stable, loaded from
SeaBIOS, or Linux payload (Heads)
Not tested:
- Fingerprint reader on USB2
- Onboard USB2 interfaces (wlan slot, wwan slot)
Change-Id: Ibbc45f22c63b77ac95c188db825d0d7e2b03d2d1
Signed-off-by: Bill XIE <persmule(a)gmail.com>
---
M Documentation/mainboard/lenovo/t4xx_series.md
A Documentation/mainboard/lenovo/thinkpad_hmm.md
A Documentation/mainboard/lenovo/x1.md
A Documentation/mainboard/lenovo/x2xx_series.md
M src/mainboard/lenovo/x220/Kconfig
M src/mainboard/lenovo/x220/Kconfig.name
M src/mainboard/lenovo/x220/Makefile.inc
A src/mainboard/lenovo/x220/variants/x1/devicetree.cb
A src/mainboard/lenovo/x220/variants/x1/gpio.c
R src/mainboard/lenovo/x220/variants/x220/devicetree.cb
R src/mainboard/lenovo/x220/variants/x220/gpio.c
11 files changed, 495 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/29434/4
--
To view, visit https://review.coreboot.org/29434
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibbc45f22c63b77ac95c188db825d0d7e2b03d2d1
Gerrit-Change-Number: 29434
Gerrit-PatchSet: 4
Gerrit-Owner: Bill XIE <persmule(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
[View Less]
Angel Pons has posted comments on this change. ( https://review.coreboot.org/29347 )
Change subject: mb/asus/p5qc: Fix spelling of Marvell
......................................................................
Patch Set 1: Code-Review+2
> Patch Set 1:
>
> Does it need to be in uppercase? I'd rather use "Marvell", since it is not an acronym.
Whatever
--
To view, visit https://review.coreboot.org/29347
To unsubscribe, or for help writing mail filters, visit https://review.…
[View More]coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I437ac0d4fc706fbb62a0a74ca74a197dba4499fb
Gerrit-Change-Number: 29347
Gerrit-PatchSet: 1
Gerrit-Owner: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sat, 03 Nov 2018 14:14:33 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
[View Less]
Marcello Sylvester Bauer has uploaded a new patch set (#2). ( https://review.coreboot.org/29445 )
Change subject: Linuxboot: check go version
......................................................................
Linuxboot: check go version
Check if the installed Golang version is >= 1.9.
Change-Id: I9fd74fa8dc5e906e8b54ff5afaf69609fe957960
Signed-off-by: Marcello Sylvester Bauer <info(a)marcellobauer.com>
---
M payloads/external/LinuxBoot/targets/u-root.mk
1 file changed, 12 …
[View More]insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/29445/2
--
To view, visit https://review.coreboot.org/29445
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9fd74fa8dc5e906e8b54ff5afaf69609fe957960
Gerrit-Change-Number: 29445
Gerrit-PatchSet: 2
Gerrit-Owner: Marcello Sylvester Bauer <info(a)marcellobauer.com>
[View Less]
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/29434 )
Change subject: mb/lenovo/x220: Add x1 as a variant
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
Please also add Documentation.
https://review.coreboot.org/#/c/29434/3/src/mainboard/lenovo/x220/variants/…
File src/mainboard/lenovo/x220/variants/x1/gpio.c:
https://review.coreboot.org/#/c/29434/3/src/mainboard/lenovo/x220/variants/……
[View More]PS3, Line 5: * Copyright (C) 2014 Vladimir Serbinenko
your copyright is missing
--
To view, visit https://review.coreboot.org/29434
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibbc45f22c63b77ac95c188db825d0d7e2b03d2d1
Gerrit-Change-Number: 29434
Gerrit-PatchSet: 3
Gerrit-Owner: Bill XIE <persmule(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sat, 03 Nov 2018 10:59:35 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: Yes
[View Less]
Hello Patrick Rudolph, Philipp Deppenwiese, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/29443
to look at the new patch set (#3).
Change subject: Linuxboot: add current kernel releases
......................................................................
Linuxboot: add current kernel releases
Add current kernel releases instead of hardcode the version number.
Also add the option to select a custom version of the kernel.
Change-Id:…
[View More] Ieb071696cfbb466521faaef045fc488a06f6db0c
Signed-off-by: Marcello Sylvester Bauer <info(a)marcellobauer.com>
---
M payloads/external/LinuxBoot/Kconfig
M payloads/external/LinuxBoot/Makefile
M payloads/external/Makefile.inc
3 files changed, 46 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/29443/3
--
To view, visit https://review.coreboot.org/29443
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieb071696cfbb466521faaef045fc488a06f6db0c
Gerrit-Change-Number: 29443
Gerrit-PatchSet: 3
Gerrit-Owner: Marcello Sylvester Bauer <info(a)marcellobauer.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
[View Less]