From nobody Thu Dec 18 04:46:26 2025 Received: from todd.t-8ch.de (todd.t-8ch.de [159.69.126.157]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 59CB215EFDA; Wed, 29 May 2024 06:27:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=159.69.126.157 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716964044; cv=none; b=q9prvmx9Xa3w4jfx5BSCzqqCUNESwj8A2PrIaDjXGJm/xoy3pI1SddrKrS/qlNYtFwa/Y/JBjzzPF2t01EWTOCWWWIoMivRVsK8ybrIxpcbu/u1cqwx4sJR/4YnI4HL423/z6jAFmsMkleqlsFvB6WTpDO+LSOCtOOrrGUgS1x0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716964044; c=relaxed/simple; bh=YXsw9Xicl8EATJ4ZrHMH0cJ6u503P6Q2iTyqNUPD3Zs=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=l3tzD5sOteAs4kiayYXQOBmdfvquLVmYX3BrNt39MebDUKW2m+4FUwalAieNOKIYrHvUojC/G9Gc6bsvivw98RDucKQ5BZMjKNFI+YgV3OwYRwt2oyTAZ6K9E/SB/dMv28RffA6cqV+sCdrL4lWuA3l0cwrQXaTXAM5qfqmxtic= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net; spf=pass smtp.mailfrom=weissschuh.net; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b=AsTYe+fa; arc=none smtp.client-ip=159.69.126.157 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b="AsTYe+fa" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=weissschuh.net; s=mail; t=1716964039; bh=YXsw9Xicl8EATJ4ZrHMH0cJ6u503P6Q2iTyqNUPD3Zs=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=AsTYe+fafq71rhigQ0nCUQ5bRYUTvFzsk8bNulgzOOQbudjBfXTVbj02dWbJ28MJS SqLL0L8Xr8/NTGvkpp4uKzGle7NovX3w7wNE2/zcIYuDR1Bpi8eR1ZchS8R+PeKhYn oUEJ+4iLrz3Y35cnSR4CFqxD4RYSC0ZvsFBVnlMM= From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= Date: Wed, 29 May 2024 08:27:11 +0200 Subject: [PATCH v4 1/3] platform/chrome: cros_ec_proto: Introduce cros_ec_cmd_readmem() Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240529-cros_ec-hwmon-v4-1-5cdf0c5db50a@weissschuh.net> References: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> In-Reply-To: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> To: Jean Delvare , Guenter Roeck , Benson Leung , Lee Jones , Tzung-Bi Shih Cc: Guenter Roeck , linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, chrome-platform@lists.linux.dev, Dustin Howett , Mario Limonciello , Moritz Fischer , Stephen Horvath , Rajas Paranjpe , =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1716964039; l=2309; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=YXsw9Xicl8EATJ4ZrHMH0cJ6u503P6Q2iTyqNUPD3Zs=; b=+sofws1B8KLLelgmW9aJk2d8gV94HqdSiWSn61jhVzcBoJ33D9eRxoGAG28lYHD8DW2KjbC4y q+ufw94H2odAqWfRlndYmxj8b3zx3INTwPR7j+ApuNKY/gyXvJNKuin X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= To read from the EC memory different mechanism are possible. ECs connected via LPC expose their memory via a ->cmd_readmem operation. Other protocols require the usage of EC_CMD_READ_MEMMAP, which on the other hand is not implemented by LPC ECs. Provide a helper that automatically selects the correct mechanism. Signed-off-by: Thomas Wei=C3=9Fschuh Reviewed-by: Guenter Roeck --- drivers/platform/chrome/cros_ec_proto.c | 27 +++++++++++++++++++++++++= ++ include/linux/platform_data/cros_ec_proto.h | 2 ++ 2 files changed, 29 insertions(+) diff --git a/drivers/platform/chrome/cros_ec_proto.c b/drivers/platform/chr= ome/cros_ec_proto.c index 945b1b15a04c..daddc8731cef 100644 --- a/drivers/platform/chrome/cros_ec_proto.c +++ b/drivers/platform/chrome/cros_ec_proto.c @@ -1035,3 +1035,30 @@ int cros_ec_cmd(struct cros_ec_device *ec_dev, return ret; } EXPORT_SYMBOL_GPL(cros_ec_cmd); + +/** + * cros_ec_cmd_readmem - Read from EC memory. + * + * @ec_dev: EC device + * @offset: Is within EC_LPC_ADDR_MEMMAP region. + * @size: Number of bytes to read. + * @dest: EC command output data + * + * Return: >=3D 0 on success, negative error number on failure. + */ +int cros_ec_cmd_readmem(struct cros_ec_device *ec_dev, u8 offset, u8 size,= void *dest) +{ + struct ec_params_read_memmap params =3D {}; + + if (!size) + return -EINVAL; + + if (ec_dev->cmd_readmem) + return ec_dev->cmd_readmem(ec_dev, offset, size, dest); + + params.offset =3D offset; + params.size =3D size; + return cros_ec_cmd(ec_dev, 0, EC_CMD_READ_MEMMAP, + ¶ms, sizeof(params), dest, size); +} +EXPORT_SYMBOL_GPL(cros_ec_cmd_readmem); diff --git a/include/linux/platform_data/cros_ec_proto.h b/include/linux/pl= atform_data/cros_ec_proto.h index 8865e350c12a..1ddc52603f9a 100644 --- a/include/linux/platform_data/cros_ec_proto.h +++ b/include/linux/platform_data/cros_ec_proto.h @@ -261,6 +261,8 @@ int cros_ec_get_sensor_count(struct cros_ec_dev *ec); int cros_ec_cmd(struct cros_ec_device *ec_dev, unsigned int version, int c= ommand, const void *outdata, size_t outsize, void *indata, size_t insize); =20 +int cros_ec_cmd_readmem(struct cros_ec_device *ec_dev, u8 offset, u8 size,= void *dest); + /** * cros_ec_get_time_ns() - Return time in ns. * --=20 2.45.1 From nobody Thu Dec 18 04:46:26 2025 Received: from todd.t-8ch.de (todd.t-8ch.de [159.69.126.157]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7AEE315F417; Wed, 29 May 2024 06:27:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=159.69.126.157 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716964044; cv=none; b=dGskMOGF/TK3ywZpXjI2Qw8pVC6Itpur65SbEWWa42ZSFS42k7N2BYzAcytpiyMlWE8lK+ynYb2/lnmJcIyKpSNo9RxxoQpI77lXDK2PpfqzLgcTprMc6Sk/MLW2nJhrHDDBbRS9ezW0lpW8cGQewRplq/AvcswSzo0xMwh2I6k= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716964044; c=relaxed/simple; bh=35Nmu3ktrJpC4c9LU6YISZcM/t9jt3cr7DDfkBZyoV8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=HuZrBckSJ+EVfGzJO9gnlgWZpAjUysQ0d16pmiMR13tGWDI57HZ+dpZD58sr6XYys7mSzbVfuwr7LeNF1ZEDbBmPZNvwR/kY69bdb9GBMn+7t4lSo+pNXe6oRqHM5aO4n1K9Dwz4fiPw5s9YToZfqTf+X/sHK+5T2Jxg8OTPPQE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net; spf=pass smtp.mailfrom=weissschuh.net; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b=PlLloM0F; arc=none smtp.client-ip=159.69.126.157 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b="PlLloM0F" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=weissschuh.net; s=mail; t=1716964039; bh=35Nmu3ktrJpC4c9LU6YISZcM/t9jt3cr7DDfkBZyoV8=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=PlLloM0FJpfFUq+IB3etWiQmES6jB7s5x8D2SpS8tV9W5L4IKVPlkSKEB/o1mznjf gz0LDaiqaAer3DRxuWBoxHmOJFQEY8TuSVyClvUbGRWCKj3RzPn80uh3OzaaCNWUgi WGqLxIjXV64w4CEAW2pJYsCyaGkKbR8d6fGkKyK4= From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= Date: Wed, 29 May 2024 08:27:12 +0200 Subject: [PATCH v4 2/3] hwmon: add ChromeOS EC driver Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240529-cros_ec-hwmon-v4-2-5cdf0c5db50a@weissschuh.net> References: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> In-Reply-To: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> To: Jean Delvare , Guenter Roeck , Benson Leung , Lee Jones , Tzung-Bi Shih Cc: Guenter Roeck , linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, chrome-platform@lists.linux.dev, Dustin Howett , Mario Limonciello , Moritz Fischer , Stephen Horvath , Rajas Paranjpe , =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1716964039; l=12617; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=35Nmu3ktrJpC4c9LU6YISZcM/t9jt3cr7DDfkBZyoV8=; b=CCVEBpRt0QiZkPuJRIM7W120A2bwTm7egruTp1pbzI0XCmz1/cSle93bA1FhBQx+VxHdM9VDN Bse4BuCjTJWC+C8iwBXiLZc2jcL5/534NrFIKq1icx1UCt2noK/7xvM X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= The ChromeOS Embedded Controller exposes fan speed and temperature readings. Expose this data through the hwmon subsystem. The driver is designed to be probed via the cros_ec mfd device. Signed-off-by: Thomas Wei=C3=9Fschuh Acked-by: Guenter Roeck --- Documentation/hwmon/cros_ec_hwmon.rst | 26 ++++ Documentation/hwmon/index.rst | 1 + MAINTAINERS | 8 + drivers/hwmon/Kconfig | 11 ++ drivers/hwmon/Makefile | 1 + drivers/hwmon/cros_ec_hwmon.c | 286 ++++++++++++++++++++++++++++++= ++++ 6 files changed, 333 insertions(+) diff --git a/Documentation/hwmon/cros_ec_hwmon.rst b/Documentation/hwmon/cr= os_ec_hwmon.rst new file mode 100644 index 000000000000..47ecae983bdb --- /dev/null +++ b/Documentation/hwmon/cros_ec_hwmon.rst @@ -0,0 +1,26 @@ +.. SPDX-License-Identifier: GPL-2.0-or-later + +Kernel driver cros_ec_hwmon +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D + +Supported chips: + + * ChromeOS embedded controllers. + + Prefix: 'cros_ec' + + Addresses scanned: - + +Author: + + - Thomas Wei=C3=9Fschuh + +Description +----------- + +This driver implements support for hardware monitoring commands exposed by= the +ChromeOS embedded controller used in Chromebooks and other devices. + +The channel labels exposed via hwmon are retrieved from the EC itself. + +Fan and temperature readings are supported. diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst index 03d313af469a..342ea5deba24 100644 --- a/Documentation/hwmon/index.rst +++ b/Documentation/hwmon/index.rst @@ -58,6 +58,7 @@ Hardware Monitoring Kernel Drivers coretemp corsair-cpro corsair-psu + cros_ec_hwmon da9052 da9055 dell-smm-hwmon diff --git a/MAINTAINERS b/MAINTAINERS index d6c90161c7bf..0c85aa5073e7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5135,6 +5135,14 @@ S: Maintained F: Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml F: sound/soc/codecs/cros_ec_codec.* =20 +CHROMEOS EC HARDWARE MONITORING +M: Thomas Wei=C3=9Fschuh +L: chrome-platform@lists.linux.dev +L: linux-hwmon@vger.kernel.org +S: Maintained +F: Documentation/hwmon/chros_ec_hwmon.rst +F: drivers/hwmon/cros_ec_hwmon.c + CHROMEOS EC SUBDRIVERS M: Benson Leung R: Guenter Roeck diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig index e14ae18a973b..702dc45ea405 100644 --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -506,6 +506,17 @@ config SENSORS_CORSAIR_PSU This driver can also be built as a module. If so, the module will be called corsair-psu. =20 +config SENSORS_CROS_EC + tristate "ChromeOS Embedded Controller sensors" + depends on MFD_CROS_EC_DEV + default MFD_CROS_EC_DEV + help + If you say yes here you get support for ChromeOS Embedded Controller + sensors. + + This driver can also be built as a module. If so, the module + will be called cros_ec_hwmon. + config SENSORS_DRIVETEMP tristate "Hard disk drives with temperature sensors" depends on SCSI && ATA diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile index e3f25475d1f0..4fb14dd1eafd 100644 --- a/drivers/hwmon/Makefile +++ b/drivers/hwmon/Makefile @@ -64,6 +64,7 @@ obj-$(CONFIG_SENSORS_CHIPCAP2) +=3D chipcap2.o obj-$(CONFIG_SENSORS_CORETEMP) +=3D coretemp.o obj-$(CONFIG_SENSORS_CORSAIR_CPRO) +=3D corsair-cpro.o obj-$(CONFIG_SENSORS_CORSAIR_PSU) +=3D corsair-psu.o +obj-$(CONFIG_SENSORS_CROS_EC) +=3D cros_ec_hwmon.o obj-$(CONFIG_SENSORS_DA9052_ADC)+=3D da9052-hwmon.o obj-$(CONFIG_SENSORS_DA9055)+=3D da9055-hwmon.o obj-$(CONFIG_SENSORS_DELL_SMM) +=3D dell-smm-hwmon.o diff --git a/drivers/hwmon/cros_ec_hwmon.c b/drivers/hwmon/cros_ec_hwmon.c new file mode 100644 index 000000000000..f503f9343364 --- /dev/null +++ b/drivers/hwmon/cros_ec_hwmon.c @@ -0,0 +1,286 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * ChromeOS EC driver for hwmon + * + * Copyright (C) 2024 Thomas Wei=C3=9Fschuh + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#define DRV_NAME "cros-ec-hwmon" + +struct cros_ec_hwmon_priv { + struct cros_ec_device *cros_ec; + const char *temp_sensor_names[EC_TEMP_SENSOR_ENTRIES + EC_TEMP_SENSOR_B_E= NTRIES]; + u8 usable_fans; +}; + +static int cros_ec_hwmon_read_fan_speed(struct cros_ec_device *cros_ec, u8= index, u16 *speed) +{ + int ret; + + ret =3D cros_ec_cmd_readmem(cros_ec, EC_MEMMAP_FAN + index * 2, 2, speed); + if (ret < 0) + return ret; + + *speed =3D le16_to_cpu(*speed); + return 0; +} + +static int cros_ec_hwmon_read_temp(struct cros_ec_device *cros_ec, u8 inde= x, u8 *temp) +{ + unsigned int offset; + int ret; + + if (index < EC_TEMP_SENSOR_ENTRIES) + offset =3D EC_MEMMAP_TEMP_SENSOR + index; + else + offset =3D EC_MEMMAP_TEMP_SENSOR_B + index - EC_TEMP_SENSOR_ENTRIES; + + ret =3D cros_ec_cmd_readmem(cros_ec, offset, 1, temp); + if (ret < 0) + return ret; + return 0; +} + +static bool cros_ec_hwmon_is_error_fan(u16 speed) +{ + return speed =3D=3D EC_FAN_SPEED_NOT_PRESENT || speed =3D=3D EC_FAN_SPEED= _STALLED; +} + +static bool cros_ec_hwmon_is_error_temp(u8 temp) +{ + return temp =3D=3D EC_TEMP_SENSOR_NOT_PRESENT || + temp =3D=3D EC_TEMP_SENSOR_ERROR || + temp =3D=3D EC_TEMP_SENSOR_NOT_POWERED || + temp =3D=3D EC_TEMP_SENSOR_NOT_CALIBRATED; +} + +static long cros_ec_hwmon_temp_to_millicelsius(u8 temp) +{ + return kelvin_to_millicelsius((((long)temp) + EC_TEMP_SENSOR_OFFSET)); +} + +static int cros_ec_hwmon_read(struct device *dev, enum hwmon_sensor_types = type, + u32 attr, int channel, long *val) +{ + struct cros_ec_hwmon_priv *priv =3D dev_get_drvdata(dev); + int ret =3D -EOPNOTSUPP; + u16 speed; + u8 temp; + + if (type =3D=3D hwmon_fan) { + if (attr =3D=3D hwmon_fan_input) { + ret =3D cros_ec_hwmon_read_fan_speed(priv->cros_ec, channel, &speed); + if (ret =3D=3D 0) { + if (cros_ec_hwmon_is_error_fan(speed)) + ret =3D -ENODATA; + else + *val =3D speed; + } + + } else if (attr =3D=3D hwmon_fan_fault) { + ret =3D cros_ec_hwmon_read_fan_speed(priv->cros_ec, channel, &speed); + if (ret =3D=3D 0) + *val =3D cros_ec_hwmon_is_error_fan(speed); + } + + } else if (type =3D=3D hwmon_temp) { + if (attr =3D=3D hwmon_temp_input) { + ret =3D cros_ec_hwmon_read_temp(priv->cros_ec, channel, &temp); + if (ret =3D=3D 0) { + if (cros_ec_hwmon_is_error_temp(temp)) + ret =3D -ENODATA; + else + *val =3D cros_ec_hwmon_temp_to_millicelsius(temp); + } + + } else if (attr =3D=3D hwmon_temp_fault) { + ret =3D cros_ec_hwmon_read_temp(priv->cros_ec, channel, &temp); + if (ret =3D=3D 0) + *val =3D cros_ec_hwmon_is_error_temp(temp); + } + } + + return ret; +} + +static int cros_ec_hwmon_read_string(struct device *dev, enum hwmon_sensor= _types type, + u32 attr, int channel, const char **str) +{ + struct cros_ec_hwmon_priv *priv =3D dev_get_drvdata(dev); + + if (type =3D=3D hwmon_temp && attr =3D=3D hwmon_temp_label) { + *str =3D priv->temp_sensor_names[channel]; + return 0; + } + + return -EOPNOTSUPP; +} + +static umode_t cros_ec_hwmon_is_visible(const void *data, enum hwmon_senso= r_types type, + u32 attr, int channel) +{ + const struct cros_ec_hwmon_priv *priv =3D data; + + if (type =3D=3D hwmon_fan) { + if (priv->usable_fans & BIT(channel)) + return 0444; + + } else if (type =3D=3D hwmon_temp) { + if (priv->temp_sensor_names[channel]) + return 0444; + } + + return 0; +} + +static const struct hwmon_channel_info * const cros_ec_hwmon_info[] =3D { + HWMON_CHANNEL_INFO(fan, + HWMON_F_INPUT | HWMON_F_FAULT, + HWMON_F_INPUT | HWMON_F_FAULT, + HWMON_F_INPUT | HWMON_F_FAULT, + HWMON_F_INPUT | HWMON_F_FAULT), + HWMON_CHANNEL_INFO(temp, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL, + HWMON_T_INPUT | HWMON_T_FAULT | HWMON_T_LABEL), + NULL +}; + +static const struct hwmon_ops cros_ec_hwmon_ops =3D { + .read =3D cros_ec_hwmon_read, + .read_string =3D cros_ec_hwmon_read_string, + .is_visible =3D cros_ec_hwmon_is_visible, +}; + +static const struct hwmon_chip_info cros_ec_hwmon_chip_info =3D { + .ops =3D &cros_ec_hwmon_ops, + .info =3D cros_ec_hwmon_info, +}; + +static void cros_ec_hwmon_probe_temp_sensors(struct device *dev, struct cr= os_ec_hwmon_priv *priv, + u8 thermal_version) +{ + struct ec_params_temp_sensor_get_info req =3D {}; + struct ec_response_temp_sensor_get_info resp; + size_t candidates, i, sensor_name_size; + int ret; + u8 temp; + + if (thermal_version < 2) + candidates =3D EC_TEMP_SENSOR_ENTRIES; + else + candidates =3D ARRAY_SIZE(priv->temp_sensor_names); + + for (i =3D 0; i < candidates; i++) { + if (cros_ec_hwmon_read_temp(priv->cros_ec, i, &temp) < 0) + continue; + + if (temp =3D=3D EC_TEMP_SENSOR_NOT_PRESENT) + continue; + + req.id =3D i; + ret =3D cros_ec_cmd(priv->cros_ec, 0, EC_CMD_TEMP_SENSOR_GET_INFO, + &req, sizeof(req), &resp, sizeof(resp)); + if (ret < 0) + continue; + + sensor_name_size =3D strnlen(resp.sensor_name, sizeof(resp.sensor_name)); + priv->temp_sensor_names[i] =3D devm_kasprintf(dev, GFP_KERNEL, "%*s", + (int)sensor_name_size, + resp.sensor_name); + } +} + +static void cros_ec_hwmon_probe_fans(struct cros_ec_hwmon_priv *priv) +{ + u16 speed; + size_t i; + int ret; + + for (i =3D 0; i < EC_FAN_SPEED_ENTRIES; i++) { + ret =3D cros_ec_hwmon_read_fan_speed(priv->cros_ec, i, &speed); + if (ret =3D=3D 0 && speed !=3D EC_FAN_SPEED_NOT_PRESENT) + priv->usable_fans |=3D BIT(i); + } +} + +static int cros_ec_hwmon_probe(struct platform_device *pdev) +{ + struct device *dev =3D &pdev->dev; + struct cros_ec_dev *ec_dev =3D dev_get_drvdata(dev->parent); + struct cros_ec_device *cros_ec =3D ec_dev->ec_dev; + struct cros_ec_hwmon_priv *priv; + struct device *hwmon_dev; + u8 thermal_version; + int ret; + + ret =3D cros_ec_cmd_readmem(cros_ec, EC_MEMMAP_THERMAL_VERSION, 1, &therm= al_version); + if (ret < 0) + return ret; + + /* Covers both fan and temp sensors */ + if (thermal_version =3D=3D 0) + return -ENODEV; + + priv =3D devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + priv->cros_ec =3D cros_ec; + + cros_ec_hwmon_probe_temp_sensors(dev, priv, thermal_version); + cros_ec_hwmon_probe_fans(priv); + + hwmon_dev =3D devm_hwmon_device_register_with_info(dev, "cros_ec", priv, + &cros_ec_hwmon_chip_info, NULL); + + return PTR_ERR_OR_ZERO(hwmon_dev); +} + +static const struct platform_device_id cros_ec_hwmon_id[] =3D { + { DRV_NAME, 0 }, + {} +}; + +static struct platform_driver cros_ec_hwmon_driver =3D { + .driver.name =3D DRV_NAME, + .probe =3D cros_ec_hwmon_probe, + .id_table =3D cros_ec_hwmon_id, +}; +module_platform_driver(cros_ec_hwmon_driver); + +MODULE_DEVICE_TABLE(platform, cros_ec_hwmon_id); +MODULE_DESCRIPTION("ChromeOS EC Hardware Monitoring Driver"); +MODULE_AUTHOR("Thomas Wei=C3=9Fschuh Date: Wed, 29 May 2024 08:27:13 +0200 Subject: [PATCH v4 3/3] mfd: cros_ec: Register hardware monitoring subdevice Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240529-cros_ec-hwmon-v4-3-5cdf0c5db50a@weissschuh.net> References: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> In-Reply-To: <20240529-cros_ec-hwmon-v4-0-5cdf0c5db50a@weissschuh.net> To: Jean Delvare , Guenter Roeck , Benson Leung , Lee Jones , Tzung-Bi Shih Cc: Guenter Roeck , linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, chrome-platform@lists.linux.dev, Dustin Howett , Mario Limonciello , Moritz Fischer , Stephen Horvath , Rajas Paranjpe , =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1716964039; l=672; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=dG6+vHBr5BNz0npKbHGTIBr20hPfJ/e4zMa9XaBnm8s=; b=JMv1OZh/Sj3Nld6VzjSeFdwMR4dGvnIOFEc33cEK1nhz0w0y3D3WSrmVMrLQ/w6UIgC1r4U1p eAzwsGqrmWODqgy94p3q+YpW/eHGwwZ89H956xlNia5I2bVtP4Uy6Rw X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= Add ChromeOS EC-based hardware monitoring as EC subdevice. Signed-off-by: Thomas Wei=C3=9Fschuh --- drivers/mfd/cros_ec_dev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/cros_ec_dev.c b/drivers/mfd/cros_ec_dev.c index a52d59cc2b1e..1262d1f8d954 100644 --- a/drivers/mfd/cros_ec_dev.c +++ b/drivers/mfd/cros_ec_dev.c @@ -130,6 +130,7 @@ static const struct cros_feature_to_cells cros_subdevic= es[] =3D { static const struct mfd_cell cros_ec_platform_cells[] =3D { { .name =3D "cros-ec-chardev", }, { .name =3D "cros-ec-debugfs", }, + { .name =3D "cros-ec-hwmon", }, { .name =3D "cros-ec-sysfs", }, }; =20 --=20 2.45.1