From patchwork Mon May 17 15:54:58 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sudeep Holla X-Patchwork-Id: 440140 Delivered-To: patch@linaro.org Received: by 2002:a02:7a1b:0:0:0:0:0 with SMTP id a27csp1214655jac; Mon, 17 May 2021 08:57:34 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx8f1IQu+VQlR0MMN3+KMzchUWs3NpQ1pUhdrjtIL029T5/XJ79tGHg86AFNb7sajotH00k X-Received: by 2002:a02:5289:: with SMTP id d131mr644588jab.121.1621267054529; Mon, 17 May 2021 08:57:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621267054; cv=none; d=google.com; s=arc-20160816; b=EaNAyE6IxeIwzVmxXvDNEG4OE0/uwNEIppgx3lfQI8rs5Vez6PkodNE9TfqdGXlp5U UqPLNtm0+yyGDA67UargpybRQwf2DUCpZo/KH+ux1V6j9IkqIkG4bphssppgJmh5/mpi Bs/odJKGZrhOUqlBhGk5vFHadnYShmCdJG9c5ZuekEwZFojEBbd1eqxIJ0STTXn0eaRc 5NeOMUs8KtEgXwvoRIAOBsMijEYgKvuEwrMXiS+4bCxSpIeRd/6jk6X7RJwswjSnCo+j hR7dUCsuScvjf10m3tsgqxzFIl2xgAdB0qORbfHfPRt00CL58+6A7fk08QRxaeObo6mM u5QQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=LeCsbVclohRKXWchrvNO70krQKQNSgdVnwcUvSUKse0=; b=vGnNow4VeYr70W+rh2YILgLwAJeEsDNN2y2QUkRpjVyd0RYTLet6w1lwtiOQgwttEh IY02axtUplKzfNWGpTzBlGYg1f06KPaLjIpV+HKE8fYmp3QLMxU1hqLgRq1CleVwPV38 a0yU/QfW5L8FJWPpnPoGobNvGtIE0EQjEcX9t6GHRhLeO08TWY96efHOpnypW1c2o3Ph 7dIKuO7dvDNSCxoRqn1ojChVQ5g2PDu8qzwJTr2ykANYpgEuDP1rqyLXg0W5LPdsOZGa RvMS5zYBnY9pBzcvHvmU30+ivKaVFBkbgBy0CoPy553NA38tZp1uRRP5ImchqnIuc4GD HiUQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id c7si9149633ioi.54.2021.05.17.08.57.34; Mon, 17 May 2021 08:57:34 -0700 (PDT) Received-SPF: pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245275AbhEQP6o (ORCPT + 7 others); Mon, 17 May 2021 11:58:44 -0400 Received: from foss.arm.com ([217.140.110.172]:56630 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1345535AbhEQP40 (ORCPT ); Mon, 17 May 2021 11:56:26 -0400 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 40CC3106F; Mon, 17 May 2021 08:55:06 -0700 (PDT) Received: from usa.arm.com (e103737-lin.cambridge.arm.com [10.1.197.49]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 062BF3F73D; Mon, 17 May 2021 08:55:04 -0700 (PDT) From: Sudeep Holla To: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Cc: Sudeep Holla , Rob Herring , Manivannan Sadhasivam , Hector Yuan , Viresh Kumar , Bjorn Andersson , Rob Herring Subject: [PATCH v4] dt-bindings: dvfs: Add support for generic performance domains Date: Mon, 17 May 2021 16:54:58 +0100 Message-Id: <20210517155458.1016707-1-sudeep.holla@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org The CLKSCREW attack [0] exposed security vulnerabilities in energy management implementations where untrusted software had direct access to clock and voltage hardware controls. In this attack, the malicious software was able to place the platform into unsafe overclocked or undervolted configurations. Such configurations then enabled the injection of predictable faults to reveal secrets. Many Arm-based systems used to or still use voltage regulator and clock frameworks in the kernel. These frameworks allow callers to independently manipulate frequency and voltage settings. Such implementations can render systems susceptible to this form of attack. Attacks such as CLKSCREW are now being mitigated by not having direct and independent control of clock and voltage in the kernel and moving that control to a trusted entity, such as the SCP firmware or secure world firmware/software which are to perform sanity checking on the requested performance levels, thereby preventing any attempted malicious programming. With the advent of such an abstraction, there is a need to replace the generic clock and regulator bindings used by such devices with a generic performance domains bindings. [0] https://www.usenix.org/conference/usenixsecurity17/technical-sessions/presentation/tang Link: https://lore.kernel.org/r/20201116181356.804590-1-sudeep.holla@arm.com Cc: Rob Herring Acked-by: Viresh Kumar Signed-off-by: Sudeep Holla --- Hi All, Sorry for yet another delay, I don't want to mist this for v5.14 as Mediatek cpufreq driver was depending on this IIRC. v3[3]->v4: - Dropped unnecessary phandle-array reference - Added maxItems = 1 for the property v2[2]->v3[3]: - Dropped required properties - Added non cpu device example - Updated cpu bindings too v1[1]->v2[2]: - Changed to Dual License - Added select: true, enum for #performance-domain-cells and $ref for performance-domain - Changed the example to use real existing compatibles instead of made-up ones Regards, Sudeep [1] https://lore.kernel.org/r/20201105173539.1426301-1-sudeep.holla@arm.com [2] https://lore.kernel.org/r/20201116181356.804590-1-sudeep.holla@arm.com [3] https://lore.kernel.org/r/20210407135913.2067694-1-sudeep.holla@arm.com .../devicetree/bindings/arm/cpus.yaml | 7 ++ .../bindings/dvfs/performance-domain.yaml | 74 +++++++++++++++++++ 2 files changed, 81 insertions(+) create mode 100644 Documentation/devicetree/bindings/dvfs/performance-domain.yaml -- 2.25.1 Reviewed-by: Rob Herring diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml index f3c7249c73d6..9a2432a88074 100644 --- a/Documentation/devicetree/bindings/arm/cpus.yaml +++ b/Documentation/devicetree/bindings/arm/cpus.yaml @@ -257,6 +257,13 @@ description: |+ where voltage is in V, frequency is in MHz. + performance-domains: + maxItems: 1 + description: + List of phandles and performance domain specifiers, as defined by + bindings of the performance domain provider. See also + dvfs/performance-domain.yaml. + power-domains: description: List of phandles and PM domain specifiers, as defined by bindings of the diff --git a/Documentation/devicetree/bindings/dvfs/performance-domain.yaml b/Documentation/devicetree/bindings/dvfs/performance-domain.yaml new file mode 100644 index 000000000000..c8b91207f34d --- /dev/null +++ b/Documentation/devicetree/bindings/dvfs/performance-domain.yaml @@ -0,0 +1,74 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/dvfs/performance-domain.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic performance domains + +maintainers: + - Sudeep Holla + +description: |+ + This binding is intended for performance management of groups of devices or + CPUs that run in the same performance domain. Performance domains must not + be confused with power domains. A performance domain is defined by a set + of devices that always have to run at the same performance level. For a given + performance domain, there is a single point of control that affects all the + devices in the domain, making it impossible to set the performance level of + an individual device in the domain independently from other devices in + that domain. For example, a set of CPUs that share a voltage domain, and + have a common frequency control, is said to be in the same performance + domain. + + This device tree binding can be used to bind performance domain consumer + devices with their performance domains provided by performance domain + providers. A performance domain provider can be represented by any node in + the device tree and can provide one or more performance domains. A consumer + node can refer to the provider by a phandle and a set of phandle arguments + (so called performance domain specifiers) of length specified by the + \#performance-domain-cells property in the performance domain provider node. + +select: true + +properties: + "#performance-domain-cells": + description: + Number of cells in a performance domain specifier. Typically 0 for nodes + representing a single performance domain and 1 for nodes providing + multiple performance domains (e.g. performance controllers), but can be + any value as specified by device tree binding documentation of particular + provider. + enum: [ 0, 1 ] + + performance-domains: + $ref: '/schemas/types.yaml#/definitions/phandle-array' + maxItems: 1 + description: + A phandle and performance domain specifier as defined by bindings of the + performance controller/provider specified by phandle. + +additionalProperties: true + +examples: + - | + performance: performance-controller@12340000 { + compatible = "qcom,cpufreq-hw"; + reg = <0x12340000 0x1000>; + #performance-domain-cells = <1>; + }; + + // The node above defines a performance controller that is a performance + // domain provider and expects one cell as its phandle argument. + + cpus { + #address-cells = <2>; + #size-cells = <0>; + + cpu@0 { + device_type = "cpu"; + compatible = "arm,cortex-a57"; + reg = <0x0 0x0>; + performance-domains = <&performance 1>; + }; + };