[PATCH v3 0/3] platform/arm64: Lenovo Yoga Slim 7x Gen 11 embedded controller

[PATCH v3 0/3] platform/arm64: Lenovo Yoga Slim 7x Gen 11 embedded controller

[Oleg Keri]

The Lenovo Yoga Slim 7x Gen 11 is a Qualcomm Snapdragon X2 Elite (glymur)
laptop. It carries a Compal EF06 embedded controller on I2C at address
0x70, which is the only source on this machine for the board thermistors,
the battery cell voltages, the fan tachometer, the keyboard backlight, and
the mic-mute and airplane-mode keys.

Patch 1 documents the binding, patch 2 adds the driver, patch 3 adds the
node to the board.

Why one driver
--------------

The automated review of v2 asked for the hwmon part to become an
auxiliary driver under drivers/hwmon/. The hwmon channels, the LED and
the input device all sit behind one command mailbox and one interrupt,
so the driver stays in one file, like huawei-gaokun-ec,
lenovo-thinkpad-t14s and qcom-hamoa-ec next to it. It is documented
under Documentation/ABI only.

Notes
-----

Fan speed is reported but not controlled, deliberately - the EC enforces no
thermal floor over a manual setpoint. Patch 2 has the measurement.

Tested on a Lenovo Yoga Slim 7x Gen 11 (DMI 83QR, "Yoga Slim 7 14Q8Y11").

Changes in v3:
- Add the board DT node as patch 3. It was posted on its own [1], and a
  node cannot be merged before its binding (Konrad).
- Add the "Why one driver" section above.
- Patch 2: name the driver after the module, and the i2c id after the
  compatible.
- Patch 2: spell out both SCUK bits the Fn-lock toggle moves instead of
  the 0xa0/0x5f masks.
- Patch 2: return -ENODATA for a thermistor reading of 0, as probe already
  treats 0 as "not fitted".
- Patch 2: ten tries for a cell-voltage sample instead of thirty.
- Patch 2: include what is used directly, devm_mutex_init(), comment the
  mutex, fix the address in MODULE_AUTHOR, say in the commit message that
  results cannot be matched to commands.
- Link to v2: https://lore.kernel.org/all/20260908183403.2107-1-okerixx@gmail.com/

Changes in v2:
- Patch 2: drop #include <linux/mod_devicetable.h>; <linux/i2c.h> already
  provides both of_device_id (via linux/of.h) and i2c_device_id (via
  linux/device-id/i2c.h), and the header is on its way out.
  Reported-by: Uwe Kleine-Koenig <u.kleine-koenig@baylibre.com>
- Patch 2: add "depends on INPUT" to EC_LENOVO_YOGA_SLIM7X_GEN11; the driver
  registers an input device unconditionally, so a config with INPUT=n did
  not link.
- Patch 1: pick up Conor Dooley's Acked-by.
- Link to v1: https://lore.kernel.org/all/20260908091752.5508-1-okerixx@gmail.com/

[1] https://lore.kernel.org/all/20260914082950.9393-1-okerixx@gmail.com/

Oleg Keri (3):
  dt-bindings: embedded-controller: add Lenovo Yoga Slim 7x Gen 11 EC
  platform/arm64: add Lenovo Yoga Slim 7x Gen 11 EC driver
  arm64: dts: qcom: glymur-lenovo-yoga-slim7x: add the embedded controller

 .../sysfs-driver-lenovo-yoga-slim7x-gen11-ec       |  21 +
 .../lenovo,yoga-slim7x-gen11-ec.yaml               |  56 ++
 MAINTAINERS                                        |   8 +
 .../boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts    |  12 +-
 drivers/platform/arm64/Kconfig                     |  23 +
 drivers/platform/arm64/Makefile                    |   1 +
 .../platform/arm64/lenovo-yoga-slim7x-gen11-ec.c   | 653 +++++++++++++++++++++
 7 files changed, 772 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
 create mode 100644 Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml
 create mode 100644 drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c

-- 
2.55.0

[PATCH v3 1/3] dt-bindings: embedded-controller: add Lenovo Yoga Slim 7x Gen 11 EC

[Oleg Keri]

Document the Compal embedded controller found on the Lenovo Yoga Slim 7x
Gen 11, a Qualcomm Snapdragon X2 Elite (glymur) laptop. It sits on I2C at
address 0x70 and is distinct from the Qualcomm reference EC that the
reference designs place at 0x76.

Signed-off-by: Oleg Keri <okerixx@gmail.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---
 .../lenovo,yoga-slim7x-gen11-ec.yaml          | 56 +++++++++++++++++++
 MAINTAINERS                                   |  8 +++
 2 files changed, 64 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml

diff --git a/Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml b/Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml
new file mode 100644
index 000000000000..37337de86b0b
--- /dev/null
+++ b/Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml
@@ -0,0 +1,56 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Lenovo Yoga Slim 7x Gen 11 Embedded Controller
+
+maintainers:
+  - Oleg Keri <okerixx@gmail.com>
+
+description:
+  The Lenovo Yoga Slim 7x Gen 11, a Qualcomm Snapdragon X2 Elite (glymur)
+  laptop, has a Compal embedded controller on I2C. It reports board
+  thermistors, battery cell voltages and CPU fan speed, and drives the
+  keyboard backlight.
+
+  This controller is not the Qualcomm reference EC that the Snapdragon
+  reference designs place at address 0x76; it speaks a different command
+  set and answers at 0x70.
+
+properties:
+  compatible:
+    const: lenovo,yoga-slim7x-gen11-ec
+
+  reg:
+    const: 0x70
+
+  interrupts:
+    maxItems: 1
+    description:
+      The controller pulses this line while it has events queued.
+
+required:
+  - compatible
+  - reg
+  - interrupts
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        embedded-controller@70 {
+            compatible = "lenovo,yoga-slim7x-gen11-ec";
+            reg = <0x70>;
+
+            interrupts-extended = <&tlmm 66 IRQ_TYPE_EDGE_FALLING>;
+        };
+    };
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index be096be5fff6..11d459c17caf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14966,6 +14966,14 @@ S:	Maintained
 F:	Documentation/ABI/testing/sysfs-driver-lenovo-yb9-kbdock
 F:	drivers/platform/x86/lenovo/yb9-kbdock.c
 
+LENOVO YOGA SLIM 7X GEN 11 EMBEDDED CONTROLLER DRIVER
+M:	Oleg Keri <okerixx@gmail.com>
+L:	linux-arm-msm@vger.kernel.org
+S:	Maintained
+F:	Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
+F:	Documentation/devicetree/bindings/embedded-controller/lenovo,yoga-slim7x-gen11-ec.yaml
+F:	drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
+
 LETSKETCH HID TABLET DRIVER
 M:	Hans de Goede <hansg@kernel.org>
 L:	linux-input@vger.kernel.org
-- 
2.55.0

[PATCH v3 2/3] platform/arm64: add Lenovo Yoga Slim 7x Gen 11 EC driver

[Oleg Keri]

The Lenovo Yoga Slim 7x Gen 11 (Qualcomm glymur / Snapdragon X2 Elite)
carries a Compal EF06 embedded controller on I2C at address 0x70. The
platform firmware describes it as an ACPI GenericSerialBus region, but the
machine boots from device tree, so nothing otherwise talks to it.

Commands are exchanged with two raw I2C block transfers: a 7 byte write to
register 0x02 carrying the command and up to six arguments, then a 6 byte
read of register 0x01 for the result, with a short delay in between before
the result register is valid. The EC offers no way to match a result to
its command; the delay is the one the vendor ACPI code uses, and the one
value that lives in a mailbox shared with other EC activity, the cell
voltages, is validated against a signature and retried.

Expose what the EC offers that the rest of the kernel does not already
provide:

  - five board thermistors as hwmon temperature channels. These are
    charger, CPU voltage regulator, 5 V rail and ambient sensors, not die
    sensors, and read considerably cooler than tsens
  - the four battery cell voltages, which live in a banked mailbox shared
    with other EC activity; readings are validated against a signature and
    retried, and cached briefly so that all four channels come from one
    coherent sample
  - CPU fan speed in RPM
  - the keyboard backlight as an LED, off/low/high
  - Fn-lock as a read/write sysfs attribute, matching the ideapad-laptop
    interface. The EC reports Fn+Esc as an event but keeps no state of its
    own, so the driver owns the bit and toggles it on the key
  - the mic-mute and airplane-mode keys as KEY_MICMUTE and KEY_RFKILL.
    These reach the host only as EC events; they are not on the keyboard's
    HID interface and have no GPIO of their own

The EC signals pending events on a dedicated line, and keeps pulsing it
until they are read - so the queue must be drained completely on every
interrupt, including codes this driver ignores. Leaving one behind makes
the controller pulse indefinitely.

Fan speed is reported but not controlled. The EC accepts a manual setpoint,
however it enforces no thermal floor over it: with the fan pinned at
roughly 2000 RPM under full load the SoC rose from 69 to 91 degrees in 30
seconds without the EC ever overriding. Exposing pwm1_enable or fan1_target
would therefore let userspace overheat the machine, so neither is
implemented.

