From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sricharan R Subject: Re: [PATCH v4 13/14] dt-bindings: cpufreq: Document operating-points-v2-kryo-cpu Date: Mon, 2 Apr 2018 20:37:54 +0530 Message-ID: <7ca35b7a-6078-620b-4772-091e53ef6790@codeaurora.org> References: <1522358807-10413-1-git-send-email-ilialin@codeaurora.org> <1522358807-10413-14-git-send-email-ilialin@codeaurora.org> <20180402093018.GC3572@vireshk-i7> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180402093018.GC3572@vireshk-i7> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Viresh Kumar , Ilia Lin Cc: mturquette@baylibre.com, sboyd@kernel.org, robh@kernel.org, mark.rutland@arm.com, rjw@rjwysocki.net, lgirdwood@gmail.com, broonie@kernel.org, andy.gross@linaro.org, david.brown@linaro.org, catalin.marinas@arm.com, will.deacon@arm.com, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, rnayak@codeaurora.org, amit.kucheria@linaro.org, nicolas.dechesne@linaro.org, celster@codeaurora.org, tfinkel@codeaurora.org List-Id: linux-pm@vger.kernel.org Hi Viresh, On 4/2/2018 3:00 PM, Viresh Kumar wrote: > +Sricharan, > > On 30-03-18, 00:26, Ilia Lin wrote: >> In Certain Qualcomm Technologies, Inc. SoCs like apq8096 and msm8996 >> that have KRYO processors, the CPU ferequencies subset and voltage value >> of each OPP varies based on the silicon variant in use. >> Qualcomm Technologies, Inc. Process Voltage Scaling Tables >> defines the voltage and frequency value based on the msm-id in SMEM >> and speedbin blown in the efuse combination. >> The qcom-cpufreq-kryo driver reads the msm-id and efuse value from the SoC >> to provide the OPP framework with required information. >> This is used to determine the voltage and frequency value for each OPP of >> operating-points-v2 table when it is parsed by the OPP framework. >> >> This change adds documentation. >> >> Change-Id: I1953f652a48249fb516d175f0e965a9510cd4209 >> Signed-off-by: Ilia Lin >> --- >> .../devicetree/bindings/cpufreq/kryo-cpufreq.txt | 693 +++++++++++++++++++++ >> 1 file changed, 693 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/cpufreq/kryo-cpufreq.txt >> >> diff --git a/Documentation/devicetree/bindings/cpufreq/kryo-cpufreq.txt b/Documentation/devicetree/bindings/cpufreq/kryo-cpufreq.txt > > This should really go in opp directory. > >> new file mode 100644 >> index 0000000..20cef9d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/cpufreq/kryo-cpufreq.txt >> @@ -0,0 +1,693 @@ >> +Qualcomm Technologies, Inc. KRYO CPUFreq and OPP bindings >> +=================================== >> + >> +In Certain Qualcomm Technologies, Inc. SoCs like apq8096 and msm8996 >> +that have KRYO processors, the CPU ferequencies subset and voltage value >> +of each OPP varies based on the silicon variant in use. >> +Qualcomm Technologies, Inc. Process Voltage Scaling Tables >> +defines the voltage and frequency value based on the msm-id in SMEM >> +and speedbin blown in the efuse combination. >> +The qcom-cpufreq-kryo driver reads the msm-id and efuse value from the SoC >> +to provide the OPP framework with required information (existing HW bitmap). >> +This is used to determine the voltage and frequency value for each OPP of >> +operating-points-v2 table when it is parsed by the OPP framework. >> + >> +Required properties: >> +-------------------- >> +In 'cpus' nodes: >> +- operating-points-v2: Phandle to the operating-points-v2 table to use. >> + >> +In 'operating-points-v2' table: >> +- compatible: Should be >> + - 'operating-points-v2-kryo-cpu' for apq8096 and msm8996. >> +- nvmem-cells: A phandle pointing to a nvmem-cells node representing the >> + efuse registers that has information about the >> + speedbin that is used to select the right frequency/voltage >> + value pair. >> + Please refer the for nvmem-cells >> + bindings Documentation/devicetree/bindings/nvmem/nvmem.txt >> + and also examples below. > > Sricharan is also working on adding these, just make sure you guys do the same > thing.. > Right, i was adding a similar one for krait cores [1]. There is code common in the init sequence across both (little). Do you intent to make them common ? Regards, Sricharan [1] https://patchwork.kernel.org/patch/10261873/ -- "QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation