From patchwork Wed Mar 30 15:05:29 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Cristian Marussi X-Patchwork-Id: 12795991 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4A218C433FE for ; Wed, 30 Mar 2022 15:08:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=aPvlEy1l8aUIJLXaeUrAbuYv5Fh+Wy1XRu8APeAgHHI=; b=ol1po1aBu7Vx2q OUM6S49VHYO8rAk1IGOiz9p7H4BG7WeN/lHsXBpqX8c9K15DwBYul8xI6CDBd27lZaTf7z9WUdWay pA50LiEPu1J7mNGCubRnSGhVVVJhOYzEsrhLRfyGb9kR7fRRpQpEL7tHx9RsvDYgzfjzOBgIPx6rP 7LyYF6TJvqoKe6lnrPGvYIaCin1vfKd82gB2jscIv2Fd+jrr0+spFa+wb17E7dr9BCyfGOnuKT8E5 M2pjm7wdXiPkNXGIeHR8duRp/zK+0XAEykgpYpDitnFft5uKMhqR6VqIRcW7dc5OWCnCU36N2oDLL JalzQXMQyG4G8mBF/NTQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZZuC-00GPfV-CJ; Wed, 30 Mar 2022 15:06:40 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZZu2-00GPdG-No for linux-arm-kernel@lists.infradead.org; Wed, 30 Mar 2022 15:06:32 +0000 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 F1E8C23A; Wed, 30 Mar 2022 08:06:28 -0700 (PDT) Received: from e120937-lin.home (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A28163F73B; Wed, 30 Mar 2022 08:06:27 -0700 (PDT) From: Cristian Marussi To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, cristian.marussi@arm.com Subject: [PATCH 00/22] SCMIv3.1 Miscellaneous changes Date: Wed, 30 Mar 2022 16:05:29 +0100 Message-Id: <20220330150551.2573938-1-cristian.marussi@arm.com> X-Mailer: git-send-email 2.32.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220330_080630_894567_5CB78197 X-CRM114-Status: GOOD ( 11.54 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi all, this series introduces a bunch of SCMIv3.1 miscellaneous changes to support basically all the SCMIv3.1 specification [1] addition with the exclusion of the Powercap protocol and driver which will be introduced later on in another series. Most notably the series adds: - supports across all protocols for long resources naming using *_NAME_GET dedicated new commands - Clock protocol Rate change pre and post notifications - Voltage protocol asynchronous voltage level set command (VOLTAGE_LEVEL_SET_COMPLETE delayed response) - Perf protocol power-cost in micro-watts (only internal support) - Perf protocol PERFORMANCE_LIMITS_SET new checks Beside this, the series starts with a few general fixes (01-08/22) and a couple of refactoring: - one (09/22) simply to split out of common.h into a new protocols.h all the structures needed only by protocol code, so that the protocol implementation can include such reduced header instead of the whole common.h which was growing insanely - another around the handling of multi-part commands. SCMI already allowed to issue some commands using a multi-message scheme through which a particularly big response, which could possibly not fit the underlying transport max payload size, could have been split into multiple chunks. Such logic, though, was scattered all across various protocols, leading to a lot of code duplication, so, before adding even more duplication with SCMIv3.1, I split out the common logic to a couple of helpers to handle such mechanism in a common way through some abstract iterators (13/22) and then ported all the protocols users of such mechanism to the common iterators (14,16-18/22). SCMIv3.1 new features have been tested with an emulated backend server, while refactoring has been chekced against a standard SCMI stack running on a JUNO. The series is based on sudeep/for-next/scmi/updates [2] on top of commit 38a0e5b735d6 ("clk: scmi: Support atomic clock enable/disable API") Any feedback very welcome, Thanks, Cristian [1]: https://developer.arm.com/documentation/den0056/d/?lang=en [2]: https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scmi/updates --- Cristian Marussi (22): firmware: arm_scmi: Fix sorting of retrieved clock rates firmware: arm_scmi: Make protocols init fail on basic errors firmware: arm_scmi: Fix Base list protocols enumeration firmware: arm_scmi: Validate BASE_DISCOVER_LIST_PROTOCOLS reply firmware: arm_scmi: Dynamically allocate protocols array firmware: arm_scmi: Make name_get operations return a const firmware: arm_scmi: Check CLOCK_RATE_SET_COMPLETE async reply firmware: arm_scmi: Remove unneeded NULL termination of clk name firmware: arm_scmi: Split protocol specific definitions in a dedicated header firmware: arm_scmi: Introduce a common SCMIv3.1 .extended_name_get helper firmware: arm_scmi: Add SCMIv3.1 extended names protocols support firmware: arm_scmi: Parse clock_enable_latency conditionally firmware: arm_scmi: Add iterators for multi-part commands firmware: arm_scmi: Use common iterators in Sensor protocol firmware: arm_scmi: Add SCMIv3.1 SENSOR_AXIS_NAME_GET support firmware: arm_scmi: Use common iterators in Clock protocol firmware: arm_scmi: Use common iterators in Voltage protocol firmware: arm_scmi: Use common iterators in Perf protocol firmware: arm_scmi: Add SCMIv3.1 Clock notifications firmware: arm_scmi: Add SCMIv3.1 VOLTAGE_LEVEL_SET_COMPLETE firmware: arm_scmi: Add SCMI v3.1 Perf power-cost in microwatts firmware: arm_scmi: Add SCMIv3.1 PERFORMANCE_LIMITS_SET checks drivers/firmware/arm_scmi/base.c | 43 +- drivers/firmware/arm_scmi/clock.c | 337 +++++++++++--- drivers/firmware/arm_scmi/common.h | 225 +-------- drivers/firmware/arm_scmi/driver.c | 165 ++++++- drivers/firmware/arm_scmi/perf.c | 162 ++++--- drivers/firmware/arm_scmi/power.c | 44 +- drivers/firmware/arm_scmi/protocols.h | 318 +++++++++++++ drivers/firmware/arm_scmi/reset.c | 40 +- drivers/firmware/arm_scmi/sensors.c | 645 +++++++++++++++----------- drivers/firmware/arm_scmi/system.c | 9 +- drivers/firmware/arm_scmi/voltage.c | 218 ++++++--- include/linux/scmi_protocol.h | 31 +- 12 files changed, 1495 insertions(+), 742 deletions(-) create mode 100644 drivers/firmware/arm_scmi/protocols.h