Note this is the Gen 11 machine specifically. Its EC differs from the
Qualcomm reference EC that the reference designs place at address 0x76,
which is not populated here, and so EC_QCOM_HAMOA does not drive it.

Signed-off-by: Oleg Keri <okerixx@gmail.com>
---
 .../sysfs-driver-lenovo-yoga-slim7x-gen11-ec  |  21 +
 drivers/platform/arm64/Kconfig                |  23 +
 drivers/platform/arm64/Makefile               |   1 +
 .../arm64/lenovo-yoga-slim7x-gen11-ec.c       | 653 ++++++++++++++++++
 4 files changed, 698 insertions(+)
 create mode 100644 Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
 create mode 100644 drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c

diff --git a/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec b/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
new file mode 100644
index 000000000000..c8ab554ad452
--- /dev/null
+++ b/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
@@ -0,0 +1,21 @@
+What:		/sys/bus/i2c/devices/*/fn_lock
+Date:		August 2026
+KernelVersion:	7.3
+Contact:	Oleg Keri <okerixx@gmail.com>
+Description:
+		Control fn-lock mode.
+
+			* 1 -> Switched On
+			* 0 -> Switched Off
+
+		When switched on, the function keys act as F1..F12 and the
+		multimedia functions require the Fn modifier.
+
+		The embedded controller reports the Fn+Esc key as an event but
+		keeps no state of its own, so this attribute is the state: the
+		driver toggles it when the key is pressed, and writing it takes
+		effect immediately.
+
+		For example::
+
+		  # echo "0" > /sys/bus/i2c/devices/9-0070/fn_lock
diff --git a/drivers/platform/arm64/Kconfig b/drivers/platform/arm64/Kconfig
index e32e01b2a9bd..7581432e69b0 100644
--- a/drivers/platform/arm64/Kconfig
+++ b/drivers/platform/arm64/Kconfig
@@ -90,6 +90,29 @@ config EC_LENOVO_THINKPAD_T14S
 
 	  Say M or Y here to include this support.
 
+config EC_LENOVO_YOGA_SLIM7X_GEN11
+	tristate "Lenovo Yoga Slim 7x Gen 11 Embedded Controller driver"
+	depends on ARCH_QCOM || COMPILE_TEST
+	depends on I2C
+	depends on INPUT
+	depends on HWMON
+	select NEW_LEDS
+	select LEDS_CLASS
+	help
+	  Say Y here to enable the EC driver for the (Snapdragon X2 Elite,
+	  glymur-based) Lenovo Yoga Slim 7x Gen 11 laptop. The EC provides
+	  thermistors, battery cell voltages and CPU fan speed as hwmon
+	  channels, and the keyboard backlight as an LED.
+
+	  This is the Gen 11 (glymur) machine specifically. Its EC is a Compal
+	  design at i2c address 0x70 and speaks a different protocol from the
+	  Qualcomm reference EC that the reference designs place at 0x76, so
+	  EC_QCOM_HAMOA does not drive it - and conversely this driver does not
+	  cover the X1E-based Yoga Slim 7x Gen 9.
+
+	  To compile this driver as a module, choose M here: the module will
+	  be called lenovo-yoga-slim7x-gen11-ec.
+
 config EC_QCOM_HAMOA
 	tristate "Embedded Controller driver for Qualcomm Hamoa/Glymur reference devices"
 	depends on ARCH_QCOM || COMPILE_TEST
diff --git a/drivers/platform/arm64/Makefile b/drivers/platform/arm64/Makefile
index 7681be4a46e9..605cca4d42a9 100644
--- a/drivers/platform/arm64/Makefile
+++ b/drivers/platform/arm64/Makefile
@@ -9,4 +9,5 @@ obj-$(CONFIG_EC_ACER_ASPIRE1)	+= acer-aspire1-ec.o
 obj-$(CONFIG_EC_HUAWEI_GAOKUN)	+= huawei-gaokun-ec.o
 obj-$(CONFIG_EC_LENOVO_YOGA_C630) += lenovo-yoga-c630.o
 obj-$(CONFIG_EC_LENOVO_THINKPAD_T14S) += lenovo-thinkpad-t14s.o
+obj-$(CONFIG_EC_LENOVO_YOGA_SLIM7X_GEN11) += lenovo-yoga-slim7x-gen11-ec.o
 obj-$(CONFIG_EC_QCOM_HAMOA) += qcom-hamoa-ec.o
diff --git a/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c b/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
new file mode 100644
index 000000000000..c8c7928475b7
--- /dev/null
+++ b/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
@@ -0,0 +1,653 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+#include <linux/bitfield.h>
+#include <linux/bitops.h>
+#include <linux/cleanup.h>
+#include <linux/container_of.h>
+#include <linux/delay.h>
+#include <linux/device.h>
+#include <linux/err.h>
+#include <linux/errno.h>
+#include <linux/gfp_types.h>
+#include <linux/hwmon.h>
+#include <linux/i2c.h>
+#include <linux/input.h>
+#include <linux/interrupt.h>
+#include <linux/jiffies.h>
+#include <linux/kstrtox.h>
+#include <linux/leds.h>
+#include <linux/module.h>
+#include <linux/mutex.h>
+#include <linux/pm.h>
+#include <linux/string.h>
+#include <linux/sysfs.h>
+#include <linux/types.h>
+
+#define YOGA_EC_BLOCK_READ	0x01
+#define YOGA_EC_BLOCK_WRITE	0x02
+#define YOGA_EC_RESP_LEN	6
+#define YOGA_EC_REQ_LEN		7
+
+#define YOGA_EC_CMD_RAM_READ	0xb0
+#define YOGA_EC_CMD_RAM_WRITE	0xb1
+#define YOGA_EC_CMD_BANK_READ	0xb4
+#define YOGA_EC_CMD_FAN		0x46
+#define YOGA_EC_FAN_GET_SPEED	0x81
+
+#define YOGA_EC_CMD_EVENT	0x84
+#define YOGA_EC_EVT_MAX_DRAIN	64
+#define YOGA_EC_EVT_FN_LOCK	0x0f
+#define YOGA_EC_EVT_MIC_MUTE	0x18
+#define YOGA_EC_EVT_FLIGHT_MODE	0x19
+
+#define YOGA_EC_SCUK		0x1c
+#define YOGA_EC_SCUK_FN_LOCK	BIT(7)
+#define YOGA_EC_SCUK_FN_LOCK_IND	BIT(5)
+#define YOGA_EC_SCUK_FN_LOCK_MASK	(YOGA_EC_SCUK_FN_LOCK | YOGA_EC_SCUK_FN_LOCK_IND)
+
+#define YOGA_EC_KBLT		0x03
+#define YOGA_EC_KBLT_LEVEL	GENMASK(1, 0)
+#define YOGA_EC_KBLT_AUTO	3
+#define YOGA_EC_KBLT_MAX	2
+
+#define YOGA_EC_TEMP_FIRST	0x12
+#define YOGA_EC_NUM_TEMP	6
+#define YOGA_EC_TEMP_MAX	120
+
+#define YOGA_EC_CELL_BANK	0x0b
+#define YOGA_EC_CELL_SIG_LO	0x02
+#define YOGA_EC_CELL_SIG_HI	0x0c
+#define YOGA_EC_CELL_FIRST	0x04
+#define YOGA_EC_NUM_CELL	4
+#define YOGA_EC_CELL_MIN_MV	2000
+#define YOGA_EC_CELL_MAX_MV	5000
+#define YOGA_EC_CELL_TRIES	10
+
+#define YOGA_EC_CACHE		(2 * HZ)
+#define YOGA_EC_STALE		(10 * HZ)
+
+struct yoga_ec {
+	struct i2c_client *client;
+	/* serializes all EC access */
+	struct mutex lock;
+	unsigned long temp_present;
+	unsigned long cell_read_at;
+	u16 cell[YOGA_EC_NUM_CELL];
+	bool cell_cached;
+	struct led_classdev kbd_led;
+	u8 kblt_level;
+	bool kblt_restore;
+	struct input_dev *keys;
+};
+
+static int yoga_ec_command(struct yoga_ec *ec, u8 cmd, u8 arg0, u8 arg1,
+			   u8 resp[YOGA_EC_RESP_LEN])
+{
+	u8 req[YOGA_EC_REQ_LEN] = { cmd, arg0, arg1 };
+	int ret;
+
+	ret = i2c_smbus_write_i2c_block_data(ec->client, YOGA_EC_BLOCK_WRITE,
+					     sizeof(req), req);
+	if (ret < 0)
+		return ret;
+
+	usleep_range(2000, 3000);
+
+	ret = i2c_smbus_read_i2c_block_data(ec->client, YOGA_EC_BLOCK_READ,
+					    YOGA_EC_RESP_LEN, resp);
+	if (ret < 0)
+		return ret;
+	if (ret != YOGA_EC_RESP_LEN)
+		return -EIO;
+
+	return 0;
+}
+
+static int yoga_ec_ram_read(struct yoga_ec *ec, u8 addr, u8 *val)
+{
+	u8 resp[YOGA_EC_RESP_LEN];
+	int ret;
+
+	ret = yoga_ec_command(ec, YOGA_EC_CMD_RAM_READ, addr, 0, resp);
+	if (ret)
+		return ret;
+
+	*val = resp[0];
+	return 0;
+}
+
+static int yoga_ec_bank_read(struct yoga_ec *ec, u8 bank, u8 addr, u8 *val)
+{
+	u8 resp[YOGA_EC_RESP_LEN];
+	int ret;
+
+	ret = yoga_ec_command(ec, YOGA_EC_CMD_BANK_READ, bank, addr, resp);
+	if (ret)
+		return ret;
+
+	*val = resp[0];
+	return 0;
+}
+
+static int yoga_ec_ram_write(struct yoga_ec *ec, u8 addr, u8 val)
+{
+	u8 resp[YOGA_EC_RESP_LEN];
+
+	return yoga_ec_command(ec, YOGA_EC_CMD_RAM_WRITE, addr, val, resp);
+}
+
+static int yoga_ec_fan_read(struct yoga_ec *ec, long *rpm)
+{
+	u8 resp[YOGA_EC_RESP_LEN];
+	int ret;
+
+	ret = yoga_ec_command(ec, YOGA_EC_CMD_FAN, YOGA_EC_FAN_GET_SPEED, 0,
+			      resp);
+	if (ret)
+		return ret;
+
+	*rpm = resp[0] * 100;
+	return 0;
+}
+
+static int yoga_ec_refresh_cells(struct yoga_ec *ec)
+{
+	unsigned int try;
+	int ret;
+
+	if (ec->cell_cached &&
+	    time_before(jiffies, ec->cell_read_at + YOGA_EC_CACHE))
+		return 0;
+
+	for (try = 0; try < YOGA_EC_CELL_TRIES; try++) {
+		u16 cell[YOGA_EC_NUM_CELL];
+		u8 sig[3];
+		unsigned int i;
+		bool ok = true;
+
+		ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
+					YOGA_EC_CELL_SIG_LO, &sig[0]);
+		if (!ret)
+			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
+						YOGA_EC_CELL_SIG_LO + 1, &sig[1]);
+		if (!ret)
+			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
+						YOGA_EC_CELL_SIG_HI, &sig[2]);
+		if (ret)
+			return ret;
+
+		if (sig[0] != 0x01 || sig[1] != 0x01 || sig[2] != 0x03)
+			continue;
+
+		for (i = 0; i < YOGA_EC_NUM_CELL; i++) {
+			u8 lsb, msb;
+
+			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
+						YOGA_EC_CELL_FIRST + i * 2, &lsb);
+			if (!ret)
+				ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
+							YOGA_EC_CELL_FIRST + i * 2 + 1,
+							&msb);
+			if (ret)
+				return ret;
+
+			cell[i] = lsb | (msb << 8);
+			if (cell[i] < YOGA_EC_CELL_MIN_MV ||
+			    cell[i] > YOGA_EC_CELL_MAX_MV)
+				ok = false;
+		}
+
+		if (!ok)
+			continue;
+
+		memcpy(ec->cell, cell, sizeof(cell));
+		ec->cell_cached = true;
+		ec->cell_read_at = jiffies;
+
+		return 0;
+	}
+
+	if (ec->cell_cached &&
+	    time_before(jiffies, ec->cell_read_at + YOGA_EC_STALE))
+		return 0;
+
+	return -EAGAIN;
+}
+
+static umode_t yoga_ec_is_visible(const void *drvdata,
+				  enum hwmon_sensor_types type,
+				  u32 attr, int channel)
+{
+	const struct yoga_ec *ec = drvdata;
+
+	switch (type) {
+	case hwmon_temp:
+		if (!(ec->temp_present & BIT(channel)))
+			return 0;
+		return 0444;
+	case hwmon_in:
+		return 0444;
+	case hwmon_fan:
+		return 0444;
+	default:
+		return 0;
+	}
+}
+
+static int yoga_ec_read(struct device *dev, enum hwmon_sensor_types type,
+			u32 attr, int channel, long *val)
+{
+	struct yoga_ec *ec = dev_get_drvdata(dev);
+	int ret;
+	u8 raw;
+
+	guard(mutex)(&ec->lock);
+
+	switch (type) {
+	case hwmon_temp:
+		ret = yoga_ec_ram_read(ec, YOGA_EC_TEMP_FIRST + channel, &raw);
+		if (ret)
+			return ret;
+		if (!raw)
+			return -ENODATA;
+		if (raw > YOGA_EC_TEMP_MAX)
+			return -EIO;
+		*val = raw * 1000;
+		return 0;
+	case hwmon_in:
+		ret = yoga_ec_refresh_cells(ec);
+		if (ret)
+			return ret;
+		*val = ec->cell[channel];
+		return 0;
+	case hwmon_fan:
+		return yoga_ec_fan_read(ec, val);
+	default:
+		return -EOPNOTSUPP;
+	}
+}
+
+static const char * const yoga_ec_temp_labels[YOGA_EC_NUM_TEMP] = {
+	"Charger A",
+	"CPU VR",
+	"5V rail",
+	"Ambient",
+	"Charger B",
+	"Thermistor 6",
+};
+
+static const char * const yoga_ec_cell_labels[YOGA_EC_NUM_CELL] = {
+	"Cell 1", "Cell 2", "Cell 3", "Cell 4",
+};
+
+static int yoga_ec_read_string(struct device *dev, enum hwmon_sensor_types type,
+			       u32 attr, int channel, const char **str)
+{
+	switch (type) {
+	case hwmon_temp:
+		*str = yoga_ec_temp_labels[channel];
+		return 0;
+	case hwmon_in:
+		*str = yoga_ec_cell_labels[channel];
+		return 0;
+	case hwmon_fan:
+		*str = "CPU fan";
+		return 0;
+	default:
+		return -EOPNOTSUPP;
+	}
+}
+
+static enum led_brightness yoga_ec_kbd_led_get(struct led_classdev *led)
+{
+	struct yoga_ec *ec = container_of(led, struct yoga_ec, kbd_led);
+	u8 raw;
+
+	guard(mutex)(&ec->lock);
+
+	if (yoga_ec_ram_read(ec, YOGA_EC_KBLT, &raw))
+		return 0;
+
+	raw = FIELD_GET(YOGA_EC_KBLT_LEVEL, raw);
+
+	if (raw == YOGA_EC_KBLT_AUTO)
+		return YOGA_EC_KBLT_MAX;
+
+	return raw;
+}
+
+static int yoga_ec_kbd_led_level(struct yoga_ec *ec, u8 level, u8 *old)
+{
+	u8 raw;
+	int ret;
+
+	ret = yoga_ec_ram_read(ec, YOGA_EC_KBLT, &raw);
+	if (ret)
+		return ret;
+
+	if (old)
+		*old = FIELD_GET(YOGA_EC_KBLT_LEVEL, raw);
+
+	raw &= ~YOGA_EC_KBLT_LEVEL;
+	raw |= FIELD_PREP(YOGA_EC_KBLT_LEVEL, level);
+
+	return yoga_ec_ram_write(ec, YOGA_EC_KBLT, raw);
+}
+
+static int yoga_ec_kbd_led_set(struct led_classdev *led,
+			       enum led_brightness brightness)
+{
+	struct yoga_ec *ec = container_of(led, struct yoga_ec, kbd_led);
+
+	guard(mutex)(&ec->lock);
+
+	return yoga_ec_kbd_led_level(ec, brightness, NULL);
+}
+
+static const struct hwmon_ops yoga_ec_hwmon_ops = {
+	.is_visible = yoga_ec_is_visible,
+	.read = yoga_ec_read,
+	.read_string = yoga_ec_read_string,
+};
+
+static const struct hwmon_channel_info * const yoga_ec_hwmon_info[] = {
+	HWMON_CHANNEL_INFO(in,
+			   HWMON_I_INPUT | HWMON_I_LABEL,
+			   HWMON_I_INPUT | HWMON_I_LABEL,
+			   HWMON_I_INPUT | HWMON_I_LABEL,
+			   HWMON_I_INPUT | HWMON_I_LABEL),
+	HWMON_CHANNEL_INFO(temp,
+			   HWMON_T_INPUT | HWMON_T_LABEL,
+			   HWMON_T_INPUT | HWMON_T_LABEL,
+			   HWMON_T_INPUT | HWMON_T_LABEL,
+			   HWMON_T_INPUT | HWMON_T_LABEL,
+			   HWMON_T_INPUT | HWMON_T_LABEL,
+			   HWMON_T_INPUT | HWMON_T_LABEL),
+	HWMON_CHANNEL_INFO(fan,
+			   HWMON_F_INPUT | HWMON_F_LABEL),
+	NULL
+};
+
+static const struct hwmon_chip_info yoga_ec_chip_info = {
+	.ops = &yoga_ec_hwmon_ops,
+	.info = yoga_ec_hwmon_info,
+};
+
+static int yoga_ec_fn_lock_toggle(struct yoga_ec *ec)
+{
+	u8 raw;
+	int ret;
+
+	ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
+	if (ret)
+		return ret;
+
+	if (raw & YOGA_EC_SCUK_FN_LOCK)
+		raw &= ~YOGA_EC_SCUK_FN_LOCK_MASK;
+	else
+		raw |= YOGA_EC_SCUK_FN_LOCK_MASK;
+
+	return yoga_ec_ram_write(ec, YOGA_EC_SCUK, raw);
+}
+
+static void yoga_ec_report_key(struct yoga_ec *ec, unsigned int code)
+{
+	if (!ec->keys)
+		return;
+
+	input_report_key(ec->keys, code, 1);
+	input_sync(ec->keys);
+	input_report_key(ec->keys, code, 0);
+	input_sync(ec->keys);
+}
+
+static irqreturn_t yoga_ec_irq(int irq, void *data)
+{
+	struct yoga_ec *ec = data;
+	unsigned int i;
+
+	guard(mutex)(&ec->lock);
+
+	for (i = 0; i < YOGA_EC_EVT_MAX_DRAIN; i++) {
+		u8 resp[YOGA_EC_RESP_LEN];
+
+		if (yoga_ec_command(ec, YOGA_EC_CMD_EVENT, 0, 0, resp))
+			break;
+		if (!resp[0])
+			break;
+
+		switch (resp[0]) {
+		case YOGA_EC_EVT_FN_LOCK:
+			yoga_ec_fn_lock_toggle(ec);
+			break;
+		case YOGA_EC_EVT_MIC_MUTE:
+			yoga_ec_report_key(ec, KEY_MICMUTE);
+			break;
+		case YOGA_EC_EVT_FLIGHT_MODE:
+			yoga_ec_report_key(ec, KEY_RFKILL);
+			break;
+		default:
+			break;
+		}
+	}
+
+	return IRQ_HANDLED;
+}
+
+static void yoga_ec_register_keys(struct yoga_ec *ec)
+{
+	struct device *dev = &ec->client->dev;
+	struct input_dev *input;
+	int ret;
+
+	input = devm_input_allocate_device(dev);
+	if (!input) {
+		dev_warn(dev, "no memory for the hotkey device\n");
+		return;
+	}
+
+	input->name = "Lenovo Yoga Slim 7x hotkeys";
+	input->phys = "lenovo-yoga-slim7x-gen11-ec/input0";
+	input->id.bustype = BUS_I2C;
+	input_set_capability(input, EV_KEY, KEY_MICMUTE);
+	input_set_capability(input, EV_KEY, KEY_RFKILL);
+
+	ret = input_register_device(input);
+	if (ret) {
+		dev_warn(dev, "cannot register the hotkey device: %d\n", ret);
+		return;
+	}
+
+	ec->keys = input;
+}
+
+static ssize_t fn_lock_show(struct device *dev, struct device_attribute *attr,
+			    char *buf)
+{
+	struct yoga_ec *ec = dev_get_drvdata(dev);
+	u8 raw;
+	int ret;
+
+	scoped_guard(mutex, &ec->lock)
+		ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
+
+	if (ret)
+		return ret;
+
+	return sysfs_emit(buf, "%u\n", !!(raw & YOGA_EC_SCUK_FN_LOCK));
+}
+
+static ssize_t fn_lock_store(struct device *dev, struct device_attribute *attr,
+			     const char *buf, size_t count)
+{
+	struct yoga_ec *ec = dev_get_drvdata(dev);
+	bool enable;
+	u8 raw;
+	int ret;
+
+	ret = kstrtobool(buf, &enable);
+	if (ret)
+		return ret;
+
+	guard(mutex)(&ec->lock);
+
+	ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
+	if (ret)
+		return ret;
+
+	if (enable)
+		raw |= YOGA_EC_SCUK_FN_LOCK_MASK;
+	else
+		raw &= ~YOGA_EC_SCUK_FN_LOCK_MASK;
+
+	ret = yoga_ec_ram_write(ec, YOGA_EC_SCUK, raw);
+	if (ret)
+		return ret;
+
+	return count;
+}
+static DEVICE_ATTR_RW(fn_lock);
+
+static struct attribute *yoga_ec_attrs[] = {
+	&dev_attr_fn_lock.attr,
+	NULL
+};
+ATTRIBUTE_GROUPS(yoga_ec);
+
+static int yoga_ec_probe(struct i2c_client *client)
+{
+	struct device *dev = &client->dev;
+	struct device *hwmon;
+	struct yoga_ec *ec;
+	unsigned int i;
+	int ret;
+
+	if (!i2c_check_functionality(client->adapter,
+				     I2C_FUNC_SMBUS_I2C_BLOCK))
+		return dev_err_probe(dev, -ENODEV,
+				     "adapter does not support SMBus block transfers\n");
+
+	ec = devm_kzalloc(dev, sizeof(*ec), GFP_KERNEL);
+	if (!ec)
+		return -ENOMEM;
+
+	ec->client = client;
+	i2c_set_clientdata(client, ec);
+
+	ret = devm_mutex_init(dev, &ec->lock);
+	if (ret)
+		return ret;
+
+	for (i = 0; i < YOGA_EC_NUM_TEMP; i++) {
+		u8 raw;
+
+		ret = yoga_ec_ram_read(ec, YOGA_EC_TEMP_FIRST + i, &raw);
+		if (ret)
+			return dev_err_probe(dev, ret,
+					     "failed to read thermistor %u\n", i);
+
+		if (raw > 0 && raw <= YOGA_EC_TEMP_MAX)
+			ec->temp_present |= BIT(i);
+	}
+
+	if (!ec->temp_present)
+		return dev_err_probe(dev, -ENODEV,
+				     "no thermistor reported a usable value\n");
+
+	dev_dbg(dev, "thermistors present: %#lx\n", ec->temp_present);
+
+	hwmon = devm_hwmon_device_register_with_info(dev, "yoga_slim7x_ec", ec,
+						     &yoga_ec_chip_info, NULL);
+	if (IS_ERR(hwmon))
+		return PTR_ERR(hwmon);
+
+	ec->kbd_led.name = "platform::kbd_backlight";
+	ec->kbd_led.max_brightness = YOGA_EC_KBLT_MAX;
+	ec->kbd_led.brightness_get = yoga_ec_kbd_led_get;
+	ec->kbd_led.brightness_set_blocking = yoga_ec_kbd_led_set;
+
+	ret = devm_led_classdev_register(dev, &ec->kbd_led);
+	if (ret)
+		return dev_err_probe(dev, ret,
+				     "failed to register keyboard backlight\n");
+
+	if (client->irq) {
+		yoga_ec_register_keys(ec);
+
+		ret = devm_request_threaded_irq(dev, client->irq, NULL,
+						yoga_ec_irq, IRQF_ONESHOT,
+						"lenovo-yoga-slim7x-gen11-ec", ec);
+		if (ret)
+			return dev_err_probe(dev, ret, "cannot request the EC interrupt\n");
+
+		scoped_guard(mutex, &ec->lock) {
+			unsigned int i;
+
+			for (i = 0; i < YOGA_EC_EVT_MAX_DRAIN; i++) {
+				u8 resp[YOGA_EC_RESP_LEN];
+
+				if (yoga_ec_command(ec, YOGA_EC_CMD_EVENT, 0, 0, resp))
+					break;
+				if (!resp[0])
+					break;
+			}
+		}
+	}
+
+	return 0;
+}
+
+static int yoga_ec_suspend(struct device *dev)
+{
+	struct yoga_ec *ec = dev_get_drvdata(dev);
+
+	guard(mutex)(&ec->lock);
+	ec->kblt_restore = !yoga_ec_kbd_led_level(ec, 0, &ec->kblt_level);
+
+	return 0;
+}
+
+static int yoga_ec_resume(struct device *dev)
+{
+	struct yoga_ec *ec = dev_get_drvdata(dev);
+
+	guard(mutex)(&ec->lock);
+	ec->cell_cached = false;
+
+	if (ec->kblt_restore) {
+		ec->kblt_restore = false;
+		yoga_ec_kbd_led_level(ec, ec->kblt_level, NULL);
+	}
+
+	return 0;
+}
+
+static DEFINE_SIMPLE_DEV_PM_OPS(yoga_ec_pm_ops, yoga_ec_suspend, yoga_ec_resume);
+
+static const struct of_device_id yoga_ec_of_match[] = {
+	{ .compatible = "lenovo,yoga-slim7x-gen11-ec" },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, yoga_ec_of_match);
+
+static const struct i2c_device_id yoga_ec_i2c_id[] = {
+	{ "yoga-slim7x-gen11-ec" },
+	{ }
+};
+MODULE_DEVICE_TABLE(i2c, yoga_ec_i2c_id);
+
+static struct i2c_driver yoga_ec_driver = {
+	.driver = {
+		.name = "lenovo-yoga-slim7x-gen11-ec",
+		.of_match_table = yoga_ec_of_match,
+		.dev_groups = yoga_ec_groups,
+		.pm = pm_sleep_ptr(&yoga_ec_pm_ops),
+	},
+	.probe = yoga_ec_probe,
+	.id_table = yoga_ec_i2c_id,
+};
+module_i2c_driver(yoga_ec_driver);
+
+MODULE_AUTHOR("Oleg Keri <okerixx@gmail.com>");
+MODULE_DESCRIPTION("Lenovo Yoga Slim 7x Gen 11 embedded controller");
+MODULE_LICENSE("GPL");
-- 
2.55.0

