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;
Date:   Tue, 10 Jul 2018 11:29:00 +0200
From:   Niklas Cassel <niklas.cassel@linaro.org>
To:     Ilia Lin <ilialin@codeaurora.org>
Cc:     andy.gross@linaro.org, david.brown@linaro.org, robh@kernel.org,
        mark.rutland@arm.com, catalin.marinas@arm.com, will.deacon@arm.com,
        linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, ilia.lin@gmail.com
Subject: Re: [PATCH v11 2/2] dt: qcom: Add qcom-cpufreq-kryo driver
 configuration
Message-ID: <20180710092900.GA26718@centauri.lan>
References: <1527081091-13389-1-git-send-email-ilialin@codeaurora.org>
 <1527081091-13389-3-git-send-email-ilialin@codeaurora.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1527081091-13389-3-git-send-email-ilialin@codeaurora.org>
User-Agent: Mutt/1.10.0 (2018-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, May 23, 2018 at 04:11:31PM +0300, Ilia Lin wrote:
> 1. Add NVMEM node for the speedbin
> 2. Add definitions for all possible MSM8996 CPU OPPs.
> The qcom-cpufreq-kryo driver will select the appropriate subset.
> 
> Signed-off-by: Ilia Lin <ilialin@codeaurora.org>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/apq8096-db820c.dts |   2 +-
>  arch/arm64/boot/dts/qcom/msm8996.dtsi       | 281 ++++++++++++++++++++++++++--
>  2 files changed, 270 insertions(+), 13 deletions(-)
> 

Hello Ilia, Viresh


I tried this patch series (together with you cpu clock driver).

The first problem is that the driver fails to create debugfs entries.

The second problem appears to be that it runs on unlisted frequency.
Is this simply because of missing entries in the opp table?

[    4.538513] cpu cpu1: opp_list_debug_create_link: Failed to create link
[    4.538567] cpu cpu1: _add_opp_dev: Failed to register opp debugfs (-12)
[    4.544514] cpufreq: cpufreq_online: CPU0: Running at unlisted freq: 614400 KHz
[    4.551441] cpufreq: cpufreq_online: CPU0: Unlisted initial frequency changed to: 652800 KHz
[    4.563219] cpu cpu3: opp_list_debug_create_link: Failed to create link
[    4.566937] cpu cpu3: _add_opp_dev: Failed to register opp debugfs (-12)
[    4.573489] cpufreq: cpufreq_online: CPU2: Running at unlisted freq: 19200 KHz
[    4.580368] cpufreq: cpufreq_online: CPU2: Unlisted initial frequency changed to: 307200 KHz


Kind regards,
Niklas