[coreboot-gerrit] New patch to review for coreboot: 86dbf6f ec/kontron/it8516e: Add option for external temperature sensor

[Nico Huber]

Nico Huber ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3772

-gerrit

commit 86dbf6f14af76882115479184252a6f0337a5a1d
Author: Nico Huber 
Date:   Fri Jul 12 14:35:00 2013 +0200

ec/kontron/it8516e: Add option for external temperature sensor

The IT8516E firmware of Kontron supports some selected external sensors
    attached to the EC via SMBUS or GPIO16.

Change-Id: I4c451c360a393e916430e3bea04a95847455cef7
    Signed-off-by: Nico Huber 
---
 src/ec/kontron/it8516e/chip.h | 10 +++++++---
 src/ec/kontron/it8516e/ec.c   | 15 +++++++++++++++
 src/ec/kontron/it8516e/ec.h   |  8 ++++++++
 3 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/src/ec/kontron/it8516e/chip.h b/src/ec/kontron/it8516e/chip.h
index dddc5e0..5638efd 100644
--- a/src/ec/kontron/it8516e/chip.h
+++ b/src/ec/kontron/it8516e/chip.h
@@ -26,10 +26,14 @@

struct ec_kontron_it8516e_config {
    /*
-	 * Per fan settings
-	 * Can be overwritten by fan1_mode, fan2_mode, fan1_target
-	 * and fan2_target options
+	 * Fan settings (fan1: CPU; fan2: System)
+	 * Can be overwritten by
+	 *   systemp_type,
+	 *   fan1_mode, fan2_mode,
+	 *   fan1_target, and fan2_target
+	 * nvram options.
     */
+	enum it8516e_systemp_types default_systemp;
    enum it8516e_fan_modes default_fan_mode[2];
    u16 default_fan_target[2]; /* PWM: % / Speed: RPM / Thermal: °C */
 };
diff --git a/src/ec/kontron/it8516e/ec.c b/src/ec/kontron/it8516e/ec.c
index fa7bbc8..3195ce9 100644
--- a/src/ec/kontron/it8516e/ec.c
+++ b/src/ec/kontron/it8516e/ec.c
@@ -30,6 +30,8 @@
 typedef struct ec_kontron_it8516e_config config_t;

enum { /* EC commands */
+	IT8516E_CMD_SET_SYSTEMP_TYPE	= 0x06,
+	IT8516E_CMD_GET_SYSTEMP_TYPE	= 0x07,
    IT8516E_CMD_GET_FAN_MODE	= 0x10,
    IT8516E_CMD_SET_FAN_MODE	= 0x11,
    IT8516E_CMD_GET_FAN_PWM		= 0x12,
@@ -40,6 +42,13 @@ enum { /* EC commands */
    IT8516E_CMD_SET_FAN_TEMP	= 0x17,
 };

+static void it8516e_set_systemp_type(const u8 type)
+{
+	if (send_ec_command(IT8516E_CMD_SET_SYSTEMP_TYPE))
+		return;
+	send_ec_data(type);
+}
+
 static void it8516e_set_fan_mode(const u8 idx, const u8 mode)
 {
    if (send_ec_command(IT8516E_CMD_SET_FAN_MODE))
@@ -138,6 +147,12 @@ static void it8516e_pm2_init(const device_t dev)
    ec_set_ports(find_resource(dev, PNP_IDX_IO1)->base,
    	     find_resource(dev, PNP_IDX_IO0)->base);

+	u8 systemp_type = config->default_systemp;
+	get_option(&systemp_type, "systemp_type");
+	if (systemp_type > 4)
+		systemp_type = IT8516E_SYSTEMP_NONE;
+	it8516e_set_systemp_type(systemp_type);
+
    it8516e_set_fan_from_options(config, 0);
    it8516e_set_fan_from_options(config, 1);
 }
diff --git a/src/ec/kontron/it8516e/ec.h b/src/ec/kontron/it8516e/ec.h
index 5bd2801..d9761e5 100644
--- a/src/ec/kontron/it8516e/ec.h
+++ b/src/ec/kontron/it8516e/ec.h
@@ -40,4 +40,12 @@ enum it8516e_fan_modes { /* Possible modes of fan control */
    IT8516E_MODE_THERMAL	= 0x03,
 };

+enum it8516e_systemp_types { /* Possible sources of system temperature */
+	IT8516E_SYSTEMP_NONE	= 0x00,
+	IT8516E_SYSTEMP_AMD	= 0x01,
+	IT8516E_SYSTEMP_LM75_90	= 0x02,
+	IT8516E_SYSTEMP_GPIO16	= 0x03,
+	IT8516E_SYSTEMP_LM75_9e	= 0x04,
+};
+
 #endif