coreboot-gerrit February 2016

coreboot-gerrit@coreboot.org

1 participants
1305 discussions

Patch merged into coreboot/master: include/device: Move inline functions from pci_def.h to pci.h
by gerrit＠coreboot.org 03 Feb '16

03 Feb '16

the following patch was just integrated into master: commit 420caafee8455bd5df5be254594f1b280c3921c4 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Tue Jan 5 17:00:27 2016 -0800 include/device: Move inline functions from pci_def.h to pci.h pci_def.h is supposed to only contain definitions, such that it may be included in assembly files. Declaration of functions in said file prevents that. Change-Id: I0f90a74291c8a2ef7a1e1027d2d2182f896050fb Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Reviewed-on: https://review.coreboot.org/13300 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See https://review.coreboot.org/13300 for details. -gerrit

1 0

Patch merged into coreboot/master: xcompile: Add a way to specify -march=i586
by gerrit＠coreboot.org 03 Feb '16

03 Feb '16

the following patch was just integrated into master: commit 0cd9ff8987f9d4f303e01aefe9bfc812916fee04 Author: Martin Roth <martinroth(a)google.com> Date: Mon Feb 1 17:33:37 2016 -0700 xcompile: Add a way to specify -march=i586 Instead of instructing users to edit xcompile when they want to build a quark platform, give the build a way to set -march=586 so that the quark code will build correctly. The Quark processor does not support the instructions introduced with the Pentium 6 architecture. Change-Id: I0ed69aadc515f86f76800180e0e33bcd75feac5a Signed-off-by: Martin Roth <martinroth(a)google.com> Reviewed-on: https://review.coreboot.org/13552 Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Tested-by: build bot (Jenkins) Reviewed-by: Leroy P Leahy <leroy.p.leahy(a)intel.com> Reviewed-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-by: FEI WANG <wangfei.jimei(a)gmail.com> See https://review.coreboot.org/13552 for details. -gerrit

1 0

New patch to review for coreboot: crossgcc: Add checksum for make
by Stefan Reinauer 03 Feb '16

03 Feb '16

Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13568 -gerrit commit e68e6161d31e997def55d98449c3431fedfdc750 Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Date: Tue Feb 2 17:52:42 2016 -0800 crossgcc: Add checksum for make I forgot to add that when I added support to buildgcc. Change-Id: I586d64805e72f9512057a4e0698bdee19cc53146 Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> --- util/crossgcc/sum/make-4.1.tar.bz2.cksum | 1 + 1 file changed, 1 insertion(+) diff --git a/util/crossgcc/sum/make-4.1.tar.bz2.cksum b/util/crossgcc/sum/make-4.1.tar.bz2.cksum new file mode 100644 index 0000000..02807c6 --- /dev/null +++ b/util/crossgcc/sum/make-4.1.tar.bz2.cksum @@ -0,0 +1 @@ +0d701882fd6fd61a9652cb8d866ad7fc7de54d58 tarballs/make-4.1.tar.bz2

1 0

New patch to review for coreboot: crossgcc: Rename x86 to x64
by Stefan Reinauer 03 Feb '16

03 Feb '16

Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13567 -gerrit commit a5680996efbcf9fad636a35a4a2026b91f32d812 Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Date: Tue Feb 2 17:51:39 2016 -0800 crossgcc: Rename x86 to x64 Idee4eb5d112e3f6bffced0681e9112101bed6763 has renamed the architecture by accident. Rename it back. Change-Id: I5509d2aa09df513789325bc24d9b696a09cb898f Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> --- util/crossgcc/Makefile.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/crossgcc/Makefile.inc b/util/crossgcc/Makefile.inc index 1c28504..c2c0890 100644 --- a/util/crossgcc/Makefile.inc +++ b/util/crossgcc/Makefile.inc @@ -13,7 +13,7 @@ ## GNU General Public License for more details. ## -TOOLCHAIN_ARCHES := i386 x86 arm aarch64 mips riscv power8 nds32le +TOOLCHAIN_ARCHES := i386 x64 arm aarch64 mips riscv power8 nds32le help_toolchain help:: @echo '*** Toolchain targets ***'

1 0

Patch merged into coreboot/master: crossgcc: Also add the nds32le architecture to the coreboot Makefile
by gerrit＠coreboot.org 03 Feb '16

03 Feb '16

