ish: Trim down the release branchstabilize-wristpin-14469.59.B-ishstabilize-voshyr-14637.B-ishstabilize-quickfix-14695.187.B-ishstabilize-quickfix-14695.124.B-ishstabilize-quickfix-14526.91.B-ishstabilize-14695.85.B-ishstabilize-14695.107.B-ishstabilize-14682.B-ishstabilize-14633.B-ishstabilize-14616.B-ishstabilize-14589.B-ishstabilize-14588.98.B-ishstabilize-14588.14.B-ishstabilize-14588.123.B-ishstabilize-14536.B-ishstabilize-14532.B-ishstabilize-14528.B-ishstabilize-14526.89.B-ishstabilize-14526.84.B-ishstabilize-14526.73.B-ishstabilize-14526.67.B-ishstabilize-14526.57.B-ishstabilize-14498.B-ishstabilize-14496.B-ishstabilize-14477.B-ishstabilize-14469.9.B-ishstabilize-14469.8.B-ishstabilize-14469.58.B-ishstabilize-14469.41.B-ishstabilize-14442.B-ishstabilize-14438.B-ishstabilize-14411.B-ishstabilize-14396.B-ishstabilize-14395.B-ishstabilize-14388.62.B-ishstabilize-14388.61.B-ishstabilize-14388.52.B-ishstabilize-14385.B-ishstabilize-14345.B-ishstabilize-14336.B-ishstabilize-14333.B-ishrelease-R99-14469.B-ishrelease-R98-14388.B-ishrelease-R102-14695.B-ishrelease-R101-14588.B-ishrelease-R100-14526.B-ishfirmware-cherry-14454.B-ishfirmware-brya-14505.B-ishfirmware-brya-14505.71.B-ishfactory-kukui-14374.B-ishfactory-guybrush-14600.B-ishfactory-cherry-14455.B-ishfactory-brya-14517.B-ish

In the interest of making long-term branch maintenance incur as little
technical debt on us as possible, we should not maintain any files on
the branch we are not actually using.

This has the added effect of making it extremely clear when merging CLs
from the main branch when changes have the possibility to affect us.

The follow-on CL adds a convenience script to actually pull updates from
the main branch and generate a CL for the update.

BUG=b:204206272
BRANCH=ish
TEST=make BOARD=arcada_ish && make BOARD=drallion_ish

Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I17e4694c38219b5a0823e0a3e55a28d1348f4b18
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262038
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>

1 files changed, 0 insertions, 399 deletions

