From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f48.google.com (mail-wm1-f48.google.com [209.85.128.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E342F48EBE8 for ; Tue, 25 Aug 2026 20:14:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787688887; cv=none; b=TBgB8e+tQnw4aob6aF1+aovfnR29LS69XRVW48CXbXNEMjBopZ8MBGk0A7kT+wPlXyiyHtNeSe2/pLcQwoCicOLoEyYk4f8f1xIjIJ+99RlzvRHzyquRE9qTwh6WLKIw+Ty1/B+q9RUvRK+xponogykB607vFfioayhiQ06B1KQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787688887; c=relaxed/simple; bh=/vTICs4rfZnQqWAY7jp6Xx8e1B3/4HZ5WGwfLBmD4E8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=bzRRrJmGRswDSALe5cSfMGPJsQ/rlQKREXtXaffAl9JxT8jFZJzupKv6rpFWTl6UezxnHFVb7iurgkgn33d1A7OLHEh7oFpAPzJpWqHcZ2Hoohe3M9JKjmPFdsb5871XqZM0imM9uSpqOgxQT+02c8EP3HPlXJjEUrDytJvC048= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=no13wnrI; arc=none smtp.client-ip=209.85.128.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="no13wnrI" Received: by mail-wm1-f48.google.com with SMTP id 5b1f17b1804b1-4953e04ef16so1123655e9.2 for ; Tue, 25 Aug 2026 13:14:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787688884; x=1788293684; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=hWvvM2eJTieYNAJ1IrJEwIoJaD0ytMJ17q0W4sd4AmA=; b=no13wnrIWtL13vXQkmEAVFZmcye2xYOSzJISNBI57Ca0fAuh4FptkhTEq0nMYKx0fF Z6MwIYliX/WttdhybRmVClrS9Z4h+G94ZpgMGul8Hbcy3ZkejzgImBezpdqYP2PoGUb2 pF3csgsWluJ3rxOcegCGElricJwL6t+E5rrGC+Csz9OHotPPJHZjj7nECzDilNVt6SAO C+xD263FqgJfde4TpFcIiujJY/YLLns0pWumU5qfoovrbxt/IfOgozxy8RCAbV/6auwB LIdxSIk4DqZX5FTr2SB7WoKHHqitAMyw8LGhE8ieeaSNCt1eVZXmclI0tODNQ0KSVyeu 8e2A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787688884; x=1788293684; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=hWvvM2eJTieYNAJ1IrJEwIoJaD0ytMJ17q0W4sd4AmA=; b=sHMCUXvdOOhHYfLRSbNnE617UJbX7S8eigIIeka6fXy0n6/nHpw2lcVtemLkvoP8E8 lYGuP0NI7IwyioDPrfbwZ5LPq4n04JLSBR/mYE0sRsDYHEUVltjuGNGcBBxF6g3V9bzB HWewJWsdTGKaF20W7oAq4iATjN0rJ1A56EwoYAsWf1HkkPQnkTXMl+pFLqB1GQNQ0Amn sxQakdatRG4xhXdWrXpdM4PEeIpBsVvo5/4Zm8FpSTgRxyMjN6r7SHkzS9bxZPTTy1YW LjESMPKb6NGM5up4sMYgUJu3s/BuFmz44TqD33V/9PlLrASWn+RQ1mH5ZBGEYrhVD6XS NjCA== X-Forwarded-Encrypted: i=1; AHgh+Roh/s1teJBW2dLO/xyYpJmH8xg3gHyJBbCDLrZG5KrOBYGw3GVyMScIPQ/5CYl+M3POWS3LX3xGeON3Ssw=@vger.kernel.org X-Gm-Message-State: AFuF++nuXzaJeq0ak9QWgq+/WtVvuKYjdJcfjbtRceC/uT+irZ/rZadV QnrhxjFPe0ntct0QihU+CCCi9jgPL5ZvcM4w0sli+qTmw/WKbXFxXoDW X-Gm-Gg: AR+sD13MxFnSpwt45X8QJ0o41GzBA/bgKQAeCvqXpp7LrF+704PYjfNJXenBYM2NxE9 cQ5a0LdTnZ0oYSfY+ze0fr3CV1bW3M2NbdpxXFZLpj7Szmi1jaS7yGAGMOmqZVaS7DymTlrDB0c RKZ0L/adYZd40f/xVVxmF0kpF70EH04MY/RAz6CuFaO6qOIYJTpH7qmzfua03YMOzM6tNJQxGMJ oGe5EeDBRDQIIe+t4noSiwkyex+/1n/BGEk+LFRl+9c8ld8xUINM2/Z3K6L+8j24HEymbTOhr43 fbXc+XPcz5VW4dXKzDm8pa8SxE+n+FISj3mWRhQCWW9OLNr4IXOH2tsNkm5upqq7ykvXiCXfZFB BXq7eQPx1yz05fIbZqpQiKptpVxWBiFyMA4oEJQL1glbVQ84EUay0am/HfiFAsgiSHtdcJOHy4O XCzl88CyyHBAzlrzwAeDDPExtKnRJHA6N9NdtsqmZWTtDsvoIknRvI4iCAcDmgFSs9GXgxP0CTU E7daY4tKi5/lJMKZk1F/2XzqCd71jHUREftfY9DsaPlMOKSHEYjL8dy3epMYMteA0spWn4oJlpI nsaUkcZLf5+n4AuATCf9sQd/TGUS9wo8OBN/rlDiKGpYF6+n X-Received: by 2002:a05:600c:811a:b0:499:db9b:f253 with SMTP id 5b1f17b1804b1-499dc6ec214mr15659295e9.3.1787688883796; Tue, 25 Aug 2026 13:14:43 -0700 (PDT) Received: from scambox.localdomain (5-198-68-184.static.kc.net.uk. [5.198.68.184]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-499dc960e80sm5050435e9.1.2026.08.25.13.14.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2026 13:14:43 -0700 (PDT) From: Edward Blair To: linux-acpi@vger.kernel.org, linux-usb@vger.kernel.org Cc: rafael@kernel.org, lenb@kernel.org, heikki.krogerus@linux.intel.com, mika.westerberg@linux.intel.com, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, Edward Blair Subject: [PATCH v2 2/2] usb: typec: ucsi: add ITE885x I2C transport driver Date: Tue, 25 Aug 2026 21:14:26 +0100 Message-ID: <20260825201426.47030-3-edward.blair@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260825201426.47030-1-edward.blair@gmail.com> References: <20260825201426.47030-1-edward.blair@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Add a UCSI transport driver for ITE8853 and ITE8800 through ITE8805 USB Type-C controllers found on desktop motherboards. These controllers expose CCI, MESSAGE_IN and CONTROL at ITE-specific I2C offsets and signal UCSI and vendor events through a shared interrupt status register. Read and cache each complete UCSI event before acknowledging it so command data remains coherent between the interrupt handler and UCSI core. The interface does not expose a VERSION register and does not accept PPM_RESET over I2C. Report UCSI 1.0, limit MESSAGE_IN to its 16-byte window and handle PPM_RESET locally, matching the vendor driver's behavior. Signed-off-by: Edward Blair --- drivers/usb/typec/ucsi/Kconfig | 11 + drivers/usb/typec/ucsi/Makefile | 1 + drivers/usb/typec/ucsi/ucsi_ite.c | 395 ++++++++++++++++++++++++++++++ 3 files changed, 407 insertions(+) create mode 100644 drivers/usb/typec/ucsi/ucsi_ite.c diff --git a/drivers/usb/typec/ucsi/Kconfig b/drivers/usb/typec/ucsi/Kconfig index 87dd992a4..3819c4f73 100644 --- a/drivers/usb/typec/ucsi/Kconfig +++ b/drivers/usb/typec/ucsi/Kconfig @@ -104,4 +104,15 @@ config UCSI_HUAWEI_GAOKUN To compile the driver as a module, choose M here: the module will be called ucsi_huawei_gaokun. +config UCSI_ITE + tristate "UCSI Interface Driver for ITE885x" + depends on ACPI && I2C + help + This driver enables UCSI support on platforms that expose an ITE8853 + or ITE8800-ITE8805 USB Type-C controller over I2C, commonly found + on ASUS Z690/Z790/X670E motherboards. + + To compile the driver as a module, choose M here: the module will be + called ucsi_ite. + endif diff --git a/drivers/usb/typec/ucsi/Makefile b/drivers/usb/typec/ucsi/Makefile index c7e38bf01..9bc1d6bbb 100644 --- a/drivers/usb/typec/ucsi/Makefile +++ b/drivers/usb/typec/ucsi/Makefile @@ -28,3 +28,4 @@ obj-$(CONFIG_UCSI_PMIC_GLINK) += ucsi_glink.o obj-$(CONFIG_CROS_EC_UCSI) += cros_ec_ucsi.o obj-$(CONFIG_UCSI_LENOVO_YOGA_C630) += ucsi_yoga_c630.o obj-$(CONFIG_UCSI_HUAWEI_GAOKUN) += ucsi_huawei_gaokun.o +obj-$(CONFIG_UCSI_ITE) += ucsi_ite.o diff --git a/drivers/usb/typec/ucsi/ucsi_ite.c b/drivers/usb/typec/ucsi/ucsi_ite.c new file mode 100644 index 000000000..16b22c77f --- /dev/null +++ b/drivers/usb/typec/ucsi/ucsi_ite.c @@ -0,0 +1,395 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * UCSI I2C transport driver for ITE885x USB-C controllers + * + * ITE8853/ITE8800-ITE8805 are UCSI-compliant USB-C controllers found on + * desktop motherboards. They communicate over I2C using UCSI registers at + * ITE-specific offsets and signal events through a vendor interrupt register. + */ + +#include +#include +#include +#include +#include + +#include "ucsi.h" + +#define ITE_REG_CCI 0x84 +#define ITE_REG_MESSAGE_IN 0x88 +#define ITE_REG_CONTROL 0x98 +#define ITE_REG_INT_ACK 0xbc +#define ITE_REG_INT_STATUS 0xbd + +#define ITE_INT_VENDOR_ALERT BIT(0) +#define ITE_INT_CCI BIT(1) +#define ITE_INT_MASK (ITE_INT_VENDOR_ALERT | ITE_INT_CCI) + +#define ITE_MESSAGE_IN_MAX_LEN 0x10 + +enum ucsi_ite_event { + ITE_EVENT_NONE, + ITE_EVENT_CCI, + ITE_EVENT_VENDOR, +}; + +struct ucsi_ite { + struct i2c_client *client; + struct ucsi *ucsi; + struct mutex event_lock; /* Serializes IRQ and polling */ + struct mutex received_lock; /* Protects CCI and message_in */ + u8 message_in[ITE_MESSAGE_IN_MAX_LEN]; + u32 cci; + bool registered; +}; + +static int ucsi_ite_read(struct ucsi_ite *ite, u8 reg, void *val, size_t len) +{ + struct i2c_client *client = ite->client; + struct i2c_msg msgs[] = { + { + .addr = client->addr, + .len = 1, + .buf = ®, + }, + { + .addr = client->addr, + .flags = I2C_M_RD, + .len = len, + .buf = val, + }, + }; + int ret; + + ret = i2c_transfer(client->adapter, msgs, ARRAY_SIZE(msgs)); + if (ret == ARRAY_SIZE(msgs)) + return 0; + + ret = ret < 0 ? ret : -EIO; + dev_err_ratelimited(&client->dev, + "register 0x%02x read failed: %d\n", reg, ret); + return ret; +} + +static int ucsi_ite_write(struct ucsi_ite *ite, u8 reg, const void *val, + size_t len) +{ + struct i2c_client *client = ite->client; + u8 buf[sizeof(u64) + 1]; + struct i2c_msg msg = { + .addr = client->addr, + .len = len + 1, + .buf = buf, + }; + int ret; + + if (len > sizeof(buf) - 1) + return -EINVAL; + + buf[0] = reg; + memcpy(&buf[1], val, len); + + ret = i2c_transfer(client->adapter, &msg, 1); + if (ret == 1) + return 0; + + ret = ret < 0 ? ret : -EIO; + dev_err_ratelimited(&client->dev, + "register 0x%02x write failed: %d\n", reg, ret); + return ret; +} + +static int ucsi_ite_process_event(struct ucsi_ite *ite, u32 *cci) +{ + u8 message_in[ITE_MESSAGE_IN_MAX_LEN] = {}; + __le32 raw_cci; + u8 status; + u8 len = 0; + int event; + int err = 0; + int ret; + + mutex_lock(&ite->event_lock); + + ret = ucsi_ite_read(ite, ITE_REG_INT_STATUS, &status, sizeof(status)); + if (ret) + goto out_unlock; + + status &= ITE_INT_MASK; + if (!status) { + mutex_lock(&ite->received_lock); + *cci = ite->cci; + mutex_unlock(&ite->received_lock); + ret = ITE_EVENT_NONE; + goto out_unlock; + } + + if (status & ITE_INT_CCI) { + err = ucsi_ite_read(ite, ITE_REG_CCI, &raw_cci, + sizeof(raw_cci)); + if (!err) { + *cci = le32_to_cpu(raw_cci); + len = UCSI_CCI_LENGTH(*cci); + + if (len > sizeof(message_in)) { + len = sizeof(message_in); + *cci &= ~GENMASK(15, 8); + *cci |= UCSI_SET_CCI_LENGTH(len); + } + if (len) { + err = ucsi_ite_read(ite, ITE_REG_MESSAGE_IN, + message_in, len); + } + } + } + + /* Acknowledge each latched event with the value expected by the PPM. */ + if (status & ITE_INT_VENDOR_ALERT) { + u8 ack = ITE_INT_VENDOR_ALERT; + + ret = ucsi_ite_write(ite, ITE_REG_INT_ACK, &ack, sizeof(ack)); + if (ret) + goto out_unlock; + } + + if ((status & ITE_INT_CCI) && !err) { + u8 ack = ITE_INT_CCI; + + ret = ucsi_ite_write(ite, ITE_REG_INT_ACK, &ack, sizeof(ack)); + if (ret) + goto out_unlock; + } + + if (err) { + ret = err; + goto out_unlock; + } + + if (status & ITE_INT_CCI) { + mutex_lock(&ite->received_lock); + ite->cci = *cci; + memset(ite->message_in, 0, sizeof(ite->message_in)); + memcpy(ite->message_in, message_in, len); + mutex_unlock(&ite->received_lock); + event = ITE_EVENT_CCI; + } else { + mutex_lock(&ite->received_lock); + *cci = ite->cci; + mutex_unlock(&ite->received_lock); + event = ITE_EVENT_VENDOR; + } + + ret = event; + +out_unlock: + mutex_unlock(&ite->event_lock); + return ret; +} + +static int ucsi_ite_read_version(struct ucsi *ucsi, u16 *version) +{ + /* The ITE interface does not expose a VERSION register. */ + *version = UCSI_VERSION_1_0; + return 0; +} + +static int ucsi_ite_read_cci(struct ucsi *ucsi, u32 *cci) +{ + struct ucsi_ite *ite = ucsi_get_drvdata(ucsi); + + mutex_lock(&ite->received_lock); + *cci = ite->cci; + mutex_unlock(&ite->received_lock); + + return 0; +} + +static int ucsi_ite_poll_cci(struct ucsi *ucsi, u32 *cci) +{ + struct ucsi_ite *ite = ucsi_get_drvdata(ucsi); + int ret; + + ret = ucsi_ite_process_event(ite, cci); + return ret < 0 ? ret : 0; +} + +static int ucsi_ite_read_message_in(struct ucsi *ucsi, void *val, size_t len) +{ + struct ucsi_ite *ite = ucsi_get_drvdata(ucsi); + + if (len > sizeof(ite->message_in)) + return -EINVAL; + + mutex_lock(&ite->received_lock); + memcpy(val, ite->message_in, len); + mutex_unlock(&ite->received_lock); + + return 0; +} + +static int ucsi_ite_async_control(struct ucsi *ucsi, u64 command) +{ + struct ucsi_ite *ite = ucsi_get_drvdata(ucsi); + __le64 raw_command = cpu_to_le64(command); + int ret; + + if (UCSI_COMMAND(command) == UCSI_PPM_RESET) { + /* The PPM handles reset internally; do not write it over I2C. */ + mutex_lock(&ite->event_lock); + mutex_lock(&ite->received_lock); + ite->cci = UCSI_CCI_RESET_COMPLETE; + memset(ite->message_in, 0, sizeof(ite->message_in)); + mutex_unlock(&ite->received_lock); + mutex_unlock(&ite->event_lock); + return 0; + } + + mutex_lock(&ite->event_lock); + mutex_lock(&ite->received_lock); + ite->cci = 0; + memset(ite->message_in, 0, sizeof(ite->message_in)); + mutex_unlock(&ite->received_lock); + ret = ucsi_ite_write(ite, ITE_REG_CONTROL, &raw_command, + sizeof(raw_command)); + mutex_unlock(&ite->event_lock); + + return ret; +} + +static const struct ucsi_operations ucsi_ite_ops = { + .read_version = ucsi_ite_read_version, + .read_cci = ucsi_ite_read_cci, + .poll_cci = ucsi_ite_poll_cci, + .read_message_in = ucsi_ite_read_message_in, + .sync_control = ucsi_sync_control_common, + .async_control = ucsi_ite_async_control, +}; + +static irqreturn_t ucsi_ite_irq(int irq, void *data) +{ + struct ucsi_ite *ite = data; + u32 cci; + int ret; + + ret = ucsi_ite_process_event(ite, &cci); + if (ret == ITE_EVENT_NONE) + return IRQ_NONE; + if (ret < 0) + return IRQ_HANDLED; + + if (ret == ITE_EVENT_CCI) + ucsi_notify_common(ite->ucsi, cci); + + return IRQ_HANDLED; +} + +static void ucsi_ite_destroy(void *data) +{ + struct ucsi_ite *ite = data; + + if (ite->registered) + ucsi_unregister(ite->ucsi); + ucsi_destroy(ite->ucsi); +} + +static int ucsi_ite_probe(struct i2c_client *client) +{ + struct device *dev = &client->dev; + struct ucsi_ite *ite; + u32 cci; + int ret; + + if (client->irq <= 0) + return dev_err_probe(dev, -ENODEV, "no IRQ provided\n"); + if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) + return dev_err_probe(dev, -EOPNOTSUPP, + "adapter does not support I2C transfers\n"); + + ite = devm_kzalloc(dev, sizeof(*ite), GFP_KERNEL); + if (!ite) + return -ENOMEM; + + ite->client = client; + mutex_init(&ite->event_lock); + mutex_init(&ite->received_lock); + i2c_set_clientdata(client, ite); + + ite->ucsi = ucsi_create(dev, &ucsi_ite_ops); + if (IS_ERR(ite->ucsi)) + return dev_err_probe(dev, PTR_ERR(ite->ucsi), + "failed to create UCSI interface\n"); + + ret = devm_add_action_or_reset(dev, ucsi_ite_destroy, ite); + if (ret) + return ret; + + ucsi_set_drvdata(ite->ucsi, ite); + + ret = devm_request_threaded_irq(dev, client->irq, NULL, + ucsi_ite_irq, IRQF_ONESHOT, + dev_name(dev), ite); + if (ret) + return dev_err_probe(dev, ret, "failed to request IRQ\n"); + + ret = ucsi_ite_process_event(ite, &cci); + if (ret < 0) + dev_warn(dev, "initial event processing failed: %d\n", ret); + + ret = ucsi_register(ite->ucsi); + if (ret) + return dev_err_probe(dev, ret, + "failed to register UCSI interface\n"); + + ite->registered = true; + return 0; +} + +static int ucsi_ite_suspend(struct device *dev) +{ + struct ucsi_ite *ite = dev_get_drvdata(dev); + int ret; + + disable_irq(ite->client->irq); + ret = ucsi_suspend(ite->ucsi); + if (ret) + enable_irq(ite->client->irq); + + return ret; +} + +static int ucsi_ite_resume(struct device *dev) +{ + struct ucsi_ite *ite = dev_get_drvdata(dev); + + enable_irq(ite->client->irq); + return ucsi_resume(ite->ucsi); +} + +static DEFINE_SIMPLE_DEV_PM_OPS(ucsi_ite_pm, ucsi_ite_suspend, + ucsi_ite_resume); + +static const struct acpi_device_id ucsi_ite_acpi_ids[] = { + { "ITE8853" }, + { "ITE8800" }, + { "ITE8801" }, + { "ITE8802" }, + { "ITE8803" }, + { "ITE8804" }, + { "ITE8805" }, + { } +}; +MODULE_DEVICE_TABLE(acpi, ucsi_ite_acpi_ids); + +static struct i2c_driver ucsi_ite_driver = { + .driver = { + .name = "ucsi_ite", + .acpi_match_table = ucsi_ite_acpi_ids, + .pm = pm_sleep_ptr(&ucsi_ite_pm), + }, + .probe = ucsi_ite_probe, +}; +module_i2c_driver(ucsi_ite_driver); + +MODULE_AUTHOR("Edward Blair "); +MODULE_DESCRIPTION("UCSI I2C transport driver for ITE885x USB-C controllers"); +MODULE_LICENSE("GPL"); -- 2.55.0