clk: qcom: apcs-msm8916: Flag a53mux instead of a53pll as critical
authorShawn Guo <shawn.guo@linaro.org>
Sun, 4 Jul 2021 02:40:29 +0000 (10:40 +0800)
committerStephen Boyd <sboyd@kernel.org>
Fri, 6 Aug 2021 01:52:11 +0000 (18:52 -0700)
commit0dfe9bf91f9f2993ae73c603571a85e3c6e6fc24
tree9a5bad9fd513b87bc7221284762df95aca9e8159
parent945cb3a105aef63af1354e0fbe10a0d1ca7a32c2
clk: qcom: apcs-msm8916: Flag a53mux instead of a53pll as critical

The clock source for MSM8916 cpu cores is like below.

                        |\
         a53pll --------| \ a53mux     +------+
                        | |------------| cpus |
     gpll0_vote --------| /            +------+
                        |/

So a53mux rather than a53pll is actually the parent clock of cpu cores.
It makes more sense to flag a53mux as critical instead, so that when
either a53pll or gpll0_vote is used by cpu cores, the clock will be kept
enabled while the other can be disabled.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Link: https://lore.kernel.org/r/20210704024032.11559-2-shawn.guo@linaro.org
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/qcom/a53-pll.c
drivers/clk/qcom/apcs-msm8916.c