diff --git a/driver/temp_sensor/tmp432.c b/driver/temp_sensor/tmp432.c
deleted file mode 100644
index 6260678dcd..0000000000
--- a/driver/temp_sensor/tmp432.c
+++ /dev/null
@@ -1,399 +0,0 @@
-/* Copyright 2013 The Chromium OS Authors. All rights reserved.
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-/* TMP432 temperature sensor module for Chrome EC */
-
-#include "common.h"
-#include "console.h"
-#include "tmp432.h"
-#include "gpio.h"
-#include "i2c.h"
-#include "hooks.h"
-#include "util.h"
-
-static int temp_val_local;
-static int temp_val_remote1;
-static int temp_val_remote2;
-#ifndef CONFIG_TEMP_SENSOR_POWER_GPIO
-static uint8_t is_sensor_shutdown;
-#endif
-static int fake_temp[TMP432_IDX_COUNT] = {-1, -1, -1};
-
-/**
- * Determine whether the sensor is powered.
- *
- * @return non-zero the tmp432 sensor is powered.
- */
-static int has_power(void)
-{
-#ifdef CONFIG_TEMP_SENSOR_POWER_GPIO
-	return gpio_get_level(CONFIG_TEMP_SENSOR_POWER_GPIO);
-#else
-	return !is_sensor_shutdown;
-#endif
-}
-
-static int raw_read8(const int offset, int *data_ptr)
-{
-	return i2c_read8(I2C_PORT_THERMAL, TMP432_I2C_ADDR_FLAGS,
-			 offset, data_ptr);
-}
-
-static int raw_write8(const int offset, int data)
-{
-	return i2c_write8(I2C_PORT_THERMAL, TMP432_I2C_ADDR_FLAGS,
-			  offset, data);
-}
-
-static int get_temp(const int offset, int *temp_ptr)
-{
-	int rv;
-	int temp_raw = 0;
-
-	rv = raw_read8(offset, &temp_raw);
-	if (rv < 0)
-		return rv;
-
-	*temp_ptr = (int)(int8_t)temp_raw;
-	return EC_SUCCESS;
-}
-
-#ifdef CONFIG_CMD_TEMP_SENSOR
-static int tmp432_set_temp(const int offset, int temp)
-{
-	if (temp < -127 || temp > 127)
-		return EC_ERROR_INVAL;
-
-	return raw_write8(offset, (uint8_t)temp);
-}
-#endif
-
-int tmp432_get_val(int idx, int *temp_ptr)
-{
-	if (!has_power())
-		return EC_ERROR_NOT_POWERED;
-
-	switch (idx) {
-	case TMP432_IDX_LOCAL:
-		*temp_ptr = temp_val_local;
-		break;
-	case TMP432_IDX_REMOTE1:
-		*temp_ptr = temp_val_remote1;
-		break;
-	case TMP432_IDX_REMOTE2:
-		*temp_ptr = temp_val_remote2;
-		break;
-	default:
-		return EC_ERROR_UNKNOWN;
-	}
-
-	return EC_SUCCESS;
-}
-
-#ifndef CONFIG_TEMP_SENSOR_POWER_GPIO
-static int tmp432_shutdown(uint8_t want_shutdown)
-{
-	int ret, value;
-
-	if (want_shutdown == is_sensor_shutdown)
-		return EC_SUCCESS;
-
-	ret = raw_read8(TMP432_CONFIGURATION1_R, &value);
-	if (ret < 0) {
-		ccprintf("ERROR: Temp sensor I2C read8 error.\n");
-		return ret;
-	}
-
-	if (want_shutdown && !(value & TMP432_CONFIG1_RUN_L)) {
-		/* tmp432 is running, and want it to shutdown */
-		/* CONFIG REG1 BIT6: 0=Run, 1=Shutdown */
-		/* shut it down */
-		value |= TMP432_CONFIG1_RUN_L;
-		ret = raw_write8(TMP432_CONFIGURATION1_R, value);
-	} else if (!want_shutdown && (value & TMP432_CONFIG1_RUN_L)) {
-		/* tmp432 is shutdown, and want turn it on */
-		value &= ~TMP432_CONFIG1_RUN_L;
-		ret = raw_write8(TMP432_CONFIGURATION1_R, value);
-	}
-	/* else, the current setting is exactly what you want */
-
-	is_sensor_shutdown = want_shutdown;
-	return ret;
-}
-#endif
-
-static int tmp432_set_therm_mode(void)
-{
-	int ret = 0;
-	int data = 0;
-
-	ret = raw_read8(TMP432_CONFIGURATION1_R, &data);
-	if (ret)
-		return EC_ERROR_UNKNOWN;
-
-	data |= TMP432_CONFIG1_MODE;
-	ret = raw_write8(TMP432_CONFIGURATION1_W, data);
-	if (ret)
-		return EC_ERROR_UNKNOWN;
-
-	return EC_SUCCESS;
-}
-
-int tmp432_set_therm_limit(int channel, int limit_c, int hysteresis)
-{
-	int ret = 0;
-	int reg = 0;
-
-	if (channel >= TMP432_CHANNEL_COUNT)
-		return EC_ERROR_INVAL;
-
-	if (hysteresis > TMP432_HYSTERESIS_HIGH_LIMIT ||
-		hysteresis < TMP432_HYSTERESIS_LOW_LIMIT)
-		return EC_ERROR_INVAL;
-
-	/* hysteresis must be less than high limit */
-	if (hysteresis > limit_c)
-		return EC_ERROR_INVAL;
-
-	if (tmp432_set_therm_mode() != EC_SUCCESS)
-		return EC_ERROR_UNKNOWN;
-
-	switch (channel) {
-	case TMP432_CHANNEL_LOCAL:
-		reg = TMP432_LOCAL_HIGH_LIMIT_W;
-		break;
-	case TMP432_CHANNEL_REMOTE1:
-		reg = TMP432_REMOTE1_HIGH_LIMIT_W;
-		break;
-	case TMP432_CHANNEL_REMOTE2:
-		reg = TMP432_REMOTE2_HIGH_LIMIT_W;
-		break;
-	}
-
-	ret = raw_write8(reg, limit_c);
-	if (ret)
-		return EC_ERROR_UNKNOWN;
-
-	ret = raw_write8(TMP432_THERM_HYSTERESIS, hysteresis);
-	if (ret)
-		return EC_ERROR_UNKNOWN;
-
-	return EC_SUCCESS;
-}
-
-static void temp_sensor_poll(void)
-{
-	int temp_c;
-
-	if (!has_power())
-		return;
-
-	if (fake_temp[TMP432_IDX_LOCAL] != -1) {
-		temp_val_local = C_TO_K(fake_temp[TMP432_IDX_LOCAL]);
-	} else {
-		if (get_temp(TMP432_LOCAL, &temp_c) == EC_SUCCESS)
-			temp_val_local = C_TO_K(temp_c);
-		/* else: Keep previous value when it fails */
-	}
-
-	if (fake_temp[TMP432_IDX_REMOTE1] != -1) {
-		temp_val_remote1 = C_TO_K(fake_temp[TMP432_IDX_REMOTE1]);
-	} else {
-		if (get_temp(TMP432_REMOTE1, &temp_c) == EC_SUCCESS)
-			temp_val_remote1 = C_TO_K(temp_c);
-		/* else: Keep previous value when it fails */
-	}
-
-	if (fake_temp[TMP432_IDX_REMOTE2] != -1) {
-		temp_val_remote2 = C_TO_K(fake_temp[TMP432_IDX_REMOTE2]);
-	} else {
-		if (get_temp(TMP432_REMOTE2, &temp_c) == EC_SUCCESS)
-			temp_val_remote2 = C_TO_K(temp_c);
-		/* else: Keep previous value when it fails */
-	}
-}
-DECLARE_HOOK(HOOK_SECOND, temp_sensor_poll, HOOK_PRIO_TEMP_SENSOR);
-
-#ifdef CONFIG_CMD_TEMP_SENSOR
-static int tmp432_set_fake_temp(int index, int degree_c)
-{
-	if ((index < 0) || (index >= TMP432_IDX_COUNT))
-		return EC_ERROR_INVAL;
-
-	fake_temp[index] = degree_c;
-	ccprintf("New degree will be updated 1 sec later\n\n");
-
-	return EC_SUCCESS;
-}
-
-static void print_temps(
-		const char *name,
-		const int tmp432_temp_reg,
-		const int tmp432_therm_limit_reg,
-		const int tmp432_high_limit_reg,
-		const int tmp432_low_limit_reg)
-{
-	int value;
-
-	if (!has_power()) {
-		ccprintf("  TMP432 is shutdown\n");
-		return;
-	}
-
-	ccprintf("%s:\n", name);
-
-	if (get_temp(tmp432_temp_reg, &value) == EC_SUCCESS)
-		ccprintf("  Temp       %3dC\n", value);
-
-	if (get_temp(tmp432_therm_limit_reg, &value) == EC_SUCCESS)
-		ccprintf("  Therm Trip %3dC\n", value);
-
-	if (get_temp(tmp432_high_limit_reg, &value) == EC_SUCCESS)
-		ccprintf("  High Alarm %3dC\n", value);
-
-	if (get_temp(tmp432_low_limit_reg, &value) == EC_SUCCESS)
-		ccprintf("  Low Alarm  %3dC\n", value);
-}
-
-static int print_status(void)
-{
-	int value, i;
-
-	print_temps("Local", TMP432_LOCAL,
-		    TMP432_LOCAL_THERM_LIMIT,
-		    TMP432_LOCAL_HIGH_LIMIT_R,
-		    TMP432_LOCAL_LOW_LIMIT_R);
-
-	print_temps("Remote1", TMP432_REMOTE1,
-		    TMP432_REMOTE1_THERM_LIMIT,
-		    TMP432_REMOTE1_HIGH_LIMIT_R,
-		    TMP432_REMOTE1_LOW_LIMIT_R);
-
-	print_temps("Remote2", TMP432_REMOTE2,
-		    TMP432_REMOTE2_THERM_LIMIT,
-		    TMP432_REMOTE2_HIGH_LIMIT_R,
-		    TMP432_REMOTE2_LOW_LIMIT_R);
-
-	ccprintf("\n");
-
-	for (i = 0; i < TMP432_IDX_COUNT; ++i) {
-		ccprintf("fake temperature[%d]= ", i);
-		if (fake_temp[i] == -1) {
-			ccprintf("Not overridden\n");
-			continue;
-		}
-
-		if (tmp432_get_val(i, &value) == EC_SUCCESS)
-			ccprintf("%d C or %d K\n", (value - 273), value);
-		else
-			ccprintf("Access error\n");
-	}
-
-	ccprintf("\n");
-
-	if (raw_read8(TMP432_STATUS, &value) == EC_SUCCESS)
-		ccprintf("STATUS:  %pb\n", BINARY_VALUE(value, 8));
-
-	if (raw_read8(TMP432_CONFIGURATION1_R, &value) == EC_SUCCESS)
-		ccprintf("CONFIG1: %pb\n", BINARY_VALUE(value, 8));
-
-	if (raw_read8(TMP432_CONFIGURATION2_R, &value) == EC_SUCCESS)
-		ccprintf("CONFIG2: %pb\n", BINARY_VALUE(value, 8));
-
-	return EC_SUCCESS;
-}
-
-static int command_tmp432(int argc, char **argv)
-{
-	char *command;
-	char *e;
-	char *power;
-	int data;
-	int offset;
-	int rv;
-
-	/* handle "power" command before checking the power status. */
-	if ((argc == 3) && !strcasecmp(argv[1], "power")) {
-		power = argv[2];
-		if (!strncasecmp(power, "on", sizeof("on"))) {
-			rv = tmp432_set_power(TMP432_POWER_ON);
-			if (!rv)
-				print_status();
-		}
-		else if (!strncasecmp(power, "off", sizeof("off")))
-			rv = tmp432_set_power(TMP432_POWER_OFF);
-		else
-			return EC_ERROR_PARAM2;
-		ccprintf("Set TMP432 %s\n", power);
-		return rv;
-	}
-
-	if (!has_power()) {
-		ccprintf("ERROR: Temp sensor not powered.\n");
-		return EC_ERROR_NOT_POWERED;
-	}
-
-	/* If no args just print status */
-	if (argc == 1)
-		return print_status();
-
-	if (argc < 3)
-		return EC_ERROR_PARAM_COUNT;
-
-	command = argv[1];
-	offset = strtoi(argv[2], &e, 0);
-	if (*e || offset < 0 || offset > 255)
-		return EC_ERROR_PARAM2;
-
-	if (!strcasecmp(command, "getbyte")) {
-		rv = raw_read8(offset, &data);
-		if (rv < 0)
-			return rv;
-		ccprintf("Byte at offset 0x%02x is %pb\n",
-			 offset, BINARY_VALUE(data, 8));
-		return rv;
-	}
-
-	/* Remaining commands are "tmp432 set-command offset data" */
-	if (argc != 4)
-		return EC_ERROR_PARAM_COUNT;
-
-	data = strtoi(argv[3], &e, 0);
-	if (*e)
-		return EC_ERROR_PARAM3;
-
-	if (!strcasecmp(command, "settemp")) {
-		ccprintf("Setting 0x%02x to %dC\n", offset, data);
-		rv = tmp432_set_temp(offset, data);
-	} else if (!strcasecmp(command, "setbyte")) {
-		ccprintf("Setting 0x%02x to 0x%02x\n", offset, data);
-		rv = raw_write8(offset, data);
-	} else if (!strcasecmp(command, "fake")) {
-		ccprintf("Hook temperature\n");
-		rv = tmp432_set_fake_temp(offset, data);
-		print_status();
-	} else
-		return EC_ERROR_PARAM1;
-
-	return rv;
-}
-DECLARE_CONSOLE_COMMAND(tmp432, command_tmp432,
-	"[settemp|setbyte <offset> <value>] or [getbyte <offset>] or"
-	"[fake <index> <value>] or [power <on|off>]. "
-	"Temps in Celsius.",
-	"Print tmp432 temp sensor status or set parameters.");
-#endif
-
-int tmp432_set_power(enum tmp432_power_state power_on)
-{
-#ifndef CONFIG_TEMP_SENSOR_POWER_GPIO
-	uint8_t shutdown = (power_on == TMP432_POWER_OFF) ? 1 : 0;
-	return tmp432_shutdown(shutdown);
-#else
-	gpio_set_level(CONFIG_TEMP_SENSOR_POWER_GPIO, power_on);
-	return EC_SUCCESS;
-#endif
-}
-