Patrick Georgi (pgeorgi(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17603
-gerrit
commit 4e7df06c8e1ebe450ddc4e9c601c898b30452c75
Author: Patrick Georgi <pgeorgi(a)chromium.org>
Date: Fri Nov 25 15:27:41 2016 +0100
util/crossgcc: fix using -D
Otherwise errors similar to "touch: cannot touch
'${TARGETDIR}/.GMP.6.1.0.success': No such file
or directory" might occur.
Change-Id: I4f24c93a25b7d567d3ce14a0415d20fd0778c9c8
Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org>
---
util/crossgcc/buildgcc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 949e17c..72e8366 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -388,20 +388,20 @@ build_for_host()
{
package="$1"
version="$(eval echo \$$package"_VERSION")"
- generic_build "$package" host "build-$package" "${TARGETDIR}/.${package}.${version}.success" "$version"
+ generic_build "$package" host "build-$package" "${DESTDIR}${TARGETDIR}/.${package}.${version}.success" "$version"
}
build_for_target()
{
package="$1"
version="$(eval echo \$$package"_VERSION")"
- generic_build "$package" target "build-${TARGETARCH}-$package" "${TARGETDIR}/.${TARGETARCH}-${package}.${version}.success" "$version"
+ generic_build "$package" target "build-${TARGETARCH}-$package" "${DESTDIR}${TARGETDIR}/.${TARGETARCH}-${package}.${version}.success" "$version"
}
build()
{
if package_uses_targetarch $1; then
- if [ $BOOTSTRAP -eq 1 -a ! -f "${TARGETDIR}/.GCC.success" ]; then
+ if [ $BOOTSTRAP -eq 1 -a ! -f "${DESTDIR}${TARGETDIR}/.GCC.success" ]; then
build_for_host GCC
fi
build_for_target $1
the following patch was just integrated into master:
commit bf618cb94ea3e4a131ea61dab3702363afa94286
Author: Jeremy Compostella <jeremy.compostella(a)gmail.com>
Date: Fri Nov 18 13:40:32 2016 +0100
libpayload: increase MAX_ARGC_COUNT
MAX_ARGC_COUNT limits the payload to ten parameters which is not
enough when used with a proprietary first stage bootloader providing
hardware description using around 20 parameters.
This patch makes the libpayload able to get up to 32 parameters.
Change-Id: I49925040d951dffb9c11425334674d8d498821f2
Signed-off-by: Jeremy Compostella <jeremy.compostella(a)gmail.com>
Reviewed-on: https://review.coreboot.org/17467
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Tested-by: Martin Roth <martinroth(a)google.com>
Reviewed-by: Nico Huber <nico.h(a)gmx.de>
See https://review.coreboot.org/17467 for details.
-gerrit
Damien Zammit (damien(a)zamaudio.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17581
-gerrit
commit 11eb933c58ea71f53aa86defc27b419fcbc56add
Author: Arthur Heymans <arthur(a)aheymans.xyz>
Date: Wed Nov 23 16:39:52 2016 +0100
sio/it8718f: Hook up common environment-controller driver
Change-Id: I25019c6323b6e9de2e0ce19325266bf3e8f2e309
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
src/superio/ite/it8718f/Kconfig | 3 +++
src/superio/ite/it8718f/chip.h | 26 ++++++++++++++++++++++++++
src/superio/ite/it8718f/superio.c | 12 +++++++++++-
3 files changed, 40 insertions(+), 1 deletion(-)
diff --git a/src/superio/ite/it8718f/Kconfig b/src/superio/ite/it8718f/Kconfig
index 0721317..3201195 100644
--- a/src/superio/ite/it8718f/Kconfig
+++ b/src/superio/ite/it8718f/Kconfig
@@ -17,3 +17,6 @@
config SUPERIO_ITE_IT8718F
bool
select SUPERIO_ITE_COMMON_ROMSTAGE
+ select SUPERIO_ITE_ENV_CTRL
+ select SUPERIO_ITE_ENV_CTRL_FAN16_CONFIG
+ select SUPERIO_ITE_ENV_CTRL_PWM_FREQ2
diff --git a/src/superio/ite/it8718f/chip.h b/src/superio/ite/it8718f/chip.h
new file mode 100644
index 0000000..23511bb
--- /dev/null
+++ b/src/superio/ite/it8718f/chip.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016 secunet Security Networks AG
+ *
+ * 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; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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.
+ */
+
+#ifndef SUPERIO_ITE_IT8718F_CHIP_H
+#define SUPERIO_ITE_IT8718F_CHIP_H
+
+#include <superio/ite/common/env_ctrl_chip.h>
+
+struct superio_ite_it8718f_config {
+ struct ite_ec_config ec;
+};
+
+#endif /* SUPERIO_ITE_IT8718F_CHIP_H */
diff --git a/src/superio/ite/it8718f/superio.c b/src/superio/ite/it8718f/superio.c
index a156aa6..a384ec0 100644
--- a/src/superio/ite/it8718f/superio.c
+++ b/src/superio/ite/it8718f/superio.c
@@ -18,10 +18,15 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
+#include <superio/ite/common/env_ctrl.h>
+
#include "it8718f.h"
+#include "chip.h"
static void init(struct device *dev)
{
+ const struct superio_ite_it8718f_config *conf;
+ const struct resource *res;
if (!dev->enabled)
return;
@@ -31,7 +36,12 @@ static void init(struct device *dev)
break;
case IT8718F_PP: /* TODO. */
break;
- case IT8718F_EC: /* TODO. */
+ case IT8718F_EC:
+ conf = dev->chip_info;
+ res = find_resource(dev, PNP_IDX_IO0);
+ if (!conf || !res)
+ break;
+ ite_ec_init(res->base, &conf->ec);
break;
case IT8718F_KBCK:
pc_keyboard_init(NO_AUX_DEVICE);
the following patch was just integrated into master:
commit f6fe2f1286f2551ae8d9732f949a51eb6c975363
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Mon Nov 21 11:26:48 2016 +0200
AGESA binaryPI: Fix cache-as-ram for x86_64
AMD_ENABLE_STACK was not called on x86_64 path for AGESA, while
it was for binaryPI.
Comments on BIST and cpu_init_detected were reversed, so fix those
too.
Change-Id: I0ddfaf51feb386a56d488c29d60171b05ff6fbc4
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: https://review.coreboot.org/17551
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-by: Edward O'Callaghan <edward.ocallaghan(a)koparo.com>
See https://review.coreboot.org/17551 for details.
-gerrit