the following patch was just integrated into master: commit a9fa0c897c378b6c4f52d7bc71a3f672af7f2ee8 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Tue Feb 2 19:55:48 2016 +0100 crossgcc: Also add the nds32le architecture to the coreboot Makefile Change-Id: Ibf3346586d188dbd5b7ab10bedfc1609b2bb1499 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Reviewed-on: https://review.coreboot.org/13565 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> See https://review.coreboot.org/13565 for details. -gerrit

1 0

New patch to review for coreboot: soc/intel/broadwell: Fix SPI driver compilation
by Stefan Reinauer 03 Feb '16

03 Feb '16

Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13566 -gerrit commit 128910d248d3b436d1e84756867440174a77646e Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Date: Tue Feb 2 17:42:33 2016 -0800 soc/intel/broadwell: Fix SPI driver compilation With the latest coreboot toolchain the cast less removal of volatile breaks compilation. Change-Id: I92010516e8e01c870b60107e20a576a75d491e4e Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> --- src/soc/intel/broadwell/spi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/soc/intel/broadwell/spi.c b/src/soc/intel/broadwell/spi.c index e07b425..fde90c4 100644 --- a/src/soc/intel/broadwell/spi.c +++ b/src/soc/intel/broadwell/spi.c @@ -187,21 +187,21 @@ static u32 readl_(const void *addr) static void writeb_(u8 b, const void *addr) { - write8(addr, b); + write8((void *)addr, b); printk(BIOS_DEBUG, "wrote %2.2x to %4.4x\n", b, ((unsigned) addr & 0xffff) - 0xf020); } static void writew_(u16 b, const void *addr) { - write16(addr, b); + write16((void *)addr, b); printk(BIOS_DEBUG, "wrote %4.4x to %4.4x\n", b, ((unsigned) addr & 0xffff) - 0xf020); } static void writel_(u32 b, const void *addr) { - write32(addr, b); + write32((void *)addr, b); printk(BIOS_DEBUG, "wrote %8.8x to %4.4x\n", b, ((unsigned) addr & 0xffff) - 0xf020); }

1 0

Patch set updated for coreboot: crossgcc: Also add the nds32le architecture to the coreboot Makefile
by Stefan Reinauer 03 Feb '16

03 Feb '16

Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13565 -gerrit commit 10efd97b672b559701b8b19abe35765a54bffbeb Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Tue Feb 2 19:55:48 2016 +0100 crossgcc: Also add the nds32le architecture to the coreboot Makefile Change-Id: Ibf3346586d188dbd5b7ab10bedfc1609b2bb1499 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> --- util/crossgcc/Makefile.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/crossgcc/Makefile.inc b/util/crossgcc/Makefile.inc index bd1b0cd..1c28504 100644 --- a/util/crossgcc/Makefile.inc +++ b/util/crossgcc/Makefile.inc @@ -13,7 +13,7 @@ ## GNU General Public License for more details. ## -TOOLCHAIN_ARCHES := i386 x86 arm aarch64 mips riscv power8 +TOOLCHAIN_ARCHES := i386 x86 arm aarch64 mips riscv power8 nds32le help_toolchain help:: @echo '*** Toolchain targets ***'

1 0

Patch merged into coreboot/master: crossgcc/Makefile.inc: deduplicate cross*-$arch rules
by gerrit＠coreboot.org 03 Feb '16

03 Feb '16

the following patch was just integrated into master: commit 6c0ccfb10d9f2785832b98734b00993ec19e8611 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Sat Jan 30 09:14:45 2016 +0100 crossgcc/Makefile.inc: deduplicate cross*-$arch rules Change-Id: Idee4eb5d112e3f6bffced0681e9112101bed6763 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Reviewed-on: https://review.coreboot.org/13524 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> See https://review.coreboot.org/13524 for details. -gerrit

1 0

Patch set updated for coreboot: crossgcc/Makefile.inc: deduplicate cross*-$arch rules
by Stefan Reinauer 03 Feb '16

