| Message ID | 20231018-msm8909-cpufreq-v2-0-0962df95f654@kernkonzept.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | cpufreq: Add basic cpufreq scaling for Qualcomm MSM8909 | expand |
On 18-10-23, 10:06, Stephan Gerhold wrote: > Add the necessary definitions to the qcom-cpufreq-nvmem driver to > support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice > the necessary power domains vary depending on the actual PMIC the SoC > was combined with. With PM8909 the VDD_APC power domain is shared with > VDD_CX so the RPM firmware handles all voltage adjustments, while with > PM8916 and PM660 Linux is responsible to do adaptive voltage scaling > of a dedicated CPU regulator using CPR. > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> Applied patch 1 and 3. Thanks.
On Thu, 19 Oct 2023 at 08:16, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > On 18-10-23, 10:06, Stephan Gerhold wrote: > > Add the necessary definitions to the qcom-cpufreq-nvmem driver to > > support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice > > the necessary power domains vary depending on the actual PMIC the SoC > > was combined with. With PM8909 the VDD_APC power domain is shared with > > VDD_CX so the RPM firmware handles all voltage adjustments, while with > > PM8916 and PM660 Linux is responsible to do adaptive voltage scaling > > of a dedicated CPU regulator using CPR. > > > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> > > Applied patch 1 and 3. Thanks. I did spend quite some time reviewing the previous version. Please allow me to have a look at this too before applying. Kind regards Uffe > > -- > viresh
On 19-10-23, 12:19, Ulf Hansson wrote: > On Thu, 19 Oct 2023 at 08:16, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > > > On 18-10-23, 10:06, Stephan Gerhold wrote: > > > Add the necessary definitions to the qcom-cpufreq-nvmem driver to > > > support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice > > > the necessary power domains vary depending on the actual PMIC the SoC > > > was combined with. With PM8909 the VDD_APC power domain is shared with > > > VDD_CX so the RPM firmware handles all voltage adjustments, while with > > > PM8916 and PM660 Linux is responsible to do adaptive voltage scaling > > > of a dedicated CPU regulator using CPR. > > > > > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> > > > > Applied patch 1 and 3. Thanks. > > I did spend quite some time reviewing the previous version. Please > allow me to have a look at this too before applying. My branch isn't immutable, I keep on changing it. Please provide your feedback :)
On 19-10-23, 11:46, Viresh Kumar wrote: > On 18-10-23, 10:06, Stephan Gerhold wrote: > > Add the necessary definitions to the qcom-cpufreq-nvmem driver to > > support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice > > the necessary power domains vary depending on the actual PMIC the SoC > > was combined with. With PM8909 the VDD_APC power domain is shared with > > VDD_CX so the RPM firmware handles all voltage adjustments, while with > > PM8916 and PM660 Linux is responsible to do adaptive voltage scaling > > of a dedicated CPU regulator using CPR. > > > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> > > Applied patch 1 and 3. Thanks. Hi Stephan, I think your platform has exactly what I am looking for. Can you please help me test this, before it lands into linux-next :) https://lore.kernel.org/cover.1697710527.git.viresh.kumar@linaro.org TIA.
On Thu, Oct 19, 2023 at 03:53:42PM +0530, Viresh Kumar wrote: > On 19-10-23, 11:46, Viresh Kumar wrote: > > On 18-10-23, 10:06, Stephan Gerhold wrote: > > > Add the necessary definitions to the qcom-cpufreq-nvmem driver to > > > support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice > > > the necessary power domains vary depending on the actual PMIC the SoC > > > was combined with. With PM8909 the VDD_APC power domain is shared with > > > VDD_CX so the RPM firmware handles all voltage adjustments, while with > > > PM8916 and PM660 Linux is responsible to do adaptive voltage scaling > > > of a dedicated CPU regulator using CPR. > > > > > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> > > > > Applied patch 1 and 3. Thanks. > > Hi Stephan, > > I think your platform has exactly what I am looking for. Can you > please help me test this, before it lands into linux-next :) > > https://lore.kernel.org/cover.1697710527.git.viresh.kumar@linaro.org > Sure, I will try to test it until end of next week, with both single and multiple power domains assigned to the CPU. Is there something particular you would like me to look for? Or just that the scaling still works correctly as before? Stephan
On 19-10-23, 15:48, Stephan Gerhold wrote: > Sure, I will try to test it until end of next week, with both single and > multiple power domains assigned to the CPU. Is there something > particular you would like me to look for? Or just that the scaling still > works correctly as before? Yeah, simple test to ensure scaling works fine is all I need. Shouldn't take a lot of time. Thanks.
Add the necessary definitions to the qcom-cpufreq-nvmem driver to support basic cpufreq scaling on the Qualcomm MSM8909 SoC. In practice the necessary power domains vary depending on the actual PMIC the SoC was combined with. With PM8909 the VDD_APC power domain is shared with VDD_CX so the RPM firmware handles all voltage adjustments, while with PM8916 and PM660 Linux is responsible to do adaptive voltage scaling of a dedicated CPU regulator using CPR. Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> --- Changes in v2: - Reword commit messages based on discussion with Uffe - Use generic power domain name "perf" (Uffe) - Fix pm_runtime error handling (Uffe) - Add allocation cleanup patch as preparation - Fix ordering of qcom,msm8909 compatible (Konrad) - cpufreq-dt-platdev blocklist/dt-bindings patches were applied already - Link to v1: https://lore.kernel.org/r/20230912-msm8909-cpufreq-v1-0-767ce66b544b@kernkonzept.com --- Stephan Gerhold (3): cpufreq: qcom-nvmem: Simplify driver data allocation cpufreq: qcom-nvmem: Enable virtual power domain devices cpufreq: qcom-nvmem: Add MSM8909 drivers/cpufreq/qcom-cpufreq-nvmem.c | 124 +++++++++++++++++++++++++---------- 1 file changed, 90 insertions(+), 34 deletions(-) --- base-commit: 2e12b516f5e6046ceabd4d24e24297e4d130b148 change-id: 20230906-msm8909-cpufreq-dff238de9ff3 Best regards,