Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/28628
to look at the new patch set (#3).
Change subject: mb/lenovo/t400: Link the gpio.c settings
......................................................................
mb/lenovo/t400: Link the gpio.c settings
Linking this file instead of including a header makes it possible to easily
change gpio settings for a variant.
Change-Id: Ifd496510d4868f5901a9dbbf7f1523ccffaf15ab
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/mainboard/lenovo/t400/Makefile.inc
R src/mainboard/lenovo/t400/gpio.c
M src/mainboard/lenovo/t400/romstage.c
3 files changed, 4 insertions(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/28628/3
--
To view, visit https://review.coreboot.org/28628
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: Ifd496510d4868f5901a9dbbf7f1523ccffaf15ab
Gerrit-Change-Number: 28628
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/28626
to look at the new patch set (#2).
Change subject: mb/lenovo/Thinkpad{T500, R400, W500}: Unify under T400
......................................................................
mb/lenovo/Thinkpad{T500, R400, W500}: Unify under T400
A negative side-effect is that those boards disappear from the board-status
output, but this is an issue on all variants.
Change-Id: Ic80804dc1f7d9c6f83ceee3db667019532c31d4c
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
D src/mainboard/lenovo/r400/Kconfig
D src/mainboard/lenovo/r400/Kconfig.name
D src/mainboard/lenovo/r400/board_info.txt
M src/mainboard/lenovo/t400/Kconfig
M src/mainboard/lenovo/t400/Kconfig.name
D src/mainboard/lenovo/t500/Kconfig
D src/mainboard/lenovo/t500/Kconfig.name
D src/mainboard/lenovo/t500/board_info.txt
D src/mainboard/lenovo/w500/Kconfig
D src/mainboard/lenovo/w500/Kconfig.name
D src/mainboard/lenovo/w500/board_info.txt
11 files changed, 15 insertions(+), 51 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/28626/2
--
To view, visit https://review.coreboot.org/28626
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: Ic80804dc1f7d9c6f83ceee3db667019532c31d4c
Gerrit-Change-Number: 28626
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/28596 )
Change subject: mb/asrock/g41m_vs3_r2: Add mainboard
......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/#/c/28596/5/src/mainboard/asrock/g41c-gs/gpio.c
File src/mainboard/asrock/g41c-gs/gpio.c:
https://review.coreboot.org/#/c/28596/5/src/mainboard/asrock/g41c-gs/gpio.c…
PS5, Line 75: #if IS_ENABLED(CONFIG_BOARD_ASROCK_G41M_VS3_R2_0)
: .gpio8 = GPIO_LEVEL_LOW,
: .gpio9 = GPIO_LEVEL_LOW,
: #endif
> this IS_ENABLED(CONFIG_BOARD_ASROCK_G41M_VS3_R2_0) is always false, since it's inside the IS_ENABLED […]
This file is becoming a mess. I'll just link them separately, while removing the preprocessor statements I think...
--
To view, visit https://review.coreboot.org/28596
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: Ibce9ecdc0e44db3703401f116c9a8bff5b66437f
Gerrit-Change-Number: 28596
Gerrit-PatchSet: 5
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
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-CC: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Sun, 16 Sep 2018 14:02:04 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Philipp Deppenwiese has submitted this change and it was merged. ( https://review.coreboot.org/28590 )
Change subject: payloads/external: Add yabits payload
......................................................................
payloads/external: Add yabits payload
Yabits (Yet another UEFI bootloader) is designed to be a slim and quick
alternative to Tianocore. It is still under heavy development.
https://web.archive.org/web/https://yabits.github.io/
Change-Id: I132970e952c605c73cfe33dc47f20170ae8aa899
Signed-off-by: Martin Roth <martin(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/28590
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi(a)google.com>
Reviewed-by: Marc Jones <marc(a)marcjonesconsulting.com>
---
M payloads/Makefile.inc
M payloads/external/Makefile.inc
A payloads/external/Yabits/Kconfig
A payloads/external/Yabits/Kconfig.name
A payloads/external/Yabits/Makefile
5 files changed, 140 insertions(+), 0 deletions(-)
Approvals:
build bot (Jenkins): Verified
Patrick Georgi: Looks good to me, approved
Marc Jones: Looks good to me, approved
diff --git a/payloads/Makefile.inc b/payloads/Makefile.inc
index 0d142d6..83d3910 100644
--- a/payloads/Makefile.inc
+++ b/payloads/Makefile.inc
@@ -36,6 +36,7 @@
payloads/external/tianocore \
payloads/external/GRUB2 \
payloads/external/LinuxBoot \
+payloads/external/Yabits \
payloads/coreinfo/build/coreinfo.elf coreinfo:
$(MAKE) -C payloads/coreinfo defaultbuild
diff --git a/payloads/external/Makefile.inc b/payloads/external/Makefile.inc
index e4d6d4f..c730bba 100644
--- a/payloads/external/Makefile.inc
+++ b/payloads/external/Makefile.inc
@@ -283,3 +283,15 @@
payloads/external/LinuxBoot/linuxboot/initramfs.cpio.xz: linuxboot
payloads/external/LinuxBoot/linuxboot/kernel/.config: linuxboot
payloads/external/LinuxBoot/linuxboot/go/src/github.com/u-root/u-root/.git: linuxboot
+
+payloads/external/Yabits/uefi/build/uefi.elf yabits:
+ $(MAKE) -C payloads/external/Yabits all \
+ CC="$(CC_x86_32)" \
+ LD="$(LD_x86_32)" \
+ OBJCOPY="$(OBJCOPY_x86_32)" \
+ AS="$(AS_x86_32)" \
+ CONFIG_YABITS_REVISION=$(CONFIG_YABITS_REVISION) \
+ CONFIG_YABITS_REVISION_ID=$(CONFIG_YABITS_REVISION_ID) \
+ CONFIG_YABITS_MASTER=$(CONFIG_YABITS_MASTER) \
+ CONFIG_YABITS_STABLE=$(CONFIG_YABITS_STABLE) \
+ MFLAGS= MAKEFLAGS=
diff --git a/payloads/external/Yabits/Kconfig b/payloads/external/Yabits/Kconfig
new file mode 100644
index 0000000..cd1ade8
--- /dev/null
+++ b/payloads/external/Yabits/Kconfig
@@ -0,0 +1,47 @@
+if PAYLOAD_YABITS
+
+choice
+ prompt "Yabits version"
+ default YABITS_STABLE
+
+config YABITS_STABLE
+ bool "Stable"
+ help
+ Stable Yabits version.
+
+ For reproducible builds, this option must be selected.
+
+config YABITS_MASTER
+ bool "Master"
+ help
+ Newest version.
+
+ This option will fetch the newest version of the Yabits code,
+ updating as new changes are committed. This makes the build
+ non-reproducible, as it can fetch different code each time.
+
+config YABITS_REVISION
+ bool "git revision"
+ help
+ Select this option if you have a specific commit or branch
+ that you want to use as the revision from which to
+ build Yabits. Using a branch name makes the build
+ non-reproducible, as it can fetch different code as the
+ branch changes.
+
+ You will be able to specify the name of a branch or a commit id
+ later.
+
+endchoice
+
+config YABITS_REVISION_ID
+ string "Insert a commit's SHA-1 or a branch name"
+ depends on YABITS_REVISION
+ default "origin/master"
+ help
+ The commit's SHA-1 or branch name of the revision to use.
+
+config PAYLOAD_FILE
+ default "payloads/external/Yabits/uefi/build/uefi.elf"
+
+endif
diff --git a/payloads/external/Yabits/Kconfig.name b/payloads/external/Yabits/Kconfig.name
new file mode 100644
index 0000000..84f860f
--- /dev/null
+++ b/payloads/external/Yabits/Kconfig.name
@@ -0,0 +1,11 @@
+config PAYLOAD_YABITS
+ bool "Yabits - Yet another UEFI Bootloader (Under Development)"
+ depends on ARCH_X86
+ help
+ Yabits - yabits.github.io
+ yabits is a pure UEFI coreboot payload. Compared with
+ TianoCore, it is fast and lightweight. yabits is based on the
+ part of Minoca OS. It can run GRUB2, Linux, OpenBSD, and other
+ UEFI applications.
+
+ It is still under the development and not ready for production.
diff --git a/payloads/external/Yabits/Makefile b/payloads/external/Yabits/Makefile
new file mode 100644
index 0000000..ec3c9c5
--- /dev/null
+++ b/payloads/external/Yabits/Makefile
@@ -0,0 +1,69 @@
+##
+## This file is part of the coreboot project.
+##
+## Copyright (C) 2016 Google Inc.
+##
+## 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.
+##
+
+TAG-$(CONFIG_YABITS_MASTER)=origin/master
+NAME-$(CONFIG_YABITS_MASTER)=Master
+TAG-$(CONFIG_YABITS_STABLE)=d25abb067431dee9af9f8a874a209730ab7f0e91
+NAME-$(CONFIG_YABITS_STABLE)=Stable
+TAG-$(CONFIG_YABITS_REVISION)=$(CONFIG_YABITS_REVISION_ID)
+
+project_name=Yabits
+project_dir=$(CURDIR)/uefi
+project_git_repo=https://github.com/yabits/uefi.git
+LIBCONFIG_PATH="../../../libpayload"
+
+all: build
+
+$(project_dir):
+ echo " Cloning $(project_name) from Git"
+ git clone $(project_git_repo) $(project_dir)
+
+fetch: $(project_dir)
+ifeq ($(TAG-y),)
+ echo "Error: The specified tag is invalid"
+ifeq ($(CONFIG_YABITS_REVISION),y)
+ echo "Error: There is no revision specified for $(project_name)"
+ false
+endif
+ false
+endif
+ -cd $(project_dir); git show $(TAG-y) >/dev/null 2>&1 ; \
+ if [ $$? -ne 0 ] || [ "$(TAG-y)" = "origin/master" ]; then \
+ echo " Fetching new commits from the $(project_name) git repo"; \
+ git fetch; fi
+
+checkout: fetch
+ echo " Checking out $(project_name) revision $(NAME-y) ($(TAG-y))"
+ cd $(project_dir); \
+ git checkout master; \
+ git branch -D coreboot 2>/dev/null; \
+ git checkout -b coreboot $(TAG-y)
+
+build: checkout
+ echo " MAKE $(project_name) $(NAME-y)"
+ $(if $(wildcard uefi/.xcompile),,$(shell bash ../../../util/xcompile/xcompile > uefi/.xcompile))
+ $(MAKE) -C $(project_dir) defconfig LIBCONFIG_PATH=$(LIBCONFIG_PATH) XGCC=$(XGCCPATH)
+ $(MAKE) -C $(project_dir) all LIBCONFIG_PATH=$(LIBCONFIG_PATH) XGCC=$(XGCCPATH)
+
+clean:
+ test -d $(project_dir) && $(MAKE) -C $(project_dir) clean LIBCONFIG_PATH=$(LIBCONFIG_PATH) XGCC=$(XGCCPATH) || exit 0
+
+distclean:
+ rm -rf $(project_dir)
+
+print-repo-info:
+ echo "$(project_git_repo) $(project_dir)"
+
+.PHONY: all build checkout clean distclean fetch print-repo-info
--
To view, visit https://review.coreboot.org/28590
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I132970e952c605c73cfe33dc47f20170ae8aa899
Gerrit-Change-Number: 28590
Gerrit-PatchSet: 5
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Philipp Deppenwiese has submitted this change and it was merged. ( https://review.coreboot.org/28616 )
Change subject: configs: add sercon port and disable pxe serial console for apu{2,3,4,5}
......................................................................
configs: add sercon port and disable pxe serial console for apu{2,3,4,5}
To avoid mangled characters on serial output from iPXE we have to disable
serial from iPXE console. More to that to have correct serial input we
have to enable SeaBIOS SERCON option with default configuration.
The only limitation of this configs is that apu5 doesn't detect iPXE -
that platform is not for public use so it doesn't affect anyone.
Change-Id: I124705bd691b3c8dcd9a2636b17c019d02732c5a
Signed-off-by: Piotr Król <piotr.krol(a)3mdeb.com>
Reviewed-on: https://review.coreboot.org/28616
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Patrick Rudolph <siro(a)das-labor.org>
---
A configs/config.pcengines_apu1
M configs/config.pcengines_apu2
A configs/config.pcengines_apu3
A configs/config.pcengines_apu4
A configs/config.pcengines_apu5
5 files changed, 48 insertions(+), 0 deletions(-)
Approvals:
build bot (Jenkins): Verified
Patrick Rudolph: Looks good to me, approved
diff --git a/configs/config.pcengines_apu1 b/configs/config.pcengines_apu1
new file mode 100644
index 0000000..e511d65
--- /dev/null
+++ b/configs/config.pcengines_apu1
@@ -0,0 +1,10 @@
+# CONFIG_COLLECT_TIMESTAMPS is not set
+CONFIG_VENDOR_PCENGINES=y
+CONFIG_BOARD_PCENGINES_APU1=y
+CONFIG_NO_GFX_INIT=y
+CONFIG_SEABIOS_ADD_SERCON_PORT_FILE=y
+CONFIG_PXE=y
+CONFIG_BUILD_IPXE=y
+CONFIG_PXE_ROM_ID="10ec,8168"
+# CONFIG_PXE_SERIAL_CONSOLE is not set
+CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu2 b/configs/config.pcengines_apu2
index 2754024..08d4d4e 100644
--- a/configs/config.pcengines_apu2
+++ b/configs/config.pcengines_apu2
@@ -4,7 +4,9 @@
CONFIG_APU2_PINMUX_UART_C=y
CONFIG_APU2_PINMUX_UART_D=y
CONFIG_NO_GFX_INIT=y
+CONFIG_SEABIOS_ADD_SERCON_PORT_FILE=y
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="8086,157b"
+# CONFIG_PXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu3 b/configs/config.pcengines_apu3
new file mode 100644
index 0000000..7acf014
--- /dev/null
+++ b/configs/config.pcengines_apu3
@@ -0,0 +1,12 @@
+# CONFIG_COLLECT_TIMESTAMPS is not set
+CONFIG_VENDOR_PCENGINES=y
+CONFIG_BOARD_PCENGINES_APU3=y
+CONFIG_APU2_PINMUX_UART_C=y
+CONFIG_APU2_PINMUX_UART_D=y
+CONFIG_NO_GFX_INIT=y
+CONFIG_SEABIOS_ADD_SERCON_PORT_FILE=y
+CONFIG_PXE=y
+CONFIG_BUILD_IPXE=y
+CONFIG_PXE_ROM_ID="8086,1539"
+# CONFIG_PXE_SERIAL_CONSOLE is not set
+CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu4 b/configs/config.pcengines_apu4
new file mode 100644
index 0000000..9e7ca5a
--- /dev/null
+++ b/configs/config.pcengines_apu4
@@ -0,0 +1,12 @@
+# CONFIG_COLLECT_TIMESTAMPS is not set
+CONFIG_VENDOR_PCENGINES=y
+CONFIG_BOARD_PCENGINES_APU4=y
+CONFIG_APU2_PINMUX_UART_C=y
+CONFIG_APU2_PINMUX_UART_D=y
+CONFIG_NO_GFX_INIT=y
+CONFIG_SEABIOS_ADD_SERCON_PORT_FILE=y
+CONFIG_PXE=y
+CONFIG_BUILD_IPXE=y
+CONFIG_PXE_ROM_ID="8086,1539"
+# CONFIG_PXE_SERIAL_CONSOLE is not set
+CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu5 b/configs/config.pcengines_apu5
new file mode 100644
index 0000000..e5ddcee
--- /dev/null
+++ b/configs/config.pcengines_apu5
@@ -0,0 +1,12 @@
+# CONFIG_COLLECT_TIMESTAMPS is not set
+CONFIG_VENDOR_PCENGINES=y
+CONFIG_BOARD_PCENGINES_APU5=y
+CONFIG_APU2_PINMUX_UART_C=y
+CONFIG_APU2_PINMUX_UART_D=y
+CONFIG_NO_GFX_INIT=y
+CONFIG_SEABIOS_ADD_SERCON_PORT_FILE=y
+CONFIG_PXE=y
+CONFIG_BUILD_IPXE=y
+CONFIG_PXE_ROM_ID="8086,1539"
+# CONFIG_PXE_SERIAL_CONSOLE is not set
+CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
--
To view, visit https://review.coreboot.org/28616
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I124705bd691b3c8dcd9a2636b17c019d02732c5a
Gerrit-Change-Number: 28616
Gerrit-PatchSet: 2
Gerrit-Owner: Piotr Król <piotr.krol(a)3mdeb.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Piotr Król <piotr.krol(a)3mdeb.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>