coreboot-gerrit March 2016

coreboot-gerrit@coreboot.org

1 participants
1119 discussions

Patch set updated for coreboot: abuild: Add option of starting with an existing defconfig file
by Martin Roth 11 Mar '16

11 Mar '16

Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14016 -gerrit commit 4d67a419562439a2460a45d6fc6d3bc34b3b213c Author: Martin Roth <martinroth(a)google.com> Date: Wed Mar 9 13:12:10 2016 -0700 abuild: Add option of starting with an existing defconfig file We want to start testing builds with additional Kconfig options to try to get more coverage. This will allow us to enable various options to test without having to add each individual option to the abuild script. Change-Id: I9bb2bb6f38589e3bcc1282dc4cad51cf6f5149aa Signed-off-by: Martin Roth <martinroth(a)google.com> --- util/abuild/abuild | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/util/abuild/abuild b/util/abuild/abuild index ffa16b5..4d64bbe 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -410,6 +410,7 @@ Options:\n" [-o|--outdir <path>] store build results in path (defaults to $TARGET) [-L|--clang] Use clang + [-K|--kconfig <name>] Prepend file to generated Kconfig [-x|--chromeos] Build with CHROMEOS enabled Skip boards without Chrome OS support [-X|--xmlfile <name>] set JUnit XML log file filename @@ -456,11 +457,11 @@ cmdline="$* -c 1" getoptbrand="`getopt -V`" if [ "${getoptbrand:0:6}" == "getopt" ]; then # Detected GNU getopt that supports long options. - args=`getopt -l version,verbose,quiet,help,all,target:,payloads:,cpus:,silent,junit,config,loglevel:,remove,prefix:,update,scan-build,ccache,blobs,clang,clean,outdir:,chromeos,xmlfile: -o Vvqhat:p:c:sJCl:rP:uyBLzo:xX: -- "$@"` || exit 1 + args=`getopt -l version,verbose,quiet,help,all,target:,payloads:,cpus:,silent,junit,config,loglevel:,remove,prefix:,update,scan-build,ccache,blobs,clang,clean,outdir:,chromeos,xmlfile:,kconfig: -o Vvqhat:p:c:sJCl:rP:uyBLzo:xX:K: -- "$@"` || exit 1 eval set -- $args else # Detected non-GNU getopt - args=`getopt Vvqhat:p:c:sJCl:rP:uyBLzo:xX: $*` + args=`getopt Vvqhat:p:c:sJCl:rP:uyBLzo:xX:K: $*` set -- $args fi @@ -538,6 +539,11 @@ while true ; do configoptions="${configoptions}CONFIG_CHROMEOS=y\n" ;; -X|--xmlfile) shift; XMLFILE=$1; REAL_XMLFILE=$1; shift;; + -K|--kconfig) shift + testclass=="$(basename $1) + customizing="${customizing}, $1 config" + configoptions="$(cat "$1")${configoptions}\n" + shift;; --) shift; break;; -*) printf "Invalid option\n\n"; myhelp; exit 1;; *) break;;

1 0

Patch merged into coreboot/master: coreinfo/Makefile: change $(obj) to $(coreinfo_obj)
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 9c3ff1ba5209220cd41bee16015ae388ac6fce39 Author: Martin Roth <martinroth(a)google.com> Date: Mon Mar 7 16:26:51 2016 -0700 coreinfo/Makefile: change $(obj) to $(coreinfo_obj) - Rename obj to coreinfo_obj so it doesn't conflict with the obj variable in libpayload. Change-Id: I2ffb06a87e30a5eeff5b0dfc0ba62b5e9ab46e26 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13938 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/13938 for details. -gerrit

1 0

Patch merged into coreboot/master: coreinfo: Remove .xcompile when doing a clean
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 143a78c9f80cf6ea5b1d9533f3f7a22a87f9a1d3 Author: Martin Roth <martinroth(a)google.com> Date: Tue Mar 8 12:29:33 2016 -0700 coreinfo: Remove .xcompile when doing a clean Change-Id: I3e719e105c4bacd5e02e055d3f00650a1e126656 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13965 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/13965 for details. -gerrit

1 0

