From patchwork Wed Sep 21 09:32:50 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 76678 Delivered-To: patch@linaro.org Received: by 10.140.106.72 with SMTP id d66csp1947584qgf; Wed, 21 Sep 2016 02:33:17 -0700 (PDT) X-Received: by 10.98.60.153 with SMTP id b25mr63910266pfk.12.1474450397582; Wed, 21 Sep 2016 02:33:17 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id sn7si38022022pac.169.2016.09.21.02.33.17; Wed, 21 Sep 2016 02:33:17 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755926AbcIUJdJ (ORCPT + 27 others); Wed, 21 Sep 2016 05:33:09 -0400 Received: from mail-pf0-f179.google.com ([209.85.192.179]:33178 "EHLO mail-pf0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755178AbcIUJdA (ORCPT ); Wed, 21 Sep 2016 05:33:00 -0400 Received: by mail-pf0-f179.google.com with SMTP id 21so17518870pfy.0 for ; Wed, 21 Sep 2016 02:32:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :in-reply-to:references; bh=OpwaKtn+ZZn7vuNUYQRC6sVZNZfUbuSVg6sm3tSS5Ec=; b=HZEIkvQzDAFoO6t4i2FkGx5tqv8HSldu8WgCzQi6adkHGBO7shGSd2kNN9djKPaoEd jdOGHO0VnyLcpvGXCUxJBnyB+W7TwwA3sJ6M+1oCryq3/kZ8iEJ0WIXaaSB3L9Quo27F lFwSGFVTfgPi0WKimB/fq+ZoEA9Ob8Hgly3d8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:in-reply-to:references; bh=OpwaKtn+ZZn7vuNUYQRC6sVZNZfUbuSVg6sm3tSS5Ec=; b=Xrf1uMAliz1hFeeTnTa5Wl4Nnt/8TbkuIu+IqgUWSdCYuq+ttSWpNEmNjXv9O1cjV8 VoaH+kq4RUvx/Uulp8PH6HfBKlypksMd2iT0c0LdEaFVZ4g2ckPy015VF0jMckZKeojl 6nWz/2n8nk61BSVocO5rB72fHhmnov6du9kbGUKyF6ZAQXJanaK1UHfrTJfRI2o4UXgw gJF07hi0xv65+X5JndcNbaGYgTtZ0oayW7q9cggvgBIuNNFOHwhgOzWekycTtryt8nnN N+k2IO5IcbpXpPSszqTyhvSFIWamSheGv0A67gUhR4Oyi7LTJKWTybTazHzf/4JKnHwH 5tiw== X-Gm-Message-State: AE9vXwO6JHB+DgYzSH03RXRdsJmmTLK/ZTZDNHCFUPqBzT/4MKC426I5fGEKa18CrhQLONyZ X-Received: by 10.98.57.136 with SMTP id u8mr62814166pfj.93.1474450379326; Wed, 21 Sep 2016 02:32:59 -0700 (PDT) Received: from localhost ([122.172.39.60]) by smtp.gmail.com with ESMTPSA id xw10sm45107337pab.0.2016.09.21.02.32.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 21 Sep 2016 02:32:58 -0700 (PDT) From: Viresh Kumar To: Rafael Wysocki , Viresh Kumar , Nishanth Menon , Stephen Boyd Cc: linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Guittot , Viresh Kumar , devicetree@vger.kernel.org Subject: [PATCH 2/2] PM / OPP: Multiple regulators aren't supported yet Date: Wed, 21 Sep 2016 15:02:50 +0530 Message-Id: X-Mailer: git-send-email 2.7.1.410.g6faf27b In-Reply-To: References: In-Reply-To: References: Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Multiple regulators per device aren't supported yet by the kernel code and the bindings provided in documentation aren't sufficient to handle that case (as there is no way for kernel code to link multiple voltage/current values to a power supply). Remove them. These can be added later if required with appropriate changes. Signed-off-by: Viresh Kumar --- Documentation/devicetree/bindings/opp/opp.txt | 68 ++------------------------- 1 file changed, 4 insertions(+), 64 deletions(-) -- 2.7.1.410.g6faf27b diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt index 5eab6f0215d1..5d0595f0a10c 100644 --- a/Documentation/devicetree/bindings/opp/opp.txt +++ b/Documentation/devicetree/bindings/opp/opp.txt @@ -86,8 +86,7 @@ properties. Single entry is for target voltage and three entries are for voltages. - Entries for multiple regulators must be present in the same order as - regulators are specified in device's DT node. + Entries for multiple regulators aren't supported. - opp-microvolt-: Named opp-microvolt property. This is exactly similar to the above opp-microvolt property, but allows multiple voltage ranges to be @@ -104,10 +103,7 @@ properties. Should only be set if opp-microvolt is set for the OPP. - Entries for multiple regulators must be present in the same order as - regulators are specified in device's DT node. If this property isn't required - for few regulators, then this should be marked as zero for them. If it isn't - required for any regulator, then this property need not be present. + Entries for multiple regulators aren't supported. - opp-microamp-: Named opp-microamp property. Similar to opp-microvolt- property, but for microamp instead. @@ -381,63 +377,7 @@ DVFS state together. }; }; -Example 4: Handling multiple regulators - -/ { - cpus { - cpu@0 { - compatible = "arm,cortex-a7"; - ... - - cpu-supply = <&cpu_supply0>, <&cpu_supply1>, <&cpu_supply2>; - operating-points-v2 = <&cpu0_opp_table>; - }; - }; - - cpu0_opp_table: opp_table0 { - compatible = "operating-points-v2"; - opp-shared; - - opp@1000000000 { - opp-hz = /bits/ 64 <1000000000>; - opp-microvolt = <970000>, /* Supply 0 */ - <960000>, /* Supply 1 */ - <960000>; /* Supply 2 */ - opp-microamp = <70000>, /* Supply 0 */ - <70000>, /* Supply 1 */ - <70000>; /* Supply 2 */ - clock-latency-ns = <300000>; - }; - - /* OR */ - - opp@1000000000 { - opp-hz = /bits/ 64 <1000000000>; - opp-microvolt = <970000 975000 985000>, /* Supply 0 */ - <960000 965000 975000>, /* Supply 1 */ - <960000 965000 975000>; /* Supply 2 */ - opp-microamp = <70000>, /* Supply 0 */ - <70000>, /* Supply 1 */ - <70000>; /* Supply 2 */ - clock-latency-ns = <300000>; - }; - - /* OR */ - - opp@1000000000 { - opp-hz = /bits/ 64 <1000000000>; - opp-microvolt = <970000 975000 985000>, /* Supply 0 */ - <960000 965000 975000>, /* Supply 1 */ - <960000 965000 975000>; /* Supply 2 */ - opp-microamp = <70000>, /* Supply 0 */ - <0>, /* Supply 1 doesn't need this */ - <70000>; /* Supply 2 */ - clock-latency-ns = <300000>; - }; - }; -}; - -Example 5: opp-supported-hw +Example 4: opp-supported-hw (example: three level hierarchy of versions: cuts, substrate and process) / { @@ -482,7 +422,7 @@ Example 5: opp-supported-hw }; }; -Example 6: opp-microvolt-, opp-microamp-: +Example 5: opp-microvolt-, opp-microamp-: (example: device with two possible microvolt ranges: slow and fast) / {