Re: [PATCH v3 2/3] platform/arm64: add Lenovo Yoga Slim 7x Gen 11 EC driver

[Ilpo Järvinen]

On Mon, 21 Sep 2026, Oleg Keri wrote:

> The Lenovo Yoga Slim 7x Gen 11 (Qualcomm glymur / Snapdragon X2 Elite)
> carries a Compal EF06 embedded controller on I2C at address 0x70. The
> platform firmware describes it as an ACPI GenericSerialBus region, but the
> machine boots from device tree, so nothing otherwise talks to it.
> 
> Commands are exchanged with two raw I2C block transfers: a 7 byte write to
> register 0x02 carrying the command and up to six arguments, then a 6 byte
> read of register 0x01 for the result, with a short delay in between before
> the result register is valid. The EC offers no way to match a result to
> its command; the delay is the one the vendor ACPI code uses, and the one
> value that lives in a mailbox shared with other EC activity, the cell
> voltages, is validated against a signature and retried.
> 
> Expose what the EC offers that the rest of the kernel does not already
> provide:
> 
>   - five board thermistors as hwmon temperature channels. These are
>     charger, CPU voltage regulator, 5 V rail and ambient sensors, not die
>     sensors, and read considerably cooler than tsens
>   - the four battery cell voltages, which live in a banked mailbox shared
>     with other EC activity; readings are validated against a signature and
>     retried, and cached briefly so that all four channels come from one
>     coherent sample
>   - CPU fan speed in RPM
>   - the keyboard backlight as an LED, off/low/high
>   - Fn-lock as a read/write sysfs attribute, matching the ideapad-laptop
>     interface. The EC reports Fn+Esc as an event but keeps no state of its
>     own, so the driver owns the bit and toggles it on the key
>   - the mic-mute and airplane-mode keys as KEY_MICMUTE and KEY_RFKILL.
>     These reach the host only as EC events; they are not on the keyboard's
>     HID interface and have no GPIO of their own
> 
> The EC signals pending events on a dedicated line, and keeps pulsing it
> until they are read - so the queue must be drained completely on every
> interrupt, including codes this driver ignores. Leaving one behind makes
> the controller pulse indefinitely.
> 
> Fan speed is reported but not controlled. The EC accepts a manual setpoint,
> however it enforces no thermal floor over it: with the fan pinned at
> roughly 2000 RPM under full load the SoC rose from 69 to 91 degrees in 30
> seconds without the EC ever overriding. Exposing pwm1_enable or fan1_target
> would therefore let userspace overheat the machine, so neither is
> implemented.
> 
> Note this is the Gen 11 machine specifically. Its EC differs from the
> Qualcomm reference EC that the reference designs place at address 0x76,
> which is not populated here, and so EC_QCOM_HAMOA does not drive it.
> 
> Signed-off-by: Oleg Keri <okerixx@gmail.com>
> ---
>  .../sysfs-driver-lenovo-yoga-slim7x-gen11-ec  |  21 +
>  drivers/platform/arm64/Kconfig                |  23 +
>  drivers/platform/arm64/Makefile               |   1 +
>  .../arm64/lenovo-yoga-slim7x-gen11-ec.c       | 653 ++++++++++++++++++
>  4 files changed, 698 insertions(+)
>  create mode 100644 Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
>  create mode 100644 drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
> 
> diff --git a/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec b/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
> new file mode 100644
> index 000000000000..c8ab554ad452
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-driver-lenovo-yoga-slim7x-gen11-ec
> @@ -0,0 +1,21 @@
> +What:		/sys/bus/i2c/devices/*/fn_lock
> +Date:		August 2026

??

> +KernelVersion:	7.3

This ship has already sailed.

> +Contact:	Oleg Keri <okerixx@gmail.com>
> +Description:
> +		Control fn-lock mode.
> +
> +			* 1 -> Switched On
> +			* 0 -> Switched Off
> +
> +		When switched on, the function keys act as F1..F12 and the
> +		multimedia functions require the Fn modifier.
> +
> +		The embedded controller reports the Fn+Esc key as an event but
> +		keeps no state of its own, so this attribute is the state: the
> +		driver toggles it when the key is pressed, and writing it takes
> +		effect immediately.
> +
> +		For example::
> +
> +		  # echo "0" > /sys/bus/i2c/devices/9-0070/fn_lock
> diff --git a/drivers/platform/arm64/Kconfig b/drivers/platform/arm64/Kconfig
> index e32e01b2a9bd..7581432e69b0 100644
> --- a/drivers/platform/arm64/Kconfig
> +++ b/drivers/platform/arm64/Kconfig
> @@ -90,6 +90,29 @@ config EC_LENOVO_THINKPAD_T14S
>  
>  	  Say M or Y here to include this support.
>  
> +config EC_LENOVO_YOGA_SLIM7X_GEN11
> +	tristate "Lenovo Yoga Slim 7x Gen 11 Embedded Controller driver"
> +	depends on ARCH_QCOM || COMPILE_TEST
> +	depends on I2C
> +	depends on INPUT
> +	depends on HWMON
> +	select NEW_LEDS
> +	select LEDS_CLASS
> +	help
> +	  Say Y here to enable the EC driver for the (Snapdragon X2 Elite,
> +	  glymur-based) Lenovo Yoga Slim 7x Gen 11 laptop. The EC provides
> +	  thermistors, battery cell voltages and CPU fan speed as hwmon
> +	  channels, and the keyboard backlight as an LED.
> +
> +	  This is the Gen 11 (glymur) machine specifically. Its EC is a Compal
> +	  design at i2c address 0x70 and speaks a different protocol from the
> +	  Qualcomm reference EC that the reference designs place at 0x76, so
> +	  EC_QCOM_HAMOA does not drive it - and conversely this driver does not
> +	  cover the X1E-based Yoga Slim 7x Gen 9.
> +
> +	  To compile this driver as a module, choose M here: the module will
> +	  be called lenovo-yoga-slim7x-gen11-ec.
> +
>  config EC_QCOM_HAMOA
>  	tristate "Embedded Controller driver for Qualcomm Hamoa/Glymur reference devices"
>  	depends on ARCH_QCOM || COMPILE_TEST
> diff --git a/drivers/platform/arm64/Makefile b/drivers/platform/arm64/Makefile
> index 7681be4a46e9..605cca4d42a9 100644
> --- a/drivers/platform/arm64/Makefile
> +++ b/drivers/platform/arm64/Makefile
> @@ -9,4 +9,5 @@ obj-$(CONFIG_EC_ACER_ASPIRE1)	+= acer-aspire1-ec.o
>  obj-$(CONFIG_EC_HUAWEI_GAOKUN)	+= huawei-gaokun-ec.o
>  obj-$(CONFIG_EC_LENOVO_YOGA_C630) += lenovo-yoga-c630.o
>  obj-$(CONFIG_EC_LENOVO_THINKPAD_T14S) += lenovo-thinkpad-t14s.o
> +obj-$(CONFIG_EC_LENOVO_YOGA_SLIM7X_GEN11) += lenovo-yoga-slim7x-gen11-ec.o
>  obj-$(CONFIG_EC_QCOM_HAMOA) += qcom-hamoa-ec.o
> diff --git a/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c b/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
> new file mode 100644
> index 000000000000..c8c7928475b7
> --- /dev/null
> +++ b/drivers/platform/arm64/lenovo-yoga-slim7x-gen11-ec.c
> @@ -0,0 +1,653 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +
> +#include <linux/bitfield.h>
> +#include <linux/bitops.h>
> +#include <linux/cleanup.h>
> +#include <linux/container_of.h>
> +#include <linux/delay.h>
> +#include <linux/device.h>
> +#include <linux/err.h>
> +#include <linux/errno.h>
> +#include <linux/gfp_types.h>
> +#include <linux/hwmon.h>
> +#include <linux/i2c.h>
> +#include <linux/input.h>
> +#include <linux/interrupt.h>
> +#include <linux/jiffies.h>
> +#include <linux/kstrtox.h>
> +#include <linux/leds.h>
> +#include <linux/module.h>
> +#include <linux/mutex.h>
> +#include <linux/pm.h>
> +#include <linux/string.h>
> +#include <linux/sysfs.h>
> +#include <linux/types.h>
> +
> +#define YOGA_EC_BLOCK_READ	0x01
> +#define YOGA_EC_BLOCK_WRITE	0x02
> +#define YOGA_EC_RESP_LEN	6
> +#define YOGA_EC_REQ_LEN		7
> +
> +#define YOGA_EC_CMD_RAM_READ	0xb0
> +#define YOGA_EC_CMD_RAM_WRITE	0xb1
> +#define YOGA_EC_CMD_BANK_READ	0xb4
> +#define YOGA_EC_CMD_FAN		0x46
> +#define YOGA_EC_FAN_GET_SPEED	0x81
> +
> +#define YOGA_EC_CMD_EVENT	0x84
> +#define YOGA_EC_EVT_MAX_DRAIN	64
> +#define YOGA_EC_EVT_FN_LOCK	0x0f
> +#define YOGA_EC_EVT_MIC_MUTE	0x18
> +#define YOGA_EC_EVT_FLIGHT_MODE	0x19
> +
> +#define YOGA_EC_SCUK		0x1c
> +#define YOGA_EC_SCUK_FN_LOCK	BIT(7)
> +#define YOGA_EC_SCUK_FN_LOCK_IND	BIT(5)

Add include for BIT()

> +#define YOGA_EC_SCUK_FN_LOCK_MASK	(YOGA_EC_SCUK_FN_LOCK | YOGA_EC_SCUK_FN_LOCK_IND)
> +
> +#define YOGA_EC_KBLT		0x03
> +#define YOGA_EC_KBLT_LEVEL	GENMASK(1, 0)
> +#define YOGA_EC_KBLT_AUTO	3
> +#define YOGA_EC_KBLT_MAX	2
> +
> +#define YOGA_EC_TEMP_FIRST	0x12
> +#define YOGA_EC_NUM_TEMP	6
> +#define YOGA_EC_TEMP_MAX	120
> +
> +#define YOGA_EC_CELL_BANK	0x0b
> +#define YOGA_EC_CELL_SIG_LO	0x02
> +#define YOGA_EC_CELL_SIG_HI	0x0c
> +#define YOGA_EC_CELL_FIRST	0x04
> +#define YOGA_EC_NUM_CELL	4
> +#define YOGA_EC_CELL_MIN_MV	2000
> +#define YOGA_EC_CELL_MAX_MV	5000
> +#define YOGA_EC_CELL_TRIES	10
> +
> +#define YOGA_EC_CACHE		(2 * HZ)
> +#define YOGA_EC_STALE		(10 * HZ)

It would be better to have time is some time unit + include the unit into 
name, and only convert to jiffies when a call requires jiffies/timeout 
input.

> +
> +struct yoga_ec {
> +	struct i2c_client *client;
> +	/* serializes all EC access */
> +	struct mutex lock;