Patch merged into coreboot/master: crossgcc/buildgcc: Update for recent arch additions
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 3e0f74d6b58e3c1e0a06e3a84923e26be36ebe1f Author: Martin Roth <martinroth(a)google.com> Date: Tue Mar 8 12:07:04 2016 -0700 crossgcc/buildgcc: Update for recent arch additions - Add powerpc64le-linux-gnu & nds32le-elf to the instructions as supported architectures - Add nds32le-elf as a supported architecture so it will stop warning when you build it. Change-Id: Ifcdbc3d082eae5b9b5f8828914e7d2b7ed1f13a4 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13961 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/13961 for details. -gerrit

1 0

Patch set updated for coreboot: crossgcc/buildgcc: Update for recent arch additions
by Martin Roth 11 Mar '16

11 Mar '16

Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13961 -gerrit commit 273356e387aab808b02674082c912b5a655f8d47 Author: Martin Roth <martinroth(a)google.com> Date: Tue Mar 8 12:07:04 2016 -0700 crossgcc/buildgcc: Update for recent arch additions - Add powerpc64le-linux-gnu & nds32le-elf to the instructions as supported architectures - Add nds32le-elf as a supported architecture so it will stop warning when you build it. Change-Id: Ifcdbc3d082eae5b9b5f8828914e7d2b7ed1f13a4 Signed-off-by: Martin Roth <martinroth(a)google.com> --- util/crossgcc/buildgcc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index df9ac4a..9266597 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -340,7 +340,8 @@ myhelp() printf " (defaults to $TARGETARCH)\n" printf " [-S|--scripting] build scripting support for GDB\n\n" printf "Platforms for GCC & GDB:\n" - printf " x86_64 i386-elf i386-mingw32 mipsel-elf riscv-elf arm aarch64\n\n" + printf " x86_64 i386-elf i386-mingw32 mipsel-elf riscv-elf arm aarch64\n" + printf " powerpc64le-linux-gnu nds32le-elf\n\n } printversion() { @@ -624,6 +625,7 @@ case "$TARGETARCH" in i386*) TARGETARCH=i386-elf;; arm*) TARGETARCH=arm-eabi;; aarch64*) TARGETARCH=aarch64-elf;; + nds32le-elf) ;; *) printf "${red}WARNING: Unsupported architecture $TARGETARCH.${NC}\n\n"; ;; esac

1 0

Patch merged into coreboot/master: payloads/external/Makefile.inc: Don't rebuild SeaBIOS every time
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 342cf7f6a4f63862a7b043b6c56fd3f5f3d7e500 Author: Martin Roth <martinroth(a)google.com> Date: Mon Mar 7 22:04:01 2016 -0700 payloads/external/Makefile.inc: Don't rebuild SeaBIOS every time Currently, if SeaBIOS is set as the payload, it gets rebuilt every single time we do a build. Change it to re-build just when there’s a config change. Change-Id: Ib141f2cbf8796d449172432bb30fa4806cf90328 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13948 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> See https://review.coreboot.org/13948 for details. -gerrit

1 0

Patch set updated for coreboot: payloads/external/Makefile.inc: Don't rebuild SeaBIOS every time
by Martin Roth 11 Mar '16

