diff options
| author | Leila Lin <leilacy_lin@compal.corp-partner.google.com> | 2022-11-25 01:09:20 +0800 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-12-13 03:01:18 +0000 |
| commit | ee253e7e5ae01be179eee005cfd1732fd46a8d42 (patch) | |
| tree | aab7f83cae49d73509b96dfe40ee213154cb7d3d | |
| parent | 66bb9ab754a109bdeae44f0376bf1a87a0e1fd10 (diff) | |
| download | chrome-ec-ee253e7e5ae01be179eee005cfd1732fd46a8d42.tar.gz | |
winterhold: Notify host when temperature change
Send "EC_HOST_EVENT_THERMAL_THRESHOLD" event to host when
temp_sensor_amb's value changes +/-1 deg C.
LOW_COVERAGE_REASON=no unit test for skyrim board yet: b/247151116
BRANCH=none
BUG=b:232946420
TEST=zmake build winterhold and confirm event
"EC_HOST_EVENT_THERMAL_THRESHOLD" is send when temp sensor's value
changes.
Change-Id: I97c64cc4a0e75566823556c0b957a474de15e5d2
Signed-off-by: Leila Lin <leilacy_lin@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4054311
Reviewed-by: Diana Z <dzigterman@chromium.org>
Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com>
Commit-Queue: LeilaCY Lin <leilacy_lin@compal.corp-partner.google.com.test-google-a.com>
Tested-by: LeilaCY Lin <leilacy_lin@compal.corp-partner.google.com.test-google-a.com>
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Tested-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com>
| -rw-r--r-- | zephyr/program/skyrim/winterhold/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | zephyr/program/skyrim/winterhold/project.overlay | 4 | ||||
| -rw-r--r-- | zephyr/program/skyrim/winterhold/src/thermal.c | 32 |
3 files changed, 35 insertions, 2 deletions
diff --git a/zephyr/program/skyrim/winterhold/CMakeLists.txt b/zephyr/program/skyrim/winterhold/CMakeLists.txt index 70d3882323..0596978c05 100644 --- a/zephyr/program/skyrim/winterhold/CMakeLists.txt +++ b/zephyr/program/skyrim/winterhold/CMakeLists.txt @@ -10,4 +10,5 @@ zephyr_library_sources( "src/sensor.c" "src/battery_present.c" "src/alt_charger.c" + "src/thermal.c" ) diff --git a/zephyr/program/skyrim/winterhold/project.overlay b/zephyr/program/skyrim/winterhold/project.overlay index 6ce9471460..6e36fade79 100644 --- a/zephyr/program/skyrim/winterhold/project.overlay +++ b/zephyr/program/skyrim/winterhold/project.overlay @@ -53,7 +53,7 @@ named-temp-sensors { compatible = "cros-ec,temp-sensors"; - soc-pct2075 { + temp_sensor_soc: soc-pct2075 { temp_host_high = <105>; temp_host_halt = <110>; temp_host_release_high = <95>; @@ -61,7 +61,7 @@ power-good-pin = <&gpio_pg_pwr_s5>; sensor = <&soc_pct2075>; }; - amb-pct2075 { + temp_sensor_amb: amb-pct2075 { temp_host_warn = <50>; temp_host_high = <105>; temp_host_halt = <110>; diff --git a/zephyr/program/skyrim/winterhold/src/thermal.c b/zephyr/program/skyrim/winterhold/src/thermal.c new file mode 100644 index 0000000000..2c9432be09 --- /dev/null +++ b/zephyr/program/skyrim/winterhold/src/thermal.c @@ -0,0 +1,32 @@ +/* Copyright 2022 The ChromiumOS Authors + * Use of this source code is governed by a BSD-style license that can be + * found in the LICENSE file. + */ + +#include "hooks.h" +#include "host_command.h" +#include "temp_sensor/temp_sensor.h" + +#define CPRINTS(format, args...) cprints(CC_THERMAL, format, ##args) +#define CPRINTF(format, args...) cprintf(CC_THERMAL, format, ##args) + +#define TEMP_AMB TEMP_SENSOR_ID(DT_NODELABEL(temp_sensor_amb)) + +static int last_amb_temp = -1; + +/* Set SCI event to host for temperature change */ +static void detect_temp_change(void) +{ + int t, rv; + + rv = temp_sensor_read(TEMP_AMB, &t); + if (rv == EC_SUCCESS) { + if (last_amb_temp != t) { + last_amb_temp = t; + host_set_single_event(EC_HOST_EVENT_THERMAL_THRESHOLD); + } + } else if (rv == EC_ERROR_INVAL) { + CPRINTS("Temp sensor: Invalid id"); + } +} +DECLARE_HOOK(HOOK_SECOND, detect_temp_change, HOOK_PRIO_TEMP_SENSOR_DONE); |