From patchwork Fri Dec 9 16:26:22 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Hasemeyer X-Patchwork-Id: 31871 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp877606wrr; Fri, 9 Dec 2022 08:40:13 -0800 (PST) X-Google-Smtp-Source: AA0mqf7o0+Jg6qgdqMpcHahZwDRjK+hHEqqADgnDnM+U30I47tRluBQih0FVr9vZKkxrlvs7cCyU X-Received: by 2002:a17:902:8c8d:b0:187:262a:7955 with SMTP id t13-20020a1709028c8d00b00187262a7955mr5823417plo.9.1670604013120; Fri, 09 Dec 2022 08:40:13 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1670604013; cv=none; d=google.com; s=arc-20160816; b=iR4rNekQe9PKHbIf8fa72VDSjRluek07rqMBgQJggCMa2B+dvENK1KVDGoVZjioo3L UhPT+PjgMhAjKAA+Xs4v258VvvMZv/PfcHMC647PM9+Vjn9zqacHULOcMUQB6INhcCR9 IIckFg5ItLAENQsMhk0dvixv/4esAL7c5lr7AFNbEEYeWTwZeYIcHd/hL4B/WHW9Bi7k QMhn4vMRfb23WFVhZtoEpFXH2imaQrKaS/28VvWoKFchPis9Jm55iKMqGG2Wc8LWsVrD XiXrpRG20rtVLkMXj0LzjgGQlGttvdrAjXbDRRM4xOLueckh022l0t7TuycoqwzxqMzN EmXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=0EGf/W/buCyYS/wIyyFWevAg0cwWh/UqOr2UJn1iLUs=; b=BCI+x944lgR07A5S0cDxnAo9UQeIUPM+yKzoW/CKGZuh9SHd/Gqgi8woX1EH3k63J0 8IsMwQQ8Ta+hJdHO8jQJkc9RMtpge214O8aWq3BNDz0Z4ifC887z2HZaRICQPCo68mPk M1SD85pghC/DdF7A3j7ucJc5va38hPQ0ycNHYfS/RcWK+/fOEPIhFh+3vPmGatQlIoWd ABAr5kWmBHLXMPUxoJ6vEVvGQmVMVIFFImM8ysNy0iIf84sUUfHu3sp/sedzG01UP26K g8cpklaHbdntSUCpCKQ+kJHLufsGeyubf0XsqXmYI3w644sw2GcMhnBSrBfAv/kuPf6S LCZg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Xn49DeHJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id z9-20020a170902834900b00178380f525bsi1781414pln.547.2022.12.09.08.39.57; Fri, 09 Dec 2022 08:40:13 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Xn49DeHJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229674AbiLIQ0c (ORCPT + 99 others); Fri, 9 Dec 2022 11:26:32 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60772 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229498AbiLIQ0b (ORCPT ); Fri, 9 Dec 2022 11:26:31 -0500 Received: from mail-il1-x132.google.com (mail-il1-x132.google.com [IPv6:2607:f8b0:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 12B2085D06 for ; Fri, 9 Dec 2022 08:26:29 -0800 (PST) Received: by mail-il1-x132.google.com with SMTP id y2so139008ily.5 for ; Fri, 09 Dec 2022 08:26:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=0EGf/W/buCyYS/wIyyFWevAg0cwWh/UqOr2UJn1iLUs=; b=Xn49DeHJeLdCg2ZnMZEoJ1sRTl0Pi1LfmUBihmbU8AWfmJLRTAWx7mvu6zWWhhz/zT 6ruuMnX8nhpVyusHdkM56a3KnAlrtbOeRkPfvZVjmudRTr3FJyPtTDUqgOif/POmS4aN +27inixQ6HSYCuotRf02WxwpCoKeVjIetC3wI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=0EGf/W/buCyYS/wIyyFWevAg0cwWh/UqOr2UJn1iLUs=; b=l/smf9v2TBP9rmUFKuxvDKl1sLJTzTYB6u9Kw0Mf4ocOSaDnjzT6o+VnO3iBtKhTUw 3NgkZM4JUzKPvfBhBtSG1vDK/lWSPkaz709AVNHYHDImgNuLAXp72TsVbuquhyrzFiTy j007qqHAw0roFKI9NE06lcCR2nUhB/I7i6zxwVg9wO242t+tIrt9M802kdI8tPREZv/a irAS9nLTxOdyWFTEwcqLcGNESbZAHoeIk5sFKwan7Vl+7YBiQH8Zb4D/Xx3oyG7DmEvW yIJkiPUG7Erzp8z3p496pbkq4Vc2Sh4hgcH80OShmgRFVxvI3sbPZaVHFraloB7ZnZUp 3EeQ== X-Gm-Message-State: ANoB5plkGhbh1WOiq8pFqrCnoUGzbs2M5hiAkWHxKurSxD5+gjpEDYth 8nFJFGzjidyzL+Pdyd6O7/kdOy3C4AYv8mKj X-Received: by 2002:a05:6e02:1a26:b0:303:5f4c:a8f8 with SMTP id g6-20020a056e021a2600b003035f4ca8f8mr4934150ile.29.1670603187894; Fri, 09 Dec 2022 08:26:27 -0800 (PST) Received: from markhas1.roam.corp.google.com (63-225-246-100.hlrn.qwest.net. [63.225.246.100]) by smtp.gmail.com with ESMTPSA id b2-20020a92dcc2000000b002e85e8b8d1dsm556531ilr.5.2022.12.09.08.26.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Dec 2022 08:26:27 -0800 (PST) From: Mark Hasemeyer To: LKML Cc: Raul Rangel , Bhanu Prakash Maiya , kernel test robot , Mark Hasemeyer , Prashant Malani , Benson Leung , Guenter Roeck , Tzung-Bi Shih , chrome-platform@lists.linux.dev Subject: [PATCH v12 1/3] platform/chrome: cros_ec_uart: Add cros-ec-uart transport layer Date: Fri, 9 Dec 2022 09:26:22 -0700 Message-Id: <20221209092619.v12.1.If7926fcbad397bc6990dd725690229bed403948c@changeid> X-Mailer: git-send-email 2.39.0.rc1.256.g54fd8350bd-goog MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1751755273899844524?= X-GMAIL-MSGID: =?utf-8?q?1751755273899844524?= From: Bhanu Prakash Maiya This patch does following: 1. Adds a new cros-ec-uart driver. This driver can send EC requests on UART and process response packets received on UART transport. 2. Once probed, this driver will initialize the serdev device based on the underlying information in the ACPI resource. After serdev device properties are set, this driver will register itself cros-ec. 3. High level driver can use this implementation to talk to ChromeOS Embedded Controller device in case it supports UART as transport. 4. When cros-ec driver initiates a request packet, outgoing message is processed in buffer and sent via serdev. Once bytes are sent, driver enables a wait_queue. 5. Since ChromeOS EC device sends response asynchronously, AP's TTY driver accumulates response bytes and calls the registered callback. TTY driver can send multiple callback for bytes ranging from 1 to MAX bytes supported by EC device. 6. Driver waits for EC_MSG_DEADLINE_MS to collect and process received bytes. It wakes wait_queue if expected bytes are received or else wait_queue timeout. Based on the error condition, driver returns data_len or error to cros_ec. Reported-by: kernel test robot Signed-off-by: Bhanu Prakash Maiya Co-developed-by: Mark Hasemeyer Signed-off-by: Mark Hasemeyer Reviewed-by: Prashant Malani --- Changes in v12: -No change Changes in v11: - Clarify error status in documentation - Join more multi-line statments onto a single line if < 100 chars - Add newline to the end of some printks - Terminate acpi_device_id with a NULL entry - Fix serdev doc warning for cros_ec_uart struct Changes in v10: - Remove superfluous comments - Remove unneeded #include - Use uninterruptible variant of wait_event - Merge error and received variables into a single status var - White space alignment - Add warning message when byes are received out of band - Join some multi-line statments onto a single line if < 100 chars Changes in v9: - Rebase onto for-next - Update contact information in MAINTAINERS - Remove superfluous comments - Invert conditional logic checking response size - Update authorship - Convert single case switch statement to if conditional - Reorder initializer list in increasing order of line length - Use wait_event_timeout return value to detect timeout - Remove unnecessary zeroizing of ec_uart struct - Set received conditional on -EMSGSIZE error case - Join dev_err msg onto single line where possible Changes in v8: - Combine for loops used for xsum calculation - Use ec_dev->dev instead of serdev->dev - Sort includes - Swap struct var order to match doc order - fix whitespace - Update contact information in MAINTAINERS - Update copyright date Changes in v7: - Use ACPI ID matching - Change MODULE_LICENSE from GPL v2 to GPL Changes in v6: - Tuning EC_MSG_DEADLINE_MS from 1 sec to 500 msec. Since the EC's UART response buffer was tuned and redeuced to 256 byte, 500 msec timeout should give enough time for 256 bytes to be recieved from SoC UART buffer. Changes in v5: - Fixed checkpatch.pl warnings. Changes in v4: - Change EC_MSG_DEADLINE_MS from 3 secs to 1 sec. 3 secs timeout prevents driver from intercepting EC in RO stage. In case of request failure, 1 sec allows driver to retry and send HC when EC is in RO. - Add more information in commit message. Changes in v3: - checkpatch.pl script warns about char len 80 even though we have relaxed view guideline for line below 100 chars. Currently sticking with 80 chars in v3. - Fixed style issues Changes in v2: - Fixed build error on v1. - Changed EC timeout for response packet to 3 Sec and added comments. - Fixed cros_ec_uart_rx_bytes function to handle rx buffer < size of response header. --- MAINTAINERS | 7 + drivers/platform/chrome/Kconfig | 10 + drivers/platform/chrome/Makefile | 1 + drivers/platform/chrome/cros_ec_uart.c | 352 +++++++++++++++++++++++++ 4 files changed, 370 insertions(+) create mode 100644 drivers/platform/chrome/cros_ec_uart.c diff --git a/MAINTAINERS b/MAINTAINERS index 4002d61291ca7..1b54bd6f28328 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4913,6 +4913,13 @@ S: Maintained F: Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml F: sound/soc/codecs/cros_ec_codec.* +CHROMEOS EC UART DRIVER +M: Bhanu Prakash Maiya +R: Benson Leung +R: Tzung-Bi Shih +S: Maintained +F: drivers/platform/chrome/cros_ec_uart.c + CHROMEOS EC SUBDRIVERS M: Benson Leung R: Guenter Roeck diff --git a/drivers/platform/chrome/Kconfig b/drivers/platform/chrome/Kconfig index c1ca247987d2d..0d272c87f32b2 100644 --- a/drivers/platform/chrome/Kconfig +++ b/drivers/platform/chrome/Kconfig @@ -119,6 +119,16 @@ config CROS_EC_SPI response time cannot be guaranteed, we support ignoring 'pre-amble' bytes before the response actually starts. +config CROS_EC_UART + tristate "ChromeOS Embedded Controller (UART)" + depends on CROS_EC && ACPI && SERIAL_DEV_BUS + help + If you say Y here, you get support for talking to the ChromeOS EC + through a UART, using a byte-level protocol. + + To compile this driver as a module, choose M here: the + module will be called cros_ec_uart. + config CROS_EC_LPC tristate "ChromeOS Embedded Controller (LPC)" depends on CROS_EC && ACPI && (X86 || COMPILE_TEST) diff --git a/drivers/platform/chrome/Makefile b/drivers/platform/chrome/Makefile index f6068d077a40a..0f39edef660dd 100644 --- a/drivers/platform/chrome/Makefile +++ b/drivers/platform/chrome/Makefile @@ -15,6 +15,7 @@ obj-$(CONFIG_CROS_EC_ISHTP) += cros_ec_ishtp.o obj-$(CONFIG_CROS_TYPEC_SWITCH) += cros_typec_switch.o obj-$(CONFIG_CROS_EC_RPMSG) += cros_ec_rpmsg.o obj-$(CONFIG_CROS_EC_SPI) += cros_ec_spi.o +obj-$(CONFIG_CROS_EC_UART) += cros_ec_uart.o cros_ec_lpcs-objs := cros_ec_lpc.o cros_ec_lpc_mec.o obj-$(CONFIG_CROS_EC_TYPEC) += cros_ec_typec.o obj-$(CONFIG_CROS_EC_LPC) += cros_ec_lpcs.o diff --git a/drivers/platform/chrome/cros_ec_uart.c b/drivers/platform/chrome/cros_ec_uart.c new file mode 100644 index 0000000000000..43a7493347a59 --- /dev/null +++ b/drivers/platform/chrome/cros_ec_uart.c @@ -0,0 +1,352 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * UART interface for ChromeOS Embedded Controller + * + * Copyright 2020-2022 Google LLC. + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include "cros_ec.h" + +/* + * EC sends contiguous bytes of response packet on UART AP RX. + * TTY driver in AP accumulates incoming bytes and calls the registered callback + * function. Byte count can range from 1 to MAX bytes supported by EC. + * This driver should wait for long time for all callbacks to be processed. + * Considering the worst case scenario, wait for 500 msec. This timeout should + * account for max latency and some additional guard time. + * Best case: Entire packet is received in ~200 ms, wait queue will be released + * and packet will be processed. + * Worst case: TTY driver sends bytes in multiple callbacks. In this case this + * driver will wait for ~1 sec beyond which it will timeout. + * This timeout value should not exceed ~500 msec because in case if + * EC_CMD_REBOOT_EC sent, high level driver should be able to intercept EC + * in RO. + */ +#define EC_MSG_DEADLINE_MS 500 + +/** + * struct response_info - Encapsulate EC response related + * information for passing between function + * cros_ec_uart_pkt_xfer() and cros_ec_uart_rx_bytes() + * callback. + * @data: Copy the data received from EC here. + * @max_size: Max size allocated for the @data buffer. If the + * received data exceeds this value, we log an error. + * @size: Actual size of data received from EC. This is also + * used to accumulate byte count with response is received + * in dma chunks. + * @exp_len: Expected bytes of response from EC including header. + * @status: Re-init to 0 before sending a cmd. Updated to 1 when + * a response is successfully received, or an error number + * on failure. + * @wait_queue: Wait queue EC response where the cros_ec sends request + * to EC and waits + */ +struct response_info { + void *data; + size_t max_size; + size_t size; + size_t exp_len; + int status; + wait_queue_head_t wait_queue; +}; + +/** + * struct cros_ec_uart - information about a uart-connected EC + * + * @serdev: serdev uart device we are connected to. + * @baudrate: UART baudrate of attached EC device. + * @flowcontrol: UART flowcontrol of attached device. + * @irq: Linux IRQ number of associated serial device. + * @response: Response info passing between cros_ec_uart_pkt_xfer() + * and cros_ec_uart_rx_bytes() + */ +struct cros_ec_uart { + struct serdev_device *serdev; + u32 baudrate; + u8 flowcontrol; + u32 irq; + struct response_info response; +}; + +static int cros_ec_uart_rx_bytes(struct serdev_device *serdev, + const u8 *data, + size_t count) +{ + struct ec_host_response *response; + struct cros_ec_device *ec_dev = serdev_device_get_drvdata(serdev); + struct cros_ec_uart *ec_uart = ec_dev->priv; + + /* Check if bytes were sent out of band */ + if (!ec_uart->response.data) { + /* Discard all bytes */ + dev_warn(ec_dev->dev, "Bytes received out of band, dropping them.\n"); + return count; + } + + /* + * Check if incoming bytes + response.size is greater than allocated + * buffer in din by cros_ec. This will ensure that if EC sends more + * bytes than max_size, waiting process will be notified with an error. + */ + if (ec_uart->response.size + count > ec_uart->response.max_size) { + ec_uart->response.status = -EMSGSIZE; + wake_up(&ec_uart->response.wait_queue); + return count; + } + + memcpy(ec_uart->response.data + ec_uart->response.size, data, count); + + ec_uart->response.size += count; + + /* Read data_len if we received response header and if exp_len was not read before. */ + if (ec_uart->response.size >= sizeof(*response) && + ec_uart->response.exp_len == 0) { + response = (struct ec_host_response *) ec_uart->response.data; + ec_uart->response.exp_len = response->data_len + sizeof(*response); + } + + /* If driver received response header and payload from EC, wake up the wait queue. */ + if (ec_uart->response.size >= sizeof(*response) && + ec_uart->response.size == ec_uart->response.exp_len) { + ec_uart->response.status = 1; + wake_up(&ec_uart->response.wait_queue); + } + + return count; +} + +static int cros_ec_uart_pkt_xfer(struct cros_ec_device *ec_dev, + struct cros_ec_command *ec_msg) +{ + struct cros_ec_uart *ec_uart = ec_dev->priv; + struct serdev_device *serdev = ec_uart->serdev; + struct ec_host_response *response; + unsigned int len; + int ret, i; + u8 sum = 0; + + len = cros_ec_prepare_tx(ec_dev, ec_msg); + dev_dbg(ec_dev->dev, "Prepared len=%d\n", len); + + /* Setup for incoming response */ + ec_uart->response.data = ec_dev->din; + ec_uart->response.max_size = ec_dev->din_size; + ec_uart->response.size = 0; + ec_uart->response.exp_len = 0; + ec_uart->response.status = 0; + + ret = serdev_device_write_buf(serdev, ec_dev->dout, len); + if (ret < len) { + dev_err(ec_dev->dev, "Unable to write data\n"); + ret = -EIO; + goto exit; + } + + ret = wait_event_timeout(ec_uart->response.wait_queue, + ec_uart->response.status, + msecs_to_jiffies(EC_MSG_DEADLINE_MS)); + if (ret == 0) { + dev_warn(ec_dev->dev, "Timed out waiting for response.\n"); + ret = -ETIMEDOUT; + goto exit; + } + + if (ec_uart->response.status < 0) { + dev_warn(ec_dev->dev, "Error response received: %d\n", ec_uart->response.status); + ret = ec_uart->response.status; + goto exit; + } + + response = (struct ec_host_response *)ec_dev->din; + ec_msg->result = response->result; + + if (response->data_len > ec_msg->insize) { + dev_err(ec_dev->dev, "Resp too long (%d bytes, expected %d)\n", response->data_len, + ec_msg->insize); + ret = -ENOSPC; + goto exit; + } + + /* Validate checksum */ + for (i = 0; i < sizeof(*response) + response->data_len; i++) + sum += ec_dev->din[i]; + + if (sum) { + dev_err(ec_dev->dev, "Bad packet checksum calculated %x\n", sum); + ret = -EBADMSG; + goto exit; + } + + memcpy(ec_msg->data, ec_dev->din + sizeof(*response), response->data_len); + + ret = response->data_len; + +exit: + /* Invalidate response buffer to guard against out of band rx data */ + ec_uart->response.data = NULL; + + if (ec_msg->command == EC_CMD_REBOOT_EC) + msleep(EC_REBOOT_DELAY_MS); + + return ret; +} + +static int cros_ec_uart_resource(struct acpi_resource *ares, void *data) +{ + struct cros_ec_uart *ec_uart = data; + struct acpi_resource_uart_serialbus *sb = &ares->data.uart_serial_bus; + + if (ares->type == ACPI_RESOURCE_TYPE_SERIAL_BUS && + sb->type == ACPI_RESOURCE_SERIAL_TYPE_UART) { + ec_uart->baudrate = sb->default_baud_rate; + dev_dbg(&ec_uart->serdev->dev, "Baudrate %d\n", ec_uart->baudrate); + + ec_uart->flowcontrol = sb->flow_control; + dev_dbg(&ec_uart->serdev->dev, "Flow control %d\n", ec_uart->flowcontrol); + } + + return 0; +} + +static int cros_ec_uart_acpi_probe(struct cros_ec_uart *ec_uart) +{ + int ret; + LIST_HEAD(resources); + struct acpi_device *adev = ACPI_COMPANION(&ec_uart->serdev->dev); + + ret = acpi_dev_get_resources(adev, &resources, cros_ec_uart_resource, ec_uart); + if (ret < 0) + return ret; + + acpi_dev_free_resource_list(&resources); + + /* Retrieve GpioInt and translate it to Linux IRQ number */ + ret = acpi_dev_gpio_irq_get(adev, 0); + if (ret < 0) + return ret; + + ec_uart->irq = ret; + dev_dbg(&ec_uart->serdev->dev, "IRQ number %d\n", ec_uart->irq); + + return 0; +} + +static const struct serdev_device_ops cros_ec_uart_client_ops = { + .receive_buf = cros_ec_uart_rx_bytes, +}; + +static int cros_ec_uart_probe(struct serdev_device *serdev) +{ + struct device *dev = &serdev->dev; + struct cros_ec_device *ec_dev; + struct cros_ec_uart *ec_uart; + int ret; + + ec_uart = devm_kzalloc(dev, sizeof(*ec_uart), GFP_KERNEL); + if (!ec_uart) + return -ENOMEM; + + ec_dev = devm_kzalloc(dev, sizeof(*ec_dev), GFP_KERNEL); + if (!ec_dev) + return -ENOMEM; + + ec_uart->serdev = serdev; + + ret = devm_serdev_device_open(dev, ec_uart->serdev); + if (ret) { + dev_err(dev, "Unable to open UART device"); + return ret; + } + + serdev_device_set_drvdata(serdev, ec_dev); + serdev_device_set_client_ops(serdev, &cros_ec_uart_client_ops); + init_waitqueue_head(&ec_uart->response.wait_queue); + + ret = cros_ec_uart_acpi_probe(ec_uart); + if (ret < 0) { + dev_err(dev, "Failed to get ACPI info (%d)", ret); + return ret; + } + + ret = serdev_device_set_baudrate(serdev, ec_uart->baudrate); + if (ret < 0) { + dev_err(dev, "Failed to set up host baud rate (%d)", ret); + return ret; + } + + serdev_device_set_flow_control(serdev, ec_uart->flowcontrol); + + /* Initialize ec_dev for cros_ec */ + ec_dev->phys_name = dev_name(dev); + ec_dev->dev = dev; + ec_dev->priv = ec_uart; + ec_dev->irq = ec_uart->irq; + ec_dev->cmd_xfer = NULL; + ec_dev->pkt_xfer = cros_ec_uart_pkt_xfer; + ec_dev->din_size = sizeof(struct ec_host_response) + + sizeof(struct ec_response_get_protocol_info); + ec_dev->dout_size = sizeof(struct ec_host_request); + + return cros_ec_register(ec_dev); +} + +static void cros_ec_uart_remove(struct serdev_device *serdev) +{ + struct cros_ec_device *ec_dev = serdev_device_get_drvdata(serdev); + + cros_ec_unregister(ec_dev); +}; + +static int __maybe_unused cros_ec_uart_suspend(struct device *dev) +{ + struct cros_ec_device *ec_dev = dev_get_drvdata(dev); + + return cros_ec_suspend(ec_dev); +} + +static int __maybe_unused cros_ec_uart_resume(struct device *dev) +{ + struct cros_ec_device *ec_dev = dev_get_drvdata(dev); + + return cros_ec_resume(ec_dev); +} + +static SIMPLE_DEV_PM_OPS(cros_ec_uart_pm_ops, cros_ec_uart_suspend, + cros_ec_uart_resume); + +#ifdef CONFIG_ACPI +static const struct acpi_device_id cros_ec_uart_acpi_id[] = { + { "GOOG0019", 0 }, + {} +}; + +MODULE_DEVICE_TABLE(acpi, cros_ec_uart_acpi_id); +#endif + +static struct serdev_device_driver cros_ec_uart_driver = { + .driver = { + .name = "cros-ec-uart", + .acpi_match_table = ACPI_PTR(cros_ec_uart_acpi_id), + .pm = &cros_ec_uart_pm_ops, + }, + .probe = cros_ec_uart_probe, + .remove = cros_ec_uart_remove, +}; + +module_serdev_device_driver(cros_ec_uart_driver); + +MODULE_LICENSE("GPL"); +MODULE_DESCRIPTION("UART interface for ChromeOS Embedded Controller"); +MODULE_AUTHOR("Bhanu Prakash Maiya "); From patchwork Fri Dec 9 16:26:23 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Hasemeyer X-Patchwork-Id: 31870 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp874908wrr; Fri, 9 Dec 2022 08:33:50 -0800 (PST) X-Google-Smtp-Source: AA0mqf7EPZ9lKukD2AGtEUCumHv4lfDmFHOHOxw0UPG6gLnJEo5Ie3T7yth4Eyco6injBKVr5/qi X-Received: by 2002:a17:906:b14b:b0:78d:f459:719e with SMTP id bt11-20020a170906b14b00b0078df459719emr8405382ejb.73.1670603629949; Fri, 09 Dec 2022 08:33:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1670603629; cv=none; d=google.com; s=arc-20160816; b=NismTlEVCPg9Jp91Qf1M72nOGIII8O9x+XQsnAGe3/G9xVO7QNkbwRiKNK2jTGdsZt yuiIcZaTOSEAzMiQXCa4ITHR4eBjJsFbFW3+tVstcFPCGoED7Vfi8XpwePxJzX0ozKhm rQ0Ak/Sdd3SME2RQMdqPv6bz1g8y2Lb6xbZLSgpF6XKtWijXGZCUUeXLEqWw4bv+6Aeh KbtnC0DpTh5ZxEwsg5LZ8V+vZAva7PseDBUN7GISRm9A0bqE11G3oxyw+aGPZxegKpe0 fVILJgHEzW+KVuPq8ai+kTc/BU1HjRytPyNL1MyWuUHo6d9LHPw9SN5+hZseL0VVHX9T 44XQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=IgG/TNRbidgEyJvAVaHHfxRU0Ng4OyfVihTTmLvE0JQ=; b=TwZ50a7pHSvGXSTloLB3dIWOZBid9bMkBFejYpuXBxo5fJ4v+JG2sqHDOLfL0h0fJK +VPADZxoJOpGR+yRQMREc09Z/5nf8DbYjZ4rtg6yJFVr6r7+BKLCW1zwkGHcxBMo0irT JxNHqpzzJ1SraWV/on1G140eyV/bv9TbzFYHgmKgh2DFKspHENsJSnIljRTXuJQ7IOIQ MBLDYwcvxsQU6q2Nt44NFPiKkZWq1cP3W1or/wbXoCew3g+5Ti7zBBJJuUKvr/blrm8M aR6vVJnkdEWzcEreE5WPyWJn7WxsCUgX/zB1mkBZOUjej/C0pITbL/z/1d7hJf5Hm/r2 sONA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=K2COfFDP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id mp8-20020a1709071b0800b007aeab6a1174si182491ejc.633.2022.12.09.08.33.26; Fri, 09 Dec 2022 08:33:49 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=K2COfFDP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229724AbiLIQ0e (ORCPT + 99 others); Fri, 9 Dec 2022 11:26:34 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60778 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229538AbiLIQ0b (ORCPT ); Fri, 9 Dec 2022 11:26:31 -0500 Received: from mail-il1-x129.google.com (mail-il1-x129.google.com [IPv6:2607:f8b0:4864:20::129]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 803BF86F4D for ; Fri, 9 Dec 2022 08:26:29 -0800 (PST) Received: by mail-il1-x129.google.com with SMTP id m15so145098ilq.2 for ; Fri, 09 Dec 2022 08:26:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; 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; bh=IgG/TNRbidgEyJvAVaHHfxRU0Ng4OyfVihTTmLvE0JQ=; b=K2COfFDPnQsv126b/kS1Mvllrw3QIpmb9WDCa0cAUFybrJ8VpQtraDrw/XddNXS3hE CGUa7htqAGx3XqePpeRi597iv9RWO6TM8JJ7TLnEN80eCZmZTxPMay9cq7dJzDL1AqnF oS2lECZHiRtfUgaLT2+k0OKBqtzpeIlqzUgn0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=IgG/TNRbidgEyJvAVaHHfxRU0Ng4OyfVihTTmLvE0JQ=; b=JyNul8+3gz0jAPAmDTtOo6YnFIHhv3wipUK14QBSZBYCoVuKAzq3zUvFE9QMpLtCE9 rXm/9mUC8OyX8THSqwtZbyMN+PQGpf/gkw0UEjge8O6yOK3ZvE0QQBJuj7NdW3Sss8HY QVyBcfmHl8sRHT8pCkq0N2b+rJ2oMqj24rT36Xf0kGZyU98BBq9WyatRaqK5vBcdixRh WC3orIMBAyv0UqKH597nVpf3jn1hTW6PLAn4I0o+PPYwW5060eVsG/5yK4FxiWEIZscy v6H6uZ155YUk32Pv0px444i/wgfja3qArxX9j2IpEeVhR4ZaOin2sKJTTStA8ktOTEaG +9xQ== X-Gm-Message-State: ANoB5plr8KqvgPWmSwh5Ko878tOoeE+vsws6SU9LIuytTdiZp3HL8n2S nVXZgzIyl9zJ+CJU7ioflgC2cEGNAWMsMfcJ X-Received: by 2002:a92:d489:0:b0:303:8cff:9841 with SMTP id p9-20020a92d489000000b003038cff9841mr3381027ilg.26.1670603188775; Fri, 09 Dec 2022 08:26:28 -0800 (PST) Received: from markhas1.roam.corp.google.com (63-225-246-100.hlrn.qwest.net. [63.225.246.100]) by smtp.gmail.com with ESMTPSA id b2-20020a92dcc2000000b002e85e8b8d1dsm556531ilr.5.2022.12.09.08.26.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Dec 2022 08:26:28 -0800 (PST) From: Mark Hasemeyer To: LKML Cc: Raul Rangel , Bhanu Prakash Maiya , Mark Hasemeyer , Krzysztof Kozlowski , Benson Leung , Guenter Roeck , Krzysztof Kozlowski , Lee Jones , Rob Herring , chrome-platform@lists.linux.dev, devicetree@vger.kernel.org Subject: [PATCH v12 2/3] dt-bindings: mfd: cros-ec: Add UART compatible string Date: Fri, 9 Dec 2022 09:26:23 -0700 Message-Id: <20221209092619.v12.2.I9e018ecb8bdf341648cb64417085978ff0d22a46@changeid> X-Mailer: git-send-email 2.39.0.rc1.256.g54fd8350bd-goog In-Reply-To: <20221209092619.v12.1.If7926fcbad397bc6990dd725690229bed403948c@changeid> References: <20221209092619.v12.1.If7926fcbad397bc6990dd725690229bed403948c@changeid> MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1751754871724918406?= X-GMAIL-MSGID: =?utf-8?q?1751754871724918406?= From: Bhanu Prakash Maiya Add a compatible string to support the UART implementation of the cros ec interface. The driver does not support the reg and interrupt properties, so exempt them from being required for UART compatible nodes. Signed-off-by: Bhanu Prakash Maiya Co-developed-by: Mark Hasemeyer Signed-off-by: Mark Hasemeyer Acked-by: Krzysztof Kozlowski --- Changes in v12: - No change Changes in v11: - Update commit message to include device Changes in v10: - No change Changes in v9: - Rebase onto for-next - Update node-name from uart0 to serial - Remove reg and interrupt property requirement for UART compatible nodes. - Update authorship Changes in v8: - Update commit message Changes in v7: - No change Changes in v6: - No change Changes in v5: - No change Changes in v4: - Changes in commit message. Changes in v3: - Rebased changes on google,cros-ec.yaml Changes in v2: - No change --- .../bindings/mfd/google,cros-ec.yaml | 20 +++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml index 3d5efa5578d1b..cdf1d719efe9d 100644 --- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml +++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml @@ -33,6 +33,9 @@ properties: - description: For implementations of the EC connected through RPMSG. const: google,cros-ec-rpmsg + - description: + For implementations of the EC connected through UART. + const: google,cros-ec-uart controller-data: true @@ -187,6 +190,15 @@ allOf: properties: mediatek,rpmsg-name: false + - if: + properties: + compatible: + not: + contains: + enum: + - google,cros-ec-rpmsg + - google,cros-ec-uart + then: required: - reg - interrupts @@ -299,4 +311,12 @@ examples: vdd-supply = <&pp3300_fp_mcu>; }; }; + + # Example for UART + - | + serial { + cros-ec { + compatible = "google,cros-ec-uart"; + }; + }; ... From patchwork Fri Dec 9 16:26:24 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Hasemeyer X-Patchwork-Id: 31869 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp872641wrr; Fri, 9 Dec 2022 08:29:46 -0800 (PST) X-Google-Smtp-Source: AA0mqf6gzK6KS0u7mRd3o4Isostt4X3e96Iv13phjG9qV7FB6D3ZcIPL7HitZ4ovZ5urh0Pxngpj X-Received: by 2002:a05:6a20:8f04:b0:a4:efdd:a9c0 with SMTP id b4-20020a056a208f0400b000a4efdda9c0mr11071666pzk.14.1670603385745; Fri, 09 Dec 2022 08:29:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1670603385; cv=none; d=google.com; s=arc-20160816; b=S2aGbBaZcNZWC8KpOk2ByEEz2bQV2MDqAyUD9nYPaFE+pFVa9x4MPmssjiYTLJ2lAp WWjOVCaakved24elUv18ZumM8ritM7if/1VjGSL+bkJPzUKBZX7wrK/LbRlxicvIhpec JazuReQb6ndWmAc58Uhu7d0eFMy3KjrKmA5KKIMaK/2gPEnkxQcmhF4u7/iu+P0uYLlk ki8CaVww8rib9y8T+2QXi9lnVS7eq/s7bDtzbMLv2iwmZpVw+sX4VBoJrOOoOZ159xRm wo1hNVgkqI5BpMqp/c6HrFtifjlrugWVwZgXeEkk1Q+I/AXVvU8mvurOEwkqt8d7iaLk gGdw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=LET7+BVLD7u2mz2p/Z3V+N5nYBkLwsfyiST0eTO7hzc=; b=zQjN5vfjfo1MFObYJXCyO4VoMX65xGsyUiqqBjGtRbUpqZsjxCnloth1I2RnBVN523 6GmTXF2lDCKHTDAHQRntRhLB8uh12h2NFuFL9UHUYxkGP5LMD/DP2Y1nCLLga9DBCakY 8hUcDBh3w/ixjXgUQnBGG8Ln1mBqFZ4/t9KSp5PPVrAlJD77y6pL+8sI7PCPf7nkFL7J EIUFB1Oo0BOIWsE5AslbvCqMLOq6BnU3sQgAZ2dSXvKsAOl1o6SExORQdD9b8vZwqfrU bTy8mE8b1iHr809AXBEJWM8/TeCTc0UFoT3Ph/OukaxSCTsAf6LI/DwgWNStcPNqbQPy u0dw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=NDNMKLLD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id d24-20020a17090ab31800b00213193e7047si1958071pjr.185.2022.12.09.08.29.24; Fri, 09 Dec 2022 08:29:45 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=NDNMKLLD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229759AbiLIQ0j (ORCPT + 99 others); Fri, 9 Dec 2022 11:26:39 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60780 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229556AbiLIQ0b (ORCPT ); Fri, 9 Dec 2022 11:26:31 -0500 Received: from mail-il1-x136.google.com (mail-il1-x136.google.com [IPv6:2607:f8b0:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5F32486F5A for ; Fri, 9 Dec 2022 08:26:30 -0800 (PST) Received: by mail-il1-x136.google.com with SMTP id x12so147225ilg.1 for ; Fri, 09 Dec 2022 08:26:30 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; 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; bh=LET7+BVLD7u2mz2p/Z3V+N5nYBkLwsfyiST0eTO7hzc=; b=NDNMKLLDBAWf3MbF+6zPgovX77wTaVAyPMqg01VPkudPh14T1CwOao5Z19mx2+2xV7 FTo99ZqyJQ83ny1hGx7ubIC4VplT5IlyRzD4g+UjgtMNKneXVSelvMsYF6GYqUyu48hU iJz/SJ0ywZgJyvxcUzTBEYtGf8wooFhu7IDSs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=LET7+BVLD7u2mz2p/Z3V+N5nYBkLwsfyiST0eTO7hzc=; b=esXlpaABUe7RF1X83D+ElMxZtYkk/vvXYMo7VJebrTcKImzYVqsSGZ4Ao3YQZ3Bsw0 BYWEWPCQlPZ46eTOh3F2NiDtChKy2Hc0sgEsMZPB5OtMM/t1fKhBB0tgMOouctBjFz0l h1/6j0/RrLhi63wVubwqngb7A/FwN5AlhMntztrnpYLNogzw9JYUFUkqp1KUHAFhcdt9 w8KC5kWFCdNMBJecXFSOZWNQuLxRNUOQlh6rN0VhuzuJxGV++4op2livaos7P5I70rWU 7tfuQo6qMPft4Pz4TDTBHbPvRKRj9hWw2JeYsj7CHdUfdrqNDu2EKAWm/M4rEganwsjp nkgA== X-Gm-Message-State: ANoB5pneL7kHta1Y98UTF6k4b6h/5olPpw7aPx+BDsUVsa9k3Q1YHhiL ZM7u6jnwEqp1Pn1gqDJ56GUikgjPt5p19HhT X-Received: by 2002:a92:d9ce:0:b0:303:2f20:1a05 with SMTP id n14-20020a92d9ce000000b003032f201a05mr3535629ilq.24.1670603189613; Fri, 09 Dec 2022 08:26:29 -0800 (PST) Received: from markhas1.roam.corp.google.com (63-225-246-100.hlrn.qwest.net. [63.225.246.100]) by smtp.gmail.com with ESMTPSA id b2-20020a92dcc2000000b002e85e8b8d1dsm556531ilr.5.2022.12.09.08.26.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Dec 2022 08:26:29 -0800 (PST) From: Mark Hasemeyer To: LKML Cc: Raul Rangel , Bhanu Prakash Maiya , Mark Hasemeyer , Prashant Malani , Benson Leung , Guenter Roeck , Tzung-Bi Shih , chrome-platform@lists.linux.dev Subject: [PATCH v12 3/3] platform/chrome: cros_ec_uart: Add DT enumeration support Date: Fri, 9 Dec 2022 09:26:24 -0700 Message-Id: <20221209092619.v12.3.Ie23c217d69ff25d7354db942613f143bbc8ef891@changeid> X-Mailer: git-send-email 2.39.0.rc1.256.g54fd8350bd-goog In-Reply-To: <20221209092619.v12.1.If7926fcbad397bc6990dd725690229bed403948c@changeid> References: <20221209092619.v12.1.If7926fcbad397bc6990dd725690229bed403948c@changeid> MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1751754615571958828?= X-GMAIL-MSGID: =?utf-8?q?1751754615571958828?= From: Bhanu Prakash Maiya Existing firmware uses the "PRP0001" _HID and an associated compatible string to enumerate the cros_ec_uart. Add DT enumeration support for already shipped firmware. Signed-off-by: Bhanu Prakash Maiya Co-developed-by: Mark Hasemeyer Signed-off-by: Mark Hasemeyer Reviewed-by: Prashant Malani --- Changes in v12: - Add Reviewed-by: Prashant Malani Changes in v11: - No change Changes in v10: - No change Changes in v9: - Rebase onto for-next - Update authorship Changes in v8: - No change Changes in v7: - Move PRP0001 enumeration support to its own commit drivers/platform/chrome/cros_ec_uart.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/drivers/platform/chrome/cros_ec_uart.c b/drivers/platform/chrome/cros_ec_uart.c index 43a7493347a59..bf0c291914725 100644 --- a/drivers/platform/chrome/cros_ec_uart.c +++ b/drivers/platform/chrome/cros_ec_uart.c @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -326,6 +327,12 @@ static int __maybe_unused cros_ec_uart_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(cros_ec_uart_pm_ops, cros_ec_uart_suspend, cros_ec_uart_resume); +static const struct of_device_id cros_ec_uart_of_match[] = { + { .compatible = "google,cros-ec-uart" }, + {} +}; +MODULE_DEVICE_TABLE(of, cros_ec_uart_of_match); + #ifdef CONFIG_ACPI static const struct acpi_device_id cros_ec_uart_acpi_id[] = { { "GOOG0019", 0 }, @@ -339,6 +346,7 @@ static struct serdev_device_driver cros_ec_uart_driver = { .driver = { .name = "cros-ec-uart", .acpi_match_table = ACPI_PTR(cros_ec_uart_acpi_id), + .of_match_table = cros_ec_uart_of_match, .pm = &cros_ec_uart_pm_ops, }, .probe = cros_ec_uart_probe,