From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 596E53BF69E; Wed, 17 Jun 2026 09:59:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690373; cv=none; b=Ly9q/ROZF/eSkS+dYs1h5ZaXt/1lVv3hs53uQu8eTCq2n1ALwpiB/simdfCtycIqe8SGZfuCqRVZ9qIuu7NGMZV8qFjW31qZSfwGrSoF+msvmrF2TPnyCN5A1erpmhZAVozEGicOh7k4pS05+SC4fc/F6qJuyDQej8RcRXPNjJU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690373; c=relaxed/simple; bh=v4PQvJxXL6yAb8eVisl3HMvO3Ev5F2132vGHAZ1Y3G8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=VEdSLRD3HkxRVJ1vhKPQiKrQFhy0+QuVDlurMycSlIHlTpsJW6I7104EHalgQhgHYWKTWnKNVtGApUBc1p7/X0U2eSaTpTKapPwXyXns2jBIyaQHC3l67+Lz6QMKhCQ59qXQ4IlLMzrHxowCSzveCTvk9jKlvRGY+Xa25AScl/k= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=ocKnKrUN; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="ocKnKrUN" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7AB124800; Wed, 17 Jun 2026 02:59:24 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id CC1E13F915; Wed, 17 Jun 2026 02:59:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690369; bh=v4PQvJxXL6yAb8eVisl3HMvO3Ev5F2132vGHAZ1Y3G8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ocKnKrUNbDSGDgA+dgTEXWVBhPgJ7ntsBDPjrnt8OqAIa8r32N4spvPd7My6PyJvx TUxBfqGpcPYMnbZlDMMzfcrE7k6FOTS8URajgVXp6ldMHyNNAvzzeqHVTrDFGOVYvG rVE5k76a00UbE9b6wqoWjPYkuEkvXnNHaUL0XZxY= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 01/13] powercap: Add enable disable control-type Date: Wed, 17 Jun 2026 10:58:58 +0100 Message-ID: <20260617095910.1963578-2-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add functionality to disable or enable the Powercap control-type by writing directly into sys/class/powercap/arm-scmi/enabled. Signed-off-by: Philip Radford --- drivers/powercap/arm_scmi_powercap.c | 155 ++++++++++++++++++++++++++- 1 file changed, 154 insertions(+), 1 deletion(-) diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index ab66e9a3b1e2..e33829d5c551 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -33,13 +33,151 @@ struct scmi_powercap_zone { =20 struct scmi_powercap_root { unsigned int num_zones; + bool enabled; + struct list_head node; struct scmi_powercap_zone *spzones; struct list_head *registered_zones; struct list_head scmi_zones; }; =20 +static LIST_HEAD(scmi_powercap_roots); +static DEFINE_MUTEX(scmi_powercap_roots_lock); + static struct powercap_control_type *scmi_top_pcntrl; =20 +static bool scmi_powercap_is_control_type_child(const struct scmi_powercap= _zone *spz) +{ + return spz->registered && !spz->invalid && + spz->info->parent_id =3D=3D SCMI_POWERCAP_ROOT_ZONE_ID; +} + +static int +scmi_powercap_read_root_children_enable_state(struct scmi_powercap_root *p= r, bool *mode) +{ + struct scmi_powercap_zone *spz; + bool enabled; + int i, ret; + + *mode =3D true; + + for (i =3D 0; i < pr->num_zones; i++) { + spz =3D &pr->spzones[i]; + + if (!scmi_powercap_is_control_type_child(spz)) + continue; + + ret =3D powercap_ops->cap_enable_get(spz->ph, spz->info->id, &enabled); + if (ret) + return ret; + + if (!enabled) { + *mode =3D false; + return 0; + } + } + + return 0; +} + +static int +scmi_powercap_set_root_children_enable_state(struct scmi_powercap_root *pr= , bool enable) +{ + struct scmi_powercap_zone *spz; + bool *prev_state; + int i, ret; + + prev_state =3D kcalloc(pr->num_zones, sizeof(*prev_state), GFP_KERNEL); + if (!prev_state) + return -ENOMEM; + + for (i =3D 0; i < pr->num_zones; i++) { + spz =3D &pr->spzones[i]; + + if (!scmi_powercap_is_control_type_child(spz)) + continue; + + ret =3D powercap_ops->cap_enable_get(spz->ph, spz->info->id, + &prev_state[i]); + + if (ret) + goto revert; + + if (prev_state[i] =3D=3D enable) + continue; + + ret =3D powercap_ops->cap_enable_set(spz->ph, spz->info->id, enable); + if (ret) + goto revert; + } + + pr->enabled =3D enable; + kfree(prev_state); + return 0; + +revert: + while (--i >=3D 0) { + spz =3D &pr->spzones[i]; + + if (!scmi_powercap_is_control_type_child(spz)) + continue; + if (!spz->info->powercap_cap_config) + continue; + if (prev_state[i] =3D=3D enable) + continue; + + powercap_ops->cap_enable_set(spz->ph, spz->info->id, prev_state[i]); + } + + kfree(prev_state); + return ret; +} + +static int +scmi_powercap_control_type_set_enable(struct powercap_control_type *pct, b= ool mode) +{ + struct scmi_powercap_root *pr; + int ret =3D 0; + + mutex_lock(&scmi_powercap_roots_lock); + list_for_each_entry(pr, &scmi_powercap_roots, node) { + ret =3D scmi_powercap_set_root_children_enable_state(pr, mode); + if (ret) + break; + } + mutex_unlock(&scmi_powercap_roots_lock); + + return ret; +} + +static int +scmi_powercap_control_type_get_enable(struct powercap_control_type *pct, b= ool *mode) +{ + struct scmi_powercap_root *pr; + int ret =3D 0; + + *mode =3D true; + + mutex_lock(&scmi_powercap_roots_lock); + list_for_each_entry(pr, &scmi_powercap_roots, node) { + ret =3D scmi_powercap_read_root_children_enable_state(pr, &pr->enabled); + + if (ret) + break; + if (!pr->enabled) { + *mode =3D false; + break; + } + } + mutex_unlock(&scmi_powercap_roots_lock); + + return ret; +} + +static const struct powercap_control_type_ops scmi_powercap_control_type_o= ps =3D { + .set_enable =3D scmi_powercap_control_type_set_enable, + .get_enable =3D scmi_powercap_control_type_get_enable, +}; + static int scmi_powercap_zone_release(struct powercap_zone *pz) { return 0; @@ -495,6 +633,16 @@ static int scmi_powercap_probe(struct scmi_device *sde= v) if (ret) return ret; =20 + INIT_LIST_HEAD(&pr->node); + + ret =3D scmi_powercap_read_root_children_enable_state(pr, &pr->enabled); + if (ret) + return ret; + + mutex_lock(&scmi_powercap_roots_lock); + list_add_tail(&pr->node, &scmi_powercap_roots); + mutex_unlock(&scmi_powercap_roots_lock); + dev_set_drvdata(dev, pr); =20 return ret; @@ -505,6 +653,10 @@ static void scmi_powercap_remove(struct scmi_device *s= dev) struct device *dev =3D &sdev->dev; struct scmi_powercap_root *pr =3D dev_get_drvdata(dev); =20 + mutex_lock(&scmi_powercap_roots_lock); + list_del(&pr->node); + mutex_unlock(&scmi_powercap_roots_lock); + scmi_powercap_unregister_all_zones(pr); } =20 @@ -525,7 +677,8 @@ static int __init scmi_powercap_init(void) { int ret; =20 - scmi_top_pcntrl =3D powercap_register_control_type(NULL, "arm-scmi", NULL= ); + scmi_top_pcntrl =3D powercap_register_control_type(NULL, "arm-scmi", + &scmi_powercap_control_type_ops); if (IS_ERR(scmi_top_pcntrl)) return PTR_ERR(scmi_top_pcntrl); =20 --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 962A33D3CEC; Wed, 17 Jun 2026 09:59:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690375; cv=none; b=f/4w4mVkX8KXX69FU6AdRmx9JJJ4hJ+6pZufNcailDfbwbtDLRwWnwTO/OTONdSDyoCaONGDdoYtUWPSVt+9ZEQib5hItetqe0OTO0PrPVMlHC043VOlfuraSxbYUwsMdyAW9B03YvMxObRYoWnec1wCuX/tqy9QnlpOtohQDjI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690375; c=relaxed/simple; bh=kb+faEBQwpa+6x5yiLQOyunnwPeV1UaGWYCXwUJdJ0k=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=VkBHqrUgD8EOMHV/akJ9c2yBu/EW89nxst6NVthK66EVjNKW3tdKiQTAk7nk8q9Ge/K5PvqLM5j9/NMitsMC8camEuvivnNfKwQWrcAIbUB1d/zdE6i4OSH/tgmRpidk8ihn8qPoyMMDxxc6ohtL99oP/+kodK+R5aZNoGrWLIg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=Kc7lGPtR; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="Kc7lGPtR" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 51F392E8E; Wed, 17 Jun 2026 02:59:27 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 7D5DE3F915; Wed, 17 Jun 2026 02:59:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690372; bh=kb+faEBQwpa+6x5yiLQOyunnwPeV1UaGWYCXwUJdJ0k=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Kc7lGPtRhEDKZSRTGxhg2k9a6y7tUwgejqrCwwnF8y1Rkk7Uow2Ns+INiI4du/GlE vU6p0y14ErcD8Yv8UQ5wvv69FObdNNK5pNktU0b7Ul9TxEo/4pq7iZcL+fZai2l0XM F37HvGbD8CkesJ98yfTG06IjJYJyg2NdQSe42I8k= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 02/13] firmware: arm_scmi: Add an optional custom parameter to fastchannel helpers Date: Wed, 17 Jun 2026 10:58:59 +0100 Message-ID: <20260617095910.1963578-3-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Starting from SCMIv4.0 the protocols DESCRIBE_FASTCHANNEL commands allow to specify one additional per-protocol custom field in the outgoing message request in order to, optionally, further narrow down the scope of the fastchannel discovery request; the related message-reply format is instead unchanged. Add an optional custom protocol parameter to the common fastchannel helper so as to enable the caller to choose the kind of message to send based on the detected protocol version. Signed-off-by: Cristian Marussi Signed-off-by: Philip Radford --- drivers/firmware/arm_scmi/driver.c | 12 ++++++++++-- drivers/firmware/arm_scmi/perf.c | 8 ++++---- drivers/firmware/arm_scmi/powercap.c | 8 ++++---- drivers/firmware/arm_scmi/protocols.h | 2 +- 4 files changed, 19 insertions(+), 11 deletions(-) diff --git a/drivers/firmware/arm_scmi/driver.c b/drivers/firmware/arm_scmi= /driver.c index f247e19cff87..53ae4cfa10b1 100644 --- a/drivers/firmware/arm_scmi/driver.c +++ b/drivers/firmware/arm_scmi/driver.c @@ -1872,6 +1872,11 @@ static int scmi_iterator_run(void *iter) struct scmi_msg_get_fc_info { __le32 domain; __le32 message_id; + __le32 custom; +#define MSG_FC_INFO_SZ_EXTENDED \ + (sizeof(struct scmi_msg_get_fc_info)) +#define MSG_FC_INFO_SZ \ + (sizeof(struct scmi_msg_get_fc_info) - sizeof(__le32)) }; =20 struct scmi_msg_resp_desc_fc { @@ -1900,7 +1905,7 @@ struct scmi_msg_resp_desc_fc { static void scmi_common_fastchannel_init(const struct scmi_protocol_handle *ph, u8 describe_id, u32 message_id, u32 valid_size, - u32 domain, void __iomem **p_addr, + u32 domain, u32 *custom, void __iomem **p_addr, struct scmi_fc_db_info **p_db, u32 *rate_limit) { int ret; @@ -1931,13 +1936,16 @@ scmi_common_fastchannel_init(const struct scmi_prot= ocol_handle *ph, } =20 ret =3D ph->xops->xfer_get_init(ph, describe_id, - sizeof(*info), sizeof(*resp), &t); + custom ? MSG_FC_INFO_SZ_EXTENDED : + MSG_FC_INFO_SZ, sizeof(*resp), &t); if (ret) goto err_out; =20 info =3D t->tx.buf; info->domain =3D cpu_to_le32(domain); info->message_id =3D cpu_to_le32(message_id); + if (custom) + info->custom =3D cpu_to_le32(*custom); =20 /* * Bail out on error leaving fc_info addresses zeroed; this includes diff --git a/drivers/firmware/arm_scmi/perf.c b/drivers/firmware/arm_scmi/p= erf.c index 4583d02bee1c..7f283f457e02 100644 --- a/drivers/firmware/arm_scmi/perf.c +++ b/drivers/firmware/arm_scmi/perf.c @@ -835,25 +835,25 @@ static void scmi_perf_domain_init_fc(const struct scm= i_protocol_handle *ph, return; =20 ph->hops->fastchannel_init(ph, PERF_DESCRIBE_FASTCHANNEL, - PERF_LEVEL_GET, 4, dom->id, + PERF_LEVEL_GET, 4, dom->id, NULL, &fc[PERF_FC_LEVEL].get_addr, NULL, &fc[PERF_FC_LEVEL].rate_limit); =20 ph->hops->fastchannel_init(ph, PERF_DESCRIBE_FASTCHANNEL, - PERF_LIMITS_GET, 8, dom->id, + PERF_LIMITS_GET, 8, dom->id, NULL, &fc[PERF_FC_LIMIT].get_addr, NULL, &fc[PERF_FC_LIMIT].rate_limit); =20 if (dom->info.set_perf) ph->hops->fastchannel_init(ph, PERF_DESCRIBE_FASTCHANNEL, - PERF_LEVEL_SET, 4, dom->id, + PERF_LEVEL_SET, 4, dom->id, NULL, &fc[PERF_FC_LEVEL].set_addr, &fc[PERF_FC_LEVEL].set_db, &fc[PERF_FC_LEVEL].rate_limit); =20 if (dom->set_limits) ph->hops->fastchannel_init(ph, PERF_DESCRIBE_FASTCHANNEL, - PERF_LIMITS_SET, 8, dom->id, + PERF_LIMITS_SET, 8, dom->id, NULL, &fc[PERF_FC_LIMIT].set_addr, &fc[PERF_FC_LIMIT].set_db, &fc[PERF_FC_LIMIT].rate_limit); diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index ab9733f4458b..22aff71c75e9 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -716,24 +716,24 @@ static void scmi_powercap_domain_init_fc(const struct= scmi_protocol_handle *ph, return; =20 ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_CAP_SET, 4, domain, + POWERCAP_CAP_SET, 4, domain, NULL, &fc[POWERCAP_FC_CAP].set_addr, &fc[POWERCAP_FC_CAP].set_db, &fc[POWERCAP_FC_CAP].rate_limit); =20 ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_CAP_GET, 4, domain, + POWERCAP_CAP_GET, 4, domain, NULL, &fc[POWERCAP_FC_CAP].get_addr, NULL, &fc[POWERCAP_FC_CAP].rate_limit); =20 ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_PAI_SET, 4, domain, + POWERCAP_PAI_SET, 4, domain, NULL, &fc[POWERCAP_FC_PAI].set_addr, &fc[POWERCAP_FC_PAI].set_db, &fc[POWERCAP_FC_PAI].rate_limit); =20 ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_PAI_GET, 4, domain, + POWERCAP_PAI_GET, 4, domain, NULL, &fc[POWERCAP_FC_PAI].get_addr, NULL, &fc[POWERCAP_FC_PAI].rate_limit); =20 diff --git a/drivers/firmware/arm_scmi/protocols.h b/drivers/firmware/arm_s= cmi/protocols.h index f51245aca259..e618d3141c95 100644 --- a/drivers/firmware/arm_scmi/protocols.h +++ b/drivers/firmware/arm_scmi/protocols.h @@ -280,7 +280,7 @@ struct scmi_proto_helpers_ops { u32 message_id, u32 *attributes); void (*fastchannel_init)(const struct scmi_protocol_handle *ph, u8 describe_id, u32 message_id, - u32 valid_size, u32 domain, + u32 valid_size, u32 domain, u32 *custom, void __iomem **p_addr, struct scmi_fc_db_info **p_db, u32 *rate_limit); --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7CBA83D45EA; Wed, 17 Jun 2026 09:59:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690381; cv=none; b=IBpgh220vyKBcMEWRIIbe904yrJHK2rdexqhiGHZ6/udwjj7vdbhcNlgE6Bbu8KTS/2zRcap5TL/iNi94RAOS9dL3Cr7dRyNPPm1uxBqcHa2dxh56r9pQoo3kuFKg7ENKtfSEkGrnGF8d+7cvFn55hDms6b+nlY9Y8CIlLJ9x5M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690381; c=relaxed/simple; bh=Xx5gG3JAOtplC5xPvLIQ7CbkmMwVA6vQATitRGWwIFA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=nxVLkuVmVGRMxBcjP8JBEGxN+w/jWTXCwq1ZH90AT7BcDBECxNmBDqu5/nVMcJms4Sot8kwenizN3R4i8SwabHraESW5lexccv4js8o3njfdV769EcBsgUsEpYvx3dy/5Y7Z50q1295QDrz5II1OXK1v3NtCYsbJcRENGFFXpW0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=fjjAU5Tc; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="fjjAU5Tc" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 401634800; Wed, 17 Jun 2026 02:59:30 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 53A853F915; Wed, 17 Jun 2026 02:59:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690374; bh=Xx5gG3JAOtplC5xPvLIQ7CbkmMwVA6vQATitRGWwIFA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fjjAU5TcgKA3ScKzJxwdMoyOrl4brL9lf+QHJxdZtUn3MyagjnKPJWQX7a0wpJads HA9AOwK7ngAdNmsLe3rDn6szoeTbl04JtipCLEShVvEUwb8cTjqT9CT9FkW9nbFl7L p3TA2oVHU900kVPmHu0jq6M1YrEJuN7A2Cbe50sE= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 03/13] firmware: arm_scmi: Refactor powercap domain layout Date: Wed, 17 Jun 2026 10:59:00 +0100 Message-ID: <20260617095910.1963578-4-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi SCMIv4.0 introduces the idea of an optional Concurrent Power Limit (CPL) for each powercap domain, where CPL0 coincides with the one and only per-domain constraint limit that was available in pre-v4.0 SCMI Powercap. Refactor the powercap domain descriptors and powercap operations to allow future v4.0 extensions to cope with multiple CPLs. While at that generalize the powercap protocol API to drop PAI references in favour of a more generic avg_ivl naming, since from v4.0 the number and types of averaging intervals will change in a non-backward compatible way, so let's bury these changes within the protocol layer. Last but not least, make the necessary changes to the ARM SCMI Powwercap driver in order to support all of these new capabilities. No functional change. Signed-off-by: Cristian Marussi [Philip: Adjusted domain_id comparitor in scmi_powercap_pai_get] Signed-off-by: Philip Radford --- drivers/firmware/arm_scmi/powercap.c | 182 +++++++++++++++++---------- drivers/powercap/arm_scmi_powercap.c | 52 ++++---- include/linux/scmi_protocol.h | 74 +++++++---- 3 files changed, 189 insertions(+), 119 deletions(-) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index 22aff71c75e9..47aa6dde4a52 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -2,7 +2,7 @@ /* * System Control and Management Interface (SCMI) Powercap Protocol * - * Copyright (C) 2022 ARM Ltd. + * Copyright (C) 2022-2026 ARM Ltd. */ =20 #define pr_fmt(fmt) "SCMI Notifications POWERCAP - " fmt @@ -20,6 +20,8 @@ /* Updated only after ALL the mandatory features for that version are merg= ed */ #define SCMI_PROTOCOL_SUPPORTED_VERSION 0x20000 =20 +#define CPL0 0 + enum scmi_powercap_protocol_cmd { POWERCAP_DOMAIN_ATTRIBUTES =3D 0x3, POWERCAP_CAP_GET =3D 0x4, @@ -192,27 +194,26 @@ scmi_powercap_validate(unsigned int min_val, unsigned= int max_val, =20 static int scmi_powercap_domain_attributes_get(const struct scmi_protocol_handle *ph, - struct powercap_info *pinfo, u32 domain) + struct powercap_info *pinfo, + struct scmi_powercap_info *dom_info) { int ret; u32 flags; struct scmi_xfer *t; - struct scmi_powercap_info *dom_info =3D pinfo->powercaps + domain; struct scmi_msg_resp_powercap_domain_attributes *resp; =20 ret =3D ph->xops->xfer_get_init(ph, POWERCAP_DOMAIN_ATTRIBUTES, - sizeof(domain), sizeof(*resp), &t); + sizeof(dom_info->id), sizeof(*resp), &t); if (ret) return ret; =20 - put_unaligned_le32(domain, t->tx.buf); + put_unaligned_le32(dom_info->id, t->tx.buf); resp =3D t->rx.buf; =20 ret =3D ph->xops->do_xfer(ph, t); if (!ret) { flags =3D le32_to_cpu(resp->attributes); =20 - dom_info->id =3D domain; if (pinfo->notify_cap_cmd) dom_info->notify_powercap_cap_change =3D SUPPORTS_POWERCAP_CAP_CHANGE_NOTIFY(flags); @@ -221,12 +222,9 @@ scmi_powercap_domain_attributes_get(const struct scmi_= protocol_handle *ph, SUPPORTS_POWERCAP_MEASUREMENTS_CHANGE_NOTIFY(flags); dom_info->async_powercap_cap_set =3D SUPPORTS_ASYNC_POWERCAP_CAP_SET(flags); - dom_info->powercap_cap_config =3D - SUPPORTS_POWERCAP_CAP_CONFIGURATION(flags); + dom_info->powercap_monitoring =3D SUPPORTS_POWERCAP_MONITORING(flags); - dom_info->powercap_pai_config =3D - SUPPORTS_POWERCAP_PAI_CONFIGURATION(flags); dom_info->powercap_scale_mw =3D SUPPORTS_POWER_UNITS_MW(flags); dom_info->powercap_scale_uw =3D @@ -236,13 +234,29 @@ scmi_powercap_domain_attributes_get(const struct scmi= _protocol_handle *ph, =20 strscpy(dom_info->name, resp->name, SCMI_SHORT_NAME_MAX_SIZE); =20 - dom_info->min_pai =3D le32_to_cpu(resp->min_pai); - dom_info->max_pai =3D le32_to_cpu(resp->max_pai); - dom_info->pai_step =3D le32_to_cpu(resp->pai_step); - ret =3D scmi_powercap_validate(dom_info->min_pai, - dom_info->max_pai, - dom_info->pai_step, - dom_info->powercap_pai_config); + dom_info->sustainable_power =3D + le32_to_cpu(resp->sustainable_power); + dom_info->accuracy =3D le32_to_cpu(resp->accuracy); + + dom_info->parent_id =3D le32_to_cpu(resp->parent_id); + if (dom_info->parent_id !=3D SCMI_POWERCAP_ROOT_ZONE_ID && + (dom_info->parent_id >=3D pinfo->num_domains || + dom_info->parent_id =3D=3D dom_info->id)) { + dev_err(ph->dev, + "Platform reported inconsistent parent ID for domain %d - %s\n", + dom_info->id, dom_info->name); + ret =3D -ENODEV; + } + + dom_info->cpli[0].avg_ivl_config =3D + SUPPORTS_POWERCAP_PAI_CONFIGURATION(flags); + dom_info->cpli[0].min_avg_ivl =3D le32_to_cpu(resp->min_pai); + dom_info->cpli[0].max_avg_ivl =3D le32_to_cpu(resp->max_pai); + dom_info->cpli[0].avg_ivl_step =3D le32_to_cpu(resp->pai_step); + ret =3D scmi_powercap_validate(dom_info->cpli[0].min_avg_ivl, + dom_info->cpli[0].max_avg_ivl, + dom_info->cpli[0].avg_ivl_step, + dom_info->cpli[0].avg_ivl_config); if (ret) { dev_err(ph->dev, "Platform reported inconsistent PAI config for domain %d - %s\n", @@ -250,13 +264,15 @@ scmi_powercap_domain_attributes_get(const struct scmi= _protocol_handle *ph, goto clean; } =20 - dom_info->min_power_cap =3D le32_to_cpu(resp->min_power_cap); - dom_info->max_power_cap =3D le32_to_cpu(resp->max_power_cap); - dom_info->power_cap_step =3D le32_to_cpu(resp->power_cap_step); - ret =3D scmi_powercap_validate(dom_info->min_power_cap, - dom_info->max_power_cap, - dom_info->power_cap_step, - dom_info->powercap_cap_config); + dom_info->cpli[0].cap_config =3D + SUPPORTS_POWERCAP_CAP_CONFIGURATION(flags); + dom_info->cpli[0].min_power_cap =3D le32_to_cpu(resp->min_power_cap); + dom_info->cpli[0].max_power_cap =3D le32_to_cpu(resp->max_power_cap); + dom_info->cpli[0].power_cap_step =3D le32_to_cpu(resp->power_cap_step); + ret =3D scmi_powercap_validate(dom_info->cpli[0].min_power_cap, + dom_info->cpli[0].max_power_cap, + dom_info->cpli[0].power_cap_step, + dom_info->cpli[0].cap_config); if (ret) { dev_err(ph->dev, "Platform reported inconsistent CAP config for domain %d - %s\n", @@ -264,19 +280,9 @@ scmi_powercap_domain_attributes_get(const struct scmi_= protocol_handle *ph, goto clean; } =20 - dom_info->sustainable_power =3D - le32_to_cpu(resp->sustainable_power); - dom_info->accuracy =3D le32_to_cpu(resp->accuracy); - - dom_info->parent_id =3D le32_to_cpu(resp->parent_id); - if (dom_info->parent_id !=3D SCMI_POWERCAP_ROOT_ZONE_ID && - (dom_info->parent_id >=3D pinfo->num_domains || - dom_info->parent_id =3D=3D dom_info->id)) { - dev_err(ph->dev, - "Platform reported inconsistent parent ID for domain %d - %s\n", - dom_info->id, dom_info->name); - ret =3D -ENODEV; - } + /* Just using same short name */ + strscpy(dom_info->cpli[0].name, dom_info->name, + SCMI_SHORT_NAME_MAX_SIZE); } =20 clean: @@ -288,12 +294,30 @@ scmi_powercap_domain_attributes_get(const struct scmi= _protocol_handle *ph, */ if (!ret && SUPPORTS_EXTENDED_NAMES(flags)) ph->hops->extended_name_get(ph, POWERCAP_DOMAIN_NAME_GET, - domain, NULL, dom_info->name, + dom_info->id, NULL, dom_info->name, SCMI_MAX_STR_SIZE); =20 return ret; } =20 +static int +scmi_powercap_domain_initialize(const struct scmi_protocol_handle *ph, + struct powercap_info *pinfo, u32 domain) +{ + struct scmi_powercap_info *dom_info =3D pinfo->powercaps + domain; + + dom_info->num_cpli =3D 1; + dom_info->cpli =3D devm_kcalloc(ph->dev, dom_info->num_cpli, + sizeof(*dom_info->cpli), GFP_KERNEL); + if (!dom_info->cpli) + return -ENOMEM; + + dom_info->id =3D domain; + dom_info->cpli[0].id =3D CPL0; + + return scmi_powercap_domain_attributes_get(ph, pinfo, dom_info); +} + static int scmi_powercap_num_domains_get(const struct scmi_protocol_handle= *ph) { struct powercap_info *pi =3D ph->get_priv(ph); @@ -335,10 +359,11 @@ static int scmi_powercap_xfer_cap_get(const struct sc= mi_protocol_handle *ph, =20 static int __scmi_powercap_cap_get(const struct scmi_protocol_handle *ph, const struct scmi_powercap_info *dom, - u32 *power_cap) + u32 cpl_id, u32 *power_cap) { - if (dom->fc_info && dom->fc_info[POWERCAP_FC_CAP].get_addr) { - *power_cap =3D ioread32(dom->fc_info[POWERCAP_FC_CAP].get_addr); + if (dom->cpli[cpl_id].fc_info && + dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_addr) { + *power_cap =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_a= ddr); trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_CAP_GET, dom->id, *power_cap, 0); return 0; @@ -348,7 +373,7 @@ static int __scmi_powercap_cap_get(const struct scmi_pr= otocol_handle *ph, } =20 static int scmi_powercap_cap_get(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 *power_cap) + u32 domain_id, u32 cpl_id, u32 *power_cap) { const struct scmi_powercap_info *dom; =20 @@ -359,12 +384,13 @@ static int scmi_powercap_cap_get(const struct scmi_pr= otocol_handle *ph, if (!dom) return -EINVAL; =20 - return __scmi_powercap_cap_get(ph, dom, power_cap); + return __scmi_powercap_cap_get(ph, dom, cpl_id, power_cap); } =20 static int scmi_powercap_xfer_cap_set(const struct scmi_protocol_handle *p= h, const struct scmi_powercap_info *pc, - u32 power_cap, bool ignore_dresp) + u32 cpl_id, u32 power_cap, + bool ignore_dresp) { int ret; struct scmi_xfer *t; @@ -406,21 +432,23 @@ static int scmi_powercap_xfer_cap_set(const struct sc= mi_protocol_handle *ph, =20 static int __scmi_powercap_cap_set(const struct scmi_protocol_handle *ph, struct powercap_info *pi, u32 domain_id, - u32 power_cap, bool ignore_dresp) + u32 cpl_id, u32 power_cap, bool ignore_dresp) { int ret =3D -EINVAL; const struct scmi_powercap_info *pc; =20 pc =3D scmi_powercap_dom_info_get(ph, domain_id); - if (!pc || !pc->powercap_cap_config) + if (!pc || !pc->cpli[cpl_id].cap_config) return ret; =20 if (power_cap && - (power_cap < pc->min_power_cap || power_cap > pc->max_power_cap)) + (power_cap < pc->cpli[cpl_id].min_power_cap || + power_cap > pc->cpli[cpl_id].max_power_cap)) return ret; =20 - if (pc->fc_info && pc->fc_info[POWERCAP_FC_CAP].set_addr) { - struct scmi_fc_info *fci =3D &pc->fc_info[POWERCAP_FC_CAP]; + if (pc->cpli[cpl_id].fc_info && + pc->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].set_addr) { + struct scmi_fc_info *fci =3D &pc->cpli[cpl_id].fc_info[POWERCAP_FC_CAP]; =20 iowrite32(power_cap, fci->set_addr); ph->hops->fastchannel_db_ring(fci->set_db); @@ -428,7 +456,7 @@ static int __scmi_powercap_cap_set(const struct scmi_pr= otocol_handle *ph, domain_id, power_cap, 0); ret =3D 0; } else { - ret =3D scmi_powercap_xfer_cap_set(ph, pc, power_cap, + ret =3D scmi_powercap_xfer_cap_set(ph, pc, cpl_id, power_cap, ignore_dresp); } =20 @@ -440,7 +468,7 @@ static int __scmi_powercap_cap_set(const struct scmi_pr= otocol_handle *ph, } =20 static int scmi_powercap_cap_set(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 power_cap, + u32 domain_id, u32 cpl_id, u32 power_cap, bool ignore_dresp) { struct powercap_info *pi =3D ph->get_priv(ph); @@ -459,7 +487,7 @@ static int scmi_powercap_cap_set(const struct scmi_prot= ocol_handle *ph, return 0; } =20 - return __scmi_powercap_cap_set(ph, pi, domain_id, + return __scmi_powercap_cap_set(ph, pi, domain_id, cpl_id, power_cap, ignore_dresp); } =20 @@ -485,7 +513,7 @@ static int scmi_powercap_xfer_pai_get(const struct scmi= _protocol_handle *ph, } =20 static int scmi_powercap_pai_get(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 *pai) + u32 domain_id, u32 cpl_id, u32 *pai) { struct scmi_powercap_info *dom; struct powercap_info *pi =3D ph->get_priv(ph); @@ -494,8 +522,11 @@ static int scmi_powercap_pai_get(const struct scmi_pro= tocol_handle *ph, return -EINVAL; =20 dom =3D pi->powercaps + domain_id; - if (dom->fc_info && dom->fc_info[POWERCAP_FC_PAI].get_addr) { - *pai =3D ioread32(dom->fc_info[POWERCAP_FC_PAI].get_addr); + if (cpl_id >=3D dom->num_cpli) + return -EINVAL; + + if (dom->cpli[cpl_id].fc_info && dom->cpli[cpl_id].fc_info[POWERCAP_FC_PA= I].get_addr) { + *pai =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_PAI].get_addr); trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_PAI_GET, domain_id, *pai, 0); return 0; @@ -504,6 +535,12 @@ static int scmi_powercap_pai_get(const struct scmi_pro= tocol_handle *ph, return scmi_powercap_xfer_pai_get(ph, domain_id, pai); } =20 +static int scmi_powercap_avg_interval_get(const struct scmi_protocol_handl= e *ph, + u32 domain_id, u32 cpl_id, u32 *val) +{ + return scmi_powercap_pai_get(ph, domain_id, cpl_id, val); +} + static int scmi_powercap_xfer_pai_set(const struct scmi_protocol_handle *p= h, u32 domain_id, u32 pai) { @@ -528,17 +565,18 @@ static int scmi_powercap_xfer_pai_set(const struct sc= mi_protocol_handle *ph, } =20 static int scmi_powercap_pai_set(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 pai) + u32 domain_id, u32 cpl_id, u32 pai) { const struct scmi_powercap_info *pc; =20 pc =3D scmi_powercap_dom_info_get(ph, domain_id); - if (!pc || !pc->powercap_pai_config || !pai || - pai < pc->min_pai || pai > pc->max_pai) + if (!pc || cpl_id >=3D pc->num_cpli || !pc->cpli[cpl_id].avg_ivl_config || + !pai || pai < pc->cpli[cpl_id].min_avg_ivl || + pai > pc->cpli[cpl_id].max_avg_ivl) return -EINVAL; =20 - if (pc->fc_info && pc->fc_info[POWERCAP_FC_PAI].set_addr) { - struct scmi_fc_info *fci =3D &pc->fc_info[POWERCAP_FC_PAI]; + if (pc->cpli[cpl_id].fc_info && pc->cpli[cpl_id].fc_info[POWERCAP_FC_PAI]= .set_addr) { + struct scmi_fc_info *fci =3D &pc->cpli[cpl_id].fc_info[POWERCAP_FC_PAI]; =20 trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_PAI_SET, domain_id, pai, 0); @@ -550,6 +588,12 @@ static int scmi_powercap_pai_set(const struct scmi_pro= tocol_handle *ph, return scmi_powercap_xfer_pai_set(ph, domain_id, pai); } =20 +static int scmi_powercap_avg_interval_set(const struct scmi_protocol_handl= e *ph, + u32 domain_id, u32 cpl_id, u32 val) +{ + return scmi_powercap_pai_set(ph, domain_id, cpl_id, val); +} + static int scmi_powercap_measurements_get(const struct scmi_protocol_handl= e *ph, u32 domain_id, u32 *average_power, u32 *pai) @@ -645,11 +689,11 @@ static int scmi_powercap_cap_enable_set(const struct = scmi_protocol_handle *ph, if (!pi->states[domain_id].last_pcap) return -EINVAL; =20 - ret =3D __scmi_powercap_cap_set(ph, pi, domain_id, + ret =3D __scmi_powercap_cap_set(ph, pi, domain_id, CPL0, pi->states[domain_id].last_pcap, true); } else { - ret =3D __scmi_powercap_cap_set(ph, pi, domain_id, 0, true); + ret =3D __scmi_powercap_cap_set(ph, pi, domain_id, CPL0, 0, true); } =20 if (ret) @@ -660,7 +704,7 @@ static int scmi_powercap_cap_enable_set(const struct sc= mi_protocol_handle *ph, * server could have ignored a disable request and kept enforcing some * powercap limit requested by other agents. */ - ret =3D scmi_powercap_cap_get(ph, domain_id, &power_cap); + ret =3D scmi_powercap_cap_get(ph, domain_id, CPL0, &power_cap); if (!ret) pi->states[domain_id].enabled =3D !!power_cap; =20 @@ -682,7 +726,7 @@ static int scmi_powercap_cap_enable_get(const struct sc= mi_protocol_handle *ph, * Report always real platform state; platform could have ignored * a previous disable request. Default true on any error. */ - ret =3D scmi_powercap_cap_get(ph, domain_id, &power_cap); + ret =3D scmi_powercap_cap_get(ph, domain_id, CPL0, &power_cap); if (!ret) *enable =3D !!power_cap; =20 @@ -699,8 +743,8 @@ static const struct scmi_powercap_proto_ops powercap_pr= oto_ops =3D { .cap_set =3D scmi_powercap_cap_set, .cap_enable_set =3D scmi_powercap_cap_enable_set, .cap_enable_get =3D scmi_powercap_cap_enable_get, - .pai_get =3D scmi_powercap_pai_get, - .pai_set =3D scmi_powercap_pai_set, + .avg_interval_get =3D scmi_powercap_avg_interval_get, + .avg_interval_set =3D scmi_powercap_avg_interval_set, .measurements_get =3D scmi_powercap_measurements_get, .measurements_threshold_set =3D scmi_powercap_measurements_threshold_set, .measurements_threshold_get =3D scmi_powercap_measurements_threshold_get, @@ -991,18 +1035,18 @@ scmi_powercap_protocol_init(const struct scmi_protoc= ol_handle *ph) * formed and correlated by sane parent-child relationship (if any). */ for (domain =3D 0; domain < pinfo->num_domains; domain++) { - ret =3D scmi_powercap_domain_attributes_get(ph, pinfo, domain); + ret =3D scmi_powercap_domain_initialize(ph, pinfo, domain); if (ret) return ret; =20 if (pinfo->powercaps[domain].fastchannels) scmi_powercap_domain_init_fc(ph, domain, - &pinfo->powercaps[domain].fc_info); + &pinfo->powercaps[domain].cpli[CPL0].fc_info); =20 /* Grab initial state when disable is supported. */ if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2) { ret =3D __scmi_powercap_cap_get(ph, - &pinfo->powercaps[domain], + &pinfo->powercaps[domain], CPL0, &pinfo->states[domain].last_pcap); if (ret) return ret; diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index e33829d5c551..90d1fa70b1d4 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -120,7 +120,7 @@ scmi_powercap_set_root_children_enable_state(struct scm= i_powercap_root *pr, bool =20 if (!scmi_powercap_is_control_type_child(spz)) continue; - if (!spz->info->powercap_cap_config) + if (!spz->info->cpli[0].cap_config) continue; if (prev_state[i] =3D=3D enable) continue; @@ -235,7 +235,7 @@ static const struct powercap_zone_ops zone_ops =3D { }; =20 static void scmi_powercap_normalize_cap(const struct scmi_powercap_zone *s= pz, - u64 power_limit_uw, u32 *norm) + u64 power_limit_uw, int cid, u32 *norm) { bool scale_mw =3D spz->info->powercap_scale_mw; u64 val; @@ -246,9 +246,9 @@ static void scmi_powercap_normalize_cap(const struct sc= mi_powercap_zone *spz, * the range [min_power_cap, max_power_cap] whose bounds are assured to * be two unsigned 32bits quantities. */ - *norm =3D clamp_t(u32, val, spz->info->min_power_cap, - spz->info->max_power_cap); - *norm =3D rounddown(*norm, spz->info->power_cap_step); + *norm =3D clamp_t(u32, val, spz->info->cpli[cid].min_power_cap, + spz->info->cpli[cid].max_power_cap); + *norm =3D rounddown(*norm, spz->info->cpli[cid].power_cap_step); =20 val =3D (scale_mw) ? *norm * 1000 : *norm; if (power_limit_uw !=3D val) @@ -263,12 +263,12 @@ static int scmi_powercap_set_power_limit_uw(struct po= wercap_zone *pz, int cid, struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); u32 norm_power; =20 - if (!spz->info->powercap_cap_config) + if (!spz->info->cpli[cid].cap_config) return -EINVAL; =20 - scmi_powercap_normalize_cap(spz, power_uw, &norm_power); + scmi_powercap_normalize_cap(spz, power_uw, cid, &norm_power); =20 - return powercap_ops->cap_set(spz->ph, spz->info->id, norm_power, false); + return powercap_ops->cap_set(spz->ph, spz->info->id, cid, norm_power, fal= se); } =20 static int scmi_powercap_get_power_limit_uw(struct powercap_zone *pz, int = cid, @@ -278,7 +278,7 @@ static int scmi_powercap_get_power_limit_uw(struct powe= rcap_zone *pz, int cid, u32 power; int ret; =20 - ret =3D powercap_ops->cap_get(spz->ph, spz->info->id, &power); + ret =3D powercap_ops->cap_get(spz->ph, spz->info->id, cid, &power); if (ret) return ret; =20 @@ -290,19 +290,20 @@ static int scmi_powercap_get_power_limit_uw(struct po= wercap_zone *pz, int cid, } =20 static void scmi_powercap_normalize_time(const struct scmi_powercap_zone *= spz, - u64 time_us, u32 *norm) + u64 time_us, int cid, u32 *norm) { /* * This cast is lossless since here @time_us is certain to be within the - * range [min_pai, max_pai] whose bounds are assured to be two unsigned - * 32bits quantities. + * range [min_avg_ivl, max_avg_ivl] whose bounds are assured to be two + * unsigned 32bits quantities. */ - *norm =3D clamp_t(u32, time_us, spz->info->min_pai, spz->info->max_pai); - *norm =3D rounddown(*norm, spz->info->pai_step); + *norm =3D clamp_t(u32, time_us, spz->info->cpli[cid].min_avg_ivl, + spz->info->cpli[cid].max_avg_ivl); + *norm =3D rounddown(*norm, spz->info->cpli[cid].avg_ivl_step); =20 if (time_us !=3D *norm) dev_dbg(spz->dev, - "Normalized %s:PAI - requested:%llu - normalized:%u\n", + "Normalized %s:AVG_IVL - requested:%llu - normalized:%u\n", spz->info->name, time_us, *norm); } =20 @@ -312,12 +313,13 @@ static int scmi_powercap_set_time_window_us(struct po= wercap_zone *pz, int cid, struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); u32 norm_pai; =20 - if (!spz->info->powercap_pai_config) + if (!spz->info->cpli[cid].avg_ivl_config) return -EINVAL; =20 - scmi_powercap_normalize_time(spz, time_window_us, &norm_pai); + scmi_powercap_normalize_time(spz, time_window_us, cid, &norm_pai); =20 - return powercap_ops->pai_set(spz->ph, spz->info->id, norm_pai); + return powercap_ops->avg_interval_set(spz->ph, spz->info->id, + cid, norm_pai); } =20 static int scmi_powercap_get_time_window_us(struct powercap_zone *pz, int = cid, @@ -327,7 +329,7 @@ static int scmi_powercap_get_time_window_us(struct powe= rcap_zone *pz, int cid, int ret; u32 pai; =20 - ret =3D powercap_ops->pai_get(spz->ph, spz->info->id, &pai); + ret =3D powercap_ops->avg_interval_get(spz->ph, spz->info->id, cid, &pai); if (ret) return ret; =20 @@ -341,7 +343,7 @@ static int scmi_powercap_get_max_power_uw(struct powerc= ap_zone *pz, int cid, { struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); =20 - *max_power_uw =3D spz->info->max_power_cap; + *max_power_uw =3D spz->info->cpli[cid].max_power_cap; if (spz->info->powercap_scale_mw) *max_power_uw *=3D 1000; =20 @@ -353,7 +355,7 @@ static int scmi_powercap_get_min_power_uw(struct powerc= ap_zone *pz, int cid, { struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); =20 - *min_power_uw =3D spz->info->min_power_cap; + *min_power_uw =3D spz->info->cpli[cid].min_power_cap; if (spz->info->powercap_scale_mw) *min_power_uw *=3D 1000; =20 @@ -365,7 +367,7 @@ static int scmi_powercap_get_max_time_window_us(struct = powercap_zone *pz, { struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); =20 - *time_window_us =3D spz->info->max_pai; + *time_window_us =3D spz->info->cpli[cid].max_avg_ivl; =20 return 0; } @@ -375,14 +377,16 @@ static int scmi_powercap_get_min_time_window_us(struc= t powercap_zone *pz, { struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); =20 - *time_window_us =3D (u64)spz->info->min_pai; + *time_window_us =3D (u64)spz->info->cpli[cid].min_avg_ivl; =20 return 0; } =20 static const char *scmi_powercap_get_name(struct powercap_zone *pz, int ci= d) { - return "SCMI power-cap"; + struct scmi_powercap_zone *spz =3D to_scmi_powercap_zone(pz); + + return spz->info->cpli[cid].name; } =20 static const struct powercap_zone_constraint_ops constraint_ops =3D { diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h index aafaac1496b0..9918fb30100c 100644 --- a/include/linux/scmi_protocol.h +++ b/include/linux/scmi_protocol.h @@ -2,7 +2,7 @@ /* * SCMI Message Protocol driver header * - * Copyright (C) 2018-2021 ARM Ltd. + * Copyright (C) 2018-2026 ARM Ltd. */ =20 #ifndef _LINUX_SCMI_PROTOCOL_H @@ -609,6 +609,35 @@ struct scmi_voltage_proto_ops { s32 *volt_uV); }; =20 +/** + * struct scmi_powercap_cpl_info - Describe one CPL - Concurrent Powercap= Limit + * + * @id: CPL ID as advertised by the platform. + * @cap_config: CAP configuration support for this CPL. + * @min_power_cap: Minimum configurable CAP. + * @max_power_cap: Maximum configurable CAP. + * @power_cap_step: Step size between two consecutive CAP values. + * @avg_ivl_config: Powercap averaging interval configuration support. + * @min_avg_ivl: Minimum configurable powercap averaging interval. + * @max_avg_ivl: Maximum configurable powercap averaging interval. + * @avg_ivl_step: Step size between two consecutive averaging intervals. + * @name: name assigned to the Powercap Domain by platform. + * @fc_info: Reference to the FastChannels descriptors supported by this C= PL + */ +struct scmi_powercap_cpl_info { + unsigned int id; + bool cap_config; + unsigned int min_power_cap; + unsigned int max_power_cap; + unsigned int power_cap_step; + bool avg_ivl_config; + unsigned int min_avg_ivl; + unsigned int max_avg_ivl; + unsigned int avg_ivl_step; + char name[SCMI_SHORT_NAME_MAX_SIZE]; + struct scmi_fc_info *fc_info; +}; + /** * struct scmi_powercap_info - Describe one available Powercap domain * @@ -617,21 +646,15 @@ struct scmi_voltage_proto_ops { * @notify_powercap_measurement_change: MEASUREMENTS change notifications * support. * @async_powercap_cap_set: Asynchronous CAP set support. - * @powercap_cap_config: CAP configuration support. * @powercap_monitoring: Monitoring (measurements) support. - * @powercap_pai_config: PAI configuration support. * @powercap_scale_mw: Domain reports power data in milliwatt units. * @powercap_scale_uw: Domain reports power data in microwatt units. * Note that, when both @powercap_scale_mw and * @powercap_scale_uw are set to false, the domain * reports power data on an abstract linear scale. + * @extended_names: Support for long names. + * @fastchannels: Support for at least one fastchannel, * @name: name assigned to the Powercap Domain by platform. - * @min_pai: Minimum configurable PAI. - * @max_pai: Maximum configurable PAI. - * @pai_step: Step size between two consecutive PAI values. - * @min_power_cap: Minimum configurable CAP. - * @max_power_cap: Maximum configurable CAP. - * @power_cap_step: Step size between two consecutive CAP values. * @sustainable_power: Maximum sustainable power consumption for this doma= in * under normal conditions. * @accuracy: The accuracy with which the power is measured and reported in @@ -639,30 +662,25 @@ struct scmi_voltage_proto_ops { * @parent_id: Identifier of the containing parent power capping domain, o= r the * value 0xFFFFFFFF if this powercap domain is a root domain not * contained in any other domain. + * @num_cpli: Number of discovered CPLs. + * @cpli: Reference to an array holding descriptors to all the discovered = CPLs. */ struct scmi_powercap_info { unsigned int id; bool notify_powercap_cap_change; bool notify_powercap_measurement_change; bool async_powercap_cap_set; - bool powercap_cap_config; bool powercap_monitoring; - bool powercap_pai_config; bool powercap_scale_mw; bool powercap_scale_uw; bool fastchannels; char name[SCMI_MAX_STR_SIZE]; - unsigned int min_pai; - unsigned int max_pai; - unsigned int pai_step; - unsigned int min_power_cap; - unsigned int max_power_cap; - unsigned int power_cap_step; unsigned int sustainable_power; unsigned int accuracy; #define SCMI_POWERCAP_ROOT_ZONE_ID 0xFFFFFFFFUL unsigned int parent_id; - struct scmi_fc_info *fc_info; + unsigned int num_cpli; + struct scmi_powercap_cpl_info *cpli; }; =20 /** @@ -691,8 +709,12 @@ struct scmi_powercap_info { * on the system: for this reason @cap_get and @cap_enable_get * will always report the final platform view of the powercaps. * @cap_enable_get: get the current CAP enable status for the specified do= main. - * @pai_get: get the current PAI value for the specified domain. - * @pai_set: set the PAI value for the specified domain to the provided va= lue. + * @avg_interval_get: get the current averaging interval value for the spe= cified + * domain. This will get the PAI or CAI depending on the used + * protocol version. + * @avg_interval_set: set the current averaging interval value for the spe= cified + * domain. This will set the PAI or CAI depending on the used + * protocol version. * @measurements_get: retrieve the current average power measurements for = the * specified domain and the related PAI upon which is * calculated. @@ -716,17 +738,17 @@ struct scmi_powercap_proto_ops { const struct scmi_powercap_info __must_check *(*info_get) (const struct scmi_protocol_handle *ph, u32 domain_id); int (*cap_get)(const struct scmi_protocol_handle *ph, u32 domain_id, - u32 *power_cap); + u32 cpl_id, u32 *power_cap); int (*cap_set)(const struct scmi_protocol_handle *ph, u32 domain_id, - u32 power_cap, bool ignore_dresp); + u32 cpl_id, u32 power_cap, bool ignore_dresp); int (*cap_enable_set)(const struct scmi_protocol_handle *ph, u32 domain_id, bool enable); int (*cap_enable_get)(const struct scmi_protocol_handle *ph, u32 domain_id, bool *enable); - int (*pai_get)(const struct scmi_protocol_handle *ph, u32 domain_id, - u32 *pai); - int (*pai_set)(const struct scmi_protocol_handle *ph, u32 domain_id, - u32 pai); + int (*avg_interval_get)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 *val); + int (*avg_interval_set)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 val); int (*measurements_get)(const struct scmi_protocol_handle *ph, u32 domain_id, u32 *average_power, u32 *pai); int (*measurements_threshold_set)(const struct scmi_protocol_handle *ph, --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7A8D73B6379; Wed, 17 Jun 2026 09:59:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690383; cv=none; b=hceKJkYp5t2GzzeBqo6DAtGO3ucW2mz114MV7kozuEkH2rt71JY8hojHfT43SKE9bwLRyV6oIOGzDC5aBuWbXWTg6Ho62eChDslfDu2ONszLA2FjVlfzOhenp3dQaBQoL9KAUbYDYxKYgnCgUlPM7yG+TN7Nucg7oHOfvc/L7Pw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690383; c=relaxed/simple; bh=Lj0sfAueifHceafcRI4/EOBaIlawexdfbCRkpR0nqqY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=EvFZYfKOW8XgNHXUdLyJG1k977IBVCN7jjXqZDP7ypMWhTB1cAT8hrGQWzQGtZLnfpVtibOaqJLFuC3LfFiY3ZgmlXYs3XcUAppdjq2uaot1GIkxfTv9p17yxHDGUf5y/AOBuDsiTeBZ9Fw+qNCwEftER1zFFlnthrIKniaipPc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=V7T5MIZc; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="V7T5MIZc" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0F4E82E8E; Wed, 17 Jun 2026 02:59:33 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 442F93F915; Wed, 17 Jun 2026 02:59:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690377; bh=Lj0sfAueifHceafcRI4/EOBaIlawexdfbCRkpR0nqqY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=V7T5MIZc8CCiCFKlwRBKoA44d56nKk3WIig7NnBrLQEhlacuUTaTgFsnJvamfnrtp +8CK4cFQQJBE/PPAz/R+NAH9shkEwYfYswOu5284A+EVfvz8PRMUFQe8PkmKeM7GQe 0kcdk1/cE1MnkC2HyXyyTWMK9+FvQFJk/4MrBwCs= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 04/13] firmware: arm_scmi: Add SCMIv4.0 Powercap basic support Date: Wed, 17 Jun 2026 10:59:01 +0100 Message-ID: <20260617095910.1963578-5-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Add SCMIv4.0 Powercap support for enumerating multiple CPLs of a domain when available. Signed-off-by: Cristian Marussi [Philip: Fixed sparse issues where int was expected] Signed-off-by: Philip Radford --- V6->V7 - prevent possible cpli[0] out-of-bounds access --- drivers/firmware/arm_scmi/powercap.c | 474 +++++++++++++++++++++------ include/linux/scmi_protocol.h | 1 + 2 files changed, 379 insertions(+), 96 deletions(-) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index 47aa6dde4a52..51771c833183 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -33,6 +33,7 @@ enum scmi_powercap_protocol_cmd { POWERCAP_CAP_NOTIFY =3D 0xa, POWERCAP_MEASUREMENTS_NOTIFY =3D 0xb, POWERCAP_DESCRIBE_FASTCHANNEL =3D 0xc, + POWERCAP_CPC_ATTRIBUTES =3D 0xd, }; =20 enum { @@ -69,19 +70,58 @@ struct scmi_msg_resp_powercap_domain_attributes { __le32 parent_id; }; =20 +struct scmi_msg_resp_powercap_domain_attributes_v3 { + __le32 attributes; +#define SUPPORTS_POWERCAP_MAI_CONFIGURATION(x) ((x) & BIT(25)) +#define SUPPORTS_POWERCAP_FASTCHANNELS(x) ((x) & BIT(22)) +#define SUPPORTS_POWERCAP_CAP_CHANGE_NOTIFY_V3(x) ((x) & BIT(21)) +#define SUPPORTS_POWERCAP_CAI_CONFIGURATION(x) ((x) & BIT(20)) + u8 name[SCMI_SHORT_NAME_MAX_SIZE]; + __le32 min_mai; + __le32 max_mai; + __le32 mai_step; + __le32 min_power_cap; + __le32 max_power_cap; + __le32 power_cap_step; + __le32 sustainable_power; + __le32 accuracy; + __le32 parent_id; + __le32 min_cai; + __le32 max_cai; + __le32 cai_step; +}; + +struct scmi_msg_powercap_get_v3 { + __le32 domain_id; + __le32 cpli; +}; + struct scmi_msg_powercap_set_cap_or_pai { - __le32 domain; + __le32 domain_id; __le32 flags; #define CAP_SET_ASYNC BIT(1) #define CAP_SET_IGNORE_DRESP BIT(0) __le32 value; }; =20 +struct scmi_msg_powercap_set_cap_v3 { + __le32 domain_id; + __le32 cpli; + __le32 flags; + __le32 power_cap; +}; + struct scmi_msg_resp_powercap_cap_set_complete { - __le32 domain; + __le32 domain_id; __le32 power_cap; }; =20 +struct scmi_msg_resp_powercap_cap_set_complete_v3 { + __le32 domain_id; + __le32 power_cap; + __le32 cpli; +}; + struct scmi_msg_resp_powercap_meas_get { __le32 power; __le32 pai; @@ -112,6 +152,33 @@ struct scmi_powercap_meas_changed_notify_payld { __le32 power; }; =20 +struct scmi_msg_powercap_cpc { + __le32 domain_id; + __le32 desc_index; +}; + +struct scmi_msg_resp_powercap_cpc { + __le32 num_cpl; +#define NUM_RETURNED(n) (le32_get_bits((n), GENMASK(15, 0))) +#define NUM_REMAINING(n) (le32_get_bits((n), GENMASK(31, 16))) + struct { + __le32 cpli; + __le32 flags; + __le32 min_power_cap; + __le32 max_power_cap; + __le32 power_cap_step; + __le32 min_cai; + __le32 max_cai; + __le32 cai_step; + u8 name[SCMI_SHORT_NAME_MAX_SIZE]; + } desc[]; +}; + +struct scmi_cpls_priv { + u32 domain_id; + struct scmi_powercap_cpl_info *cpli; +}; + struct scmi_powercap_state { bool enabled; u32 last_pcap; @@ -129,6 +196,11 @@ struct powercap_info { bool notify_measurements_cmd; struct scmi_powercap_state *states; struct scmi_powercap_info *powercaps; + int (*xfer_cap_get)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 *power_cap); + int (*xfer_cap_set)(const struct scmi_protocol_handle *ph, + const struct scmi_powercap_info *pc, + u32 cpl_id, u32 power_cap, bool ignore_dresp); }; =20 static enum scmi_powercap_protocol_cmd evt_2_cmd[] =3D { @@ -192,111 +264,244 @@ scmi_powercap_validate(unsigned int min_val, unsign= ed int max_val, return 0; } =20 +static void iter_powercap_cpls_prepare_message(void *message, + unsigned int desc_index, + const void *priv) +{ + struct scmi_msg_powercap_cpc *msg =3D message; + const struct scmi_cpls_priv *p =3D priv; + + msg->domain_id =3D cpu_to_le32(p->domain_id); + msg->desc_index =3D cpu_to_le32(desc_index); +} + +static int iter_powercap_cpls_update_state(struct scmi_iterator_state *st, + const void *response, void *priv) +{ + const struct scmi_msg_resp_powercap_cpc *r =3D response; + + st->num_returned =3D NUM_RETURNED(r->num_cpl); + st->num_remaining =3D NUM_REMAINING(r->num_cpl); + + return 0; +} + static int -scmi_powercap_domain_attributes_get(const struct scmi_protocol_handle *ph, - struct powercap_info *pinfo, - struct scmi_powercap_info *dom_info) +iter_powercap_cpls_process_response(const struct scmi_protocol_handle *ph, + const void *response, + struct scmi_iterator_state *st, void *priv) { + const struct scmi_msg_resp_powercap_cpc *r =3D response; + struct scmi_cpls_priv *p =3D priv; + struct scmi_powercap_cpl_info *cpl; + + cpl =3D &p->cpli[st->desc_index + st->loop_idx]; + + cpl->id =3D le32_to_cpu(r->desc[st->loop_idx].cpli); + cpl->cap_config =3D le32_to_cpu(r->desc[st->loop_idx].flags) & BIT(0); + + cpl->min_power_cap =3D le32_to_cpu(r->desc[st->loop_idx].min_power_cap); + cpl->max_power_cap =3D le32_to_cpu(r->desc[st->loop_idx].max_power_cap); + cpl->power_cap_step =3D le32_to_cpu(r->desc[st->loop_idx].power_cap_step); + if (!cpl->power_cap_step && cpl->min_power_cap !=3D cpl->max_power_cap) + return -EINVAL; + + cpl->min_avg_ivl =3D le32_to_cpu(r->desc[st->loop_idx].min_cai); + cpl->max_avg_ivl =3D le32_to_cpu(r->desc[st->loop_idx].max_cai); + cpl->avg_ivl_step =3D le32_to_cpu(r->desc[st->loop_idx].cai_step); + if (!cpl->avg_ivl_step && cpl->min_avg_ivl !=3D cpl->max_avg_ivl) + return -EINVAL; + + cpl->avg_ivl_config =3D cpl->min_avg_ivl !=3D cpl->max_avg_ivl; + + strscpy(cpl->name, r->desc[st->loop_idx].name, SCMI_SHORT_NAME_MAX_SIZE); + + return 0; +} + +static int scmi_powercap_cpls_enumerate(const struct scmi_protocol_handle = *ph, + struct scmi_powercap_info *dom_info) +{ + void *iter; + struct scmi_iterator_ops ops =3D { + .prepare_message =3D iter_powercap_cpls_prepare_message, + .update_state =3D iter_powercap_cpls_update_state, + .process_response =3D iter_powercap_cpls_process_response, + }; + struct scmi_cpls_priv cpriv =3D { + .domain_id =3D dom_info->id, + .cpli =3D dom_info->cpli, + }; + + iter =3D ph->hops->iter_response_init(ph, &ops, dom_info->num_cpli, + POWERCAP_CPC_ATTRIBUTES, + sizeof(struct scmi_msg_powercap_cpc), + &cpriv); + if (IS_ERR(iter)) + return PTR_ERR(iter); + + return ph->hops->iter_response_run(iter); +} + +static int +scmi_powercap_domain_attrs_process(const struct scmi_protocol_handle *ph, + struct powercap_info *pinfo, + struct scmi_powercap_info *dom_info, void *r) +{ + struct scmi_msg_resp_powercap_domain_attributes *resp =3D r; + u32 flags =3D le32_to_cpu(resp->attributes); + bool cap_config; int ret; - u32 flags; - struct scmi_xfer *t; - struct scmi_msg_resp_powercap_domain_attributes *resp; =20 - ret =3D ph->xops->xfer_get_init(ph, POWERCAP_DOMAIN_ATTRIBUTES, - sizeof(dom_info->id), sizeof(*resp), &t); - if (ret) - return ret; + cap_config =3D SUPPORTS_POWERCAP_CAP_CONFIGURATION(flags); + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { + dom_info->num_cpli =3D 1; + } else { + dom_info->num_cpli =3D le32_get_bits(resp->attributes, + GENMASK(18, 15)); =20 - put_unaligned_le32(dom_info->id, t->tx.buf); - resp =3D t->rx.buf; + if (!dom_info->num_cpli) + dom_info->num_cpli =3D 1; =20 - ret =3D ph->xops->do_xfer(ph, t); - if (!ret) { - flags =3D le32_to_cpu(resp->attributes); + if (cap_config && !dom_info->num_cpli) + return -EINVAL; + } + + dom_info->cpli =3D devm_kcalloc(ph->dev, dom_info->num_cpli, + sizeof(*dom_info->cpli), GFP_KERNEL); + if (!dom_info->cpli) + return -ENOMEM; =20 - if (pinfo->notify_cap_cmd) + if (pinfo->notify_cap_cmd) { + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) dom_info->notify_powercap_cap_change =3D SUPPORTS_POWERCAP_CAP_CHANGE_NOTIFY(flags); - if (pinfo->notify_measurements_cmd) - dom_info->notify_powercap_measurement_change =3D - SUPPORTS_POWERCAP_MEASUREMENTS_CHANGE_NOTIFY(flags); - dom_info->async_powercap_cap_set =3D - SUPPORTS_ASYNC_POWERCAP_CAP_SET(flags); - - dom_info->powercap_monitoring =3D - SUPPORTS_POWERCAP_MONITORING(flags); - dom_info->powercap_scale_mw =3D - SUPPORTS_POWER_UNITS_MW(flags); - dom_info->powercap_scale_uw =3D - SUPPORTS_POWER_UNITS_UW(flags); - dom_info->fastchannels =3D - SUPPORTS_POWERCAP_FASTCHANNELS(flags); - - strscpy(dom_info->name, resp->name, SCMI_SHORT_NAME_MAX_SIZE); - - dom_info->sustainable_power =3D - le32_to_cpu(resp->sustainable_power); - dom_info->accuracy =3D le32_to_cpu(resp->accuracy); - - dom_info->parent_id =3D le32_to_cpu(resp->parent_id); - if (dom_info->parent_id !=3D SCMI_POWERCAP_ROOT_ZONE_ID && - (dom_info->parent_id >=3D pinfo->num_domains || - dom_info->parent_id =3D=3D dom_info->id)) { - dev_err(ph->dev, - "Platform reported inconsistent parent ID for domain %d - %s\n", - dom_info->id, dom_info->name); - ret =3D -ENODEV; - } + else + dom_info->notify_powercap_cap_change =3D + SUPPORTS_POWERCAP_CAP_CHANGE_NOTIFY_V3(flags); + } + + if (pinfo->notify_measurements_cmd) + dom_info->notify_powercap_measurement_change =3D + SUPPORTS_POWERCAP_MEASUREMENTS_CHANGE_NOTIFY(flags); + + dom_info->extended_names =3D SUPPORTS_EXTENDED_NAMES(flags); + + dom_info->async_powercap_cap_set =3D + SUPPORTS_ASYNC_POWERCAP_CAP_SET(flags); + + dom_info->powercap_monitoring =3D + SUPPORTS_POWERCAP_MONITORING(flags); + dom_info->powercap_scale_mw =3D + SUPPORTS_POWER_UNITS_MW(flags); + dom_info->powercap_scale_uw =3D + SUPPORTS_POWER_UNITS_UW(flags); + dom_info->fastchannels =3D + SUPPORTS_POWERCAP_FASTCHANNELS(flags); + + strscpy(dom_info->name, resp->name, SCMI_SHORT_NAME_MAX_SIZE); + + dom_info->sustainable_power =3D + le32_to_cpu(resp->sustainable_power); + dom_info->accuracy =3D le32_to_cpu(resp->accuracy); + + dom_info->parent_id =3D le32_to_cpu(resp->parent_id); + if (dom_info->parent_id !=3D SCMI_POWERCAP_ROOT_ZONE_ID && + (dom_info->parent_id >=3D pinfo->num_domains || + dom_info->parent_id =3D=3D dom_info->id)) { + dev_err(ph->dev, + "Platform reported inconsistent parent ID for domain %d - %s\n", + dom_info->id, dom_info->name); + return -ENODEV; + } =20 + dom_info->cpli[0].id =3D CPL0; + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) dom_info->cpli[0].avg_ivl_config =3D SUPPORTS_POWERCAP_PAI_CONFIGURATION(flags); + else + dom_info->cpli[0].avg_ivl_config =3D + SUPPORTS_POWERCAP_CAI_CONFIGURATION(flags); + + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { dom_info->cpli[0].min_avg_ivl =3D le32_to_cpu(resp->min_pai); dom_info->cpli[0].max_avg_ivl =3D le32_to_cpu(resp->max_pai); dom_info->cpli[0].avg_ivl_step =3D le32_to_cpu(resp->pai_step); - ret =3D scmi_powercap_validate(dom_info->cpli[0].min_avg_ivl, - dom_info->cpli[0].max_avg_ivl, - dom_info->cpli[0].avg_ivl_step, - dom_info->cpli[0].avg_ivl_config); - if (ret) { - dev_err(ph->dev, - "Platform reported inconsistent PAI config for domain %d - %s\n", - dom_info->id, dom_info->name); - goto clean; - } + } else { + struct scmi_msg_resp_powercap_domain_attributes_v3 *resp =3D r; =20 - dom_info->cpli[0].cap_config =3D - SUPPORTS_POWERCAP_CAP_CONFIGURATION(flags); - dom_info->cpli[0].min_power_cap =3D le32_to_cpu(resp->min_power_cap); - dom_info->cpli[0].max_power_cap =3D le32_to_cpu(resp->max_power_cap); - dom_info->cpli[0].power_cap_step =3D le32_to_cpu(resp->power_cap_step); - ret =3D scmi_powercap_validate(dom_info->cpli[0].min_power_cap, - dom_info->cpli[0].max_power_cap, - dom_info->cpli[0].power_cap_step, - dom_info->cpli[0].cap_config); - if (ret) { - dev_err(ph->dev, - "Platform reported inconsistent CAP config for domain %d - %s\n", - dom_info->id, dom_info->name); - goto clean; - } + dom_info->cpli[0].min_avg_ivl =3D le32_to_cpu(resp->min_cai); + dom_info->cpli[0].max_avg_ivl =3D le32_to_cpu(resp->max_cai); + dom_info->cpli[0].avg_ivl_step =3D le32_to_cpu(resp->cai_step); + } + + ret =3D scmi_powercap_validate(dom_info->cpli[0].min_avg_ivl, + dom_info->cpli[0].max_avg_ivl, + dom_info->cpli[0].avg_ivl_step, + dom_info->cpli[0].avg_ivl_config); + if (ret) { + dev_err(ph->dev, + "Platform reported inconsistent PAI config for domain %d - %s\n", + dom_info->id, dom_info->name); + return ret; + } =20 - /* Just using same short name */ - strscpy(dom_info->cpli[0].name, dom_info->name, - SCMI_SHORT_NAME_MAX_SIZE); + dom_info->cpli[0].cap_config =3D cap_config; + dom_info->cpli[0].min_power_cap =3D le32_to_cpu(resp->min_power_cap); + dom_info->cpli[0].max_power_cap =3D le32_to_cpu(resp->max_power_cap); + dom_info->cpli[0].power_cap_step =3D le32_to_cpu(resp->power_cap_step); + ret =3D scmi_powercap_validate(dom_info->cpli[0].min_power_cap, + dom_info->cpli[0].max_power_cap, + dom_info->cpli[0].power_cap_step, + dom_info->cpli[0].cap_config); + if (ret) { + dev_err(ph->dev, + "Platform reported inconsistent CAP config for domain %d - %s\n", + dom_info->id, dom_info->name); + return ret; } + /* Just using same short name */ + strscpy(dom_info->cpli[0].name, dom_info->name, SCMI_SHORT_NAME_MAX_SIZE); + + return 0; +} + +static int +scmi_powercap_domain_attributes_get(const struct scmi_protocol_handle *ph, + struct powercap_info *pinfo, + struct scmi_powercap_info *dom_info) +{ + int ret; + struct scmi_xfer *t; + struct scmi_msg_resp_powercap_domain_attributes *resp; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_DOMAIN_ATTRIBUTES, + sizeof(dom_info->id), 0, &t); + if (ret) + return ret; + + put_unaligned_le32(dom_info->id, t->tx.buf); + resp =3D t->rx.buf; + + ret =3D ph->xops->do_xfer(ph, t); + if (!ret) + ret =3D scmi_powercap_domain_attrs_process(ph, pinfo, dom_info, resp); =20 -clean: ph->xops->xfer_put(ph, t); =20 /* * If supported overwrite short name with the extended one; * on error just carry on and use already provided short name. */ - if (!ret && SUPPORTS_EXTENDED_NAMES(flags)) + if (!ret && dom_info->extended_names) ph->hops->extended_name_get(ph, POWERCAP_DOMAIN_NAME_GET, dom_info->id, NULL, dom_info->name, SCMI_MAX_STR_SIZE); =20 + /* When protocol version > 0x3 there can possibly be more than 1 CPLs */ + if (!ret && dom_info->num_cpli > 1) + ret =3D scmi_powercap_cpls_enumerate(ph, dom_info); + return ret; } =20 @@ -306,14 +511,7 @@ scmi_powercap_domain_initialize(const struct scmi_prot= ocol_handle *ph, { struct scmi_powercap_info *dom_info =3D pinfo->powercaps + domain; =20 - dom_info->num_cpli =3D 1; - dom_info->cpli =3D devm_kcalloc(ph->dev, dom_info->num_cpli, - sizeof(*dom_info->cpli), GFP_KERNEL); - if (!dom_info->cpli) - return -ENOMEM; - dom_info->id =3D domain; - dom_info->cpli[0].id =3D CPL0; =20 return scmi_powercap_domain_attributes_get(ph, pinfo, dom_info); } @@ -337,7 +535,7 @@ scmi_powercap_dom_info_get(const struct scmi_protocol_h= andle *ph, u32 domain_id) } =20 static int scmi_powercap_xfer_cap_get(const struct scmi_protocol_handle *p= h, - u32 domain_id, u32 *power_cap) + u32 domain_id, u32 cpl_id, u32 *power_cap) { int ret; struct scmi_xfer *t; @@ -348,6 +546,33 @@ static int scmi_powercap_xfer_cap_get(const struct scm= i_protocol_handle *ph, return ret; =20 put_unaligned_le32(domain_id, t->tx.buf); + + ret =3D ph->xops->do_xfer(ph, t); + if (!ret) + *power_cap =3D get_unaligned_le32(t->rx.buf); + + ph->xops->xfer_put(ph, t); + + return ret; +} + +static int scmi_powercap_xfer_cap_get_v3(const struct scmi_protocol_handle= *ph, + u32 domain_id, u32 cpl_id, + u32 *power_cap) +{ + int ret; + struct scmi_xfer *t; + struct scmi_msg_powercap_get_v3 *msg; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAP_GET, sizeof(*msg), + sizeof(u32), &t); + if (ret) + return ret; + + msg =3D t->tx.buf; + msg->domain_id =3D cpu_to_le32(domain_id); + msg->cpli =3D cpu_to_le32(cpl_id); + ret =3D ph->xops->do_xfer(ph, t); if (!ret) *power_cap =3D get_unaligned_le32(t->rx.buf); @@ -361,6 +586,8 @@ static int __scmi_powercap_cap_get(const struct scmi_pr= otocol_handle *ph, const struct scmi_powercap_info *dom, u32 cpl_id, u32 *power_cap) { + struct powercap_info *pi =3D ph->get_priv(ph); + if (dom->cpli[cpl_id].fc_info && dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_addr) { *power_cap =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_a= ddr); @@ -369,7 +596,7 @@ static int __scmi_powercap_cap_get(const struct scmi_pr= otocol_handle *ph, return 0; } =20 - return scmi_powercap_xfer_cap_get(ph, dom->id, power_cap); + return pi->xfer_cap_get(ph, dom->id, cpl_id, power_cap); } =20 static int scmi_powercap_cap_get(const struct scmi_protocol_handle *ph, @@ -402,7 +629,7 @@ static int scmi_powercap_xfer_cap_set(const struct scmi= _protocol_handle *ph, return ret; =20 msg =3D t->tx.buf; - msg->domain =3D cpu_to_le32(pc->id); + msg->domain_id =3D cpu_to_le32(pc->id); msg->flags =3D cpu_to_le32(FIELD_PREP(CAP_SET_ASYNC, pc->async_powercap_cap_set) | FIELD_PREP(CAP_SET_IGNORE_DRESP, ignore_dresp)); @@ -416,7 +643,7 @@ static int scmi_powercap_xfer_cap_set(const struct scmi= _protocol_handle *ph, struct scmi_msg_resp_powercap_cap_set_complete *resp; =20 resp =3D t->rx.buf; - if (le32_to_cpu(resp->domain) =3D=3D pc->id) + if (le32_to_cpu(resp->domain_id) =3D=3D pc->id) dev_dbg(ph->dev, "Powercap ID %d CAP set async to %u\n", pc->id, @@ -430,6 +657,51 @@ static int scmi_powercap_xfer_cap_set(const struct scm= i_protocol_handle *ph, return ret; } =20 +static int scmi_powercap_xfer_cap_set_v3(const struct scmi_protocol_handle= *ph, + const struct scmi_powercap_info *pc, + u32 cpl_id, u32 power_cap, + bool ignore_dresp) +{ + int ret; + struct scmi_xfer *t; + struct scmi_msg_powercap_set_cap_v3 *msg; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAP_SET, + sizeof(*msg), 0, &t); + if (ret) + return ret; + + msg =3D t->tx.buf; + msg->domain_id =3D cpu_to_le32(pc->id); + msg->cpli =3D cpu_to_le32(cpl_id); + msg->flags =3D + cpu_to_le32(FIELD_PREP(CAP_SET_ASYNC, pc->async_powercap_cap_set) | + FIELD_PREP(CAP_SET_IGNORE_DRESP, ignore_dresp)); + msg->power_cap =3D cpu_to_le32(power_cap); + + if (!pc->async_powercap_cap_set || ignore_dresp) { + ret =3D ph->xops->do_xfer(ph, t); + } else { + ret =3D ph->xops->do_xfer_with_response(ph, t); + if (!ret) { + struct scmi_msg_resp_powercap_cap_set_complete_v3 *resp; + + resp =3D t->rx.buf; + if (le32_to_cpu(resp->domain_id) =3D=3D pc->id && + le32_to_cpu(resp->cpli) =3D=3D pc->cpli[cpl_id].id) + dev_dbg(ph->dev, + "Powercap ID:%d/CPLI:%d CAP set async to %u\n", + pc->id, cpl_id, + get_unaligned_le32(&resp->power_cap)); + else + ret =3D -EPROTO; + } + } + + ph->xops->xfer_put(ph, t); + return ret; +} + static int __scmi_powercap_cap_set(const struct scmi_protocol_handle *ph, struct powercap_info *pi, u32 domain_id, u32 cpl_id, u32 power_cap, bool ignore_dresp) @@ -456,12 +728,12 @@ static int __scmi_powercap_cap_set(const struct scmi_= protocol_handle *ph, domain_id, power_cap, 0); ret =3D 0; } else { - ret =3D scmi_powercap_xfer_cap_set(ph, pc, cpl_id, power_cap, - ignore_dresp); + ret =3D pi->xfer_cap_set(ph, pc, cpl_id, power_cap, ignore_dresp); } =20 - /* Save the last explicitly set non-zero powercap value */ - if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2 && !ret && power_cap) + /* Save the last explicitly set non-zero powercap value for CPL0 */ + if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2 && !ret && + cpl_id =3D=3D CPL0 && power_cap) pi->states[domain_id].last_pcap =3D power_cap; =20 return ret; @@ -480,8 +752,8 @@ static int scmi_powercap_cap_set(const struct scmi_prot= ocol_handle *ph, if (!power_cap) return -EINVAL; =20 - /* Just log the last set request if acting on a disabled domain */ - if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2 && + /* Just log the last set request on CPL0 on a disabled domain */ + if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2 && cpl_id =3D=3D CPL0 && !pi->states[domain_id].enabled) { pi->states[domain_id].last_pcap =3D power_cap; return 0; @@ -554,7 +826,7 @@ static int scmi_powercap_xfer_pai_set(const struct scmi= _protocol_handle *ph, return ret; =20 msg =3D t->tx.buf; - msg->domain =3D cpu_to_le32(domain_id); + msg->domain_id =3D cpu_to_le32(domain_id); msg->flags =3D cpu_to_le32(0); msg->value =3D cpu_to_le32(pai); =20 @@ -1013,6 +1285,16 @@ scmi_powercap_protocol_init(const struct scmi_protoc= ol_handle *ph) if (!pinfo) return -ENOMEM; =20 + ph->set_priv(ph, pinfo); + + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { + pinfo->xfer_cap_get =3D scmi_powercap_xfer_cap_get; + pinfo->xfer_cap_set =3D scmi_powercap_xfer_cap_set; + } else { + pinfo->xfer_cap_get =3D scmi_powercap_xfer_cap_get_v3; + pinfo->xfer_cap_set =3D scmi_powercap_xfer_cap_set_v3; + } + ret =3D scmi_powercap_attributes_get(ph, pinfo); if (ret) return ret; diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h index 9918fb30100c..547ab4763a63 100644 --- a/include/linux/scmi_protocol.h +++ b/include/linux/scmi_protocol.h @@ -673,6 +673,7 @@ struct scmi_powercap_info { bool powercap_monitoring; bool powercap_scale_mw; bool powercap_scale_uw; + bool extended_names; bool fastchannels; char name[SCMI_MAX_STR_SIZE]; unsigned int sustainable_power; --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 46D623D75D7; Wed, 17 Jun 2026 09:59:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690386; cv=none; b=d9Af4k0GeoBShViMxOh4NKIjltS1sPTSRkJ/C5Sjp+qqa0Ym7o1K7krD3AHMLfh8t1WTF5lPNsSmYIf2Co6ZVSOfIBp8h/Tdha1tk2TF5h8qgN6bXZY+r6BYoHMZw8Dq00Aq7ftiby5XIixvq1kvDHhEKHVF34lOijPIgDQNFyI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690386; c=relaxed/simple; bh=VxVvfA/O0bDWTZSPKQE6oMFyraJ2in/ah63Mny9pjdM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=keTQNL4SwdHKEE2dPQiC0TqnztcAoqeJJJ73qHyT0t3N8AXDakUBUUmdb+LpiPYSACzYeuYzphJ7atv5m6pG0HfKfXlmjFh9ZQuDeZxhkVsSqawOtljlP/RTEhbOUKHvwezySPNwU4/+Lp+lZxYi51JhdrduaPoySoIogYLqn5M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=g0KADwKJ; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="g0KADwKJ" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D9F31483F; Wed, 17 Jun 2026 02:59:35 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 127F63F915; Wed, 17 Jun 2026 02:59:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690380; bh=VxVvfA/O0bDWTZSPKQE6oMFyraJ2in/ah63Mny9pjdM=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=g0KADwKJuMvttAAIfvxdy1yGEp3M+UhpQP9AcLNgOGD0VMYmtbwPLns6ai52K/M5V v9H96/Jb2lGtSqQ4I205zcizzDrR9CCTHEPwrRgH4jm/sMvk/Bc+saCTYDhmN9ZRh/ pbrn+D0jaZ25BawpSftagV0HGnlvNcX36eTPuR2k= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 05/13] firmware: arm_scmi: Add SCMIv4.0 Powercap FCs support Date: Wed, 17 Jun 2026 10:59:02 +0100 Message-ID: <20260617095910.1963578-6-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Add support for new SCMIv4.0 Powercap Fastchannels. Signed-off-by: Cristian Marussi [Philip: removed reference to old versioning logic] Signed-off-by: Philip Radford --- V2->V4 - moved fastchannel inits outside of loop - renamed arguments for consistency --- drivers/firmware/arm_scmi/powercap.c | 331 ++++++++++++++++++--------- 1 file changed, 229 insertions(+), 102 deletions(-) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index 51771c833183..ba037e906640 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -27,19 +27,25 @@ enum scmi_powercap_protocol_cmd { POWERCAP_CAP_GET =3D 0x4, POWERCAP_CAP_SET =3D 0x5, POWERCAP_PAI_GET =3D 0x6, + POWERCAP_MAI_GET =3D POWERCAP_PAI_GET, POWERCAP_PAI_SET =3D 0x7, + POWERCAP_MAI_SET =3D POWERCAP_PAI_SET, POWERCAP_DOMAIN_NAME_GET =3D 0x8, POWERCAP_MEASUREMENTS_GET =3D 0x9, POWERCAP_CAP_NOTIFY =3D 0xa, POWERCAP_MEASUREMENTS_NOTIFY =3D 0xb, POWERCAP_DESCRIBE_FASTCHANNEL =3D 0xc, POWERCAP_CPC_ATTRIBUTES =3D 0xd, + POWERCAP_CAI_GET =3D 0xe, + POWERCAP_CAI_SET =3D 0xf, }; =20 enum { POWERCAP_FC_CAP, - POWERCAP_FC_PAI, - POWERCAP_FC_MAX, + POWERCAP_FC_XAI, + POWERCAP_FC_MAI, + POWERCAP_FC_MEASUREMENT, + POWERCAP_FC_MAX }; =20 struct scmi_msg_resp_powercap_domain_attributes { @@ -91,12 +97,12 @@ struct scmi_msg_resp_powercap_domain_attributes_v3 { __le32 cai_step; }; =20 -struct scmi_msg_powercap_get_v3 { +struct scmi_msg_powercap_cap_or_cai_get_v3 { __le32 domain_id; __le32 cpli; }; =20 -struct scmi_msg_powercap_set_cap_or_pai { +struct scmi_msg_powercap_cap_or_pai_set { __le32 domain_id; __le32 flags; #define CAP_SET_ASYNC BIT(1) @@ -104,13 +110,20 @@ struct scmi_msg_powercap_set_cap_or_pai { __le32 value; }; =20 -struct scmi_msg_powercap_set_cap_v3 { +struct scmi_msg_powercap_cap_set_v3 { __le32 domain_id; __le32 cpli; __le32 flags; __le32 power_cap; }; =20 +struct scmi_msg_powercap_cai_set { + __le32 domain_id; + __le32 flags; + __le32 cai; + __le32 cpli; +}; + struct scmi_msg_resp_powercap_cap_set_complete { __le32 domain_id; __le32 power_cap; @@ -201,6 +214,10 @@ struct powercap_info { int (*xfer_cap_set)(const struct scmi_protocol_handle *ph, const struct scmi_powercap_info *pc, u32 cpl_id, u32 power_cap, bool ignore_dresp); + int (*xfer_avg_ivl_get)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 *ivl); + int (*xfer_avg_ivl_set)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 ivl); }; =20 static enum scmi_powercap_protocol_cmd evt_2_cmd[] =3D { @@ -505,17 +522,6 @@ scmi_powercap_domain_attributes_get(const struct scmi_= protocol_handle *ph, return ret; } =20 -static int -scmi_powercap_domain_initialize(const struct scmi_protocol_handle *ph, - struct powercap_info *pinfo, u32 domain) -{ - struct scmi_powercap_info *dom_info =3D pinfo->powercaps + domain; - - dom_info->id =3D domain; - - return scmi_powercap_domain_attributes_get(ph, pinfo, dom_info); -} - static int scmi_powercap_num_domains_get(const struct scmi_protocol_handle= *ph) { struct powercap_info *pi =3D ph->get_priv(ph); @@ -562,7 +568,7 @@ static int scmi_powercap_xfer_cap_get_v3(const struct s= cmi_protocol_handle *ph, { int ret; struct scmi_xfer *t; - struct scmi_msg_powercap_get_v3 *msg; + struct scmi_msg_powercap_cap_or_cai_get_v3 *msg; =20 ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAP_GET, sizeof(*msg), sizeof(u32), &t); @@ -621,7 +627,7 @@ static int scmi_powercap_xfer_cap_set(const struct scmi= _protocol_handle *ph, { int ret; struct scmi_xfer *t; - struct scmi_msg_powercap_set_cap_or_pai *msg; + struct scmi_msg_powercap_cap_or_pai_set *msg; =20 ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAP_SET, sizeof(*msg), 0, &t); @@ -664,7 +670,7 @@ static int scmi_powercap_xfer_cap_set_v3(const struct s= cmi_protocol_handle *ph, { int ret; struct scmi_xfer *t; - struct scmi_msg_powercap_set_cap_v3 *msg; + struct scmi_msg_powercap_cap_set_v3 *msg; =20 ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAP_SET, sizeof(*msg), 0, &t); @@ -763,8 +769,9 @@ static int scmi_powercap_cap_set(const struct scmi_prot= ocol_handle *ph, power_cap, ignore_dresp); } =20 -static int scmi_powercap_xfer_pai_get(const struct scmi_protocol_handle *p= h, - u32 domain_id, u32 *pai) +static int +scmi_powercap_xfer_avg_interval_get(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 *ivl) { int ret; struct scmi_xfer *t; @@ -777,58 +784,105 @@ static int scmi_powercap_xfer_pai_get(const struct s= cmi_protocol_handle *ph, put_unaligned_le32(domain_id, t->tx.buf); ret =3D ph->xops->do_xfer(ph, t); if (!ret) - *pai =3D get_unaligned_le32(t->rx.buf); + *ivl =3D get_unaligned_le32(t->rx.buf); + + ph->xops->xfer_put(ph, t); + + return ret; +} + +static int +scmi_powercap_xfer_avg_interval_get_v3(const struct scmi_protocol_handle *= ph, + u32 domain_id, u32 cpl_id, u32 *ivl) +{ + int ret; + struct scmi_xfer *t; + struct scmi_msg_powercap_cap_or_cai_get_v3 *msg; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAI_GET, sizeof(*msg), + sizeof(u32), &t); + if (ret) + return ret; + + msg =3D t->tx.buf; + msg->domain_id =3D cpu_to_le32(domain_id); + msg->cpli =3D cpu_to_le32(cpl_id); + + ret =3D ph->xops->do_xfer(ph, t); + if (!ret) + *ivl =3D get_unaligned_le32(t->rx.buf); =20 ph->xops->xfer_put(ph, t); =20 return ret; } =20 -static int scmi_powercap_pai_get(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 cpl_id, u32 *pai) +static int scmi_powercap_avg_interval_get(const struct scmi_protocol_handl= e *ph, + u32 domain_id, u32 cpl_id, u32 *val) { struct scmi_powercap_info *dom; struct powercap_info *pi =3D ph->get_priv(ph); =20 - if (!pai || domain_id >=3D pi->num_domains) + if (!val || domain_id >=3D pi->num_domains) return -EINVAL; =20 dom =3D pi->powercaps + domain_id; if (cpl_id >=3D dom->num_cpli) return -EINVAL; =20 - if (dom->cpli[cpl_id].fc_info && dom->cpli[cpl_id].fc_info[POWERCAP_FC_PA= I].get_addr) { - *pai =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_PAI].get_addr); - trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_PAI_GET, - domain_id, *pai, 0); + if (dom->cpli[cpl_id].fc_info && + dom->cpli[cpl_id].fc_info[POWERCAP_FC_XAI].get_addr) { + int trace_cmd =3D (PROTOCOL_REV_MAJOR(ph->version) < 0x3) ? + POWERCAP_PAI_GET : POWERCAP_CAI_GET; + + *val =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_XAI].get_addr); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, *val, 0= ); return 0; } =20 - return scmi_powercap_xfer_pai_get(ph, domain_id, pai); + return pi->xfer_avg_ivl_get(ph, domain_id, cpl_id, val); } =20 -static int scmi_powercap_avg_interval_get(const struct scmi_protocol_handl= e *ph, - u32 domain_id, u32 cpl_id, u32 *val) +static int +scmi_powercap_xfer_avg_interval_set(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 cpl_id, u32 ivl) { - return scmi_powercap_pai_get(ph, domain_id, cpl_id, val); + int ret; + struct scmi_xfer *t; + struct scmi_msg_powercap_cap_or_pai_set *msg; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_PAI_SET, sizeof(*msg), 0, &t= ); + if (ret) + return ret; + + msg =3D t->tx.buf; + msg->domain_id =3D cpu_to_le32(domain_id); + msg->flags =3D cpu_to_le32(0); + msg->value =3D cpu_to_le32(ivl); + + ret =3D ph->xops->do_xfer(ph, t); + + ph->xops->xfer_put(ph, t); + return ret; } =20 -static int scmi_powercap_xfer_pai_set(const struct scmi_protocol_handle *p= h, - u32 domain_id, u32 pai) +static int +scmi_powercap_xfer_avg_interval_set_v3(const struct scmi_protocol_handle *= ph, + u32 domain_id, u32 cpl_id, u32 ivl) { int ret; struct scmi_xfer *t; - struct scmi_msg_powercap_set_cap_or_pai *msg; + struct scmi_msg_powercap_cai_set *msg; =20 - ret =3D ph->xops->xfer_get_init(ph, POWERCAP_PAI_SET, - sizeof(*msg), 0, &t); + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_CAI_SET, sizeof(*msg), 0, &t= ); if (ret) return ret; =20 msg =3D t->tx.buf; msg->domain_id =3D cpu_to_le32(domain_id); msg->flags =3D cpu_to_le32(0); - msg->value =3D cpu_to_le32(pai); + msg->cai =3D cpu_to_le32(ivl); + msg->cpli =3D cpu_to_le32(cpl_id); =20 ret =3D ph->xops->do_xfer(ph, t); =20 @@ -836,48 +890,42 @@ static int scmi_powercap_xfer_pai_set(const struct sc= mi_protocol_handle *ph, return ret; } =20 -static int scmi_powercap_pai_set(const struct scmi_protocol_handle *ph, - u32 domain_id, u32 cpl_id, u32 pai) +static int scmi_powercap_avg_interval_set(const struct scmi_protocol_handl= e *ph, + u32 domain_id, u32 cpl_id, u32 ivl) { const struct scmi_powercap_info *pc; + struct powercap_info *pi =3D ph->get_priv(ph); =20 pc =3D scmi_powercap_dom_info_get(ph, domain_id); if (!pc || cpl_id >=3D pc->num_cpli || !pc->cpli[cpl_id].avg_ivl_config || - !pai || pai < pc->cpli[cpl_id].min_avg_ivl || - pai > pc->cpli[cpl_id].max_avg_ivl) + !ivl || ivl < pc->cpli[cpl_id].min_avg_ivl || + ivl > pc->cpli[cpl_id].max_avg_ivl) return -EINVAL; =20 - if (pc->cpli[cpl_id].fc_info && pc->cpli[cpl_id].fc_info[POWERCAP_FC_PAI]= .set_addr) { - struct scmi_fc_info *fci =3D &pc->cpli[cpl_id].fc_info[POWERCAP_FC_PAI]; + /* Note that fc_info descriptors for any unsupported FC will be NULL */ + if (pc->cpli[cpl_id].fc_info && + pc->cpli[cpl_id].fc_info[POWERCAP_FC_XAI].set_addr) { + int trace_cmd =3D (PROTOCOL_REV_MAJOR(ph->version) < 0x3) ? + POWERCAP_PAI_SET : POWERCAP_CAI_SET; + struct scmi_fc_info *fci =3D &pc->cpli[cpl_id].fc_info[POWERCAP_FC_XAI]; =20 - trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_PAI_SET, - domain_id, pai, 0); - iowrite32(pai, fci->set_addr); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, ivl, 0); + iowrite32(ivl, fci->set_addr); ph->hops->fastchannel_db_ring(fci->set_db); return 0; } =20 - return scmi_powercap_xfer_pai_set(ph, domain_id, pai); -} - -static int scmi_powercap_avg_interval_set(const struct scmi_protocol_handl= e *ph, - u32 domain_id, u32 cpl_id, u32 val) -{ - return scmi_powercap_pai_set(ph, domain_id, cpl_id, val); + return pi->xfer_avg_ivl_set(ph, domain_id, cpl_id, ivl); } =20 -static int scmi_powercap_measurements_get(const struct scmi_protocol_handl= e *ph, - u32 domain_id, u32 *average_power, - u32 *pai) +static int +scmi_powercap_xfer_measurements_get(const struct scmi_protocol_handle *ph, + const struct scmi_powercap_info *pc, + u32 *avg_power, u32 *avg_ivl) { int ret; struct scmi_xfer *t; struct scmi_msg_resp_powercap_meas_get *resp; - const struct scmi_powercap_info *pc; - - pc =3D scmi_powercap_dom_info_get(ph, domain_id); - if (!pc || !pc->powercap_monitoring || !pai || !average_power) - return -EINVAL; =20 ret =3D ph->xops->xfer_get_init(ph, POWERCAP_MEASUREMENTS_GET, sizeof(u32), sizeof(*resp), &t); @@ -885,17 +933,42 @@ static int scmi_powercap_measurements_get(const struc= t scmi_protocol_handle *ph, return ret; =20 resp =3D t->rx.buf; - put_unaligned_le32(domain_id, t->tx.buf); + put_unaligned_le32(pc->id, t->tx.buf); ret =3D ph->xops->do_xfer(ph, t); if (!ret) { - *average_power =3D le32_to_cpu(resp->power); - *pai =3D le32_to_cpu(resp->pai); + *avg_power =3D le32_to_cpu(resp->power); + *avg_ivl =3D le32_to_cpu(resp->pai); } =20 ph->xops->xfer_put(ph, t); return ret; } =20 +static int scmi_powercap_measurements_get(const struct scmi_protocol_handl= e *ph, + u32 domain_id, u32 *avg_power, + u32 *avg_ivl) +{ + const struct scmi_powercap_info *pc; + struct scmi_fc_info *fci; + + pc =3D scmi_powercap_dom_info_get(ph, domain_id); + if (!pc || !pc->powercap_monitoring || !avg_ivl || !avg_power) + return -EINVAL; + + /* Note that fc_info descriptors for any unsupported FC will be NULL */ + fci =3D pc->cpli[CPL0].fc_info; + if (fci && fci[POWERCAP_FC_MEASUREMENT].get_addr) { + *avg_power =3D ioread32(fci[POWERCAP_FC_MEASUREMENT].get_addr); + /* See SCMIv4.0 3.10.2 - Payload is 32bit ONLY avg_power */ + *avg_ivl =3D 0; + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_MEASUREMENTS_GET, + pc->id, *avg_power, *avg_ivl); + return 0; + } + + return scmi_powercap_xfer_measurements_get(ph, pc, avg_power, avg_ivl); +} + static int scmi_powercap_measurements_threshold_get(const struct scmi_protocol_handle= *ph, u32 domain_id, u32 *power_thresh_low, @@ -1023,37 +1096,85 @@ static const struct scmi_powercap_proto_ops powerca= p_proto_ops =3D { }; =20 static void scmi_powercap_domain_init_fc(const struct scmi_protocol_handle= *ph, - u32 domain, struct scmi_fc_info **p_fc) + struct scmi_powercap_info *dom_info) { - struct scmi_fc_info *fc; - - fc =3D devm_kcalloc(ph->dev, POWERCAP_FC_MAX, sizeof(*fc), GFP_KERNEL); - if (!fc) - return; - - ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_CAP_SET, 4, domain, NULL, - &fc[POWERCAP_FC_CAP].set_addr, - &fc[POWERCAP_FC_CAP].set_db, - &fc[POWERCAP_FC_CAP].rate_limit); - - ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_CAP_GET, 4, domain, NULL, - &fc[POWERCAP_FC_CAP].get_addr, NULL, - &fc[POWERCAP_FC_CAP].rate_limit); - - ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_PAI_SET, 4, domain, NULL, - &fc[POWERCAP_FC_PAI].set_addr, - &fc[POWERCAP_FC_PAI].set_db, - &fc[POWERCAP_FC_PAI].rate_limit); - - ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, - POWERCAP_PAI_GET, 4, domain, NULL, - &fc[POWERCAP_FC_PAI].get_addr, NULL, - &fc[POWERCAP_FC_PAI].rate_limit); - - *p_fc =3D fc; + struct scmi_fc_info *fc_cpl0; + + for (int id =3D 0; id < dom_info->num_cpli; id++) { + struct scmi_fc_info *fc; + u32 *cpl_id, zero_cpl_id =3D 0; + + fc =3D devm_kcalloc(ph->dev, POWERCAP_FC_MAX, sizeof(*fc), GFP_KERNEL); + if (!fc) + return; + + /* NOTE THAT when num_cpli =3D=3D 1 the arg *cpl_id is 0 */ + cpl_id =3D (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x3) ? &id : NULL; + + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_CAP_SET, 4, dom_info->id, + cpl_id, + &fc[POWERCAP_FC_CAP].set_addr, + &fc[POWERCAP_FC_CAP].set_db, + &fc[POWERCAP_FC_CAP].rate_limit); + + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_CAP_GET, 4, dom_info->id, + cpl_id, + &fc[POWERCAP_FC_CAP].get_addr, NULL, + &fc[POWERCAP_FC_CAP].rate_limit); + + if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x3) { + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_CAI_SET, 4, + dom_info->id, cpl_id, + &fc[POWERCAP_FC_XAI].set_addr, + &fc[POWERCAP_FC_XAI].set_db, + &fc[POWERCAP_FC_XAI].rate_limit); + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_CAI_GET, 4, + dom_info->id, cpl_id, + &fc[POWERCAP_FC_XAI].get_addr, NULL, + &fc[POWERCAP_FC_XAI].rate_limit); + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_MAI_SET, 4, + dom_info->id, &zero_cpl_id, + &fc[POWERCAP_FC_MAI].set_addr, + &fc[POWERCAP_FC_MAI].set_db, + &fc[POWERCAP_FC_MAI].rate_limit); + + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_MAI_GET, 4, + dom_info->id, &zero_cpl_id, + &fc[POWERCAP_FC_MAI].get_addr, NULL, + &fc[POWERCAP_FC_MAI].rate_limit); + + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_MEASUREMENTS_GET, 4, + dom_info->id, &zero_cpl_id, + &fc[POWERCAP_FC_MEASUREMENT].get_addr, NULL, + &fc[POWERCAP_FC_MEASUREMENT].rate_limit); + } + + dom_info->cpli[id].fc_info =3D fc; + } + + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { + fc_cpl0 =3D dom_info->cpli[CPL0].fc_info; + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_PAI_SET, 4, + dom_info->id, NULL, + &fc_cpl0[POWERCAP_FC_XAI].set_addr, + &fc_cpl0[POWERCAP_FC_XAI].set_db, + &fc_cpl0[POWERCAP_FC_XAI].rate_limit); + + ph->hops->fastchannel_init(ph, POWERCAP_DESCRIBE_FASTCHANNEL, + POWERCAP_PAI_GET, 4, + dom_info->id, NULL, + &fc_cpl0[POWERCAP_FC_XAI].get_addr, NULL, + &fc_cpl0[POWERCAP_FC_XAI].rate_limit); + } + } =20 static int scmi_powercap_notify(const struct scmi_protocol_handle *ph, @@ -1290,9 +1411,14 @@ scmi_powercap_protocol_init(const struct scmi_protoc= ol_handle *ph) if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { pinfo->xfer_cap_get =3D scmi_powercap_xfer_cap_get; pinfo->xfer_cap_set =3D scmi_powercap_xfer_cap_set; + pinfo->xfer_avg_ivl_get =3D scmi_powercap_xfer_avg_interval_get; + pinfo->xfer_avg_ivl_set =3D scmi_powercap_xfer_avg_interval_set; + } else { pinfo->xfer_cap_get =3D scmi_powercap_xfer_cap_get_v3; pinfo->xfer_cap_set =3D scmi_powercap_xfer_cap_set_v3; + pinfo->xfer_avg_ivl_get =3D scmi_powercap_xfer_avg_interval_get_v3; + pinfo->xfer_avg_ivl_set =3D scmi_powercap_xfer_avg_interval_set_v3; } =20 ret =3D scmi_powercap_attributes_get(ph, pinfo); @@ -1317,18 +1443,19 @@ scmi_powercap_protocol_init(const struct scmi_proto= col_handle *ph) * formed and correlated by sane parent-child relationship (if any). */ for (domain =3D 0; domain < pinfo->num_domains; domain++) { - ret =3D scmi_powercap_domain_initialize(ph, pinfo, domain); + struct scmi_powercap_info *dom_info =3D pinfo->powercaps + domain; + + dom_info->id =3D domain; + ret =3D scmi_powercap_domain_attributes_get(ph, pinfo, dom_info); if (ret) return ret; =20 - if (pinfo->powercaps[domain].fastchannels) - scmi_powercap_domain_init_fc(ph, domain, - &pinfo->powercaps[domain].cpli[CPL0].fc_info); + if (dom_info->fastchannels) + scmi_powercap_domain_init_fc(ph, dom_info); =20 /* Grab initial state when disable is supported. */ if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x2) { - ret =3D __scmi_powercap_cap_get(ph, - &pinfo->powercaps[domain], CPL0, + ret =3D __scmi_powercap_cap_get(ph, dom_info, CPL0, &pinfo->states[domain].last_pcap); if (ret) return ret; --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id AB00C3D904C; Wed, 17 Jun 2026 09:59:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690386; cv=none; b=B389mOg+k9+aJqJBTNkqeuRO6St2REWrVACwd6yGZb8rFVC0X2oHbr3SnNlx8b6P1fMjKiuFhkKVfNajCVrGYSUDhwsIYG58O050x3XD0kqSPlGWKnnAi+plq6rLur6PYqz53Zx3w9eztO0sg368TAvT/rzxuMrLKmmIu05eZZY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690386; c=relaxed/simple; bh=ATBrsW2D3HgXXET57HMCGq3nDiZ26OtuLIrk0GitQpQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=UwxWHlLj8TTaXmLu+/UKWSWQS2ZowiDknt8lAW39Sld6RE8WAoUZfaJsnB/k+ROc6hX1RdW3Y2zXrukuTSsN47qHy6kcn70mvdrwTAfrUbkQXTXPMaszl/yKS0bs5VuGg7+Xwc8xEFw5NKFKeO14r0OnCzE26cNoSNBZ5plIAgQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=IFMF0i+m; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="IFMF0i+m" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A3FBE2E8E; Wed, 17 Jun 2026 02:59:38 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D8AFF3F915; Wed, 17 Jun 2026 02:59:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690383; bh=ATBrsW2D3HgXXET57HMCGq3nDiZ26OtuLIrk0GitQpQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=IFMF0i+m+aOXu5QwYwfhX/7bLpvGrGt159a/cg8S898UrfKOt6zMKFH9sAYfQQBJF vhMGdpvVdAV2QDD+4LbwEZoCig5fljgRhjlvsDPRxtDPgLYklZy54FPtZY9YQtnqAb 8s0bQv1Xa+Q2bMSctaZNTtZq5WPYlyg39fsrgrHw= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 06/13] firmware: arm_scmi: Add SCMIV4.0 Powercap notifications support Date: Wed, 17 Jun 2026 10:59:03 +0100 Message-ID: <20260617095910.1963578-7-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Extend notification support to the new SCMIv4.0 Powercap format that carry also a CPL identifier where specified. Since this addition completes SCMIv4.0 Powercap support bump also the protocol version define. Signed-off-by: Cristian Marussi Signed-off-by: Philip Radford --- drivers/firmware/arm_scmi/powercap.c | 13 +++++++++---- include/linux/scmi_protocol.h | 3 ++- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index ba037e906640..7ed6b6467813 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -18,7 +18,7 @@ #include "notify.h" =20 /* Updated only after ALL the mandatory features for that version are merg= ed */ -#define SCMI_PROTOCOL_SUPPORTED_VERSION 0x20000 +#define SCMI_PROTOCOL_SUPPORTED_VERSION 0x30000 =20 #define CPL0 0 =20 @@ -156,7 +156,8 @@ struct scmi_powercap_cap_changed_notify_payld { __le32 agent_id; __le32 domain_id; __le32 power_cap; - __le32 pai; + __le32 avg_ivl; + __le32 cpli; }; =20 struct scmi_powercap_meas_changed_notify_payld { @@ -1316,14 +1317,18 @@ scmi_powercap_fill_custom_report(const struct scmi_= protocol_handle *ph, const struct scmi_powercap_cap_changed_notify_payld *p =3D payld; struct scmi_powercap_cap_changed_report *r =3D report; =20 - if (sizeof(*p) !=3D payld_sz) + if (sizeof(*p) > payld_sz) break; =20 r->timestamp =3D timestamp; r->agent_id =3D le32_to_cpu(p->agent_id); r->domain_id =3D le32_to_cpu(p->domain_id); r->power_cap =3D le32_to_cpu(p->power_cap); - r->pai =3D le32_to_cpu(p->pai); + r->avg_ivl =3D le32_to_cpu(p->avg_ivl); + if (sizeof(*p) =3D=3D payld_sz) + r->cpli =3D le32_to_cpu(p->cpli); + else + r->cpli =3D 0; *src_id =3D r->domain_id; rep =3D r; break; diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h index 547ab4763a63..299fa8499b3f 100644 --- a/include/linux/scmi_protocol.h +++ b/include/linux/scmi_protocol.h @@ -1125,7 +1125,8 @@ struct scmi_powercap_cap_changed_report { unsigned int agent_id; unsigned int domain_id; unsigned int power_cap; - unsigned int pai; + unsigned int avg_ivl; + unsigned int cpli; }; =20 struct scmi_powercap_meas_changed_report { --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7CEAA3DA5DB; Wed, 17 Jun 2026 09:59:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690391; cv=none; b=sQCwcrcB4ZdUtoaONtdftzCtBo4YhaG9nKbUBUD5vpIocrygjvxNIF7KPWImsw7+xdeyAruDUxM/zroApWZZ8vkNUs3dYxPiEoin9/jHHNsLZIXxwYpETuhFc2fKezb0eLLS4Jzg3bAXXtD+J7xI2pyl45Mst+Q7RgPKHm/UP0E= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690391; c=relaxed/simple; bh=/CeAtOEh4M61oY+dk3U1E0yxSoq1Hc3+PQLhLb6FPS0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=PGa4SxZ9MU35tIXSSulNMu/t+YuIRUOYdDaSQQOiEd4d/kx9Lw9QlRY3GAfR/O5qXFurMl3AEEhH0BSvsYUmWoL0FBCFYO1LEYn7BRszvOKOYTrXJB+oOIlswbE1Zx/asd82LZC5amcB7IaA7jps0wYsvzXdpG6j9KgUlGh0lRA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=s9EsxCar; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="s9EsxCar" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5500D4800; Wed, 17 Jun 2026 02:59:41 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A391D3F915; Wed, 17 Jun 2026 02:59:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690386; bh=/CeAtOEh4M61oY+dk3U1E0yxSoq1Hc3+PQLhLb6FPS0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=s9EsxCart84qEWHe5y/UkVIayVPOTLJ/KVJuSsYz9j/iUueJN+9+rIMtDeW3PHrgB XqNlMf+M6BGg8FCWMubNgW2J7MK6eVf2+J7Ezwup7c5Z7dyzrqjqiurkGTGU780qNm YW9HXJYIPUzI7l6YU3ur43M7b+sKFTdorhx6dBwg= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 07/13] firmware: arm_scmi: Extend powercap report to include MAI Date: Wed, 17 Jun 2026 10:59:04 +0100 Message-ID: <20260617095910.1963578-8-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Extend scmi_powercap_meas_changed_report to include MAI change notifications. Signed-off-by: Philip Radford --- V6->V7 - Corrected naming of define sizes V5->V6 - Re-worded existing comment for POWERCAP_MEASUREMENTS_NOTIFY - Added define for V2/V3 sizes - Used new definitions --- drivers/firmware/arm_scmi/powercap.c | 28 +++++++++++++++------------- include/linux/scmi_protocol.h | 1 + 2 files changed, 16 insertions(+), 13 deletions(-) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index 7ed6b6467813..f6f9161a0138 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -11,6 +11,7 @@ #include #include #include +#include =20 #include =20 @@ -21,6 +22,8 @@ #define SCMI_PROTOCOL_SUPPORTED_VERSION 0x30000 =20 #define CPL0 0 +#define SZ_V3 (sizeof(struct scmi_powercap_meas_changed_notify_payld)) +#define SZ_V2 (SZ_V3 - sizeof(__le32)) =20 enum scmi_powercap_protocol_cmd { POWERCAP_DOMAIN_ATTRIBUTES =3D 0x3, @@ -164,6 +167,7 @@ struct scmi_powercap_meas_changed_notify_payld { __le32 agent_id; __le32 domain_id; __le32 power; + __le32 mai; }; =20 struct scmi_msg_powercap_cpc { @@ -1205,24 +1209,18 @@ static int scmi_powercap_notify(const struct scmi_p= rotocol_handle *ph, struct scmi_msg_powercap_notify_thresh *notify; =20 /* - * Note that we have to pick the most recently configured - * thresholds to build a proper POWERCAP_MEASUREMENTS_NOTIFY - * enable request and we fail, complaining, if no thresholds - * were ever set, since this is an indication the API has been - * used wrongly. + * Build the POWERCAP_MEASUREMENTS_NOTIFY enable request using the + * most recently configured thresholds. + * + * The absence of thresholds is not considered an error: + * notifications can still be generated to report MAI changes, even + * when low and high are set to zero. */ ret =3D scmi_powercap_measurements_threshold_get(ph, domain, &low, &high); if (ret) return ret; =20 - if (enable && !low && !high) { - dev_err(ph->dev, - "Invalid Measurements Notify thresholds: %u/%u\n", - low, high); - return -EINVAL; - } - ret =3D ph->xops->xfer_get_init(ph, message_id, sizeof(*notify), 0, &t); if (ret) @@ -1338,13 +1336,17 @@ scmi_powercap_fill_custom_report(const struct scmi_= protocol_handle *ph, const struct scmi_powercap_meas_changed_notify_payld *p =3D payld; struct scmi_powercap_meas_changed_report *r =3D report; =20 - if (sizeof(*p) !=3D payld_sz) + if (payld_sz !=3D SZ_V2 && payld_sz !=3D SZ_V3) break; =20 r->timestamp =3D timestamp; r->agent_id =3D le32_to_cpu(p->agent_id); r->domain_id =3D le32_to_cpu(p->domain_id); r->power =3D le32_to_cpu(p->power); + r->mai =3D 0; + if (payld_sz =3D=3D SZ_V3 && PROTOCOL_REV_MAJOR(ph->version) >=3D 0x3) + r->mai =3D le32_to_cpu(p->mai); + *src_id =3D r->domain_id; rep =3D r; break; diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h index 299fa8499b3f..d0f6c0102559 100644 --- a/include/linux/scmi_protocol.h +++ b/include/linux/scmi_protocol.h @@ -1134,5 +1134,6 @@ struct scmi_powercap_meas_changed_report { unsigned int agent_id; unsigned int domain_id; unsigned int power; + unsigned int mai; }; #endif /* _LINUX_SCMI_PROTOCOL_H */ --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 931843D9DC5; Wed, 17 Jun 2026 09:59:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690394; cv=none; b=GkR1iLjoarDxJbumQEHPU0j7e59WKHRzDisneqp5npPinhTndNKxnCSC/0tJiigIDadFhkD6FlI7Tgh+HDwl733AZfSRhUHVPoWy0axCAey6GOc4KqKTiUltsVTRbOqIlKOjPTZf8YbMHULKbAE5T227Sw7XNdhHgGnjCzAxMSQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690394; c=relaxed/simple; bh=q17AbxeYczbVwWYsFQLnHIZgYvaMz+yjOX55cezBKpQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=meUzmaTp8j2l/JbaecrAohLKboOTWxn2rNJZIOrgezvrxnFnHSIooZaY6Q9yt/DYUuaYlrKDOYcJzmvaLnnej11hcFi3xMquLJKJ4mqN4RqQvB+Lo8lLUMt4RvhnnutX2qAqnyHCFRpiBcI06ak411a6gwKok85IK05DGxiHhLI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=C/T8DX9N; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="C/T8DX9N" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 20D542E8E; Wed, 17 Jun 2026 02:59:44 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 559653F915; Wed, 17 Jun 2026 02:59:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690388; bh=q17AbxeYczbVwWYsFQLnHIZgYvaMz+yjOX55cezBKpQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=C/T8DX9NACo6tDgpPxsXx/RFKpRam4stJxsHtWnLnoXlhg908oxp8o2o+n66qebMX +/Kn470ITxr12Sq265hwCzbv+Qtu+8r5+v3/8vLBllqpAdD5jgnaLr9FMSKB73EPjj TYTUIH7ViciSH3qYdoRhzEXzD8mJ9G/56qROUeiE= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , Philip Radford Subject: [PATCH v7 08/13] include: trace: Add new parameter to trace_scmi_fc_call Date: Wed, 17 Jun 2026 10:59:05 +0100 Message-ID: <20260617095910.1963578-9-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Since SCMIv4.0 some of the supported Fastchannels can be configured using an additional parameter like CPL_ID or Capability_ID. Add equivalent support in the SCMI fastchannel traces to printout also such parameter and fix all the existent call sites. When such parameter is not used, it will simply show up as zero. Signed-off-by: Cristian Marussi Signed-off-by: Philip Radford --- drivers/firmware/arm_scmi/perf.c | 8 ++++---- drivers/firmware/arm_scmi/powercap.c | 12 +++++++----- include/trace/events/scmi.h | 12 +++++++----- 3 files changed, 18 insertions(+), 14 deletions(-) diff --git a/drivers/firmware/arm_scmi/perf.c b/drivers/firmware/arm_scmi/p= erf.c index 7f283f457e02..88d614e3184b 100644 --- a/drivers/firmware/arm_scmi/perf.c +++ b/drivers/firmware/arm_scmi/perf.c @@ -552,7 +552,7 @@ static int __scmi_perf_limits_set(const struct scmi_pro= tocol_handle *ph, struct scmi_fc_info *fci =3D &dom->fc_info[PERF_FC_LIMIT]; =20 trace_scmi_fc_call(SCMI_PROTOCOL_PERF, PERF_LIMITS_SET, - dom->id, min_perf, max_perf); + dom->id, 0, min_perf, max_perf); iowrite32(max_perf, fci->set_addr); iowrite32(min_perf, fci->set_addr + 4); ph->hops->fastchannel_db_ring(fci->set_db); @@ -636,7 +636,7 @@ static int __scmi_perf_limits_get(const struct scmi_pro= tocol_handle *ph, *max_perf =3D ioread32(fci->get_addr); *min_perf =3D ioread32(fci->get_addr + 4); trace_scmi_fc_call(SCMI_PROTOCOL_PERF, PERF_LIMITS_GET, - dom->id, *min_perf, *max_perf); + dom->id, 0, *min_perf, *max_perf); return 0; } =20 @@ -706,7 +706,7 @@ static int __scmi_perf_level_set(const struct scmi_prot= ocol_handle *ph, struct scmi_fc_info *fci =3D &dom->fc_info[PERF_FC_LEVEL]; =20 trace_scmi_fc_call(SCMI_PROTOCOL_PERF, PERF_LEVEL_SET, - dom->id, level, 0); + dom->id, 0, level, 0); iowrite32(level, fci->set_addr); ph->hops->fastchannel_db_ring(fci->set_db); return 0; @@ -769,7 +769,7 @@ static int __scmi_perf_level_get(const struct scmi_prot= ocol_handle *ph, if (dom->fc_info && dom->fc_info[PERF_FC_LEVEL].get_addr) { *level =3D ioread32(dom->fc_info[PERF_FC_LEVEL].get_addr); trace_scmi_fc_call(SCMI_PROTOCOL_PERF, PERF_LEVEL_GET, - dom->id, *level, 0); + dom->id, 0, *level, 0); return 0; } =20 diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index f6f9161a0138..1800ee295b07 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -603,7 +603,7 @@ static int __scmi_powercap_cap_get(const struct scmi_pr= otocol_handle *ph, dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_addr) { *power_cap =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_CAP].get_a= ddr); trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_CAP_GET, - dom->id, *power_cap, 0); + dom->id, cpl_id, *power_cap, 0); return 0; } =20 @@ -736,7 +736,7 @@ static int __scmi_powercap_cap_set(const struct scmi_pr= otocol_handle *ph, iowrite32(power_cap, fci->set_addr); ph->hops->fastchannel_db_ring(fci->set_db); trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_CAP_SET, - domain_id, power_cap, 0); + domain_id, cpl_id, power_cap, 0); ret =3D 0; } else { ret =3D pi->xfer_cap_set(ph, pc, cpl_id, power_cap, ignore_dresp); @@ -841,7 +841,8 @@ static int scmi_powercap_avg_interval_get(const struct = scmi_protocol_handle *ph, POWERCAP_PAI_GET : POWERCAP_CAI_GET; =20 *val =3D ioread32(dom->cpli[cpl_id].fc_info[POWERCAP_FC_XAI].get_addr); - trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, *val, 0= ); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, + cpl_id, *val, 0); return 0; } =20 @@ -914,7 +915,8 @@ static int scmi_powercap_avg_interval_set(const struct = scmi_protocol_handle *ph, POWERCAP_PAI_SET : POWERCAP_CAI_SET; struct scmi_fc_info *fci =3D &pc->cpli[cpl_id].fc_info[POWERCAP_FC_XAI]; =20 - trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, ivl, 0); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, trace_cmd, domain_id, + cpl_id, ivl, 0); iowrite32(ivl, fci->set_addr); ph->hops->fastchannel_db_ring(fci->set_db); return 0; @@ -967,7 +969,7 @@ static int scmi_powercap_measurements_get(const struct = scmi_protocol_handle *ph, /* See SCMIv4.0 3.10.2 - Payload is 32bit ONLY avg_power */ *avg_ivl =3D 0; trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_MEASUREMENTS_GET, - pc->id, *avg_power, *avg_ivl); + pc->id, 0, *avg_power, *avg_ivl); return 0; } =20 diff --git a/include/trace/events/scmi.h b/include/trace/events/scmi.h index 703b7bb68e44..b03da7323d04 100644 --- a/include/trace/events/scmi.h +++ b/include/trace/events/scmi.h @@ -10,13 +10,14 @@ #define TRACE_SCMI_MAX_TAG_LEN 6 =20 TRACE_EVENT(scmi_fc_call, - TP_PROTO(u8 protocol_id, u8 msg_id, u32 res_id, u32 val1, u32 val2), - TP_ARGS(protocol_id, msg_id, res_id, val1, val2), + TP_PROTO(u8 protocol_id, u8 msg_id, u32 res_id, u32 sub_id, u32 val1, u32= val2), + TP_ARGS(protocol_id, msg_id, res_id, sub_id, val1, val2), =20 TP_STRUCT__entry( __field(u8, protocol_id) __field(u8, msg_id) __field(u32, res_id) + __field(u32, sub_id) __field(u32, val1) __field(u32, val2) ), @@ -25,13 +26,14 @@ TRACE_EVENT(scmi_fc_call, __entry->protocol_id =3D protocol_id; __entry->msg_id =3D msg_id; __entry->res_id =3D res_id; + __entry->sub_id =3D sub_id; __entry->val1 =3D val1; __entry->val2 =3D val2; ), =20 - TP_printk("pt=3D%02X msg_id=3D%02X res_id:%u vals=3D%u:%u", - __entry->protocol_id, __entry->msg_id, - __entry->res_id, __entry->val1, __entry->val2) + TP_printk("pt=3D%02X msg_id=3D%02X res_id:%u sub_id:%u vals=3D%u:%u", + __entry->protocol_id, __entry->msg_id, + __entry->res_id, __entry->sub_id, __entry->val1, __entry->val2) ); =20 TRACE_EVENT(scmi_xfer_begin, --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 425423DEAD3; Wed, 17 Jun 2026 09:59:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690395; cv=none; b=A2uHoGT4Np2UpG4rl9AxpAs486h93TTe1lI0RYNT2GPblHshxJpKI22mD+wZTUV+W2UOD8sHdICluutOhJX1M7wsw2XnnCiYJ0mcW600+E62BFqFvG/kwco8egF20PpoBk2rbCoHdEmSbW/CJyNUZcOFQi+crUgKp1QSjZRm07U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690395; c=relaxed/simple; bh=DaBtTc44ZyRVxzGxVQBX/VonOPX9QiFrI2KUSG4ItFM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=I7T0VARSacikeln1GtGBpnbbUwRDZmsw8K4/HUzk6UHG0ocdllnmToTAc5xZE6QJGMJpdbrkATDz3zVh8FJMDRsn6v75QkP76YfKCM/sSLTotnLPSaXmkZq//ODk5cSW4UQz+hX1jpG3AaD0X6nDWTI7uBH+wopSIX7J1aJyJCo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=relyCZ52; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="relyCZ52" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 12AEF483F; Wed, 17 Jun 2026 02:59:47 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 23E2E3F915; Wed, 17 Jun 2026 02:59:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690391; bh=DaBtTc44ZyRVxzGxVQBX/VonOPX9QiFrI2KUSG4ItFM=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=relyCZ526Tbi6T4WX9R/x+kZMFgqaAVZYrre+ucHVrkrdskmjbn43fXUyBasArqUV ldXbBwNDdtYPwFn1Oa9PgHamyyviB4EPeTOTpKPyuIZMxL1//uDn/eUYRcpHrRiecV LZEBHlPuhCAxuVNcqZV1A9w0LUFQnTsSwFG/r66k= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Cristian Marussi , "Rafael J. Wysocki" , Philip Radford Subject: [PATCH v7 09/13] powercap: arm_scmi: Enable multiple constraints support Date: Wed, 17 Jun 2026 10:59:06 +0100 Message-ID: <20260617095910.1963578-10-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Cristian Marussi Initialize the domains with all the discovered available constraints, making available multiple per-domain constraints when the platform has advertised support for multiple concurrent power limits. CC: "Rafael J. Wysocki" CC: linux-pm@vger.kernel.org Signed-off-by: Cristian Marussi [Philip: Amended Copyright] Signed-off-by: Philip Radford --- drivers/powercap/arm_scmi_powercap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index 90d1fa70b1d4..2f8a1e0ecccf 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -2,7 +2,7 @@ /* * SCMI Powercap support. * - * Copyright (C) 2022 ARM Ltd. + * Copyright (C) 2022-2026 ARM Ltd. */ =20 #include @@ -449,7 +449,7 @@ static int scmi_powercap_register_zone(struct scmi_powe= rcap_root *pr, =20 z =3D powercap_register_zone(&spz->zone, scmi_top_pcntrl, spz->info->name, parent ? &parent->zone : NULL, - &zone_ops, 1, &constraint_ops); + &zone_ops, spz->info->num_cpli, &constraint_ops); if (!IS_ERR(z)) { spz->height =3D scmi_powercap_get_zone_height(spz); spz->registered =3D true; --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id AABDC3DBD47; Wed, 17 Jun 2026 09:59:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690397; cv=none; b=A+X0KC3mLNddMw/qhhdHtPMO/0ifCy6/hhpsvYAaw+QaAVE1JoDP2aFWRHxCLscmAKDqwQf9Ngx6CoSMxi5IGNLELUtSLOkhh2OQbsMDUzRKNs1vTxVLuaDTa7xRuqPQ6eQqzdwzubLB2r3Mu08JgAKoz5mtJcEivTco7hlulII= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690397; c=relaxed/simple; bh=uHOz5ORKFey5MkQv6J72tUxFO1GdRC0tk8q6upiT31o=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=CPnoDLQPZNjLRgBpfC6FQLMlSPOl2x8UwvXIAUHxsToFdCtqUyz6M2S9sNOCllO98+Iw6SFmNJYi0Q12zjtkZ2Y3dmqditkoq6i12Jjh6FbDE3fcrDLgCZwr676IfNOUktcQzmel7sU7d/DiYuxsHn9M4ahJ+utcmu9l8Oe1nTM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=N5UmhY/B; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="N5UmhY/B" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B486D2E8E; Wed, 17 Jun 2026 02:59:49 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 145BC3F915; Wed, 17 Jun 2026 02:59:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690394; bh=uHOz5ORKFey5MkQv6J72tUxFO1GdRC0tk8q6upiT31o=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=N5UmhY/BgX58SNeyva122ajvksMr/g/o+INdFxIJFi3L1ONWHdNXvMJqiajDd1Qf+ Q6s34EIMF1IHfSeb7JLKyHWnDS8YC5vb5cQY3I9tRyO7Eseay+hgOIr0zg/aZcLQqU 1CIrkKeROpgaJLxH05HYpBJWv9Uwsvx0dme/sw/0= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 10/13] firmware: arm_scmi: add Powercap MAI get/set support Date: Wed, 17 Jun 2026 10:59:07 +0100 Message-ID: <20260617095910.1963578-11-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add support for Power Measurement Averaging Interval (MAI) get and set operations to the SCMI powercap protocol driver. Extends scmi_powercap_info to store MAI configuration and implement MAI get/set via xfer and optional fast-channel support. Signed-off-by: Philip Radford --- V5-V6 - Fixed commit length - Changed warning message wording - Fixed line lengths and alignment - Updated docs for new fields --- drivers/firmware/arm_scmi/powercap.c | 127 +++++++++++++++++++++++++++ include/linux/scmi_protocol.h | 18 ++++ 2 files changed, 145 insertions(+) diff --git a/drivers/firmware/arm_scmi/powercap.c b/drivers/firmware/arm_sc= mi/powercap.c index 1800ee295b07..03ce02b1b83b 100644 --- a/drivers/firmware/arm_scmi/powercap.c +++ b/drivers/firmware/arm_scmi/powercap.c @@ -407,6 +407,34 @@ scmi_powercap_domain_attrs_process(const struct scmi_p= rotocol_handle *ph, dom_info->notify_powercap_measurement_change =3D SUPPORTS_POWERCAP_MEASUREMENTS_CHANGE_NOTIFY(flags); =20 + if (PROTOCOL_REV_MAJOR(ph->version) >=3D 0x3) { + struct scmi_msg_resp_powercap_domain_attributes_v3 *resp_v3 =3D r; + + flags =3D le32_to_cpu(resp_v3->attributes); + if (pinfo->notify_measurements_cmd) + dom_info->notify_powercap_measurement_change =3D + SUPPORTS_POWERCAP_MEASUREMENTS_CHANGE_NOTIFY(flags); + + dom_info->mai_config =3D SUPPORTS_POWERCAP_MAI_CONFIGURATION(flags); + dom_info->min_mai =3D le32_to_cpu(resp_v3->min_mai); + dom_info->max_mai =3D le32_to_cpu(resp_v3->max_mai); + dom_info->mai_step =3D le32_to_cpu(resp_v3->mai_step); + + if (dom_info->mai_config) { + ret =3D scmi_powercap_validate(dom_info->min_mai, + dom_info->max_mai, + dom_info->mai_step, + dom_info->mai_config); + + if (ret) { + dev_warn(ph->dev, "Platform reported invalid MAI config for domain %d = - %s\n", + dom_info->id, dom_info->name); + + return ret; + } + } + } + dom_info->extended_names =3D SUPPORTS_EXTENDED_NAMES(flags); =20 dom_info->async_powercap_cap_set =3D @@ -1088,6 +1116,103 @@ static int scmi_powercap_cap_enable_get(const struc= t scmi_protocol_handle *ph, return 0; } =20 +static int scmi_powercap_xfer_mai_get(const struct scmi_protocol_handle *p= h, + u32 domain_id, u32 *mai) +{ + int ret; + struct scmi_xfer *t; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_MAI_GET, sizeof(u32), + sizeof(u32), &t); + + if (ret) + return ret; + + put_unaligned_le32(domain_id, t->tx.buf); + + ret =3D ph->xops->do_xfer(ph, t); + if (!ret) + *mai =3D get_unaligned_le32(t->rx.buf); + + ph->xops->xfer_put(ph, t); + return ret; +} + +static int scmi_powercap_xfer_mai_set(const struct scmi_protocol_handle *p= h, + u32 domain_id, u32 mai) +{ + int ret; + struct scmi_xfer *t; + struct scmi_msg_powercap_cap_or_pai_set *msg; + + ret =3D ph->xops->xfer_get_init(ph, POWERCAP_MAI_SET, sizeof(*msg), + 0, &t); + if (ret) + return ret; + + msg =3D t->tx.buf; + msg->domain_id =3D cpu_to_le32(domain_id); + msg->flags =3D cpu_to_le32(0); + msg->value =3D cpu_to_le32(mai); + + ret =3D ph->xops->do_xfer(ph, t); + + ph->xops->xfer_put(ph, t); + return ret; +} + +static int +scmi_powercap_measurements_interval_get(const struct scmi_protocol_handle = *ph, + u32 domain_id, + u32 *val) +{ + const struct scmi_powercap_info *pc; + struct scmi_fc_info *fci; + + if (!val) + return -EINVAL; + + pc =3D scmi_powercap_dom_info_get(ph, domain_id); + if (!pc) + return -EINVAL; + + fci =3D pc->cpli[CPL0].fc_info; + if (fci && fci[POWERCAP_FC_MAI].get_addr) { + *val =3D ioread32(fci[POWERCAP_FC_MAI].get_addr); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_MAI_GET, + domain_id, 0, *val, 0); + return 0; + } + + return scmi_powercap_xfer_mai_get(ph, domain_id, val); +} + +static int +scmi_powercap_measurements_interval_set(const struct scmi_protocol_handle = *ph, + u32 domain_id, + u32 val) +{ + const struct scmi_powercap_info *pc; + struct scmi_fc_info *fci; + + pc =3D scmi_powercap_dom_info_get(ph, domain_id); + if (!pc) + return -EINVAL; + + if (!pc->mai_config || !val || val < pc->min_mai || val > pc->max_mai) + return -EINVAL; + + fci =3D pc->cpli[CPL0].fc_info; + if (fci && fci[POWERCAP_FC_MAI].set_addr) { + iowrite32(val, fci[POWERCAP_FC_MAI].set_addr); + ph->hops->fastchannel_db_ring(fci[POWERCAP_FC_MAI].set_db); + trace_scmi_fc_call(SCMI_PROTOCOL_POWERCAP, POWERCAP_MAI_SET, domain_id, = 0, val, 0); + return 0; + } + + return scmi_powercap_xfer_mai_set(ph, domain_id, val); +} + static const struct scmi_powercap_proto_ops powercap_proto_ops =3D { .num_domains_get =3D scmi_powercap_num_domains_get, .info_get =3D scmi_powercap_dom_info_get, @@ -1100,6 +1225,8 @@ static const struct scmi_powercap_proto_ops powercap_= proto_ops =3D { .measurements_get =3D scmi_powercap_measurements_get, .measurements_threshold_set =3D scmi_powercap_measurements_threshold_set, .measurements_threshold_get =3D scmi_powercap_measurements_threshold_get, + .measurements_interval_get =3D scmi_powercap_measurements_interval_get, + .measurements_interval_set =3D scmi_powercap_measurements_interval_set, }; =20 static void scmi_powercap_domain_init_fc(const struct scmi_protocol_handle= *ph, diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h index d0f6c0102559..90615611be4a 100644 --- a/include/linux/scmi_protocol.h +++ b/include/linux/scmi_protocol.h @@ -654,6 +654,12 @@ struct scmi_powercap_cpl_info { * reports power data on an abstract linear scale. * @extended_names: Support for long names. * @fastchannels: Support for at least one fastchannel, + * @mai_config: MAI configuration support. + * @min_mai: Minimum supported Power Measurement Averaging Interval in + * microseconds. + * @max_mai: Maximum supporte Power Measurement Averaging Interval in + microseconds. + * @mai_step: Step size between supported MAI values in microseconds. * @name: name assigned to the Powercap Domain by platform. * @sustainable_power: Maximum sustainable power consumption for this doma= in * under normal conditions. @@ -675,6 +681,10 @@ struct scmi_powercap_info { bool powercap_scale_uw; bool extended_names; bool fastchannels; + bool mai_config; + u32 min_mai; + u32 max_mai; + u32 mai_step; char name[SCMI_MAX_STR_SIZE]; unsigned int sustainable_power; unsigned int accuracy; @@ -733,6 +743,10 @@ struct scmi_powercap_info { * @measurements_threshold_get: get the currently configured low and high = power * thresholds used when registering callbacks for * notification POWERCAP_MEASUREMENTS_NOTIFY. + * @measurements_interval_get: get the current Power Measurement Averaging + * Interval (MAI) value for the specified domain. + * @measurements_interval_set: set the Power Measurement Averaging Interval + * (MAI) value for the specified domain. */ struct scmi_powercap_proto_ops { int (*num_domains_get)(const struct scmi_protocol_handle *ph); @@ -758,6 +772,10 @@ struct scmi_powercap_proto_ops { int (*measurements_threshold_get)(const struct scmi_protocol_handle *ph, u32 domain_id, u32 *power_thresh_low, u32 *power_thresh_high); + int (*measurements_interval_get)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 *val); + int (*measurements_interval_set)(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 val); }; =20 enum scmi_pinctrl_selector_type { --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A34EA3D3CEC; Wed, 17 Jun 2026 09:59:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690400; cv=none; b=rmh+gUocYHovHVuxiTztDzymWvCpHHy53sPVCLsklFAiXW3tAyE4a/EwYkkTudiDcod/po/9twn9nU77/kPTfBJaEhToCqcEhQykSbbl5Na/aHfTu42Fx/2tiQQqvMWkz0A1Bd5eoOqR4WkYIJ8p4qDNTswn82y5NtbPTxXTxaI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690400; c=relaxed/simple; bh=DnbYuP1p4H3RSxY3W/5qQW7k9Y1jiRH6deGHPh5cxWg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=W4piBYaGzVpXe6X5ui5iDAyZITrWZH3ySwEQaVyQ25p7XwNLKkoZyA6xpsc1q8yAUeX2TpnrWepbrVbp/v0xxnDLt5WRs07zizMh5cH1iKExk2WN+DgKZaNDG/hHZXhjR0HyT8e5fz7Lv8RjtYZ1HpsgZRy+v3oKItcoG6jRSXY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=feVdqgGF; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="feVdqgGF" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 621954800; Wed, 17 Jun 2026 02:59:52 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B69213F915; Wed, 17 Jun 2026 02:59:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690397; bh=DnbYuP1p4H3RSxY3W/5qQW7k9Y1jiRH6deGHPh5cxWg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=feVdqgGFGwuLhRyVjcb/65JsIPkMvgkUBEmH4HK+QOuho/uHgbMw9Kdz/w5ssPLN4 RxvujQTO9NyNPPF7A83itJLWtcZKR+riPOYAW3lqdCiTTDSGPMpa9qOBvDxDvb1P84 dhvm+GMgzeHbmA3jm8bYLjU08+WWQGlkoOi0cCGw= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 11/13] powercap: arm_scmi: Create synthetic parent node for multi-instance Date: Wed, 17 Jun 2026 10:59:08 +0100 Message-ID: <20260617095910.1963578-12-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" An SCMI powercap instance may expose a hierarchy of domains, or even a forest of multiple domain trees rooted at SCMI_POWERCAP_ROOT_ZONE_ID. Those hierarchies are valid within the namespace of a single SCMI instance. Currently, the powercap framework has no notion of SCMI instances. If root domains from multiple SCMI instances are registered directly under the same Linux powercap control type, the per-instance boundaries are lost and the resulting Linux hierarchy becomes a merge of otherwise independent SCMI topologies. Add a synthetic top-level powercap zone per SCMI instance and register that instance's SCMI root domains beneath it. This keeps each instance's SCMI hierarchy grouped together. Signed-off-by: Philip Radford --- V6->V7 - Added cleanup for zones during scmi_powercap_probe V5->V6 - Amended omission of spz initialization - Tested unloading and loading powercap module - Re-wrote commit message --- drivers/powercap/arm_scmi_powercap.c | 93 +++++++++++++++++++++++++--- 1 file changed, 86 insertions(+), 7 deletions(-) diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index 2f8a1e0ecccf..d45e4af0cdc7 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -38,6 +38,7 @@ struct scmi_powercap_root { struct scmi_powercap_zone *spzones; struct list_head *registered_zones; struct list_head scmi_zones; + struct scmi_powercap_zone instance_root; }; =20 static LIST_HEAD(scmi_powercap_roots); @@ -401,18 +402,61 @@ static const struct powercap_zone_constraint_ops cons= traint_ops =3D { .get_name =3D scmi_powercap_get_name, }; =20 +/* + * Multi-instance constraints to meet driver requrements due to the fact + * that full zone semantics aren't available for the synthetic zone. + */ +static int instance_root_release(struct powercap_zone *pz) +{ + return 0; +} + +static int instance_root_get_power_uw(struct powercap_zone *pz, u64 *v) +{ + *v =3D 0; + return 0; +} + +static int instance_root_set_constraint(struct powercap_zone *pz, int cid,= u64 v) +{ + return -EOPNOTSUPP; +} + +static int instance_root_get_constraint(struct powercap_zone *pz, int cid,= u64 *v) +{ + return -EOPNOTSUPP; +} + +static const struct powercap_zone_ops instance_root_ops =3D { + .get_max_power_range_uw =3D scmi_powercap_get_max_power_range_uw, + .get_power_uw =3D instance_root_get_power_uw, + .release =3D instance_root_release, +}; + +static const struct powercap_zone_constraint_ops instance_root_const_ops = =3D { + .set_power_limit_uw =3D instance_root_set_constraint, + .get_power_limit_uw =3D instance_root_get_constraint, + .set_time_window_us =3D instance_root_set_constraint, + .get_time_window_us =3D instance_root_get_constraint, +}; + static void scmi_powercap_unregister_all_zones(struct scmi_powercap_root *= pr) { int i; =20 /* Un-register children zones first starting from the leaves */ - for (i =3D pr->num_zones - 1; i >=3D 0; i--) { + for (i =3D pr->num_zones; i >=3D 0; i--) { if (!list_empty(&pr->registered_zones[i])) { struct scmi_powercap_zone *spz; =20 - list_for_each_entry(spz, &pr->registered_zones[i], node) + list_for_each_entry(spz, &pr->registered_zones[i], node) { + if (!spz->registered) + continue; + + spz->registered =3D false; powercap_unregister_zone(scmi_top_pcntrl, &spz->zone); + } } } } @@ -451,7 +495,10 @@ static int scmi_powercap_register_zone(struct scmi_pow= ercap_root *pr, parent ? &parent->zone : NULL, &zone_ops, spz->info->num_cpli, &constraint_ops); if (!IS_ERR(z)) { - spz->height =3D scmi_powercap_get_zone_height(spz); + if (parent) + spz->height =3D parent->height + 1; + else + spz->height =3D 0; spz->registered =3D true; list_move(&spz->node, &pr->registered_zones[spz->height]); dev_dbg(spz->dev, "Registered node %s - parent %s - height:%d\n", @@ -522,6 +569,8 @@ static int scmi_zones_register(struct device *dev, struct scmi_powercap_zone *parent; =20 parent =3D scmi_powercap_get_parent_zone(spz); + if (!parent) + parent =3D &pr->instance_root; if (parent && !parent->registered) { zones_stack[sp++] =3D spz; spz =3D parent; @@ -562,8 +611,11 @@ static int scmi_powercap_probe(struct scmi_device *sde= v) int ret, i; struct scmi_powercap_root *pr; struct scmi_powercap_zone *spz; + struct scmi_powercap_zone *ir; struct scmi_protocol_handle *ph; struct device *dev =3D &sdev->dev; + char *instance_name; + struct powercap_zone *z; =20 if (!sdev->handle) return -ENODEV; @@ -591,7 +643,7 @@ static int scmi_powercap_probe(struct scmi_device *sdev) return -ENOMEM; =20 /* Allocate for worst possible scenario of maximum tree height. */ - pr->registered_zones =3D devm_kcalloc(dev, pr->num_zones, + pr->registered_zones =3D devm_kcalloc(dev, pr->num_zones + 1, sizeof(*pr->registered_zones), GFP_KERNEL); if (!pr->registered_zones) @@ -599,6 +651,9 @@ static int scmi_powercap_probe(struct scmi_device *sdev) =20 INIT_LIST_HEAD(&pr->scmi_zones); =20 + for (i =3D 0; i <=3D pr->num_zones; i++) + INIT_LIST_HEAD(&pr->registered_zones[i]); + for (i =3D 0, spz =3D pr->spzones; i < pr->num_zones; i++, spz++) { /* * Powercap domains are validate by the protocol layer, i.e. @@ -611,7 +666,6 @@ static int scmi_powercap_probe(struct scmi_device *sdev) spz->ph =3D ph; spz->spzones =3D pr->spzones; INIT_LIST_HEAD(&spz->node); - INIT_LIST_HEAD(&pr->registered_zones[i]); =20 list_add_tail(&spz->node, &pr->scmi_zones); /* @@ -629,19 +683,44 @@ static int scmi_powercap_probe(struct scmi_device *sd= ev) } } =20 + ir =3D &pr->instance_root; + ir->dev =3D dev; + INIT_LIST_HEAD(&ir->node); + instance_name =3D devm_kasprintf(dev, GFP_KERNEL, "instance_%s", dev_name= (dev)); + if (!instance_name) + return -ENOMEM; + + z =3D powercap_register_zone(&ir->zone, scmi_top_pcntrl, + instance_name, NULL, &instance_root_ops, 0, + &instance_root_const_ops); + + if (IS_ERR(z)) { + ret =3D PTR_ERR(z); + dev_err(dev, "Failed to register sysnthetic instance root: %d\n", ret); + return ret; + } + + ir->registered =3D true; + ir->height =3D 0; + list_add_tail(&ir->node, &pr->registered_zones[0]); + /* * Scan array of retrieved SCMI powercap domains and register them * recursively starting from the root domains. */ ret =3D scmi_zones_register(dev, pr); - if (ret) + if (ret) { + scmi_powercap_unregister_all_zones(pr); return ret; + } =20 INIT_LIST_HEAD(&pr->node); =20 ret =3D scmi_powercap_read_root_children_enable_state(pr, &pr->enabled); - if (ret) + if (ret) { + scmi_powercap_unregister_all_zones(pr); return ret; + } =20 mutex_lock(&scmi_powercap_roots_lock); list_add_tail(&pr->node, &scmi_powercap_roots); --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5F57A3E1216; Wed, 17 Jun 2026 10:00:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690402; cv=none; b=BJ2gor1NO0p+X49vvymTw2VHun9jdU26+acKOzby9VO+tGKkD2IDQi9Zm5bazt4tjw2z1Ft8RVWSycbr/Jp3HqesrFJqZw3szr6IYMNsMFsgLW34orvwq+jd66Qfua7UHtF9pa1E/nIfhjCiwbEyY1oyIQO+WOX2l1U472NaTCc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690402; c=relaxed/simple; bh=uQ2IxnI5PBbEK+vSVRZY0b0+hKZufqL3dFY/NKgEynE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Iiy3BpX4CiFHwD1GyJSx8hIBHllaWU7k2iTrtooylgG9yHL67xOvEhMDNKpnQ84TGVFjJsx3epnIA64wGlHd+wIT2N4xE6In7CkATeORj4drH4uKsGNZvABcpggCdPTgKltY/Q3zOoiSMQo+v9W9jVkbHVNY/TcXVeWCPkBNAM0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=fb4KH2eg; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="fb4KH2eg" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 125F92E8E; Wed, 17 Jun 2026 02:59:55 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 63E6A3F915; Wed, 17 Jun 2026 02:59:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690399; bh=uQ2IxnI5PBbEK+vSVRZY0b0+hKZufqL3dFY/NKgEynE=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fb4KH2egIRDRjwAOvCEK9JUixNmbOr4tnpIYvgtFlsXP0LvcktPpwHXUNErbHHcpr MJDY3O2zl8+vRW01sdRkB0sT9m7TlKVq10SmkeKi+FmBxCDG9zA8bZfHVN+ZO0/eko qU5wbefLXSocyDoXhnqPoG0PuG1jh2k0KNiSd7Uo= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 12/13] powercap: arm_scmi: Add get_power_uw to synthetic node Date: Wed, 17 Jun 2026 10:59:09 +0100 Message-ID: <20260617095910.1963578-13-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Exposes the current power usage from the immediate children of the synthetic (root) powercap node. Iterates over pr->spzones and sums per-zone power. Signed-off-by: Philip Radford --- V6->V7 - Fixed commit message length V5->V6 - Moved multiple u64 values to the same line - Reworked logic to walk SCMI zones and filter by parent - Added to_scmi_powercap_root macro --- drivers/powercap/arm_scmi_powercap.c | 35 ++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index d45e4af0cdc7..e1bad4b19990 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -17,6 +17,9 @@ #define to_scmi_powercap_zone(z) \ container_of(z, struct scmi_powercap_zone, zone) =20 +#define to_scmi_powercap_root(z) \ + container_of(z, struct scmi_powercap_root, instance_root.zone) + static const struct scmi_powercap_proto_ops *powercap_ops; =20 struct scmi_powercap_zone { @@ -411,9 +414,37 @@ static int instance_root_release(struct powercap_zone = *pz) return 0; } =20 -static int instance_root_get_power_uw(struct powercap_zone *pz, u64 *v) +static int instance_root_get_power_uw(struct powercap_zone *pz, u64 *power= _uw) { - *v =3D 0; + struct scmi_powercap_root *pr =3D to_scmi_powercap_root(pz); + struct scmi_powercap_zone *child; + + u64 p, acc =3D 0; + int i, ret; + + if (!pz || !power_uw) + return -EINVAL; + + if (!pr) + return -ENODEV; + + for (i =3D 0; i < pr->num_zones; i++) { + child =3D &pr->spzones[i]; + + if (!child->registered || child->invalid) + continue; + + if (child->info->parent_id !=3D SCMI_POWERCAP_ROOT_ZONE_ID) + continue; + + ret =3D scmi_powercap_get_power_uw(&child->zone, &p); + if (!ret) + acc +=3D p; + else + dev_dbg(child->dev, "Failed to read child power: %u\n", ret); + } + + *power_uw =3D acc; return 0; } =20 --=20 2.47.3 From nobody Thu Jun 18 19:22:54 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 1C84F3E3D86; Wed, 17 Jun 2026 10:00:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690404; cv=none; b=KXjj3egYxk6CKerEcoxjdGU9s9N8+iJFyiuq7oyDAAm2d7KuHSgXwNMhv6JdYGNAs5NXEejJymYPU0MOTJbMxaf/cR+VP4DCC4foEt33FRZPSWOr1gkI/UiHn1QWI6uL8ZnCLFd2F6QPbQ54LoKzNzmo7cr/UQTV6qDsRqezUEc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781690404; c=relaxed/simple; bh=ysAvUv+uQKD7SvxfHb5LUEwIT9/VWVtRXW6Ysv0mpjY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=X+i65xNS5fVbt5oIQKK1ThTllELejPTKMaqpXUDesFuM9Z3VKoWTCQFc0VJOoOTQlGXCBgr6/6Rdk7kcjnLa0Uuka/ogOoa4IKYrCNXYFbp2bnZrZM6GpZS8yDJ1gYHVadpbB6YYXerW76qOikdQj79XgX1hERSZdTNB6Y4XmUQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=XBaCmaZH; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="XBaCmaZH" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B7C1B2E8E; Wed, 17 Jun 2026 02:59:57 -0700 (PDT) Received: from donnerap.manchester.arm.com (donnerap.manchester.arm.com [10.33.8.81]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 1511E3F915; Wed, 17 Jun 2026 02:59:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1781690402; bh=ysAvUv+uQKD7SvxfHb5LUEwIT9/VWVtRXW6Ysv0mpjY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=XBaCmaZHYV/1Vq+XaHmiTF7v+sLjlJed8tDn3PmgDLPVjLmboaRsxR3PdwzgOgoxJ 18r1ZNOu5Yh1oALXt7Kwxd2kpwFzKm9F+2VH0iOkNYkDUMJlP704J2jwHaZV/XROXk I3H6rYvqa6I8YXINDWyKIvx9ydyrqp0x1gxmAOS4= From: Philip Radford To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, linux-pm@vger.kernel.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, dan.carpenter@linaro.org, d-gole@ti.com, souvik.chakravarty@arm.com, Philip Radford Subject: [PATCH v7 13/13] powercap: arm_scmi: Synthetic zone enable/disable Date: Wed, 17 Jun 2026 10:59:10 +0100 Message-ID: <20260617095910.1963578-14-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260617095910.1963578-1-philip.radford@arm.com> References: <20260617095910.1963578-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The synthetic instance root contols the same set of top-level SCMI domains already handled by the control-type enable/disable helpers previously introduced. Add synthetic zone enabled attribute to the existing per-instance helpers instead of duplicating the enable-state tracking and rollback logic. Signed-off-by: Philip Radford --- V6->V7 - Re-factored due to new control-type enable disable patch at the beginning of the series - Prevent instance_root_get_enable() reporting a stale state V5->V6 - Added use of to_scmi_powercap_root macro - Changed instance_root_set_)enable_state to bail out on any error - Changed logic in instance_root_get_enable to not check child states --- drivers/powercap/arm_scmi_powercap.c | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/drivers/powercap/arm_scmi_powercap.c b/drivers/powercap/arm_sc= mi_powercap.c index e1bad4b19990..23708f9934d4 100644 --- a/drivers/powercap/arm_scmi_powercap.c +++ b/drivers/powercap/arm_scmi_powercap.c @@ -458,10 +458,34 @@ static int instance_root_get_constraint(struct powerc= ap_zone *pz, int cid, u64 * return -EOPNOTSUPP; } =20 +static int instance_root_set_enable(struct powercap_zone *pz, bool mode) +{ + struct scmi_powercap_root *pr =3D to_scmi_powercap_root(pz); + + return scmi_powercap_set_root_children_enable_state(pr, mode); +} + +static int instance_root_get_enable(struct powercap_zone *pz, bool *mode) +{ + struct scmi_powercap_root *pr =3D to_scmi_powercap_root(pz); + int ret; + + if (!mode) + return -EINVAL; + + ret =3D scmi_powercap_read_root_children_enable_state(pr, mode); + if (!ret) + pr->enabled =3D *mode; + + return ret; +} + static const struct powercap_zone_ops instance_root_ops =3D { .get_max_power_range_uw =3D scmi_powercap_get_max_power_range_uw, .get_power_uw =3D instance_root_get_power_uw, .release =3D instance_root_release, + .set_enable =3D instance_root_set_enable, + .get_enable =3D instance_root_get_enable, }; =20 static const struct powercap_zone_constraint_ops instance_root_const_ops = =3D { --=20 2.47.3