03 Feb '16

Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13524 -gerrit commit efd035f29aecae0cc229f0959e3f15b46b4e95b6 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Sat Jan 30 09:14:45 2016 +0100 crossgcc/Makefile.inc: deduplicate cross*-$arch rules Change-Id: Idee4eb5d112e3f6bffced0681e9112101bed6763 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> --- util/crossgcc/Makefile.inc | 48 +++++++--------------------------------------- 1 file changed, 7 insertions(+), 41 deletions(-) diff --git a/util/crossgcc/Makefile.inc b/util/crossgcc/Makefile.inc index 04eb70b..bd1b0cd 100644 --- a/util/crossgcc/Makefile.inc +++ b/util/crossgcc/Makefile.inc @@ -13,6 +13,8 @@ ## GNU General Public License for more details. ## +TOOLCHAIN_ARCHES := i386 x86 arm aarch64 mips riscv power8 + help_toolchain help:: @echo '*** Toolchain targets ***' @echo ' crossgcc - Build coreboot cross-compilers for all platforms' @@ -23,7 +25,7 @@ help_toolchain help:: @echo ' test-toolchain - Reports if toolchain components are out of date' @echo ' crossgcc-ARCH - Build cross-compiler for specific architecture' @echo ' crosstools-ARCH - Build cross-compiler with GDB for specific architecture' - @echo ' ARCH can be "i386", "x64", "arm", "aarch64", "mips", "riscv", or "power8"' + @echo ' ARCH can be "$(subst $(spc),"$(comma) ",$(TOOLCHAIN_ARCHES))"' @echo ' Use "make [target] CPUS=#" to build toolchain using multiple cores' @echo @@ -38,26 +40,8 @@ crossgcc: clean-for-update clang crosstools-i386 crosstools-x64 crosstools-arm \ crosstools-aarch64 crosstools-mips crosstools-riscv crosstools-power8 -crossgcc-i386: clean-for-update - $(MAKE) -C util/crossgcc build-i386 SKIP_GDB=1 - -crossgcc-x64: clean-for-update - $(MAKE) -C util/crossgcc build-x64 SKIP_GDB=1 - -crossgcc-arm: clean-for-update - $(MAKE) -C util/crossgcc build-arm SKIP_GDB=1 - -crossgcc-aarch64: clean-for-update - $(MAKE) -C util/crossgcc build-aarch64 SKIP_GDB=1 - -crossgcc-mips: clean-for-update - $(MAKE) -C util/crossgcc build-mips SKIP_GDB=1 - -crossgcc-riscv: clean-for-update - $(MAKE) -C util/crossgcc build-riscv SKIP_GDB=1 - -crossgcc-power8: clean-for-update - $(MAKE) -C util/crossgcc build-power8 SKIP_GDB=1 +$(foreach arch,$(TOOLCHAIN_ARCHES),crossgcc-$(arch)): clean-for-update + $(MAKE) -C util/crossgcc $(patsubst crossgcc-%,build-%,$@) SKIP_GDB=1 crosstools: clean-for-update $(MAKE) -C util/crossgcc all_with_gdb SKIP_CLANG=1 @@ -68,26 +52,8 @@ iasl: clean-for-update clang: clean-for-update $(MAKE) -C util/crossgcc build_clang -crosstools-i386: clean-for-update - $(MAKE) -C util/crossgcc build-i386 - -crosstools-x64: clean-for-update - $(MAKE) -C util/crossgcc build-x64 - -crosstools-arm: clean-for-update - $(MAKE) -C util/crossgcc build-arm - -crosstools-aarch64: clean-for-update - $(MAKE) -C util/crossgcc build-aarch64 - -crosstools-mips: clean-for-update - $(MAKE) -C util/crossgcc build-mips - -crosstools-riscv: clean-for-update - $(MAKE) -C util/crossgcc build-riscv - -crosstools-power8: clean-for-update - $(MAKE) -C util/crossgcc build-power8 +$(foreach arch,$(TOOLCHAIN_ARCHES),crosstools-$(arch)): clean-for-update + $(MAKE) -C util/crossgcc $(patsubst crosstools-%,build-%,$@) crossgcc-clean: clean-for-update $(MAKE) -C util/crossgcc clean

1 0

Patch merged into coreboot/master: buildgcc: Move all toolchain build targets to util/crossgcc
by gerrit＠coreboot.org 03 Feb '16

03 Feb '16

the following patch was just integrated into master: commit d5779c15eb297bd544ac0dd5144b09e6ac0e9023 Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Date: Fri Jan 29 17:48:47 2016 -0800 buildgcc: Move all toolchain build targets to util/crossgcc There is a lot of potential to completely get rid of Makefile and keep everything in Makefile.inc, but for now this declutters the main Makefile.inc. Change-Id: I653313c74207f955514c036c81efcbfd988827c9 Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Reviewed-on: https://review.coreboot.org/13518 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth(a)google.com> See https://review.coreboot.org/13518 for details. -gerrit

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit February 2016