Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14231
-gerrit
commit a44e044b60f3e5da6c58afd7880e64056d308e4f
Author: Martin Roth <martinroth(a)google.com>
Date: Sat Apr 2 11:10:08 2016 -0600
crossgcc/Makefile.inc: Point jenkins build to the correct toolchain
Because the builders have the coreboot cross-compilers in their path,
the XGCCPATH variable needs to be set after building the new toolchain
before it will be used.
Change-Id: I7c270dab94be7e8f801d527169767018a24986e4
Signed-off-by: Martin Roth <martinroth(a)google.com>
---
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 5be4c10..752cab6 100644
--- a/util/crossgcc/Makefile.inc
+++ b/util/crossgcc/Makefile.inc
@@ -72,4 +72,4 @@ jenkins-build-toolchain:
$(MAKE) crosstools clang \
BUILDGCC_OPTIONS='-y --nocolor'
rm -f .xcompile
- $(MAKE) what-jenkins-does
+ $(MAKE) what-jenkins-does XGCCPATH=$(top)/util/crossgcc/xgcc/bin
Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14231
-gerrit
commit dd3fb7ba15f772628288578f79117655b7a22565
Author: Martin Roth <martinroth(a)google.com>
Date: Sat Apr 2 11:10:08 2016 -0600
crossgcc/Makefile.inc: Point jenkins build to the correct toolchain
Because the builders have the coreboot cross-compilers in their path,
the XGCCPATH variable needs to be set after building the new toolchain
before it will be used.
Change-Id: I7c270dab94be7e8f801d527169767018a24986e4
Signed-off-by: Martin Roth <martinroth(a)google.com>
---
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 5be4c10..752cab6 100644
--- a/util/crossgcc/Makefile.inc
+++ b/util/crossgcc/Makefile.inc
@@ -72,4 +72,4 @@ jenkins-build-toolchain:
$(MAKE) crosstools clang \
BUILDGCC_OPTIONS='-y --nocolor'
rm -f .xcompile
- $(MAKE) what-jenkins-does
+ $(MAKE) what-jenkins-does XGCCPATH=$(top)/util/crossgcc/xgcc/bin
the following patch was just integrated into master:
commit 87eacacb35ee45128a6aac61fb657e6f2e374f09
Author: Paul Menzel <paulepanter(a)users.sourceforge.net>
Date: Thu Jun 5 07:52:49 2014 +0200
superio/nuvoton: Use official spelling of Nuvoton in `CHIP_NAME`
The official spelling of Nuvoton is not all uppercase. Only the first
letter is uppercase. See the footer of the Nuvoton Web site.
Change-Id: I6ccd4194d7be0c89f8b332fcca5feb2420a4de1e
Signed-off-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-on: https://review.coreboot.org/5928
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
See https://review.coreboot.org/5928 for details.
-gerrit
Paul Menzel (paulepanter(a)users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/5928
-gerrit
commit 800f738b558b4907e30b993f243c4f5ed8e6a53a
Author: Paul Menzel <paulepanter(a)users.sourceforge.net>
Date: Thu Jun 5 07:52:49 2014 +0200
superio/nuvoton: Use official spelling of Nuvoton in `CHIP_NAME`
The official spelling of Nuvoton is not all uppercase. Only the first
letter is uppercase. See the footer of the Nuvoton Web site.
Change-Id: I6ccd4194d7be0c89f8b332fcca5feb2420a4de1e
Signed-off-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
---
src/superio/nuvoton/nct5104d/superio.c | 2 +-
src/superio/nuvoton/wpcm450/superio.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/superio/nuvoton/nct5104d/superio.c b/src/superio/nuvoton/nct5104d/superio.c
index 5eb8991..4205d75 100644
--- a/src/superio/nuvoton/nct5104d/superio.c
+++ b/src/superio/nuvoton/nct5104d/superio.c
@@ -53,6 +53,6 @@ static void enable_dev(struct device *dev)
}
struct chip_operations superio_nuvoton_nct5104d_ops = {
- CHIP_NAME("NUVOTON NCT5104D Super I/O")
+ CHIP_NAME("Nuvoton NCT5104D Super I/O")
.enable_dev = enable_dev,
};
diff --git a/src/superio/nuvoton/wpcm450/superio.c b/src/superio/nuvoton/wpcm450/superio.c
index ff815cf..2e355d3 100644
--- a/src/superio/nuvoton/wpcm450/superio.c
+++ b/src/superio/nuvoton/wpcm450/superio.c
@@ -60,6 +60,6 @@ static void enable_dev(struct device *dev)
}
struct chip_operations superio_nuvoton_wpcm450_ops = {
- CHIP_NAME("NUVOTON WPCM450 Super I/O")
+ CHIP_NAME("Nuvoton WPCM450 Super I/O")
.enable_dev = enable_dev,
};
the following patch was just integrated into master:
commit faa74b0fb81b3a1dd3e37f05774a24446057228a
Author: Aaron Durbin <adurbin(a)chromium.org>
Date: Thu Mar 31 14:00:47 2016 -0500
soc/intel/apollolake: use platform_segment_loaded() for CAR coherency
Instead of using arch_segment_loaded() implement
platform_segment_loaded() so as not to tangle the notion of
arch and the chipset. Lastly, add a TODO to allow filtering
of the L1D to L2 flush depending on the region loaded.
Change-Id: I52e7cd2ae6e2d95f21bdd2fe1a471a10565309cb
Signed-off-by: Aaron Durbin <adurbin(a)chromium.org>
Reviewed-on: https://review.coreboot.org/14215
Reviewed-by: Furquan Shaikh <furquan(a)google.com>
Tested-by: build bot (Jenkins)
Reviewed-by: Andrey Petrov <andrey.petrov(a)intel.com>
See https://review.coreboot.org/14215 for details.
-gerrit
Aaron Durbin (adurbin(a)chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14215
-gerrit
commit 7b8174f32aa79a62716efb068dd10b9b2ba45cb8
Author: Aaron Durbin <adurbin(a)chromium.org>
Date: Thu Mar 31 14:00:47 2016 -0500
soc/intel/apollolake: use platform_segment_loaded() for CAR coherency
Instead of using arch_segment_loaded() implement
platform_segment_loaded() so as not to tangle the notion of
arch and the chipset. Lastly, add a TODO to allow filtering
of the L1D to L2 flush depending on the region loaded.
Change-Id: I52e7cd2ae6e2d95f21bdd2fe1a471a10565309cb
Signed-off-by: Aaron Durbin <adurbin(a)chromium.org>
---
src/soc/intel/apollolake/car.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/soc/intel/apollolake/car.c b/src/soc/intel/apollolake/car.c
index c49d7ef..b0a5b45 100644
--- a/src/soc/intel/apollolake/car.c
+++ b/src/soc/intel/apollolake/car.c
@@ -26,8 +26,10 @@ static void flush_l1d_to_l2(void)
wrmsr(MSR_POWER_MISC, msr);
}
-void arch_segment_loaded(uintptr_t start, size_t size, int flags)
+void platform_segment_loaded(uintptr_t start, size_t size, int flags)
{
+ /* TODO: filter on address to see if L1D flushing required. */
+
/* Flush L1D cache to L2 on final segment loaded */
if (flags & SEG_FINAL)
flush_l1d_to_l2();
the following patch was just integrated into master:
commit 096f45792670acbd3c68cf01c07aad679f831d3d
Author: Aaron Durbin <adurbin(a)chromium.org>
Date: Thu Mar 31 13:49:00 2016 -0500
lib/prog_loading: introduce prog_segment_loaded()
In order to not muddle arch vs chipset implementations provide
a generic prog_segment_loaded() which calls platform_segment_loaded()
and arch_segment_loaded() in that order. This allows the arch variants
to live in src/arch while the chipset/platform code can implement
their own.
Change-Id: I17b6497219ec904d92bd286f18c9ec96b2b7af25
Signed-off-by: Aaron Durbin <adurbin(a)chromium.org>
Reviewed-on: https://review.coreboot.org/14214
Reviewed-by: Furquan Shaikh <furquan(a)google.com>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-by: Andrey Petrov <andrey.petrov(a)intel.com>
See https://review.coreboot.org/14214 for details.
-gerrit