Consider adding lockdep asserts to the EC access functions below to ensure 
the locking rules are never violated.

> +	unsigned long temp_present;
> +	unsigned long cell_read_at;
> +	u16 cell[YOGA_EC_NUM_CELL];
> +	bool cell_cached;
> +	struct led_classdev kbd_led;
> +	u8 kblt_level;
> +	bool kblt_restore;
> +	struct input_dev *keys;
> +};
> +
> +static int yoga_ec_command(struct yoga_ec *ec, u8 cmd, u8 arg0, u8 arg1,
> +			   u8 resp[YOGA_EC_RESP_LEN])
> +{
> +	u8 req[YOGA_EC_REQ_LEN] = { cmd, arg0, arg1 };
> +	int ret;
> +
> +	ret = i2c_smbus_write_i2c_block_data(ec->client, YOGA_EC_BLOCK_WRITE,
> +					     sizeof(req), req);
> +	if (ret < 0)
> +		return ret;
> +
> +	usleep_range(2000, 3000);
> +
> +	ret = i2c_smbus_read_i2c_block_data(ec->client, YOGA_EC_BLOCK_READ,
> +					    YOGA_EC_RESP_LEN, resp);
> +	if (ret < 0)
> +		return ret;
> +	if (ret != YOGA_EC_RESP_LEN)
> +		return -EIO;
> +
> +	return 0;
> +}
> +
> +static int yoga_ec_ram_read(struct yoga_ec *ec, u8 addr, u8 *val)
> +{
> +	u8 resp[YOGA_EC_RESP_LEN];
> +	int ret;
> +
> +	ret = yoga_ec_command(ec, YOGA_EC_CMD_RAM_READ, addr, 0, resp);
> +	if (ret)
> +		return ret;
> +
> +	*val = resp[0];
> +	return 0;
> +}
> +
> +static int yoga_ec_bank_read(struct yoga_ec *ec, u8 bank, u8 addr, u8 *val)
> +{
> +	u8 resp[YOGA_EC_RESP_LEN];
> +	int ret;
> +
> +	ret = yoga_ec_command(ec, YOGA_EC_CMD_BANK_READ, bank, addr, resp);
> +	if (ret)
> +		return ret;
> +
> +	*val = resp[0];
> +	return 0;
> +}
> +
> +static int yoga_ec_ram_write(struct yoga_ec *ec, u8 addr, u8 val)
> +{
> +	u8 resp[YOGA_EC_RESP_LEN];
> +
> +	return yoga_ec_command(ec, YOGA_EC_CMD_RAM_WRITE, addr, val, resp);
> +}
> +
> +static int yoga_ec_fan_read(struct yoga_ec *ec, long *rpm)
> +{
> +	u8 resp[YOGA_EC_RESP_LEN];
> +	int ret;
> +
> +	ret = yoga_ec_command(ec, YOGA_EC_CMD_FAN, YOGA_EC_FAN_GET_SPEED, 0,
> +			      resp);
> +	if (ret)
> +		return ret;
> +
> +	*rpm = resp[0] * 100;
> +	return 0;
> +}
> +
> +static int yoga_ec_refresh_cells(struct yoga_ec *ec)
> +{
> +	unsigned int try;
> +	int ret;
> +
> +	if (ec->cell_cached &&
> +	    time_before(jiffies, ec->cell_read_at + YOGA_EC_CACHE))
> +		return 0;
> +
> +	for (try = 0; try < YOGA_EC_CELL_TRIES; try++) {
> +		u16 cell[YOGA_EC_NUM_CELL];
> +		u8 sig[3];
> +		unsigned int i;
> +		bool ok = true;
> +
> +		ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
> +					YOGA_EC_CELL_SIG_LO, &sig[0]);
> +		if (!ret)
> +			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
> +						YOGA_EC_CELL_SIG_LO + 1, &sig[1]);
> +		if (!ret)
> +			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
> +						YOGA_EC_CELL_SIG_HI, &sig[2]);
> +		if (ret)
> +			return ret;