11 Mar '16

Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13948 -gerrit commit 2f8626b30a418ed317d530eb6989264ec85c02b8 Author: Martin Roth <martinroth(a)google.com> Date: Mon Mar 7 22:04:01 2016 -0700 payloads/external/Makefile.inc: Don't rebuild SeaBIOS every time Currently, if SeaBIOS is set as the payload, it gets rebuilt every single time we do a build. Change it to re-build just when there’s a config change. Change-Id: Ib141f2cbf8796d449172432bb30fa4806cf90328 Signed-off-by: Martin Roth <martinroth(a)google.com> --- payloads/external/Makefile.inc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/payloads/external/Makefile.inc b/payloads/external/Makefile.inc index 557de2a..bbad643 100644 --- a/payloads/external/Makefile.inc +++ b/payloads/external/Makefile.inc @@ -35,10 +35,11 @@ payload_revision-file := $(PAYLOAD_VERSION) payload_revision-type := raw SEABIOS_CC_OFFSET=$(if $(filter %ccache,$(HOSTCC)),2,1) -seabios: +payloads/external/SeaBIOS/seabios/out/bios.bin.elf seabios: $(top)/$(DOTCONFIG) $(MAKE) -C payloads/external/SeaBIOS -f Makefile.inc \ HOSTCC="$(HOSTCC)" \ - CC=$(word $(SEABIOS_CC_OFFSET),$(CC_x86_32)) CFLAGS="$(patsubst $(word $(SEABIOS_CC_OFFSET),$(CC_x86_32))%,,$(wordlist $(SEABIOS_CC_OFFSET),9999,$(CC_x86_32)))" \ + CC=$(word $(SEABIOS_CC_OFFSET),$(CC_x86_32)) \ + CFLAGS="$(patsubst $(word $(SEABIOS_CC_OFFSET),$(CC_x86_32))%,,$(wordlist $(SEABIOS_CC_OFFSET),9999,$(CC_x86_32)))" \ LD=$(word 1,$(LD_x86_32)) LDFLAGS="$(patsubst $(word 1,$(LD_x86_32))%,,$(LD_x86_32))" \ OBJDUMP="$(OBJDUMP_x86_32)" \ OBJCOPY="$(OBJCOPY_x86_32)" STRIP="$(STRIP_x86_32)" \ @@ -53,9 +54,8 @@ seabios: CONFIG_TTYS0_BASE=$(CONFIG_TTYS0_BASE) payloads/external/SeaBIOS/seabios/out/vgabios.bin: seabios -payloads/external/SeaBIOS/seabios/out/bios.bin.elf: seabios -payloads/external/SeaBIOS/seabios/.config: seabios -payloads/external/SeaBIOS/seabios/out/autoversion.h: seabios +payloads/external/SeaBIOS/seabios/.config: payloads/external/SeaBIOS/seabios/out/bios.bin.elf +payloads/external/SeaBIOS/seabios/out/autoversion.h: payloads/external/SeaBIOS/seabios/out/bios.bin.elf filo: $(MAKE) -C payloads/external/FILO -f Makefile.inc \

1 0

Patch merged into coreboot/master: nvramcui: Add distclean target
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit c3fbda4e73ff4d4470331a09779872d1973e0065 Author: Martin Roth <martinroth(a)google.com> Date: Mon Mar 7 16:40:14 2016 -0700 nvramcui: Add distclean target This doesn't do anything more than the clean target, but having both clean and distclean targets in all makefiles makes standardizing the cleaning routines easier. Change-Id: I41578de371a8f767ee23266c30e65e928f0985c4 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13939 Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Tested-by: build bot (Jenkins) See https://review.coreboot.org/13939 for details. -gerrit

1 0

Patch merged into coreboot/master: SeaBIOS: Update SeaBIOS from repo when using master branch
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 3f1abb84e87704eac45570bdc89e786022c06678 Author: Martin Roth <martinroth(a)google.com> Date: Mon Mar 7 21:56:24 2016 -0700 SeaBIOS: Update SeaBIOS from repo when using master branch Previously the SeaBIOS directory would never get updated after the initial clone because the tag would always match. This can be shown by noticing that the text 'Fetching new commits from the SeaBIOS git repo' is never seen. This change will always try to pull the latest code if 'Master' is selected. Change-Id: I460e2fb0c6f683a0f85343d164880c2d9e6d95cc Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13947 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/13947 for details. -gerrit

1 0

Patch merged into coreboot/master: crossgcc/buildgcc: Add comment about URLs and jenkins builder
by gerrit＠coreboot.org 11 Mar '16

11 Mar '16

the following patch was just integrated into master: commit 1484c03916ec7672a053f04bf0b53741189bf649 Author: Martin Roth <martinroth(a)google.com> Date: Tue Mar 8 09:37:14 2016 -0700 crossgcc/buildgcc: Add comment about URLs and jenkins builder Add a comment to try to lower possible confusion later if the jenkins tool builder fails to build a new tool. The URLs for the packages that are downloaded are checked against known locations so that someone can't maliciously download a package from somewhere and run it on the build server. This provides a little bit of security, but could confuse someone if they don't realize it. Change-Id: I7858e3d86fc705b480f6792b6adf3d5349580e01 Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13955 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/13955 for details. -gerrit

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit March 2016