The Snapdragon X2 series-based Zenbooks come with a bespoke embedded
controller firmware, bearing some resemblence to the reference Qualcomm
implementation (qcom-hamoa-ec.c), albeit with too many changes made to
consider them anywhere near "compatible".
This series introduces a new driver to let the user retrieve current
RPM of the two fans, the temperature readouts from 2 temperature
sensors and get/let keyboard brightness levels. The driver also
notifies the EC of system suspend entry/exit.
Two separate compatibles are maintained since the two laptops' ECs seem
to be generally alike, however assuming full compatibility with the
limited available information does not spark confidence.
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
Changes in v3:
- Cover Ilpo's feedback: includes, braces, dev_info->dev_dbg, line
wrapping, reuse the timeout logic
- Pick up tags
- Reabse on next-20260922 (NOP)
- Link to v2: https://patch.msgid.link/20260922-topic-asus_ec-v2-0-0bd923d1b853@oss.qualcomm.com
Changes in v2:
- Add rationale for separate compatibles
- Drop `wakeup-source` from `required` in dt-bindings
- Fix bindings example indentation
- Rebase on next-20260921 (NOP)
- Link to v1: https://patch.msgid.link/20260917-topic-asus_ec-v1-0-373516d347ae@oss.qualcomm.com
To: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzk+dt@kernel.org>
To: Conor Dooley <conor+dt@kernel.org>
To: Konrad Dybcio <konradybcio@kernel.org>
To: Hans de Goede <hansg@kernel.org>
To: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
To: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
To: Guenter Roeck <linux@roeck-us.net>
To: Bjorn Andersson <andersson@kernel.org>
To: Abel Vesa <abelvesa@kernel.org>
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: platform-driver-x86@vger.kernel.org
Cc: linux-hwmon@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org
---
Konrad Dybcio (3):
dt-bindings: embedded-controller: Add ASUS Zenbook A16 EC
platform: arm64: Add a driver for the EC found on ASUS Glymur machines
arm64: dts: qcom: glymur-zenbook-a16: Add Embedded Controller
.../asus,zenbook-a16-ux3607oa-ec.yaml | 64 +++
.../dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts | 15 +-
drivers/platform/arm64/Kconfig | 16 +
drivers/platform/arm64/Makefile | 1 +
drivers/platform/arm64/asus-glymur-ec.c | 600 +++++++++++++++++++++
5 files changed, 694 insertions(+), 2 deletions(-)
---
base-commit: a8c591ed6b672915e0be57843f943a2a723aff40
change-id: 20260917-topic-asus_ec-065764978e77
Best regards,
--
Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>