From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A5D0A3D47BF; Wed, 1 Apr 2026 09:54:12 +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=1775037259; cv=none; b=QUVWPpVeLT4UQBpKql1lWHtao33YRce3orj1sghqKGbxQgajdm9TnV+dPzxnzKl6wxkOYOXnBsMn8lBl6l5iMpoOAUjisKfg3gMGLy1UDPY3AFq1D4QLWGMhVshya0h/aouPRX6v/ibRN2tfjRsbF4C/C7IDwjvL+EqBlFhZ/oA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775037259; c=relaxed/simple; bh=ZHCAt9+eIMERbg9jOsZvNaQAOAX4yqjz9F2iiciZA6w=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=mEJZXzTBmPnsXL2PdnkhGmYDGG5I6gWM6HJrfgJx8M4G2pRdgesO92/eyZDo+7NZJXVJck7CZAhTOFjuCh5vNhWmSgWGPZ5HePz9nIJ6Uhqvd0GQBA+j3R/5eNAc5uLs3IDK5ZVonDgRhsCSmj8uCC7LcoHG+ZmMuGcCa2p4q4w= 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=I9oJnh70; 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="I9oJnh70" 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 014E4288E; Wed, 1 Apr 2026 02:54:05 -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 CAF113F7D8; Wed, 1 Apr 2026 02:54:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1775037250; bh=ZHCAt9+eIMERbg9jOsZvNaQAOAX4yqjz9F2iiciZA6w=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=I9oJnh70qK8c0G1rgNzRFMFnAz4Kt3f1EfHoFHkZlw4mydAporhNApPYDnZlH6ILC MZy15OlpCv5og257vzFIwbn5MrDP5N8ccbg3Txe9TcXqT5vjX4DpZ145yW9xB2Brfp VXwTYf1IvBKPkNQI79IDVdQab4+d1vToWXDbHqok= 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 v4 04/11] firmware: arm_scmi: Add SCMIv4.0 Powercap FCs support Date: Wed, 1 Apr 2026 10:53:42 +0100 Message-ID: <20260401095349.2217840-5-philip.radford@arm.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260401095349.2217840-1-philip.radford@arm.com> References: <20260401095349.2217840-1-philip.radford@arm.com> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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_scmi/powercap.c index 3291bde78367..db5bc4f38ea4 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 = 0x4, POWERCAP_CAP_SET = 0x5, POWERCAP_PAI_GET = 0x6, + POWERCAP_MAI_GET = POWERCAP_PAI_GET, POWERCAP_PAI_SET = 0x7, + POWERCAP_MAI_SET = POWERCAP_PAI_SET, POWERCAP_DOMAIN_NAME_GET = 0x8, POWERCAP_MEASUREMENTS_GET = 0x9, POWERCAP_CAP_NOTIFY = 0xa, POWERCAP_MEASUREMENTS_NOTIFY = 0xb, POWERCAP_DESCRIBE_FASTCHANNEL = 0xc, POWERCAP_CPC_ATTRIBUTES = 0xd, + POWERCAP_CAI_GET = 0xe, + POWERCAP_CAI_SET = 0xf, }; enum { POWERCAP_FC_CAP, - POWERCAP_FC_PAI, - POWERCAP_FC_MAX, + POWERCAP_FC_XAI, + POWERCAP_FC_MAI, + POWERCAP_FC_MEASUREMENT, + POWERCAP_FC_MAX }; struct scmi_msg_resp_powercap_domain_attributes { @@ -91,12 +97,12 @@ struct scmi_msg_resp_powercap_domain_attributes_v3 { __le32 cai_step; }; -struct scmi_msg_powercap_get_v3 { +struct scmi_msg_powercap_cap_or_cai_get_v3 { __le32 domain_id; __le32 cpli; }; -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; }; -struct scmi_msg_powercap_set_cap_v3 { +struct scmi_msg_powercap_cap_set_v3 { __le32 domain_id; __le32 cpli; __le32 flags; __le32 power_cap; }; +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); }; static enum scmi_powercap_protocol_cmd evt_2_cmd[] = { @@ -501,17 +518,6 @@ scmi_powercap_domain_attributes_get(const struct scmi_protocol_handle *ph, return ret; } -static int -scmi_powercap_domain_initialize(const struct scmi_protocol_handle *ph, - struct powercap_info *pinfo, u32 domain) -{ - struct scmi_powercap_info *dom_info = pinfo->powercaps + domain; - - dom_info->id = 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 = ph->get_priv(ph); @@ -558,7 +564,7 @@ static int scmi_powercap_xfer_cap_get_v3(const struct scmi_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; ret = ph->xops->xfer_get_init(ph, POWERCAP_CAP_GET, sizeof(*msg), sizeof(u32), &t); @@ -617,7 +623,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; ret = ph->xops->xfer_get_init(ph, POWERCAP_CAP_SET, sizeof(*msg), 0, &t); @@ -660,7 +666,7 @@ static int scmi_powercap_xfer_cap_set_v3(const struct scmi_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; ret = ph->xops->xfer_get_init(ph, POWERCAP_CAP_SET, sizeof(*msg), 0, &t); @@ -759,8 +765,9 @@ static int scmi_powercap_cap_set(const struct scmi_protocol_handle *ph, power_cap, ignore_dresp); } -static int scmi_powercap_xfer_pai_get(const struct scmi_protocol_handle *ph, - 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; @@ -773,58 +780,105 @@ static int scmi_powercap_xfer_pai_get(const struct scmi_protocol_handle *ph, put_unaligned_le32(domain_id, t->tx.buf); ret = ph->xops->do_xfer(ph, t); if (!ret) - *pai = get_unaligned_le32(t->rx.buf); + *ivl = 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 = ph->xops->xfer_get_init(ph, POWERCAP_CAI_GET, sizeof(*msg), + sizeof(u32), &t); + if (ret) + return ret; + + msg = t->tx.buf; + msg->domain_id = cpu_to_le32(domain_id); + msg->cpli = cpu_to_le32(cpl_id); + + ret = ph->xops->do_xfer(ph, t); + if (!ret) + *ivl = get_unaligned_le32(t->rx.buf); ph->xops->xfer_put(ph, t); return ret; } -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_handle *ph, + u32 domain_id, u32 cpl_id, u32 *val) { struct scmi_powercap_info *dom; struct powercap_info *pi = ph->get_priv(ph); - if (!pai || domain_id >= pi->num_domains) + if (!val || domain_id >= pi->num_domains) return -EINVAL; dom = pi->powercaps + domain_id; if (cpl_id >= dom->num_cpli) return -EINVAL; - if (dom->cpli[cpl_id].fc_info && dom->cpli[cpl_id].fc_info[POWERCAP_FC_PAI].get_addr) { - *pai = 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 = (PROTOCOL_REV_MAJOR(ph->version) < 0x3) ? + POWERCAP_PAI_GET : POWERCAP_CAI_GET; + + *val = 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; } - return scmi_powercap_xfer_pai_get(ph, domain_id, pai); + return pi->xfer_avg_ivl_get(ph, domain_id, cpl_id, val); } -static int scmi_powercap_avg_interval_get(const struct scmi_protocol_handle *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 = ph->xops->xfer_get_init(ph, POWERCAP_PAI_SET, sizeof(*msg), 0, &t); + if (ret) + return ret; + + msg = t->tx.buf; + msg->domain_id = cpu_to_le32(domain_id); + msg->flags = cpu_to_le32(0); + msg->value = cpu_to_le32(ivl); + + ret = ph->xops->do_xfer(ph, t); + + ph->xops->xfer_put(ph, t); + return ret; } -static int scmi_powercap_xfer_pai_set(const struct scmi_protocol_handle *ph, - 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; - ret = ph->xops->xfer_get_init(ph, POWERCAP_PAI_SET, - sizeof(*msg), 0, &t); + ret = ph->xops->xfer_get_init(ph, POWERCAP_CAI_SET, sizeof(*msg), 0, &t); if (ret) return ret; msg = t->tx.buf; msg->domain_id = cpu_to_le32(domain_id); msg->flags = cpu_to_le32(0); - msg->value = cpu_to_le32(pai); + msg->cai = cpu_to_le32(ivl); + msg->cpli = cpu_to_le32(cpl_id); ret = ph->xops->do_xfer(ph, t); @@ -832,48 +886,42 @@ static int scmi_powercap_xfer_pai_set(const struct scmi_protocol_handle *ph, return ret; } -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_handle *ph, + u32 domain_id, u32 cpl_id, u32 ivl) { const struct scmi_powercap_info *pc; + struct powercap_info *pi = ph->get_priv(ph); pc = scmi_powercap_dom_info_get(ph, domain_id); if (!pc || cpl_id >= 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; - if (pc->cpli[cpl_id].fc_info && pc->cpli[cpl_id].fc_info[POWERCAP_FC_PAI].set_addr) { - struct scmi_fc_info *fci = &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 = (PROTOCOL_REV_MAJOR(ph->version) < 0x3) ? + POWERCAP_PAI_SET : POWERCAP_CAI_SET; + struct scmi_fc_info *fci = &pc->cpli[cpl_id].fc_info[POWERCAP_FC_XAI]; - 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; } - return scmi_powercap_xfer_pai_set(ph, domain_id, pai); -} - -static int scmi_powercap_avg_interval_set(const struct scmi_protocol_handle *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); } -static int scmi_powercap_measurements_get(const struct scmi_protocol_handle *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 = scmi_powercap_dom_info_get(ph, domain_id); - if (!pc || !pc->powercap_monitoring || !pai || !average_power) - return -EINVAL; ret = ph->xops->xfer_get_init(ph, POWERCAP_MEASUREMENTS_GET, sizeof(u32), sizeof(*resp), &t); @@ -881,17 +929,42 @@ static int scmi_powercap_measurements_get(const struct scmi_protocol_handle *ph, return ret; resp = t->rx.buf; - put_unaligned_le32(domain_id, t->tx.buf); + put_unaligned_le32(pc->id, t->tx.buf); ret = ph->xops->do_xfer(ph, t); if (!ret) { - *average_power = le32_to_cpu(resp->power); - *pai = le32_to_cpu(resp->pai); + *avg_power = le32_to_cpu(resp->power); + *avg_ivl = le32_to_cpu(resp->pai); } ph->xops->xfer_put(ph, t); return ret; } +static int scmi_powercap_measurements_get(const struct scmi_protocol_handle *ph, + u32 domain_id, u32 *avg_power, + u32 *avg_ivl) +{ + const struct scmi_powercap_info *pc; + struct scmi_fc_info *fci; + + pc = 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 = pc->cpli[CPL0].fc_info; + if (fci && fci[POWERCAP_FC_MEASUREMENT].get_addr) { + *avg_power = ioread32(fci[POWERCAP_FC_MEASUREMENT].get_addr); + /* See SCMIv4.0 3.10.2 - Payload is 32bit ONLY avg_power */ + *avg_ivl = 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, @@ -1019,37 +1092,85 @@ static const struct scmi_powercap_proto_ops powercap_proto_ops = { }; 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 = 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 = fc; + struct scmi_fc_info *fc_cpl0; + + for (int id = 0; id < dom_info->num_cpli; id++) { + struct scmi_fc_info *fc; + u32 *cpl_id, zero_cpl_id = 0; + + fc = devm_kcalloc(ph->dev, POWERCAP_FC_MAX, sizeof(*fc), GFP_KERNEL); + if (!fc) + return; + + /* NOTE THAT when num_cpli == 1 the arg *cpl_id is 0 */ + cpl_id = (PROTOCOL_REV_MAJOR(ph->version) >= 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) >= 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 = fc; + } + + if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { + fc_cpl0 = 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); + } + } static int scmi_powercap_notify(const struct scmi_protocol_handle *ph, @@ -1286,9 +1407,14 @@ scmi_powercap_protocol_init(const struct scmi_protocol_handle *ph) if (PROTOCOL_REV_MAJOR(ph->version) < 0x3) { pinfo->xfer_cap_get = scmi_powercap_xfer_cap_get; pinfo->xfer_cap_set = scmi_powercap_xfer_cap_set; + pinfo->xfer_avg_ivl_get = scmi_powercap_xfer_avg_interval_get; + pinfo->xfer_avg_ivl_set = scmi_powercap_xfer_avg_interval_set; + } else { pinfo->xfer_cap_get = scmi_powercap_xfer_cap_get_v3; pinfo->xfer_cap_set = scmi_powercap_xfer_cap_set_v3; + pinfo->xfer_avg_ivl_get = scmi_powercap_xfer_avg_interval_get_v3; + pinfo->xfer_avg_ivl_set = scmi_powercap_xfer_avg_interval_set_v3; } ret = scmi_powercap_attributes_get(ph, pinfo); @@ -1313,18 +1439,19 @@ scmi_powercap_protocol_init(const struct scmi_protocol_handle *ph) * formed and correlated by sane parent-child relationship (if any). */ for (domain = 0; domain < pinfo->num_domains; domain++) { - ret = scmi_powercap_domain_initialize(ph, pinfo, domain); + struct scmi_powercap_info *dom_info = pinfo->powercaps + domain; + + dom_info->id = domain; + ret = scmi_powercap_domain_attributes_get(ph, pinfo, dom_info); if (ret) return ret; - 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); /* Grab initial state when disable is supported. */ if (PROTOCOL_REV_MAJOR(ph->version) >= 0x2) { - ret = __scmi_powercap_cap_get(ph, - &pinfo->powercaps[domain], CPL0, + ret = __scmi_powercap_cap_get(ph, dom_info, CPL0, &pinfo->states[domain].last_pcap); if (ret) return ret; -- 2.47.3