Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp6503304ybl; Mon, 23 Dec 2019 06:59:58 -0800 (PST) X-Google-Smtp-Source: APXvYqxM0doXI4kQdIm4jluCgPxS/t+A/zLwz9fNKYDkl1641w7HSp8LbuStPttegss05vTrixfK X-Received: by 2002:a9d:367:: with SMTP id 94mr31125082otv.329.1577113198768; Mon, 23 Dec 2019 06:59:58 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577113198; cv=none; d=google.com; s=arc-20160816; b=De6itqQOtoM2uMsY2rdmQbI82xrMbFxuyqMhCkEJwGl7Lvesd3xlftLdKy8nMybWb4 yp6wfzgfwi6rrZ+aUdJw+N2+sjpM3JnDgld57JOQOzMzek32JtdFkkeUThkcHC6BOUDH tP+P+ZQI8hP5Y8hRPX9pipHLgmZqR3ifH09+aLpafES1smotkC1W1odJX/GpMe7hQ36X niYA034Fy0E1GSSr9p3q07AU+2P4gVZ9PslAhhVIr/Q/YqZ7DTZU+PDH+8UElBuU0ei5 DmpcxN/46AwsU2Uohyxf8gKYCbxPth9nWqY3yIJ5DiqnRKmJEIFtdWfAT0Mn7KHjtF6+ 21QQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=PJZqQ9ZL1BQ0WM0bKMnF161ozdrug0XMBQ43BZdidcE=; b=j7Wk0OLjdzWZMVaMunb6NWwHqYaXEa8VaImFT6xaL6ftqegFLI626ZkMnsTxRbbtLI LVnAbXYD0v4Npo+EtrxJ0A4imJjrWcI4hyJd7Qyn5z14Z6k30pEo9ZuY4K5NDX2A7MuB R2SL0KzDoguQjZ7Vl/quUW7/UUIi2M87srC8YUDSE3AXn5/rr0qkPr2b4bIppyLbNGXO zaYbOggKhvdMFjvh715psM/K6/sa+b/RUMcc1tiQXscQAgN8g1Bll26S+uAOGE5/jbpx XIFJivFKVy6JIZ2JaPMHMihnZJRgA6MjayoQmCoUQ2vhHyf/BYNfjU6DKPHf6df1o81G aaWQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@flawful.org header.s=mail header.b=typ7neJr; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n139si9706046oig.121.2019.12.23.06.59.45; Mon, 23 Dec 2019 06:59:58 -0800 (PST) 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 (test mode) header.i=@flawful.org header.s=mail header.b=typ7neJr; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726866AbfLWO7G (ORCPT + 99 others); Mon, 23 Dec 2019 09:59:06 -0500 Received: from pio-pvt-msa1.bahnhof.se ([79.136.2.40]:35640 "EHLO pio-pvt-msa1.bahnhof.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726733AbfLWO7G (ORCPT ); Mon, 23 Dec 2019 09:59:06 -0500 Received: from localhost (localhost [127.0.0.1]) by pio-pvt-msa1.bahnhof.se (Postfix) with ESMTP id 1BB1640735; Mon, 23 Dec 2019 15:59:01 +0100 (CET) Authentication-Results: pio-pvt-msa1.bahnhof.se; dkim=pass (1024-bit key; unprotected) header.d=flawful.org header.i=@flawful.org header.b="typ7neJr"; dkim-atps=neutral X-Virus-Scanned: Debian amavisd-new at bahnhof.se X-Spam-Flag: NO X-Spam-Score: -2.099 X-Spam-Level: X-Spam-Status: No, score=-2.099 tagged_above=-999 required=6.31 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no Received: from pio-pvt-msa1.bahnhof.se ([127.0.0.1]) by localhost (pio-pvt-msa1.bahnhof.se [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id iL9FOQN669re; Mon, 23 Dec 2019 15:58:58 +0100 (CET) Received: from flawful.org (ua-84-217-220-205.bbcust.telenor.se [84.217.220.205]) (Authenticated sender: mb274189) by pio-pvt-msa1.bahnhof.se (Postfix) with ESMTPA id 89A694071F; Mon, 23 Dec 2019 15:58:55 +0100 (CET) Received: by flawful.org (Postfix, from userid 1001) id B16761451; Mon, 23 Dec 2019 15:58:53 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=flawful.org; s=mail; t=1577113133; bh=PJZqQ9ZL1BQ0WM0bKMnF161ozdrug0XMBQ43BZdidcE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=typ7neJrcWsg+AlhaXvwYvGu0B07MJ4VwlFr/TgfknnKp24QvzUVlCIxs8lWZtdCa uDotbdbsrIDcKupr5/AZ6qRaaAGxarzcOefmeopR8U/qYZ/Y15Qrg0wVKOO2ljeSZ4 uQ5ydnWBCKLRBxa3HXjoYLBC9CTpx6wl+0Tdvuzc= Date: Mon, 23 Dec 2019 15:58:53 +0100 From: Niklas Cassel To: "Rafael J. Wysocki" Cc: Niklas Cassel , linux-arm-msm@vger.kernel.org, amit.kucheria@linaro.org, sboyd@kernel.org, vireshk@kernel.org, bjorn.andersson@linaro.org, ulf.hansson@linaro.org, Rob Herring , Mark Rutland , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v7 1/5] dt-bindings: power: avs: Add support for CPR (Core Power Reduction) Message-ID: <20191223145853.eaw23k2c5oyh7j44@flawful.org> References: <20191129213917.1301110-1-niklas.cassel@linaro.org> <20191129213917.1301110-2-niklas.cassel@linaro.org> <121319954.uyNvbQYpoT@kreacher> <20191220143356.cprp55jmuhtcx7wr@flawful.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191220143356.cprp55jmuhtcx7wr@flawful.org> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 20, 2019 at 03:33:56PM +0100, Niklas Cassel wrote: > On Fri, Dec 20, 2019 at 10:31:53AM +0100, Rafael J. Wysocki wrote: > > On Friday, November 29, 2019 10:39:11 PM CET Niklas Cassel wrote: > > > Add DT bindings to describe the CPR HW found on certain Qualcomm SoCs. > > > > > > Co-developed-by: Jorge Ramirez-Ortiz > > > Signed-off-by: Jorge Ramirez-Ortiz > > > Signed-off-by: Niklas Cassel > > > Reviewed-by: Rob Herring > > > Reviewed-by: Bjorn Andersson > > > Reviewed-by: Ulf Hansson > > > --- > > > Changes since v6: > > > -Picked up Bjorn's and Ulf's Reviewed-by. > > > > > > .../bindings/power/avs/qcom,cpr.txt | 130 ++++++++++++++++++ > > > 1 file changed, 130 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/power/avs/qcom,cpr.txt > > > > > > diff --git a/Documentation/devicetree/bindings/power/avs/qcom,cpr.txt b/Documentation/devicetree/bindings/power/avs/qcom,cpr.txt > > > new file mode 100644 > > > index 000000000000..ab0d5ebbad4e > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/power/avs/qcom,cpr.txt > > > @@ -0,0 +1,130 @@ > > > +QCOM CPR (Core Power Reduction) > > > + > > > +CPR (Core Power Reduction) is a technology to reduce core power on a CPU > > > +or other device. Each OPP of a device corresponds to a "corner" that has > > > +a range of valid voltages for a particular frequency. While the device is > > > +running at a particular frequency, CPR monitors dynamic factors such as > > > +temperature, etc. and suggests adjustments to the voltage to save power > > > +and meet silicon characteristic requirements. > > > + > > > +- compatible: > > > + Usage: required > > > + Value type: > > > + Definition: should be "qcom,qcs404-cpr", "qcom,cpr" for qcs404 > > > + > > > +- reg: > > > + Usage: required > > > + Value type: > > > + Definition: base address and size of the rbcpr register region > > > + > > > +- interrupts: > > > + Usage: required > > > + Value type: > > > + Definition: should specify the CPR interrupt > > > + > > > +- clocks: > > > + Usage: required > > > + Value type: > > > + Definition: phandle to the reference clock > > > + > > > +- clock-names: > > > + Usage: required > > > + Value type: > > > + Definition: must be "ref" > > > + > > > +- vdd-apc-supply: > > > + Usage: required > > > + Value type: > > > + Definition: phandle to the vdd-apc-supply regulator > > > + > > > +- #power-domain-cells: > > > + Usage: required > > > + Value type: > > > + Definition: should be 0 > > > + > > > +- operating-points-v2: > > > + Usage: required > > > + Value type: > > > + Definition: A phandle to the OPP table containing the > > > + performance states supported by the CPR > > > + power domain > > > + > > > +- acc-syscon: > > > + Usage: optional > > > + Value type: > > > + Definition: phandle to syscon for writing ACC settings > > > + > > > +- nvmem-cells: > > > + Usage: required > > > + Value type: > > > + Definition: phandle to nvmem cells containing the data > > > + that makes up a fuse corner, for each fuse corner. > > > + As well as the CPR fuse revision. > > > + > > > +- nvmem-cell-names: > > > + Usage: required > > > + Value type: > > > + Definition: should be "cpr_quotient_offset1", "cpr_quotient_offset2", > > > + "cpr_quotient_offset3", "cpr_init_voltage1", > > > + "cpr_init_voltage2", "cpr_init_voltage3", "cpr_quotient1", > > > + "cpr_quotient2", "cpr_quotient3", "cpr_ring_osc1", > > > + "cpr_ring_osc2", "cpr_ring_osc3", "cpr_fuse_revision" > > > + for qcs404. > > > + > > > +Example: > > > + > > > + cpr_opp_table: cpr-opp-table { > > > + compatible = "operating-points-v2-qcom-level"; > > > + > > > + cpr_opp1: opp1 { > > > + opp-level = <1>; > > > + qcom,opp-fuse-level = <1>; > > > + }; > > > + cpr_opp2: opp2 { > > > + opp-level = <2>; > > > + qcom,opp-fuse-level = <2>; > > > + }; > > > + cpr_opp3: opp3 { > > > + opp-level = <3>; > > > + qcom,opp-fuse-level = <3>; > > > + }; > > > + }; > > > + > > > + power-controller@b018000 { > > > + compatible = "qcom,qcs404-cpr", "qcom,cpr"; > > > + reg = <0x0b018000 0x1000>; > > > + interrupts = <0 15 IRQ_TYPE_EDGE_RISING>; > > > + clocks = <&xo_board>; > > > + clock-names = "ref"; > > > + vdd-apc-supply = <&pms405_s3>; > > > + #power-domain-cells = <0>; > > > + operating-points-v2 = <&cpr_opp_table>; > > > + acc-syscon = <&tcsr>; > > > + > > > + nvmem-cells = <&cpr_efuse_quot_offset1>, > > > + <&cpr_efuse_quot_offset2>, > > > + <&cpr_efuse_quot_offset3>, > > > + <&cpr_efuse_init_voltage1>, > > > + <&cpr_efuse_init_voltage2>, > > > + <&cpr_efuse_init_voltage3>, > > > + <&cpr_efuse_quot1>, > > > + <&cpr_efuse_quot2>, > > > + <&cpr_efuse_quot3>, > > > + <&cpr_efuse_ring1>, > > > + <&cpr_efuse_ring2>, > > > + <&cpr_efuse_ring3>, > > > + <&cpr_efuse_revision>; > > > + nvmem-cell-names = "cpr_quotient_offset1", > > > + "cpr_quotient_offset2", > > > + "cpr_quotient_offset3", > > > + "cpr_init_voltage1", > > > + "cpr_init_voltage2", > > > + "cpr_init_voltage3", > > > + "cpr_quotient1", > > > + "cpr_quotient2", > > > + "cpr_quotient3", > > > + "cpr_ring_osc1", > > > + "cpr_ring_osc2", > > > + "cpr_ring_osc3", > > > + "cpr_fuse_revision"; > > > + }; > > > > > > > I have queued up this one and the [2/5] for 5.6, but if you'd rather want them > > to go in via a different patch, please let me know and I'll drop them. > > > > Thanks a lot Rafael! > > I would very much prefer them to go via your tree. > > Unfortunately it seems like kbuild test robot > found an incorrect printk format specifier in > one of the debug prints. > > Line 838 > dev_dbg(dev, "efuse read(%s) = %x, bytes %ld\n", cname, *data, len); > > should be > dev_dbg(dev, "efuse read(%s) = %x, bytes %zd\n", cname, *data, len); > > So %zd rather than %ld. > > This was obviously an error, but didn't show when > compiling on arm64 or x86_64. > > Sorry for this inconvenience. > > Could you fix up the commit or do I need to do a respin? Hello Rafael, Since the intel test robot found another problem, I decided to cook up a series that can be applied (or squashed) on top of your bleeding-edge branch here instead: https://patchwork.kernel.org/project/linux-pm/list/?series=220955 Again, sorry for the inconvenience. Kind regards, Niklas