Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-24 Thread Niklas Cassel
On Tue, Aug 14, 2018 at 05:42:33PM +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current value of each OPP > varies based on the silicon variant in use. > > The required OPP related data is determined based on >

Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-24 Thread Niklas Cassel
On Tue, Aug 14, 2018 at 05:42:33PM +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current value of each OPP > varies based on the silicon variant in use. > > The required OPP related data is determined based on >

Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-24 Thread Niklas Cassel
On Tue, Aug 14, 2018 at 05:42:33PM +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current value of each OPP > varies based on the silicon variant in use. > > The required OPP related data is determined based on >

Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-24 Thread Niklas Cassel
On Tue, Aug 14, 2018 at 05:42:33PM +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current value of each OPP > varies based on the silicon variant in use. > > The required OPP related data is determined based on >

Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-17 Thread Rob Herring
Hi, this email is from Rob's (experimental) review bot. I found a couple of common problems with your patch. Please see below. On Tue, 14 Aug 2018 17:42:33 +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current

Re: [PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-17 Thread Rob Herring
Hi, this email is from Rob's (experimental) review bot. I found a couple of common problems with your patch. Please see below. On Tue, 14 Aug 2018 17:42:33 +0530, Sricharan R wrote: > In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 > that has KRAIT processors the voltage/current

[PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-14 Thread Sricharan R
In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 that has KRAIT processors the voltage/current value of each OPP varies based on the silicon variant in use. The required OPP related data is determined based on the efuse value. This is similar to the existing code for kryo cores. So

[PATCH v12 14/14] cpufreq: qcom: Add support for krait based socs

2018-08-14 Thread Sricharan R
In Certain QCOM SoCs like ipq8064, apq8064, msm8960, msm8974 that has KRAIT processors the voltage/current value of each OPP varies based on the silicon variant in use. The required OPP related data is determined based on the efuse value. This is similar to the existing code for kryo cores. So