summaryrefslogtreecommitdiff
path: root/src/ec
diff options
context:
space:
mode:
authorPatrick Rudolph <siro@das-labor.org>2017-05-24 18:14:43 +0200
committerMartin Roth <martinroth@google.com>2017-07-18 19:23:26 +0000
commita959a1456b8f5fcc641f10e6da14006712a9b0c1 (patch)
tree64275c169706a31b71a6834c8cf8d5a36d3192b4 /src/ec
parent67716456785f0bfa54f76cfb49396d091892f440 (diff)
downloadcoreboot-a959a1456b8f5fcc641f10e6da14006712a9b0c1.tar.xz
ec/lenovo/h8/h8: Always enable tp-smapi and thermal
Always enable tp-smapi and thermal managment. The devicetree already configures the correct values. This patch makes sure that invalid user-settings are ignored. The tp-smapi bit is required for the SMM handler. The thermal bit should be set to allow the EC to monitor thermal state of the platform. Change-Id: Ia5aa50e0b1148a7cc8e51480623368ee62edb849 Signed-off-by: Patrick Rudolph <siro@das-labor.org> Reviewed-on: https://review.coreboot.org/19864 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/ec')
-rw-r--r--src/ec/lenovo/h8/h8.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/src/ec/lenovo/h8/h8.c b/src/ec/lenovo/h8/h8.c
index b259d9ae21..5ea4a2cbec 100644
--- a/src/ec/lenovo/h8/h8.c
+++ b/src/ec/lenovo/h8/h8.c
@@ -209,21 +209,25 @@ static void h8_enable(struct device *dev)
{
struct ec_lenovo_h8_config *conf = dev->chip_info;
u8 val;
- u8 beepmask0, beepmask1, config1;
+ u8 beepmask0, beepmask1, reg8;
dev->ops = &h8_dev_ops;
h8_log_ec_version();
- ec_write(H8_CONFIG0, conf->config0);
- config1 = conf->config1;
+ /* Always enable I/O range 0x1600-0x160f and thermal management */
+ reg8 = conf->config0;
+ reg8 |= H8_CONFIG0_SMM_H8_ENABLE;
+ reg8 |= H8_CONFIG0_TC_ENABLE;
+ ec_write(H8_CONFIG0, reg8);
+ reg8 = conf->config1;
if (conf->has_keyboard_backlight) {
if (get_option(&val, "backlight") != CB_SUCCESS)
val = 0; /* Both backlights. */
- config1 = (config1 & 0xf3) | ((val & 0x3) << 2);
+ reg8 = (reg8 & 0xf3) | ((val & 0x3) << 2);
}
- ec_write(H8_CONFIG1, config1);
+ ec_write(H8_CONFIG1, reg8);
ec_write(H8_CONFIG2, conf->config2);
ec_write(H8_CONFIG3, conf->config3);