Please don't complicate error handling flow but handle each error right 
after the first call.

> +
> +		if (sig[0] != 0x01 || sig[1] != 0x01 || sig[2] != 0x03)
> +			continue;
> +
> +		for (i = 0; i < YOGA_EC_NUM_CELL; i++) {

ARRAY_SIZE() + don't forget the include.

> +			u8 lsb, msb;
> +
> +			ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
> +						YOGA_EC_CELL_FIRST + i * 2, &lsb);
> +			if (!ret)
> +				ret = yoga_ec_bank_read(ec, YOGA_EC_CELL_BANK,
> +							YOGA_EC_CELL_FIRST + i * 2 + 1,
> +							&msb);
> +			if (ret)
> +				return ret;

Simplify error handling similar to above.

> +
> +			cell[i] = lsb | (msb << 8);
> +			if (cell[i] < YOGA_EC_CELL_MIN_MV ||
> +			    cell[i] > YOGA_EC_CELL_MAX_MV)
> +				ok = false;
> +		}
> +
> +		if (!ok)
> +			continue;
> +
> +		memcpy(ec->cell, cell, sizeof(cell));
> +		ec->cell_cached = true;
> +		ec->cell_read_at = jiffies;
> +
> +		return 0;
> +	}
> +
> +	if (ec->cell_cached &&
> +	    time_before(jiffies, ec->cell_read_at + YOGA_EC_STALE))
> +		return 0;
> +
> +	return -EAGAIN;
> +}
> +
> +static umode_t yoga_ec_is_visible(const void *drvdata,
> +				  enum hwmon_sensor_types type,
> +				  u32 attr, int channel)
> +{
> +	const struct yoga_ec *ec = drvdata;
> +
> +	switch (type) {
> +	case hwmon_temp:
> +		if (!(ec->temp_present & BIT(channel)))
> +			return 0;
> +		return 0444;
> +	case hwmon_in:
> +		return 0444;
> +	case hwmon_fan:
> +		return 0444;
> +	default:
> +		return 0;
> +	}
> +}
> +
> +static int yoga_ec_read(struct device *dev, enum hwmon_sensor_types type,
> +			u32 attr, int channel, long *val)
> +{
> +	struct yoga_ec *ec = dev_get_drvdata(dev);
> +	int ret;
> +	u8 raw;
> +
> +	guard(mutex)(&ec->lock);
> +
> +	switch (type) {
> +	case hwmon_temp:
> +		ret = yoga_ec_ram_read(ec, YOGA_EC_TEMP_FIRST + channel, &raw);
> +		if (ret)
> +			return ret;
> +		if (!raw)
> +			return -ENODATA;
> +		if (raw > YOGA_EC_TEMP_MAX)
> +			return -EIO;
> +		*val = raw * 1000;

Can something from units.h replace this literal? Don't forget to add the 
include if you start to use a define from there.

> +		return 0;
> +	case hwmon_in:
> +		ret = yoga_ec_refresh_cells(ec);
> +		if (ret)
> +			return ret;
> +		*val = ec->cell[channel];
> +		return 0;
> +	case hwmon_fan:
> +		return yoga_ec_fan_read(ec, val);
> +	default:
> +		return -EOPNOTSUPP;
> +	}
> +}
> +
> +static const char * const yoga_ec_temp_labels[YOGA_EC_NUM_TEMP] = {
> +	"Charger A",
> +	"CPU VR",
> +	"5V rail",
> +	"Ambient",
> +	"Charger B",
> +	"Thermistor 6",
> +};
> +
> +static const char * const yoga_ec_cell_labels[YOGA_EC_NUM_CELL] = {
> +	"Cell 1", "Cell 2", "Cell 3", "Cell 4",
> +};
> +
> +static int yoga_ec_read_string(struct device *dev, enum hwmon_sensor_types type,
> +			       u32 attr, int channel, const char **str)
> +{
> +	switch (type) {
> +	case hwmon_temp:
> +		*str = yoga_ec_temp_labels[channel];
> +		return 0;
> +	case hwmon_in:
> +		*str = yoga_ec_cell_labels[channel];
> +		return 0;
> +	case hwmon_fan:
> +		*str = "CPU fan";
> +		return 0;
> +	default:
> +		return -EOPNOTSUPP;
> +	}
> +}
> +
> +static enum led_brightness yoga_ec_kbd_led_get(struct led_classdev *led)
> +{
> +	struct yoga_ec *ec = container_of(led, struct yoga_ec, kbd_led);
> +	u8 raw;
> +
> +	guard(mutex)(&ec->lock);
> +
> +	if (yoga_ec_ram_read(ec, YOGA_EC_KBLT, &raw))
> +		return 0;
> +
> +	raw = FIELD_GET(YOGA_EC_KBLT_LEVEL, raw);
> +
> +	if (raw == YOGA_EC_KBLT_AUTO)
> +		return YOGA_EC_KBLT_MAX;
> +
> +	return raw;
> +}
> +
> +static int yoga_ec_kbd_led_level(struct yoga_ec *ec, u8 level, u8 *old)
> +{
> +	u8 raw;
> +	int ret;
> +
> +	ret = yoga_ec_ram_read(ec, YOGA_EC_KBLT, &raw);
> +	if (ret)
> +		return ret;
> +
> +	if (old)
> +		*old = FIELD_GET(YOGA_EC_KBLT_LEVEL, raw);
> +
> +	raw &= ~YOGA_EC_KBLT_LEVEL;
> +	raw |= FIELD_PREP(YOGA_EC_KBLT_LEVEL, level);
> +
> +	return yoga_ec_ram_write(ec, YOGA_EC_KBLT, raw);
> +}
> +
> +static int yoga_ec_kbd_led_set(struct led_classdev *led,
> +			       enum led_brightness brightness)
> +{
> +	struct yoga_ec *ec = container_of(led, struct yoga_ec, kbd_led);
> +
> +	guard(mutex)(&ec->lock);
> +
> +	return yoga_ec_kbd_led_level(ec, brightness, NULL);
> +}
> +
> +static const struct hwmon_ops yoga_ec_hwmon_ops = {
> +	.is_visible = yoga_ec_is_visible,
> +	.read = yoga_ec_read,
> +	.read_string = yoga_ec_read_string,
> +};
> +
> +static const struct hwmon_channel_info * const yoga_ec_hwmon_info[] = {
> +	HWMON_CHANNEL_INFO(in,
> +			   HWMON_I_INPUT | HWMON_I_LABEL,
> +			   HWMON_I_INPUT | HWMON_I_LABEL,
> +			   HWMON_I_INPUT | HWMON_I_LABEL,
> +			   HWMON_I_INPUT | HWMON_I_LABEL),
> +	HWMON_CHANNEL_INFO(temp,
> +			   HWMON_T_INPUT | HWMON_T_LABEL,
> +			   HWMON_T_INPUT | HWMON_T_LABEL,
> +			   HWMON_T_INPUT | HWMON_T_LABEL,
> +			   HWMON_T_INPUT | HWMON_T_LABEL,
> +			   HWMON_T_INPUT | HWMON_T_LABEL,
> +			   HWMON_T_INPUT | HWMON_T_LABEL),
> +	HWMON_CHANNEL_INFO(fan,
> +			   HWMON_F_INPUT | HWMON_F_LABEL),
> +	NULL
> +};
> +
> +static const struct hwmon_chip_info yoga_ec_chip_info = {
> +	.ops = &yoga_ec_hwmon_ops,
> +	.info = yoga_ec_hwmon_info,
> +};
> +
> +static int yoga_ec_fn_lock_toggle(struct yoga_ec *ec)
> +{
> +	u8 raw;
> +	int ret;
> +
> +	ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
> +	if (ret)
> +		return ret;
> +
> +	if (raw & YOGA_EC_SCUK_FN_LOCK)
> +		raw &= ~YOGA_EC_SCUK_FN_LOCK_MASK;
> +	else
> +		raw |= YOGA_EC_SCUK_FN_LOCK_MASK;
> +
> +	return yoga_ec_ram_write(ec, YOGA_EC_SCUK, raw);
> +}
> +
> +static void yoga_ec_report_key(struct yoga_ec *ec, unsigned int code)
> +{
> +	if (!ec->keys)
> +		return;
> +
> +	input_report_key(ec->keys, code, 1);
> +	input_sync(ec->keys);
> +	input_report_key(ec->keys, code, 0);
> +	input_sync(ec->keys);
> +}
> +
> +static irqreturn_t yoga_ec_irq(int irq, void *data)
> +{
> +	struct yoga_ec *ec = data;
> +	unsigned int i;
> +
> +	guard(mutex)(&ec->lock);
> +
> +	for (i = 0; i < YOGA_EC_EVT_MAX_DRAIN; i++) {
> +		u8 resp[YOGA_EC_RESP_LEN];
> +
> +		if (yoga_ec_command(ec, YOGA_EC_CMD_EVENT, 0, 0, resp))
> +			break;
> +		if (!resp[0])
> +			break;
> +
> +		switch (resp[0]) {
> +		case YOGA_EC_EVT_FN_LOCK:
> +			yoga_ec_fn_lock_toggle(ec);
> +			break;
> +		case YOGA_EC_EVT_MIC_MUTE:
> +			yoga_ec_report_key(ec, KEY_MICMUTE);
> +			break;
> +		case YOGA_EC_EVT_FLIGHT_MODE:
> +			yoga_ec_report_key(ec, KEY_RFKILL);
> +			break;
> +		default:
> +			break;
> +		}
> +	}
> +
> +	return IRQ_HANDLED;
> +}
> +
> +static void yoga_ec_register_keys(struct yoga_ec *ec)
> +{
> +	struct device *dev = &ec->client->dev;
> +	struct input_dev *input;
> +	int ret;
> +
> +	input = devm_input_allocate_device(dev);
> +	if (!input) {
> +		dev_warn(dev, "no memory for the hotkey device\n");

Add include.

> +		return;
> +	}
> +
> +	input->name = "Lenovo Yoga Slim 7x hotkeys";
> +	input->phys = "lenovo-yoga-slim7x-gen11-ec/input0";
> +	input->id.bustype = BUS_I2C;
> +	input_set_capability(input, EV_KEY, KEY_MICMUTE);
> +	input_set_capability(input, EV_KEY, KEY_RFKILL);
> +
> +	ret = input_register_device(input);
> +	if (ret) {
> +		dev_warn(dev, "cannot register the hotkey device: %d\n", ret);
> +		return;
> +	}
> +
> +	ec->keys = input;
> +}
> +
> +static ssize_t fn_lock_show(struct device *dev, struct device_attribute *attr,
> +			    char *buf)
> +{
> +	struct yoga_ec *ec = dev_get_drvdata(dev);
> +	u8 raw;
> +	int ret;
> +
> +	scoped_guard(mutex, &ec->lock)
> +		ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
> +
> +	if (ret)
> +		return ret;
> +
> +	return sysfs_emit(buf, "%u\n", !!(raw & YOGA_EC_SCUK_FN_LOCK));
> +}
> +
> +static ssize_t fn_lock_store(struct device *dev, struct device_attribute *attr,
> +			     const char *buf, size_t count)
> +{
> +	struct yoga_ec *ec = dev_get_drvdata(dev);
> +	bool enable;
> +	u8 raw;
> +	int ret;
> +
> +	ret = kstrtobool(buf, &enable);
> +	if (ret)
> +		return ret;
> +
> +	guard(mutex)(&ec->lock);
> +
> +	ret = yoga_ec_ram_read(ec, YOGA_EC_SCUK, &raw);
> +	if (ret)
> +		return ret;
> +
> +	if (enable)
> +		raw |= YOGA_EC_SCUK_FN_LOCK_MASK;
> +	else
> +		raw &= ~YOGA_EC_SCUK_FN_LOCK_MASK;
> +
> +	ret = yoga_ec_ram_write(ec, YOGA_EC_SCUK, raw);
> +	if (ret)
> +		return ret;
> +
> +	return count;
> +}
> +static DEVICE_ATTR_RW(fn_lock);
> +
> +static struct attribute *yoga_ec_attrs[] = {
> +	&dev_attr_fn_lock.attr,
> +	NULL
> +};
> +ATTRIBUTE_GROUPS(yoga_ec);
> +
> +static int yoga_ec_probe(struct i2c_client *client)
> +{
> +	struct device *dev = &client->dev;
> +	struct device *hwmon;
> +	struct yoga_ec *ec;
> +	unsigned int i;
> +	int ret;
> +
> +	if (!i2c_check_functionality(client->adapter,
> +				     I2C_FUNC_SMBUS_I2C_BLOCK))
> +		return dev_err_probe(dev, -ENODEV,
> +				     "adapter does not support SMBus block transfers\n");
> +
> +	ec = devm_kzalloc(dev, sizeof(*ec), GFP_KERNEL);
> +	if (!ec)
> +		return -ENOMEM;
> +
> +	ec->client = client;
> +	i2c_set_clientdata(client, ec);
> +
> +	ret = devm_mutex_init(dev, &ec->lock);
> +	if (ret)
> +		return ret;
> +
> +	for (i = 0; i < YOGA_EC_NUM_TEMP; i++) {
> +		u8 raw;
> +
> +		ret = yoga_ec_ram_read(ec, YOGA_EC_TEMP_FIRST + i, &raw);
> +		if (ret)
> +			return dev_err_probe(dev, ret,
> +					     "failed to read thermistor %u\n", i);
> +
> +		if (raw > 0 && raw <= YOGA_EC_TEMP_MAX)
> +			ec->temp_present |= BIT(i);
> +	}
> +
> +	if (!ec->temp_present)
> +		return dev_err_probe(dev, -ENODEV,
> +				     "no thermistor reported a usable value\n");

Please use braces in multi-line constructs.

> +
> +	dev_dbg(dev, "thermistors present: %#lx\n", ec->temp_present);
> +
> +	hwmon = devm_hwmon_device_register_with_info(dev, "yoga_slim7x_ec", ec,
> +						     &yoga_ec_chip_info, NULL);
> +	if (IS_ERR(hwmon))
> +		return PTR_ERR(hwmon);
> +
> +	ec->kbd_led.name = "platform::kbd_backlight";
> +	ec->kbd_led.max_brightness = YOGA_EC_KBLT_MAX;
> +	ec->kbd_led.brightness_get = yoga_ec_kbd_led_get;
> +	ec->kbd_led.brightness_set_blocking = yoga_ec_kbd_led_set;
> +
> +	ret = devm_led_classdev_register(dev, &ec->kbd_led);
> +	if (ret)
> +		return dev_err_probe(dev, ret,
> +				     "failed to register keyboard backlight\n");
> +
> +	if (client->irq) {
> +		yoga_ec_register_keys(ec);
> +
> +		ret = devm_request_threaded_irq(dev, client->irq, NULL,
> +						yoga_ec_irq, IRQF_ONESHOT,
> +						"lenovo-yoga-slim7x-gen11-ec", ec);
> +		if (ret)
> +			return dev_err_probe(dev, ret, "cannot request the EC interrupt\n");
> +
> +		scoped_guard(mutex, &ec->lock) {
> +			unsigned int i;
> +
> +			for (i = 0; i < YOGA_EC_EVT_MAX_DRAIN; i++) {
> +				u8 resp[YOGA_EC_RESP_LEN];
> +
> +				if (yoga_ec_command(ec, YOGA_EC_CMD_EVENT, 0, 0, resp))
> +					break;
> +				if (!resp[0])
> +					break;
> +			}
> +		}
> +	}
> +
> +	return 0;
> +}
> +
> +static int yoga_ec_suspend(struct device *dev)
> +{
> +	struct yoga_ec *ec = dev_get_drvdata(dev);
> +
> +	guard(mutex)(&ec->lock);
> +	ec->kblt_restore = !yoga_ec_kbd_led_level(ec, 0, &ec->kblt_level);
> +
> +	return 0;
> +}
> +
> +static int yoga_ec_resume(struct device *dev)
> +{
> +	struct yoga_ec *ec = dev_get_drvdata(dev);
> +
> +	guard(mutex)(&ec->lock);
> +	ec->cell_cached = false;
> +
> +	if (ec->kblt_restore) {
> +		ec->kblt_restore = false;
> +		yoga_ec_kbd_led_level(ec, ec->kblt_level, NULL);
> +	}
> +
> +	return 0;
> +}
> +
> +static DEFINE_SIMPLE_DEV_PM_OPS(yoga_ec_pm_ops, yoga_ec_suspend, yoga_ec_resume);
> +
> +static const struct of_device_id yoga_ec_of_match[] = {
> +	{ .compatible = "lenovo,yoga-slim7x-gen11-ec" },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(of, yoga_ec_of_match);
> +
> +static const struct i2c_device_id yoga_ec_i2c_id[] = {
> +	{ "yoga-slim7x-gen11-ec" },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(i2c, yoga_ec_i2c_id);
> +
> +static struct i2c_driver yoga_ec_driver = {
> +	.driver = {
> +		.name = "lenovo-yoga-slim7x-gen11-ec",
> +		.of_match_table = yoga_ec_of_match,
> +		.dev_groups = yoga_ec_groups,
> +		.pm = pm_sleep_ptr(&yoga_ec_pm_ops),
> +	},
> +	.probe = yoga_ec_probe,
> +	.id_table = yoga_ec_i2c_id,
> +};
> +module_i2c_driver(yoga_ec_driver);
> +
> +MODULE_AUTHOR("Oleg Keri <okerixx@gmail.com>");
> +MODULE_DESCRIPTION("Lenovo Yoga Slim 7x Gen 11 embedded controller");
> +MODULE_LICENSE("GPL");
> 

-- 
 i.

Re: [PATCH v3 2/3] platform/arm64: add Lenovo Yoga Slim 7x Gen 11 EC driver

[Oleg Keri]

Hi Ilpo,

On Mon, Sep 21, 2026, Ilpo Järvinen wrote:
> > +Date:		August 2026
>
> ??
>
> > +KernelVersion:	7.3
>
> This ship has already sailed.

Will be September 2026 and 7.4.

All the other points will be in v4 as well:

- the missing includes added where the symbols are used
- the cache and stale periods defined in milliseconds, with the unit in
  the name, and converted to jiffies at the call
- lockdep_assert_held() in the command path every EC access goes through
- each EC read checked right after the call
- ARRAY_SIZE() for the cell loop
- MILLIDEGREE_PER_DEGREE from units.h instead of the literal
- braces on the multi-line returns

Plus two things not from your review: a shorter i2c id that fits
I2C_NAME_SIZE (the bot's finding), and the symbol prefix renamed to
slim7x_ec to match the driver name, which makes the v4 diff larger
than the changes are.

Thanks,
Oleg

[PATCH v3 3/3] arm64: dts: qcom: glymur-lenovo-yoga-slim7x: add the embedded controller

[Oleg Keri]

Describe the Compal embedded controller on I2C at address 0x70. It
provides the board thermistors, the battery cell voltages and the CPU fan
speed, and drives the keyboard backlight. Its interrupt line is TLMM
gpio66, falling edge, for which the file already carries a pin state.

That state leaves the pin without a bias. gpio66 has no external pull
and floats when the EC is not driving it, which shows up as a steady
stream of spurious edges; the vendor ACPI tables describe the same pin
as GpioInt(Edge, ActiveLow, Exclusive, PullUp). Enable the pull-up.

Signed-off-by: Oleg Keri <okerixx@gmail.com>
---
 .../boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts      | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts
index 769f195c99d9..bc840fd8e458 100644
--- a/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts
+++ b/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts
@@ -659,7 +659,15 @@ &i2c9 {
 
 	status = "okay";
 
-	/* EC @ 0x70, irq = TLMM 66 */
+	embedded-controller@70 {
+		compatible = "lenovo,yoga-slim7x-gen11-ec";
+		reg = <0x70>;
+
+		interrupts-extended = <&tlmm 66 IRQ_TYPE_EDGE_FALLING>;
+
+		pinctrl-0 = <&ec_int_n_default>;
+		pinctrl-names = "default";
+	};
 };
 
 &lpass_vamacro {
@@ -990,7 +998,7 @@ reset-n-pins {
 	ec_int_n_default: ec-int-n-state {
 		pins = "gpio66";
 		function = "gpio";
-		bias-disable;
+		bias-pull-up;
 	};
 
 	kybd_default: kybd-default-state {
base-commit: 5c4d4169604b335c38bbc79bc1fc03042981fc6f
-- 
2.55.0

Re: [PATCH v3 3/3] arm64: dts: qcom: glymur-lenovo-yoga-slim7x: add the embedded controller

[Konrad Dybcio]

On 9/21/26 4:49 PM, Oleg Keri wrote:
> Describe the Compal embedded controller on I2C at address 0x70. It
> provides the board thermistors, the battery cell voltages and the CPU fan
> speed, and drives the keyboard backlight. Its interrupt line is TLMM
> gpio66, falling edge, for which the file already carries a pin state.
> 
> That state leaves the pin without a bias. gpio66 has no external pull
> and floats when the EC is not driving it, which shows up as a steady
> stream of spurious edges; the vendor ACPI tables describe the same pin
> as GpioInt(Edge, ActiveLow, Exclusive, PullUp). Enable the pull-up.
> 
> Signed-off-by: Oleg Keri <okerixx@gmail.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad