From nobody Mon Feb 9 11:08:19 2026 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 6C2F51B3F20 for ; Wed, 14 Aug 2024 15:39:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649997; cv=none; b=cPs/bZYmnIKkgssPnijd4fX6NNBC7sG4qmZkePRzRZXlo0yEjm1aNnCtHyR3muSwasNI9hCTYBuz6HtBGxajBGoXh0lEhJyyD13jUSSIetSWkE0wg3BtiGgY+uHWEZ2ZjopMLvuNU0HlgI+nCbDvHFHWiFl7MdZQOgwlfcJ0hmQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649997; c=relaxed/simple; bh=k0IWkc/XYUwY9mbwNF4Mom1JEa3IN6gBW+n97jZ0ZXI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=HPNGmuicx8SYf5GpDSev+Aa0vywQ+rEx1vhzpsfh3ifYtaSC9Fry1MbADVoHmIaaCXPtjlwMsgI42GOIhVWAaGkg5SqJgZGeZceLXUBxzIR2+enUKdOKWThBnsn/u5PMsusGovTgDFsujhA3KhXQzKxnWr1/Y1gZz0B7MZm1Yjc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=rhCbOKIU; arc=none smtp.client-ip=209.85.214.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="rhCbOKIU" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-201d6ac1426so8786175ad.1 for ; Wed, 14 Aug 2024 08:39:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1723649994; x=1724254794; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=TZwoHHCn+atSlmqhOT1+A6eF8trrUBAMiXzmjVgxULI=; b=rhCbOKIUItN0Jg/BPMrfXLcrN9p/fhM6FGDbXY6FhXZqecjWgiRudQ7c9SNLHqdM3S 3DcO0AHZkV3uIMQJyDvVx90gCF5yWKSZyqvv6gIxTRsEDz76lgLWogra+zVGP+x27PF+ LoUaHgzSqx7MgFuTbMO1HM0O48fTXFBqa37Kq12N2po/vyjx4mhCSwGnqy/6OBy17Kfb pf92YGg79p4755GuMlXEZLvpyeQwpZEb9MFXFnG+gbrwV4Xf7v96NuJnFcKsNICe9Djv 0nqvCw3Xz4Dq/aMmVuXjXL4t3jtPKl9iKgri4EZYAcSjK9POpggHUu2l0fpxGLEGE60t 6cQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723649994; x=1724254794; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=TZwoHHCn+atSlmqhOT1+A6eF8trrUBAMiXzmjVgxULI=; b=UeOAVegroKnkcoU3ftHqw8MQa2pSVkUTipLclil7pUbNcKMmjaryTBZsxSJlDZL0zD tM+T3yn+J7RS8Xbn6dBv76NnkXQ7EMc0NDSsliCkMLCiul25zLimE7ww9+ZrM00+zlXl 3uLQGUnodK28ZFVZzgUZfYXgBi8MnKCnC/moM/SOH8L4u7arH7omclstvHOoxTUifMlD qDPHNI7YsA9Zl35HfDcdKEA+n3Ru2zsvoobnkihAdfWbWMyLcDRD5kTYdrL1k+goFMju wJ84zqcxwvZRfQGr2wABD+R8C9/RiE/9yifAACJS/sfEjPg1Js65pGcolQYKVzTHwOjv GVHw== X-Forwarded-Encrypted: i=1; AJvYcCWWvkseoIeMPARssP5RNwmWcJUXlaxtRZ8qaQLT2L3rKfqhiSj8cFUuDCaT3ka8b+iAc3mUuKyYFfkALzM=@vger.kernel.org X-Gm-Message-State: AOJu0YzDUZjLhRGfNgrOzH+HIvhchvtKa2UcaG0Dt+1pgU9du2cpjCFs Jf94HI9yJNtNARmb8dpeBOL4ACjWVpWJOwzqxfgiUl13W95jC4orXXiE2ECnE2E= X-Google-Smtp-Source: AGHT+IEUPe8sQHa+iqRLGa5scCLBoeNHtfI88S/tHB5GkHGLOTZ8UPXhxFKv8w4yT/cAAfNK/if8vQ== X-Received: by 2002:a17:902:d508:b0:201:e792:d6ff with SMTP id d9443c01a7336-201e792d9f9mr14530265ad.10.1723649994415; Wed, 14 Aug 2024 08:39:54 -0700 (PDT) Received: from localhost ([71.212.170.185]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-201cd14e084sm30927495ad.110.2024.08.14.08.39.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Aug 2024 08:39:54 -0700 (PDT) From: Kevin Hilman Date: Wed, 14 Aug 2024 08:39:28 -0700 Subject: [PATCH v10 1/4] firmware: ti_sci: Add support for querying the firmware caps Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240814-lpm-constraints-firmware-msp-v10-1-bee4314bbdc8@baylibre.com> References: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> In-Reply-To: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> To: Nishanth Menon , Tero Kristo , Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Vibhore Vardhan , Dhruva Gole , Akashdeep Kaur , Markus Schneider-Pargmann X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=6409; i=khilman@baylibre.com; h=from:subject:message-id; bh=2Z3NwIB8KsuUQoqYJK95phdpT243RgM8VSUOwStsjBA=; b=owEBbQKS/ZANAwAKAVk3GJrT+8ZlAcsmYgBmvM/IW3kpcUhWKX5oP42+OPxGmBZFXfXtly4KZ x6h92RsvRuJAjMEAAEKAB0WIQR7h0YOFpJ/qfW/8QxZNxia0/vGZQUCZrzPyAAKCRBZNxia0/vG ZfA7D/4+dft8JswMtTMnCvUH5buU3yl8cMIyqydnQ9Bo0n1DPsiBsILbIb3JOq/ULRWLUdoEmGh D7DnnOTfD0XjTcQbg2vVFYBcG6VdAxGEDPYqKBrumB1cloDamMXaVVbVuZsenBZO4/12jTjMsUD nNx6dH8dXdJBLBgB/RmxJPfYzAHaV+B1EzM0UFDrDZEtQIqrt6RA83wEc6I/UQpmMXHOsrnAwNo ewSRv+GaLBvi97sEsgv0T12K0Zrf5lsBIqANqsY66U7K2+eQeWm8wNQmFr5Fu42WnsbKxACZNYX 2jKT16JDGtMQAG5NbgOQIssWbetfOCOsobDObiEf7mCiXdY2dLbdVWJT3c/IVB2Un8CN+9a1nUb eYJvO5XriSXOdO9xlCUogtL+muiR25oz/WqioIDAs9rk4p+FKHb++jueA0kN09LnQmJ+Rstnd0R 2+0q7cxjTmpbEqmUr68elc1HXMXeT9ZBkIdM4rxRHZOnCbXoYxphDcSC0jZLp0EkH9lwgFgBOoi PYXTX/7QU94RbSZqJZL0Q1tHMIDn+5rAGky2GYOs1Cg+C5HrT8biLw945ptBXqAnG8FTctfrzTx 0K5jMBHg+GY9HQD8hTg9YB/1B/KtDdgAhR6jaotcnVXm3b17Qd3+mICwCmIciVU0X5EBzUq2a9W AiEbNrEZXvH3cKw== X-Developer-Key: i=khilman@baylibre.com; a=openpgp; fpr=7B87460E16927FA9F5BFF10C5937189AD3FBC665 From: Georgi Vlaev Add support for the TISCI_MSG_QUERY_FW_CAPS message, used to retrieve the firmware capabilities of the currently running system firmware. The message belongs to the TISCI general core message API [1] and is available in SysFW version 08.04.03 and above. Currently, the message is supported on devices with split architecture of the system firmware (DM + TIFS) like AM62x. Old revisions or not yet supported platforms will NACK this request. We're using this message locally in ti_sci.c to get the low power featutes of the FW/SoC. As there's no other kernel consumers yet, this is not added to struct ti_sci_core_ops. Sysfw version >=3D 10.00.04 support LPM_DM_MANAGED capability [2], where Device Mgr firmware now manages which low power mode is chosen. Going forward, this is the default configuration supported for TI AM62 family of devices. The state chosen by the DM can be influenced by sending constraints using the new LPM constraint APIs. [1] https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/general/core.h= tml [2] https://downloads.ti.com/tisci/esd/latest/2_tisci_msgs/general/core.htm= l#tisci-msg-query-fw-caps Signed-off-by: Georgi Vlaev [vibhore@ti.com: Support for LPM_DM_MANAGED mode] Signed-off-by: Vibhore Vardhan Signed-off-by: Markus Schneider-Pargmann Signed-off-by: Kevin Hilman Tested-by: Dhruva Gole --- drivers/firmware/ti_sci.c | 69 +++++++++++++++++++++++++++++++++++++++++++= +++++++++++++++++++++++++- drivers/firmware/ti_sci.h | 22 ++++++++++++++++++++++ 2 files changed, 90 insertions(+), 1 deletion(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 160968301b1f..f77e13577eb8 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware/ti_sci.c @@ -2,7 +2,7 @@ /* * Texas Instruments System Control Interface Protocol Driver * - * Copyright (C) 2015-2022 Texas Instruments Incorporated - https://www.ti= .com/ + * Copyright (C) 2015-2024 Texas Instruments Incorporated - https://www.ti= .com/ * Nishanth Menon */ =20 @@ -24,6 +24,7 @@ #include #include #include +#include #include =20 #include "ti_sci.h" @@ -98,6 +99,7 @@ struct ti_sci_desc { * @minfo: Message info * @node: list head * @host_id: Host ID + * @fw_caps: FW/SoC low power capabilities * @users: Number of users of this instance */ struct ti_sci_info { @@ -114,6 +116,7 @@ struct ti_sci_info { struct ti_sci_xfers_info minfo; struct list_head node; u8 host_id; + u64 fw_caps; /* protected by ti_sci_list_mutex */ int users; }; @@ -1651,6 +1654,62 @@ static int ti_sci_cmd_clk_get_freq(const struct ti_s= ci_handle *handle, return ret; } =20 +/** + * ti_sci_msg_cmd_query_fw_caps() - Get the FW/SoC capabilities + * @handle: Pointer to TI SCI handle + * @fw_caps: Each bit in fw_caps indicating one FW/SOC capability + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_msg_cmd_query_fw_caps(const struct ti_sci_handle *handle, + u64 *fw_caps) +{ + struct ti_sci_info *info; + struct ti_sci_xfer *xfer; + struct ti_sci_msg_resp_query_fw_caps *resp; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_QUERY_FW_CAPS, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(struct ti_sci_msg_hdr), + sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_resp_query_fw_caps *)xfer->xfer_buf; + + if (!ti_sci_is_response_ack(resp)) { + ret =3D -ENODEV; + goto fail; + } + + if (fw_caps) + *fw_caps =3D resp->fw_caps; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + static int ti_sci_cmd_core_reboot(const struct ti_sci_handle *handle) { struct ti_sci_info *info; @@ -3390,6 +3449,14 @@ static int ti_sci_probe(struct platform_device *pdev) goto out; } =20 + /* + * Check if the firmware supports any optional low power modes. + * Old revisions of TIFS (< 08.04) will NACK the request. + */ + ret =3D ti_sci_msg_cmd_query_fw_caps(&info->handle, &info->fw_caps); + if (ret || !(info->fw_caps & MSG_FLAG_CAPS_GENERIC)) + pr_debug("Unable to detect LPM capabilities in fw_caps\n"); + ti_sci_setup_ops(info); =20 ret =3D devm_register_restart_handler(dev, tisci_reboot_handler, info); diff --git a/drivers/firmware/ti_sci.h b/drivers/firmware/ti_sci.h index 5846c60220f5..73ca9503606b 100644 --- a/drivers/firmware/ti_sci.h +++ b/drivers/firmware/ti_sci.h @@ -19,6 +19,7 @@ #define TI_SCI_MSG_WAKE_REASON 0x0003 #define TI_SCI_MSG_GOODBYE 0x0004 #define TI_SCI_MSG_SYS_RESET 0x0005 +#define TI_SCI_MSG_QUERY_FW_CAPS 0x0022 =20 /* Device requests */ #define TI_SCI_MSG_SET_DEVICE_STATE 0x0200 @@ -132,6 +133,27 @@ struct ti_sci_msg_req_reboot { struct ti_sci_msg_hdr hdr; } __packed; =20 +/** + * struct ti_sci_msg_resp_query_fw_caps - Response for query firmware caps + * @hdr: Generic header + * @fw_caps: Each bit in fw_caps indicating one FW/SOC capability + * MSG_FLAG_CAPS_GENERIC: Generic capability (LPM not supported) + * MSG_FLAG_CAPS_LPM_PARTIAL_IO: Partial IO in LPM + * MSG_FLAG_CAPS_LPM_DM_MANAGED: LPM can be managed by DM + * + * Response to a generic message with message type TI_SCI_MSG_QUERY_FW_CAPS + * providing currently available SOC/firmware capabilities. SoC that don't + * support low power modes return only MSG_FLAG_CAPS_GENERIC capability. + */ +struct ti_sci_msg_resp_query_fw_caps { + struct ti_sci_msg_hdr hdr; +#define MSG_FLAG_CAPS_GENERIC TI_SCI_MSG_FLAG(0) +#define MSG_FLAG_CAPS_LPM_PARTIAL_IO TI_SCI_MSG_FLAG(4) +#define MSG_FLAG_CAPS_LPM_DM_MANAGED TI_SCI_MSG_FLAG(5) +#define MSG_MASK_CAPS_LPM GENMASK_ULL(4, 1) + u64 fw_caps; +} __packed; + /** * struct ti_sci_msg_req_set_device_state - Set the desired state of the d= evice * @hdr: Generic header --=20 2.46.0 From nobody Mon Feb 9 11:08:19 2026 Received: from mail-pl1-f169.google.com (mail-pl1-f169.google.com [209.85.214.169]) (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 689DE1B4C45 for ; Wed, 14 Aug 2024 15:39:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.169 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649998; cv=none; b=SAHx2jlfAucXflH8d0mXtzpAroeFfrJ1grhEVQFgtqec0WQsqXNZEflYpmZZqNSAaEaRFl99cN6zhfx/DzOqjZm7BzEYuudkg8lKe/U/g5lZekmX0ZjKlY6qixhsLBuN7FkXRM5j2Wz6z1hrSzhg1o2I85WDQkNCLkfb/2irNUY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649998; c=relaxed/simple; bh=fJGypyEgJLhCF7Ad1296756iOFBRzBXWK1WwIuco6Ps=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=crYFNABcYbEIHF00/2OLT1xWisj4ZI/5a0uRlZcMJLXVYGi5PWFIThwHEv/j+qYTDPtn5MfQDav07UomSJRWyFsP9FnI7xxCGlcsNC7VqbVR6KSvQK3UCKESV9T+g+df8/LeQfmjx/vIq2JVc+uxOC/lIm002jTmTagICZZOxa0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=TxcFUWr8; arc=none smtp.client-ip=209.85.214.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="TxcFUWr8" Received: by mail-pl1-f169.google.com with SMTP id d9443c01a7336-1fc566ac769so61153355ad.1 for ; Wed, 14 Aug 2024 08:39:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1723649996; x=1724254796; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=SVC8+R3o277w5fNoilij6jhy0Uz3vrCP4aj5epdO9AI=; b=TxcFUWr8eMwJkLl9JQBOzy2lTVApzPvV93zttxbadcCnbq1HGBPEq5QcfzONU6kKzD i/frzxU5iX0N37V2Lt1gvk4SRi05RY+zgfSd8PGsDI392Ipmy8y2WRITRl9CsOQdhdz4 qRI5Z4mR3/btw3WWWZBXu6SWRdlAPqMPnuAMyfF6K632wh8+e6+qG/MwoGgjk+2vwUgP Zgd/ochW8iY4dVM6vSKFSmGpPEdcCvHTIdCxBSeYeXl4mtmaz1UUq/j2Riz7N+v5HmW1 EMAYwVGBSgnuxwoVtQsyhbmoMi6vRQRIJM6r7GSKm02veQp+QJBYGwhFMbtoDa0atNKR QrMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723649996; x=1724254796; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=SVC8+R3o277w5fNoilij6jhy0Uz3vrCP4aj5epdO9AI=; b=n48/oiH2d/arLBfHivsPMqYJWIa1hRkgJI7iVEfxWjTI6Uivs38r9ci4BhhVVosv7B zVo6+l3JUs5igrnCqtu2IL/2Zf8BEmMBSHBHvzdzbs53/+Y4dKPuNcIhfyp02eh3VwbW SD25oAmmIZ6tSLdJivMbWiN7ZCxI7EJUB0suPUQJ2p1gNJISPa/fGpywf6KJUGjKFEZ5 aSNvxFZEexXFnCK5dzqqjHv9ZXXR+7tLQOXaicQEelBirADrXy/I2Xhxjaw0Z7Tz+M09 7UyyltMG7vUOP008/oAiYNhTn1unYg6QwOWrNEGDJKe+Oi26RxHyafqXpLJhJKg2wmfq ecFA== X-Forwarded-Encrypted: i=1; AJvYcCVVwTGTfImenfnWiDkA9pEEdaS48CIZ2DY5yd3deM9DDNtkXKYIVYrQ+LJ4sQF2U57ShwTEE3MYM/9fz9fwMIfQGNRZtCWtxf2puD21 X-Gm-Message-State: AOJu0YxrkSCBmCTIfZZ1ZOEauPlZ9ZeEXoYwjodhKsxBqJOyVoHjRPlR 5DDtRVdzabXFCr+M/FbXdBAK7zMknaqlp5Hd+kfF3DXu5CUMT4K8E5kxOtNW3pA= X-Google-Smtp-Source: AGHT+IF5oDNr91igBSg2NfRwuOvbCVJoL4vmoO1/1J8Fnt1BqFQUkyXIHnt6DufRL2VaEduL5OBEIg== X-Received: by 2002:a17:902:dad2:b0:1ff:4a1e:865c with SMTP id d9443c01a7336-201d642a47amr34790655ad.29.1723649995515; Wed, 14 Aug 2024 08:39:55 -0700 (PDT) Received: from localhost ([71.212.170.185]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-201cd1cfae8sm30998135ad.264.2024.08.14.08.39.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Aug 2024 08:39:54 -0700 (PDT) From: Kevin Hilman Date: Wed, 14 Aug 2024 08:39:29 -0700 Subject: [PATCH v10 2/4] firmware: ti_sci: Add system suspend and resume call Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240814-lpm-constraints-firmware-msp-v10-2-bee4314bbdc8@baylibre.com> References: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> In-Reply-To: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> To: Nishanth Menon , Tero Kristo , Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Vibhore Vardhan , Dhruva Gole , Akashdeep Kaur , Markus Schneider-Pargmann X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=11683; i=khilman@baylibre.com; h=from:subject:message-id; bh=SApvXlKvTU624fFIvy/O4UZ3820V3is+SI1BRhZolQg=; b=owEBbQKS/ZANAwAKAVk3GJrT+8ZlAcsmYgBmvM/Ie+n+Nf3UcgIudixyFrsttDD2YnNWjE6RM noHtGd/VMWJAjMEAAEKAB0WIQR7h0YOFpJ/qfW/8QxZNxia0/vGZQUCZrzPyAAKCRBZNxia0/vG ZUH9D/oDFbGSy0yrvCYC2a9tvln5b9lk1Oe4O1iqylXwy6bOQ4Q9kp+7CV4OModr9zxJPQrm8Y+ Rb90Wexniq/T233MTyydLh6miZhaMsYKxyX3PdIiTL5J8wJXgcrUk/InT0z3Q5ACGDrIlYvnWpF S9nDb6GP5rh67UHzIdJNUPi8ygqCwKRQgIbFQkfx8lJGpV3Qh9DqH+4Vu8i+7ByJFF7GD+J9f6z gruedE4zauyliLjq5mlnv6ekts9gtvYW4/9bm8qiaRkAkuW5K9daubXmtbEU1e6Cl+E4pXAwl/f V8xVy+ScqJtOYn/O5U9NwiyjQU2vCSZmw/lu3NIpFpQL5JMtXFTXmy3ikmO+879qVVN7/aIemzz Q8JIsyEjGryo6Cad4LEze3QiJkoZtRQh7+EslA9VMdovvunCf0GLiF9gJhTKn1MA7uAbgPwBSUt QIyyH+EWlZUN2NrR1DDKYWmoJZId0TPKc0cSfswI7XpTPYfIqDw3i1TKpOogX6ZeQVKTkn914gl mWAU1Cu6Y6pGTDxkrOSPtJKaotdXYz0EDkW+zWQP5OHKprCgeBh3BlzK8OOc/oJFkByA8ioEkJe 9kgkKgRweIMsR4Lh5JErMLullMGfpjyVj2RQ80eMPEa+fj1tpkwTk3D2dRvr6Z0qf9JTNHqayPS CULLTb0VnvNGw/Q== X-Developer-Key: i=khilman@baylibre.com; a=openpgp; fpr=7B87460E16927FA9F5BFF10C5937189AD3FBC665 From: Vibhore Vardhan Introduce system suspend call that enables the ti_sci driver to support low power mode when the user space issues a suspend to mem. The following power management operations defined in the TISCI Low Power Mode API [1] are implemented to support suspend and resume: 1) TISCI_MSG_PREPARE_SLEEP Prepare the SOC for entering into a low power mode and provide details to firmware about the state being entered. 2) TISCI_MSG_SET_IO_ISOLATION Control the IO isolation for Low Power Mode. Also, write a ti_sci_prepare_system_suspend call to be used in the driver suspend handler to allow the system to identify the low power mode being entered and if necessary, send TISCI_MSG_PREPARE_SLEEP with information about the mode being entered. Sysfw version >=3D 10.00.04 support LPM_DM_MANAGED capability [2], where Device Mgr firmware now manages which low power mode is chosen. Going forward, this is the default configuration supported for TI AM62 family of devices. The state chosen by the DM can be influenced by sending constraints using the new LPM constraint APIs. In case the firmware does not support LPM_DM_MANAGED mode, the mode selection logic can be extended as needed. If no suspend-to-RAM modes are supported, return without taking any action. We're using "pm_suspend_target_state" to map the kernel's target suspend state to SysFW low power mode. Make sure this is available only when CONFIG_SUSPEND is enabled. Suspend has to be split into two parts, ti_sci_suspend() will send the prepare sleep message to prepare suspend. ti_sci_suspend_noirq() sets IO isolation which needs to be done as late as possible to avoid any issues. On resume this has to be done as early as possible. [1] https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/pm/lpm.html Co-developed-by: Dave Gerlach Signed-off-by: Dave Gerlach Signed-off-by: Georgi Vlaev Signed-off-by: Dhruva Gole Signed-off-by: Vibhore Vardhan Signed-off-by: Markus Schneider-Pargmann Signed-off-by: Kevin Hilman Tested-by: Dhruva Gole --- drivers/firmware/ti_sci.c | 188 +++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/firmware/ti_sci.h | 45 +++++++++++++++++++++++++++++= +++++++++++- include/linux/soc/ti/ti_sci_protocol.h | 4 ++++ 3 files changed, 236 insertions(+), 1 deletion(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index f77e13577eb8..808149dcc635 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware/ti_sci.c @@ -24,6 +24,7 @@ #include #include #include +#include #include #include =20 @@ -1654,6 +1655,65 @@ static int ti_sci_cmd_clk_get_freq(const struct ti_s= ci_handle *handle, return ret; } =20 +/** + * ti_sci_cmd_prepare_sleep() - Prepare system for system suspend + * @handle: pointer to TI SCI handle + * @mode: Device identifier + * @ctx_lo: Low part of address for context save + * @ctx_hi: High part of address for context save + * @debug_flags: Debug flags to pass to firmware + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_cmd_prepare_sleep(const struct ti_sci_handle *handle, u8= mode, + u32 ctx_lo, u32 ctx_hi, u32 debug_flags) +{ + struct ti_sci_info *info; + struct ti_sci_msg_req_prepare_sleep *req; + struct ti_sci_msg_hdr *resp; + struct ti_sci_xfer *xfer; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_PREPARE_SLEEP, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(*req), sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + + req =3D (struct ti_sci_msg_req_prepare_sleep *)xfer->xfer_buf; + req->mode =3D mode; + req->ctx_lo =3D ctx_lo; + req->ctx_hi =3D ctx_hi; + req->debug_flags =3D debug_flags; + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_hdr *)xfer->xfer_buf; + + ret =3D ti_sci_is_response_ack(resp) ? 0 : -ENODEV; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + /** * ti_sci_msg_cmd_query_fw_caps() - Get the FW/SoC capabilities * @handle: Pointer to TI SCI handle @@ -1710,6 +1770,58 @@ static int ti_sci_msg_cmd_query_fw_caps(const struct= ti_sci_handle *handle, return ret; } =20 +/** + * ti_sci_cmd_set_io_isolation() - Enable IO isolation in LPM + * @handle: Pointer to TI SCI handle + * @state: The desired state of the IO isolation + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_cmd_set_io_isolation(const struct ti_sci_handle *handle, + u8 state) +{ + struct ti_sci_info *info; + struct ti_sci_msg_req_set_io_isolation *req; + struct ti_sci_msg_hdr *resp; + struct ti_sci_xfer *xfer; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_SET_IO_ISOLATION, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(*req), sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + req =3D (struct ti_sci_msg_req_set_io_isolation *)xfer->xfer_buf; + req->state =3D state; + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_hdr *)xfer->xfer_buf; + + ret =3D ti_sci_is_response_ack(resp) ? 0 : -ENODEV; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + static int ti_sci_cmd_core_reboot(const struct ti_sci_handle *handle) { struct ti_sci_info *info; @@ -3321,6 +3433,79 @@ static int tisci_reboot_handler(struct sys_off_data = *data) return NOTIFY_BAD; } =20 +#ifdef CONFIG_SUSPEND +static int ti_sci_prepare_system_suspend(struct ti_sci_info *info) +{ + u8 mode; + + /* + * Map and validate the target Linux suspend state to TISCI LPM. + * Default is to let Device Manager select the low power mode. + */ + switch (pm_suspend_target_state) { + case PM_SUSPEND_MEM: + if (info->fw_caps & MSG_FLAG_CAPS_LPM_DM_MANAGED) + mode =3D TISCI_MSG_VALUE_SLEEP_MODE_DM_MANAGED; + else + /* DM Managed is not supported by the firmware. */ + return -EOPNOTSUPP; + break; + default: + /* + * Do not fail if we don't have action to take for a + * specific suspend mode. + */ + return 0; + } + + return ti_sci_cmd_prepare_sleep(&info->handle, mode, 0, 0, 0); +} + +static int ti_sci_suspend(struct device *dev) +{ + struct ti_sci_info *info =3D dev_get_drvdata(dev); + int ret; + + ret =3D ti_sci_prepare_system_suspend(info); + if (ret) + return ret; + + return 0; +} + +static int ti_sci_suspend_noirq(struct device *dev) +{ + struct ti_sci_info *info =3D dev_get_drvdata(dev); + int ret =3D 0; + + ret =3D ti_sci_cmd_set_io_isolation(&info->handle, TISCI_MSG_VALUE_IO_ENA= BLE); + if (ret) + return ret; + dev_dbg(dev, "%s: set isolation: %d\n", __func__, ret); + + return 0; +} + +static int ti_sci_resume_noirq(struct device *dev) +{ + struct ti_sci_info *info =3D dev_get_drvdata(dev); + int ret =3D 0; + + ret =3D ti_sci_cmd_set_io_isolation(&info->handle, TISCI_MSG_VALUE_IO_DIS= ABLE); + if (ret) + return ret; + dev_dbg(dev, "%s: disable isolation: %d\n", __func__, ret); + + return 0; +} + +static const struct dev_pm_ops ti_sci_pm_ops =3D { + .suspend =3D ti_sci_suspend, + .suspend_noirq =3D ti_sci_suspend_noirq, + .resume_noirq =3D ti_sci_resume_noirq, +}; +#endif /* CONFIG_SUSPEND */ + /* Description for K2G */ static const struct ti_sci_desc ti_sci_pmmc_k2g_desc =3D { .default_host_id =3D 2, @@ -3490,6 +3675,9 @@ static struct platform_driver ti_sci_driver =3D { .name =3D "ti-sci", .of_match_table =3D of_match_ptr(ti_sci_of_match), .suppress_bind_attrs =3D true, +#ifdef CONFIG_SUSPEND + .pm =3D &ti_sci_pm_ops, +#endif }, }; module_platform_driver(ti_sci_driver); diff --git a/drivers/firmware/ti_sci.h b/drivers/firmware/ti_sci.h index 73ca9503606b..8efe4d0e61fb 100644 --- a/drivers/firmware/ti_sci.h +++ b/drivers/firmware/ti_sci.h @@ -6,7 +6,7 @@ * The system works in a message response protocol * See: https://software-dl.ti.com/tisci/esd/latest/index.html for details * - * Copyright (C) 2015-2016 Texas Instruments Incorporated - https://www.t= i.com/ + * Copyright (C) 2015-2024 Texas Instruments Incorporated - https://www.t= i.com/ */ =20 #ifndef __TI_SCI_H @@ -36,6 +36,10 @@ #define TI_SCI_MSG_QUERY_CLOCK_FREQ 0x010d #define TI_SCI_MSG_GET_CLOCK_FREQ 0x010e =20 +/* Low Power Mode Requests */ +#define TI_SCI_MSG_PREPARE_SLEEP 0x0300 +#define TI_SCI_MSG_SET_IO_ISOLATION 0x0307 + /* Resource Management Requests */ #define TI_SCI_MSG_GET_RESOURCE_RANGE 0x1500 =20 @@ -567,6 +571,45 @@ struct ti_sci_msg_resp_get_clock_freq { u64 freq_hz; } __packed; =20 +/** + * struct tisci_msg_req_prepare_sleep - Request for TISCI_MSG_PREPARE_SLEE= P. + * + * @hdr TISCI header to provide ACK/NAK flags to the host. + * @mode Low power mode to enter. + * @ctx_lo Low 32-bits of physical pointer to address to use for context= save. + * @ctx_hi High 32-bits of physical pointer to address to use for contex= t save. + * @debug_flags Flags that can be set to halt the sequence during suspen= d or + * resume to allow JTAG connection and debug. + * + * This message is used as the first step of entering a low power mode. It + * allows configurable information, including which state to enter to be + * easily shared from the application, as this is a non-secure message and + * therefore can be sent by anyone. + */ +struct ti_sci_msg_req_prepare_sleep { + struct ti_sci_msg_hdr hdr; + +#define TISCI_MSG_VALUE_SLEEP_MODE_DM_MANAGED 0xfd + u8 mode; + u32 ctx_lo; + u32 ctx_hi; + u32 debug_flags; +} __packed; + +/** + * struct tisci_msg_set_io_isolation_req - Request for TI_SCI_MSG_SET_IO_I= SOLATION. + * + * @hdr: Generic header + * @state: The deseared state of the IO isolation. + * + * This message is used to enable/disable IO isolation for low power modes. + * Response is generic ACK / NACK message. + */ +struct ti_sci_msg_req_set_io_isolation { + struct ti_sci_msg_hdr hdr; + u8 state; +} __packed; + #define TI_SCI_IRQ_SECONDARY_HOST_INVALID 0xff =20 /** diff --git a/include/linux/soc/ti/ti_sci_protocol.h b/include/linux/soc/ti/= ti_sci_protocol.h index bd0d11af76c5..1f1871e23f76 100644 --- a/include/linux/soc/ti/ti_sci_protocol.h +++ b/include/linux/soc/ti/ti_sci_protocol.h @@ -195,6 +195,10 @@ struct ti_sci_clk_ops { u64 *current_freq); }; =20 +/* TISCI LPM IO isolation control values */ +#define TISCI_MSG_VALUE_IO_ENABLE 1 +#define TISCI_MSG_VALUE_IO_DISABLE 0 + /** * struct ti_sci_resource_desc - Description of TI SCI resource instance r= ange. * @start: Start index of the first resource range. --=20 2.46.0 From nobody Mon Feb 9 11:08:19 2026 Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) (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 9D71A1B4C46 for ; Wed, 14 Aug 2024 15:39:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.180 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649999; cv=none; b=h6qZtj9HIVHit5Cg4xcRC6JKwAA+D5XXZx2Fk1tHBGbEorvONZM356jua+3psdE+zFRY6Gw6Z1BrgX1Mi17VPAfIi9vCg2OnGBjGwRdv7ubd2QXxNoRuAYaQB5Ucfu8Rr0DClatDmuVzmL0c20RBT2+EZr9UZ6m4UR7x1hiABGg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723649999; c=relaxed/simple; bh=xyA5dliytXUpbWO7v2epgoP88Le1MsYfwoAy1nBVk+0=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=KqBqnhy34/oJrkAnyKflScmk1jBShvJs2wMCFbs76CJSsMoCGe+uyDGIwG+6vtyYWIxJMf5Dwh6GJcxiqLf/QmGOj/tCZTI6DwN0TPQaramMynY46VSTydT6M9mDSjrgcww5mvEaGUlqPr/nUa7wIqZmg37cMAmg581p9CPWTvE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=NuYKvywH; arc=none smtp.client-ip=209.85.214.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="NuYKvywH" Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-1fc52394c92so61471245ad.1 for ; Wed, 14 Aug 2024 08:39:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1723649997; x=1724254797; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=uO4MyaKUc9bmjiuuyPvs+rb8eIrbgGn8c1dFfmNv0w4=; b=NuYKvywH3cl1JDgasNRjXnkjU0v+ed64sd4G04jnctvGd7ft/0hiQuKwp4ut5ty5nn erAAz0LXLvEeKyZ6BbIvbOCZPII6NlnAESqxPdPsZ+vVe73mYJ7g1V3DFhfmsV0E4uqz eJbcDlmudZ0S2s/ua6ZVsOWl3IV4HQ10YOF4s0sx3r/2Ha+OXJ2WLoWoO2Oxyx+2y6kF 41psxgrjQHs4gi/eEtjHuZGQA9JJIXFNexpq0j703JsH+NMlZ56Zv3yz1cJca0HsIMff zenuyDnDDkJAFkeE29aXsgNpxp6VX7dwI+uZC6J3+Oi38lACJ4Vnlo8cRQkRIj79r52C QNKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723649997; x=1724254797; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uO4MyaKUc9bmjiuuyPvs+rb8eIrbgGn8c1dFfmNv0w4=; b=NL0JpXa1XBLkCpc0UsromXb99f3akawn6+Vyj5/mfyKjM7zwNVk2JjU+WkSy8Ncegp IvvynbpeN6zK9NYBKDsyx/hK957s6PM7hJRvn8LBlZZe+iSjtZQncbw1FCU4F3UAeYiV AN29FmndSJqbqQL0k5h5pFDle2csjRHR+YfgfXAB9Pk4RdBFTqngtyBbC6+uYK8m7Nso xN1xdxNKlVKtcr7IGy/Mzhw04GeYABAVT3IvN6ri6JbfOPQ97MVY81eF5uOoDYNl4Dhs 2hXI0YcLK1E017hL/n2KN7ZsjwZL+GLCXPfXDqskoI53THkzb33WLUSrM5G+8GUUqcD6 07/Q== X-Forwarded-Encrypted: i=1; AJvYcCUU6SwuEN89LywwkT5KYqJkhr/aoI9L3fJPd4P7YuUXvHc2txWYA+NXcBcRPM6IRM5dVpKZoKpDA6+CZ4kE9d4klhuZf/H0VehOZUf1 X-Gm-Message-State: AOJu0YxhqslLlA47gRjQ+L/sddcqUHrXT2UV4BAQ54fEG03p0hlS6DIF vBCQ2IpCTZGYe6gg96IZp0oYX28BcLIg92m56SNYKWpJAei4+WZ2FbHjPDTBQ3M= X-Google-Smtp-Source: AGHT+IFG2MHs0YyP2eBi5ugkdj+fq/r1ld/Md39wf06Uk2jsQ6U9aRM7t0s5xZi33X1KxVfIsJ/HhQ== X-Received: by 2002:a17:903:1211:b0:1ff:39d4:1663 with SMTP id d9443c01a7336-201d648b6damr33819125ad.47.1723649996645; Wed, 14 Aug 2024 08:39:56 -0700 (PDT) Received: from localhost ([71.212.170.185]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-201cd14c729sm31057595ad.118.2024.08.14.08.39.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Aug 2024 08:39:55 -0700 (PDT) From: Kevin Hilman Date: Wed, 14 Aug 2024 08:39:30 -0700 Subject: [PATCH v10 3/4] firmware: ti_sci: Introduce Power Management Ops Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240814-lpm-constraints-firmware-msp-v10-3-bee4314bbdc8@baylibre.com> References: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> In-Reply-To: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> To: Nishanth Menon , Tero Kristo , Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Vibhore Vardhan , Dhruva Gole , Akashdeep Kaur , Markus Schneider-Pargmann X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=14911; i=khilman@baylibre.com; h=from:subject:message-id; bh=fSvGYH6bEljQwJMgBnVzuMSmfecbP7FGCrchyJkV3Mc=; b=owEBbQKS/ZANAwAKAVk3GJrT+8ZlAcsmYgBmvM/Ieqt73LbDKz7vJLiUye7cptks5xH3Y5cfJ byoACHUCrGJAjMEAAEKAB0WIQR7h0YOFpJ/qfW/8QxZNxia0/vGZQUCZrzPyAAKCRBZNxia0/vG ZYv6EACXja8sFYXgAl8bQ0L9T5cU5B6aedtosv1B5CFvLTD+mWrrHLINeZoA3Tmkg2D7CznVLTn 0w64fCw7T84BIZJdmz4qtf09CURprpN3v7grNpD1y44wsIvPWDxdz4jgabHrYESaRNHN65DDwFm 9Lv7h/loeP1OvazU7azAtfS61uJO8dq/oIKNk8x/lY5MR47IiXMQjEJ5blcROoRZSN3AG/S19vb 55XktjEg+c9G/gAkZLYvnUPMuxlvrHr59541HkH+wgz2KA+O1EvJIG6O/FQRgxWQ/2cHtkQPt+D T92P2N4IMo6mAb7WnJpZecVgYMzJH3UChAKcZV806iaksRh7TASarvn1gWGYkuGH2Gs8uTER0z/ M/MbA8/3pjeYFk0yMRUV0jZ0VRsdrLN/dKJfQBB+rKy1prX44cV7KIg+ScAh+aCX13o79Soz8Ib I3VhVSmnZQJTgHtims/2EIYRvnPWaM5r9oPlfSSAt5GFroEoU+YV7043mPte0WCY0H5PG/Zdiap qkhOSHF5Ay2PkFSaQatTeQqgK073pS/WWjH3Hi5o915duZMIdey2vKNOoDrsoD7BmcS/MUXjHab ZddE0Mj6+YXbiDz28UZEz13VRL1NKXzwG/L+qEKeGQdVx/rcomGa+2GH8rLdB51nY2QuaOMyrBg w7Ah9tTjIcHrcNw== X-Developer-Key: i=khilman@baylibre.com; a=openpgp; fpr=7B87460E16927FA9F5BFF10C5937189AD3FBC665 From: Dave Gerlach Introduce power management ops supported by the TISCI Low Power Mode API [1]. 1) TISCI_MSG_LPM_WAKE_REASON Get which wake up source woke the SoC from Low Power Mode. The wake up source IDs will be common for all K3 platforms. 2) TISCI_MSG_LPM_SET_DEVICE_CONSTRAINT Set LPM constraint on behalf of a device. By setting a constraint, the device ensures that it will not be powered off or reset in the selected mode. 3) TISCI_MSG_LPM_SET_LATENCY_CONSTRAINT Set LPM resume latency constraint. By setting a constraint, the host ensures that the resume time from selected mode will be less than the constraint value. [1] https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/pm/lpm.html Signed-off-by: Dave Gerlach [g-vlaev@ti.com: LPM_WAKE_REASON and IO_ISOLATION support] Signed-off-by: Georgi Vlaev [a-kaur@ti.com: SET_DEVICE_CONSTRAINT support] Signed-off-by: Akashdeep Kaur [vibhore@ti.com: SET_LATENCY_CONSTRAINT support] Signed-off-by: Vibhore Vardhan Signed-off-by: Markus Schneider-Pargmann Signed-off-by: Kevin Hilman Reviewed-by: Akashdeep Kaur Tested-by: Dhruva Gole --- drivers/firmware/ti_sci.c | 189 +++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/firmware/ti_sci.h | 76 +++++++++++++++++++++++++++++= ++++++++++++++++++++++++++++++++++++++++ include/linux/soc/ti/ti_sci_protocol.h | 42 +++++++++++++++++++++++++++++= +++++++++ 3 files changed, 307 insertions(+) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 808149dcc635..107494406fa5 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware/ti_sci.c @@ -1822,6 +1822,179 @@ static int ti_sci_cmd_set_io_isolation(const struct= ti_sci_handle *handle, return ret; } =20 +/** + * ti_sci_msg_cmd_lpm_wake_reason() - Get the wakeup source from LPM + * @handle: Pointer to TI SCI handle + * @source: The wakeup source that woke the SoC from LPM + * @timestamp: Timestamp of the wakeup event + * @pin: The pin that has triggered wake up + * @mode: The last entered low power mode + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_msg_cmd_lpm_wake_reason(const struct ti_sci_handle *hand= le, + u32 *source, u64 *timestamp, u8 *pin, u8 *mode) +{ + struct ti_sci_info *info; + struct ti_sci_xfer *xfer; + struct ti_sci_msg_resp_lpm_wake_reason *resp; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_LPM_WAKE_REASON, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(struct ti_sci_msg_hdr), + sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_resp_lpm_wake_reason *)xfer->xfer_buf; + + if (!ti_sci_is_response_ack(resp)) { + ret =3D -ENODEV; + goto fail; + } + + if (source) + *source =3D resp->wake_source; + if (timestamp) + *timestamp =3D resp->wake_timestamp; + if (pin) + *pin =3D resp->wake_pin; + if (mode) + *mode =3D resp->mode; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + +/** + * ti_sci_cmd_set_device_constraint() - Set LPM constraint on behalf of a = device + * @handle: pointer to TI SCI handle + * @id: Device identifier + * @state: The desired state of device constraint: set or clear + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_cmd_set_device_constraint(const struct ti_sci_handle *ha= ndle, + u32 id, u8 state) +{ + struct ti_sci_info *info; + struct ti_sci_msg_req_lpm_set_device_constraint *req; + struct ti_sci_msg_hdr *resp; + struct ti_sci_xfer *xfer; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_LPM_SET_DEVICE_CONSTRAINT, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(*req), sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + req =3D (struct ti_sci_msg_req_lpm_set_device_constraint *)xfer->xfer_buf; + req->id =3D id; + req->state =3D state; + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_hdr *)xfer->xfer_buf; + + ret =3D ti_sci_is_response_ack(resp) ? 0 : -ENODEV; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + +/** + * ti_sci_cmd_set_latency_constraint() - Set LPM resume latency constraint + * @handle: pointer to TI SCI handle + * @latency: maximum acceptable latency (in ms) to wake up from LPM + * @state: The desired state of latency constraint: set or clear + * + * Return: 0 if all went well, else returns appropriate error value. + */ +static int ti_sci_cmd_set_latency_constraint(const struct ti_sci_handle *h= andle, + u16 latency, u8 state) +{ + struct ti_sci_info *info; + struct ti_sci_msg_req_lpm_set_latency_constraint *req; + struct ti_sci_msg_hdr *resp; + struct ti_sci_xfer *xfer; + struct device *dev; + int ret =3D 0; + + if (IS_ERR(handle)) + return PTR_ERR(handle); + if (!handle) + return -EINVAL; + + info =3D handle_to_ti_sci_info(handle); + dev =3D info->dev; + + xfer =3D ti_sci_get_one_xfer(info, TI_SCI_MSG_LPM_SET_LATENCY_CONSTRAINT, + TI_SCI_FLAG_REQ_ACK_ON_PROCESSED, + sizeof(*req), sizeof(*resp)); + if (IS_ERR(xfer)) { + ret =3D PTR_ERR(xfer); + dev_err(dev, "Message alloc failed(%d)\n", ret); + return ret; + } + req =3D (struct ti_sci_msg_req_lpm_set_latency_constraint *)xfer->xfer_bu= f; + req->latency =3D latency; + req->state =3D state; + + ret =3D ti_sci_do_xfer(info, xfer); + if (ret) { + dev_err(dev, "Mbox send fail %d\n", ret); + goto fail; + } + + resp =3D (struct ti_sci_msg_hdr *)xfer->xfer_buf; + + ret =3D ti_sci_is_response_ack(resp) ? 0 : -ENODEV; + +fail: + ti_sci_put_one_xfer(&info->minfo, xfer); + + return ret; +} + static int ti_sci_cmd_core_reboot(const struct ti_sci_handle *handle) { struct ti_sci_info *info; @@ -2964,6 +3137,7 @@ static void ti_sci_setup_ops(struct ti_sci_info *info) struct ti_sci_core_ops *core_ops =3D &ops->core_ops; struct ti_sci_dev_ops *dops =3D &ops->dev_ops; struct ti_sci_clk_ops *cops =3D &ops->clk_ops; + struct ti_sci_pm_ops *pmops =3D &ops->pm_ops; struct ti_sci_rm_core_ops *rm_core_ops =3D &ops->rm_core_ops; struct ti_sci_rm_irq_ops *iops =3D &ops->rm_irq_ops; struct ti_sci_rm_ringacc_ops *rops =3D &ops->rm_ring_ops; @@ -3003,6 +3177,13 @@ static void ti_sci_setup_ops(struct ti_sci_info *inf= o) cops->set_freq =3D ti_sci_cmd_clk_set_freq; cops->get_freq =3D ti_sci_cmd_clk_get_freq; =20 + if (info->fw_caps & MSG_FLAG_CAPS_LPM_DM_MANAGED) { + pr_debug("detected DM managed LPM in fw_caps\n"); + pmops->lpm_wake_reason =3D ti_sci_msg_cmd_lpm_wake_reason; + pmops->set_device_constraint =3D ti_sci_cmd_set_device_constraint; + pmops->set_latency_constraint =3D ti_sci_cmd_set_latency_constraint; + } + rm_core_ops->get_range =3D ti_sci_cmd_get_resource_range; rm_core_ops->get_range_from_shost =3D ti_sci_cmd_get_resource_range_from_shost; @@ -3490,12 +3671,20 @@ static int ti_sci_resume_noirq(struct device *dev) { struct ti_sci_info *info =3D dev_get_drvdata(dev); int ret =3D 0; + u32 source; + u64 time; + u8 pin; + u8 mode; =20 ret =3D ti_sci_cmd_set_io_isolation(&info->handle, TISCI_MSG_VALUE_IO_DIS= ABLE); if (ret) return ret; dev_dbg(dev, "%s: disable isolation: %d\n", __func__, ret); =20 + ti_sci_msg_cmd_lpm_wake_reason(&info->handle, &source, &time, &pin, &mode= ); + dev_info(dev, "ti_sci: wakeup source:0x%x, pin:0x%x, mode:0x%x\n", + source, pin, mode); + return 0; } =20 diff --git a/drivers/firmware/ti_sci.h b/drivers/firmware/ti_sci.h index 8efe4d0e61fb..053387d7baa0 100644 --- a/drivers/firmware/ti_sci.h +++ b/drivers/firmware/ti_sci.h @@ -38,7 +38,10 @@ =20 /* Low Power Mode Requests */ #define TI_SCI_MSG_PREPARE_SLEEP 0x0300 +#define TI_SCI_MSG_LPM_WAKE_REASON 0x0306 #define TI_SCI_MSG_SET_IO_ISOLATION 0x0307 +#define TI_SCI_MSG_LPM_SET_DEVICE_CONSTRAINT 0x0309 +#define TI_SCI_MSG_LPM_SET_LATENCY_CONSTRAINT 0x030A =20 /* Resource Management Requests */ #define TI_SCI_MSG_GET_RESOURCE_RANGE 0x1500 @@ -610,6 +613,79 @@ struct ti_sci_msg_req_set_io_isolation { u8 state; } __packed; =20 +/** + * struct ti_sci_msg_resp_lpm_wake_reason - Response for TI_SCI_MSG_LPM_WA= KE_REASON. + * + * @hdr: Generic header. + * @wake_source: The wake up source that woke soc from LPM. + * @wake_timestamp: Timestamp at which soc woke. + * @wake_pin: The pin that has triggered wake up. + * @mode: The last entered low power mode. + * @rsvd: Reserved for future use. + * + * Response to a generic message with message type TI_SCI_MSG_LPM_WAKE_REA= SON, + * used to query the wake up source, pin and entered low power mode. + */ +struct ti_sci_msg_resp_lpm_wake_reason { + struct ti_sci_msg_hdr hdr; + u32 wake_source; + u64 wake_timestamp; + u8 wake_pin; + u8 mode; + u32 rsvd[2]; +} __packed; + +/** + * struct ti_sci_msg_req_lpm_set_device_constraint - Request for + * TISCI_MSG_LPM_SET_DEVICE_CONSTRAINT. + * + * @hdr: TISCI header to provide ACK/NAK flags to the host. + * @id: Device ID of device whose constraint has to be modified. + * @state: The desired state of device constraint: set or clear. + * @rsvd: Reserved for future use. + * + * This message is used by host to set constraint on the device. This can = be + * sent anytime after boot before prepare sleep message. Any device can se= t a + * constraint on the low power mode that the SoC can enter. It allows + * configurable information to be easily shared from the application, as t= his + * is a non-secure message and therefore can be sent by anyone. By setting= a + * constraint, the device ensures that it will not be powered off or reset= in + * the selected mode. Note: Access Restriction: Exclusivity flag of Device= will + * be honored. If some other host already has constraint on this device ID, + * NACK will be returned. + */ +struct ti_sci_msg_req_lpm_set_device_constraint { + struct ti_sci_msg_hdr hdr; + u32 id; + u8 state; + u32 rsvd[2]; +} __packed; + +/** + * struct ti_sci_msg_req_lpm_set_latency_constraint - Request for + * TISCI_MSG_LPM_SET_LATENCY_CONSTRAINT. + * + * @hdr: TISCI header to provide ACK/NAK flags to the host. + * @wkup_latency: The maximum acceptable latency to wake up from low power= mode + * in milliseconds. The deeper the state, the higher the latency. + * @state: The desired state of wakeup latency constraint: set or clear. + * @rsvd: Reserved for future use. + * + * This message is used by host to set wakeup latency from low power mode.= This can + * be sent anytime after boot before prepare sleep message, and can be sen= t after + * current low power mode is exited. Any device can set a constraint on th= e low power + * mode that the SoC can enter. It allows configurable information to be e= asily shared + * from the application, as this is a non-secure message and therefore can= be sent by + * anyone. By setting a wakeup latency constraint, the host ensures that t= he resume time + * from selected low power mode will be less than the constraint value. + */ +struct ti_sci_msg_req_lpm_set_latency_constraint { + struct ti_sci_msg_hdr hdr; + u16 latency; + u8 state; + u32 rsvd; +} __packed; + #define TI_SCI_IRQ_SECONDARY_HOST_INVALID 0xff =20 /** diff --git a/include/linux/soc/ti/ti_sci_protocol.h b/include/linux/soc/ti/= ti_sci_protocol.h index 1f1871e23f76..4ba9e520a28f 100644 --- a/include/linux/soc/ti/ti_sci_protocol.h +++ b/include/linux/soc/ti/ti_sci_protocol.h @@ -199,6 +199,47 @@ struct ti_sci_clk_ops { #define TISCI_MSG_VALUE_IO_ENABLE 1 #define TISCI_MSG_VALUE_IO_DISABLE 0 =20 +/* TISCI LPM wake up sources */ +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_I2C0 0x00 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_UART0 0x10 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_MCU_GPIO0 0x20 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_ICEMELTER0 0x30 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_TIMER0 0x40 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_TIMER1 0x41 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_WKUP_RTC0 0x50 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_RESET 0x60 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_USB0 0x70 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_USB1 0x71 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_MAIN_IO 0x80 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_MCU_IO 0x81 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_CAN_IO 0x82 +#define TISCI_MSG_VALUE_LPM_WAKE_SOURCE_INVALID 0xFF + +/* TISCI LPM constraint state values */ +#define TISCI_MSG_CONSTRAINT_SET 1 +#define TISCI_MSG_CONSTRAINT_CLR 0 + +/** + * struct ti_sci_pm_ops - Low Power Mode (LPM) control operations + * @lpm_wake_reason: Get the wake up source that woke the SoC from LPM + * - source: The wake up source that woke soc from LPM. + * - timestamp: Timestamp at which soc woke. + * @set_device_constraint: Set LPM constraint on behalf of a device + * - id: Device Identifier + * - state: The desired state of device constraint: set or clear. + * @set_latency_constraint: Set LPM resume latency constraint + * - latency: maximum acceptable latency to wake up from low power mode + * - state: The desired state of latency constraint: set or clear. + */ +struct ti_sci_pm_ops { + int (*lpm_wake_reason)(const struct ti_sci_handle *handle, + u32 *source, u64 *timestamp, u8 *pin, u8 *mode); + int (*set_device_constraint)(const struct ti_sci_handle *handle, + u32 id, u8 state); + int (*set_latency_constraint)(const struct ti_sci_handle *handle, + u16 latency, u8 state); +}; + /** * struct ti_sci_resource_desc - Description of TI SCI resource instance r= ange. * @start: Start index of the first resource range. @@ -543,6 +584,7 @@ struct ti_sci_ops { struct ti_sci_core_ops core_ops; struct ti_sci_dev_ops dev_ops; struct ti_sci_clk_ops clk_ops; + struct ti_sci_pm_ops pm_ops; struct ti_sci_rm_core_ops rm_core_ops; struct ti_sci_rm_irq_ops rm_irq_ops; struct ti_sci_rm_ringacc_ops rm_ring_ops; --=20 2.46.0 From nobody Mon Feb 9 11:08:19 2026 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 B045D1B5812 for ; Wed, 14 Aug 2024 15:39:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723650001; cv=none; b=mOyo5T47L7EzQIK5RXUncAaSAoHR7uzUMPnl2Bt4OaqsNnOppi1zP919/kDCFrevDIdDnxE84Zmr5nObfv5dBgCR4pCv1ih11d/CLIinuuinYFag6PV4blLEFhDGZAnQkNPZpoYjuo1YYTxXcGGrLdFypbomzw1ax/k53uVtqIs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723650001; c=relaxed/simple; bh=KJIXodqaL/PlsDNGovQcl2iggmyHiixGfjVtxIDXn9g=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=i1Gco/WbIpyWSVKPkSl5WStg7R4ms7EpK7WegCg7ucIZ/aWE++7Gf/gy8RakGVYvzMfZ9ktwevCQH2gak5AcpwQtFCWWNhNDvF5VVadeN59Xvw6v4NOAEuB4ElWzmyyFxdvFIdza/vVk2jCMCZ89uoCyW0GlNBYDB3iwJ2xVuwI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=BE9AjM89; arc=none smtp.client-ip=209.85.214.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="BE9AjM89" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1fd70ba6a15so52935565ad.0 for ; Wed, 14 Aug 2024 08:39:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1723649998; x=1724254798; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=y2mHcNudHQ2J7KK+PvZEplwN1eJNrfpSWSE1XYB762U=; b=BE9AjM8905Xmsonh7g6XGECRszAmTdTh7+JYHCY6iGtmQ3KQuxmohIAwJMth2gSuIi RCQQOiTlPeuTW/DBmOiU2x561DfNxTmVbtZV9r/Q5XAGR2wpgl3bJnvxf+Ok5h6XIIzk xtFjh5aAVxGXF+MxdtD//SZQNiK2VcrqBYKgmqmQ8sWYG2uFo1/WxADbf3+scWGr6SCS XjdB4bNojTn5qZju0Z3ddwytqF2M6NP50uywMYNrUNZQO73fvxweagoJGEaLhiiFVR3W R+sxPKo5/aaImenjTl8iDgdHmgoVBZkU1wG35QTVNTmq0X+P9imynxVZ+PPanNOmYqFl LOpw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723649998; x=1724254798; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=y2mHcNudHQ2J7KK+PvZEplwN1eJNrfpSWSE1XYB762U=; b=SkCa7hxhY4Py2lP6o0yA084UKKuKydrUbVgACqGSJuchuLeTGANqN0spSgGFjYQTJv vZdMS/vnPgZb9VA2m/zZlnpnxZ6an5b4LTmPLo1HZH1lvhU1kF/fiDPuyt5YO6YdI6mu dQ8oXNuc2VHM5CWHei8JTp029D85cYUmZKZSBXBsc8RBwtbk9KxPt90sicGYnMQLftxx rJl6fZhkC6sU5NFrqwOvPRfKpt1O0SrW4rOhN9jQuc/je3KGXqCDsKZU0cmCQtogR/AX DXoTRDiDN/BoAu8Kh5okIDETiuZe8ZtUS6lNS0uO/1ESLVnxMzLTfSx3KfQVVsMN1Pkl XwVQ== X-Forwarded-Encrypted: i=1; AJvYcCVzG51fCIW3dEXKCkUHbzvX0UVtdHQbbUfeI0AYEmk0n3dZ9WnVdbUIlvqLaILNboFLgDHQZDeWWiBkIf3xs1U51yTwFJDBhqZlbQ0d X-Gm-Message-State: AOJu0YxdDNBaP7Ms6be75R8oQE4c7hR859iVL5202/Luqr+t7R8WbD6A OMqj2FvyuDfsJelDoXPQxTRUu9j0M3OPkTzsuJ9f2BI/wNfP4SlcLxlnlLHqs4k= X-Google-Smtp-Source: AGHT+IFFuFlXHcCpgUXl2SWCuQLztcXNYrz5ohSNtrXnZ2RsGSm/ZnWNqQZ7tX0OfV01B0yJywvE6g== X-Received: by 2002:a17:902:dac3:b0:1fd:6c5b:afbc with SMTP id d9443c01a7336-201d64b075amr39750105ad.49.1723649997979; Wed, 14 Aug 2024 08:39:57 -0700 (PDT) Received: from localhost ([71.212.170.185]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-201cd1e2947sm30963465ad.306.2024.08.14.08.39.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Aug 2024 08:39:57 -0700 (PDT) From: Kevin Hilman Date: Wed, 14 Aug 2024 08:39:31 -0700 Subject: [PATCH v10 4/4] firmware: ti_sci: add CPU latency constraint management Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240814-lpm-constraints-firmware-msp-v10-4-bee4314bbdc8@baylibre.com> References: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> In-Reply-To: <20240814-lpm-constraints-firmware-msp-v10-0-bee4314bbdc8@baylibre.com> To: Nishanth Menon , Tero Kristo , Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Vibhore Vardhan , Dhruva Gole , Akashdeep Kaur , Markus Schneider-Pargmann X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=2353; i=khilman@baylibre.com; h=from:subject:message-id; bh=KJIXodqaL/PlsDNGovQcl2iggmyHiixGfjVtxIDXn9g=; b=owEBbQKS/ZANAwAKAVk3GJrT+8ZlAcsmYgBmvM/Io/sKs8c84RWhexj+Zi6QBwTej+YUooUjb s97x27p1lOJAjMEAAEKAB0WIQR7h0YOFpJ/qfW/8QxZNxia0/vGZQUCZrzPyAAKCRBZNxia0/vG Zbj3D/9C07sip72O4Z5THjZtYagRVIR6SubwIE6eJYGg/7jzPwBA/CNELi2giSqGsSVsRNCRF9r HUKM+fzJapObF5XTNoJYRD8KcxNhAvQdXZcB+y+29jcnrw2GPH6Go3+Rb1MDXkI288gXSbUolpW hKWU1bN1B894RlxHQJa9jvrs5DT00kZzF4xjM67h7q4aI1LE/tnuXCLtlG7llK/KImGUp297WhG YE26VWO9fTwOsboD9t/fsjRHH7rpw0wYJXKYi16Ei0kKC0kGPumYtSl64nqYkKnVUzGn2opz0w8 7vJjKRGooIwZOdSRPY2EO2IgVqGssgm+8fDPOIdEBBTVgbfoukyrOBxXQGq8gdNqCt6YJ1jgaE7 eY+qyjImHQYAScEa0JqRFYJvMgTebgsfY6K/fVs7AZ5ptN+iHGPXFUsHj58LeGg7suQSu8IrBBg ad4f9PiW8r194/lvYEqYlzPfIR070N3+wR+2IVjVEs1D/DNVmhVUfT/9T4mQCnymGwxtkDSPdh6 tGQycTDPSs5rpyAs1Z7mZ71Li6g9YoQ2T1rasfUMxID2E1+zcUUBQ5ZJ9mtr3xP1TAGjxMiKkTV zgQ6+CMzN4iqW7naaYkBJeYsPl/eUjP76+jCqKWYUImnLSv2omR/KnZb31N1wnMEydl2HTenJ/+ 3wXwJlCd5vnMpeA== X-Developer-Key: i=khilman@baylibre.com; a=openpgp; fpr=7B87460E16927FA9F5BFF10C5937189AD3FBC665 During system-wide suspend, check if any of the CPUs have PM QoS resume latency constraints set. If so, set TI SCI constraint. TI SCI has a single system-wide latency constraint, so use the max of any of the CPU latencies as the system-wide value. Note: DM firmware clears all constraints at resume time, so constraints need to be checked/updated/sent at each system suspend. Co-developed-by: Vibhore Vardhan Signed-off-by: Vibhore Vardhan Reviewed-by: Dhruva Gole Signed-off-by: Dhruva Gole Signed-off-by: Markus Schneider-Pargmann Signed-off-by: Kevin Hilman Tested-by: Dhruva Gole --- drivers/firmware/ti_sci.c | 24 +++++++++++++++++++++++- 1 file changed, 23 insertions(+), 1 deletion(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 107494406fa5..e0b11ca30af5 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware/ti_sci.c @@ -9,6 +9,7 @@ #define pr_fmt(fmt) "%s: " fmt, __func__ =20 #include +#include #include #include #include @@ -19,6 +20,7 @@ #include #include #include +#include #include #include #include @@ -3645,7 +3647,27 @@ static int ti_sci_prepare_system_suspend(struct ti_s= ci_info *info) static int ti_sci_suspend(struct device *dev) { struct ti_sci_info *info =3D dev_get_drvdata(dev); - int ret; + struct device *cpu_dev, *cpu_dev_max =3D NULL; + s32 val, cpu_lat =3D 0; + int i, ret; + + if (info->fw_caps & MSG_FLAG_CAPS_LPM_DM_MANAGED) { + for_each_possible_cpu(i) { + cpu_dev =3D get_cpu_device(i); + val =3D dev_pm_qos_read_value(cpu_dev, DEV_PM_QOS_RESUME_LATENCY); + if (val !=3D PM_QOS_RESUME_LATENCY_NO_CONSTRAINT) { + cpu_lat =3D max(cpu_lat, val); + cpu_dev_max =3D cpu_dev; + } + } + if (cpu_dev_max) { + dev_dbg(cpu_dev_max, "%s: sending max CPU latency=3D%u\n", __func__, cp= u_lat); + ret =3D ti_sci_cmd_set_latency_constraint(&info->handle, + cpu_lat, TISCI_MSG_CONSTRAINT_SET); + if (ret) + return ret; + } + } =20 ret =3D ti_sci_prepare_system_suspend(info); if (ret) --=20 2.46.0