Re: [PATCH v3 0/7] Clock changes to support cpufreq on QCS404

From: Niklas Cassel
Date: Wed Dec 18 2019 - 07:21:54 EST


On Mon, Nov 25, 2019 at 02:59:02PM +0100, Niklas Cassel wrote:
> The following clock changes are required to enable cpufreq support on
> the QCS404.
>
> Changes since v2:
> -Addressed Stephen Boyd's comment regarding apcs-msm8916
> should use new way of specifying clock parents.
> -DT binding now has "pll" as first clock, in order to
> not break DT backwards compatibility (in case no clock-names
> are given).
> -Moved EPROBE_DEFER error handling to its own patch.
>
> Jorge Ramirez-Ortiz (6):
> dt-bindings: mailbox: qcom: Add clock-name optional property
> clk: qcom: gcc: limit GPLL0_AO_OUT operating frequency
> clk: qcom: hfpll: register as clock provider
> clk: qcom: hfpll: CLK_IGNORE_UNUSED
> clk: qcom: hfpll: use clk_parent_data to specify the parent
> clk: qcom: apcs-msm8916: silently error out on EPROBE_DEFER
>
> Niklas Cassel (1):
> clk: qcom: apcs-msm8916: use clk_parent_data to specify the parent
>
> .../mailbox/qcom,apcs-kpss-global.txt | 24 ++++++++++++++---
> drivers/clk/qcom/apcs-msm8916.c | 26 ++++++++++++++-----
> drivers/clk/qcom/clk-alpha-pll.c | 8 ++++++
> drivers/clk/qcom/clk-alpha-pll.h | 1 +
> drivers/clk/qcom/gcc-qcs404.c | 2 +-
> drivers/clk/qcom/hfpll.c | 21 +++++++++++++--
> 6 files changed, 70 insertions(+), 12 deletions(-)
>
> --
> 2.23.0
>

Hello Stephen,

I have adressed your review comments
on the previous patch series version.

Could you please have a look?

If it looks good, could you please
consider taking them via your tree?

Kind regards,
Niklas