Lee Leahy (leroy.p.leahy(a)intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17967
-gerrit
commit 7c6fee0ae263fac7b52c9dccbc2c249c0f41c1c1
Author: Lee Leahy <leroy.p.leahy(a)intel.com>
Date: Mon Dec 26 10:58:16 2016 -0800
console: Enable do_printk_va_list for VBOOT
Use CONFIG_VBOOT to enable do_printk_va_list to match the conditionals
in include/console/console.h and the only caller is vboot/vboot_logic.c.
CONFIG_VBOOT is also selected for CONFIG_CHROMEOS.
TEST=Build and run on Galileo Gen2
Change-Id: Ia115c74afa498a14d5edd6f7940ec2edc124516f
Signed-off-by: Lee Leahy <leroy.p.leahy(a)intel.com>
---
src/console/printk.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/console/printk.c b/src/console/printk.c
index 855c841..2f7e277 100644
--- a/src/console/printk.c
+++ b/src/console/printk.c
@@ -79,7 +79,7 @@ int do_printk(int msg_level, const char *fmt, ...)
return i;
}
-#if IS_ENABLED (CONFIG_CHROMEOS)
+#if IS_ENABLED (CONFIG_VBOOT)
void do_printk_va_list(int msg_level, const char *fmt, va_list args)
{
if (!console_log_level(msg_level))
@@ -87,4 +87,4 @@ void do_printk_va_list(int msg_level, const char *fmt, va_list args)
vtxprintf(wrap_putchar, fmt, args, NULL);
console_tx_flush();
}
-#endif /* CONFIG_CHROMEOS */
+#endif /* CONFIG_VBOOT */
Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17966
-gerrit
commit 07117304b797b41359c6641e6b9875ce38cfebcf
Author: Martin Roth <martinroth(a)google.com>
Date: Sun Dec 18 11:02:56 2016 -0700
drivers/uart/oxpcie_early.c: remove uart_fill_lb()
uart_fill_lb() was added to drivers/uart/uart8250mem.c, so when the
Oxford OXPCIe952 Kconfig option is enabled, we were getting an error.
"multiple definition of `uart_fill_lb'"
The new version of uart_fill_lb sets the regwidth depending on the
Kconfig symbol DRIVERS_UART_8250MEM_32, so if that's selected, don't
give DRIVERS_UART_OXPCIE as a choice.
Change-Id: Ife24ab390553b10b2266809595c2e06463de708c
Signed-off-by: Martin Roth <martinroth(a)google.com>
---
src/drivers/uart/Kconfig | 2 +-
src/drivers/uart/oxpcie_early.c | 14 --------------
2 files changed, 1 insertion(+), 15 deletions(-)
diff --git a/src/drivers/uart/Kconfig b/src/drivers/uart/Kconfig
index 518e7ca..54f591d 100644
--- a/src/drivers/uart/Kconfig
+++ b/src/drivers/uart/Kconfig
@@ -47,7 +47,7 @@ config HAVE_UART_SPECIAL
config DRIVERS_UART_OXPCIE
bool "Oxford OXPCIe952"
default n
- depends on PCI
+ depends on PCI && !DRIVERS_UART_8250MEM_32
select DRIVERS_UART_8250MEM
select EARLY_PCI_BRIDGE
select UART_OVERRIDE_REFCLK
diff --git a/src/drivers/uart/oxpcie_early.c b/src/drivers/uart/oxpcie_early.c
index 0a778d9..f44e3e8 100644
--- a/src/drivers/uart/oxpcie_early.c
+++ b/src/drivers/uart/oxpcie_early.c
@@ -84,20 +84,6 @@ void oxford_remap(u32 new_base)
{
uart0_base = new_base + 0x1000;
}
-
-void uart_fill_lb(void *data)
-{
- struct lb_serial serial;
- serial.type = LB_SERIAL_TYPE_MEMORY_MAPPED;
- serial.baseaddr = uart_platform_base(CONFIG_UART_FOR_CONSOLE);
- serial.baud = default_baudrate();
- serial.regwidth = 1;
- serial.input_hertz = uart_platform_refclk();
- serial.uart_pci_addr = CONFIG_UART_PCI_ADDR;
- lb_add_serial(&serial, data);
-
- lb_add_console(LB_TAG_CONSOLE_SERIAL8250MEM, data);
-}
#endif
unsigned int uart_platform_refclk(void)
the following patch was just integrated into master:
commit 2a5897526463dfe00feb06d99f850c2874d1d257
Author: Duncan Laurie <dlaurie(a)chromium.org>
Date: Sat Dec 24 16:05:52 2016 -0800
google/eve: Enable internal pull-up on PWRBTN#
Enable an internal pull-up on the power button input as a quick
press is resulting in power button override being asserted.
BUG=chrome-os-partner:61312
TEST=tested on eve P0b to ensure quick power button press does
not result in a shutdown due to power button override.
Change-Id: I3028cf7faef309cf4d60c3585b48adab6e1549d4
Signed-off-by: Duncan Laurie <dlaurie(a)chromium.org>
Reviewed-on: https://review.coreboot.org/17962
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin(a)chromium.org>
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
See https://review.coreboot.org/17962 for details.
-gerrit
the following patch was just integrated into master:
commit a8025db49f6a8bf8c7fc1f1302e73e6a1ec30688
Author: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Date: Fri Dec 16 13:17:34 2016 -0500
amd-based mainboards: Fix whitespace in _PTS comments
Correct tabs that were intended as spaces.
Change-Id: Idcf33d829f87a866b5ed880527102918d5b93842
Signed-off-by: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Reviewed-on: https://review.coreboot.org/17905
Reviewed-by: Martin Roth <martinroth(a)google.com>
Tested-by: build bot (Jenkins)
See https://review.coreboot.org/17905 for details.
-gerrit
the following patch was just integrated into master:
commit 88766be48806a10d870e7a3e3fddbf7e6998056c
Author: Sumeet Pawnikar <sumeet.r.pawnikar(a)intel.com>
Date: Thu Dec 22 13:53:16 2016 +0530
mainboard/google/chell: Set TCC activation offset to 10 degree C
With the default TCC activation offset value as 0 and Tjmax
temperature value as 100 degree C, Pcode firmware starts taking
prochot action at 100 degree C [Tjmax-Offset].
But before Pcode firmware starts prochot action at 100 degree C,
device is getting shutdown at 99 degree C due to DPTF critical
CPU temperature.
This patch sets TCC activation offset value to 10 degree C for
thermal throttle action to prevent this kind of shutdown.
BUG=chrome-os-partner:59397
BRANCH=None.
TEST=Built, booted on skylake and verified target offset value.
Change-Id: I0811ef481a4b3ce4bd6ef24f2aa8160f44f9c990
Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar(a)intel.com>
Reviewed-on: https://review.coreboot.org/17921
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin(a)chromium.org>
See https://review.coreboot.org/17921 for details.
-gerrit
the following patch was just integrated into master:
commit 9d2f3de48c3f9fbca0124b2f851e6057e2f43a1b
Author: Sumeet Pawnikar <sumeet.r.pawnikar(a)intel.com>
Date: Thu Dec 22 13:48:46 2016 +0530
soc/intel/skylake: set TCC activation by BSP only
TCC activation functionality has package scope. It was set
for all CPU in the system which is unnecessary.
In this patch TCC activation is being set by the BSP only.
BUG=chrome-os-partner:59397
BRANCH=None.
TEST=Built for skylake platform and verified the TCC activation
value before and after S3.
Change-Id: Iacf64cbc40871bbec3bede65f196bf292e0149a6
Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar(a)intel.com>
Reviewed-on: https://review.coreboot.org/17889
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin(a)chromium.org>
See https://review.coreboot.org/17889 for details.
-gerrit