diff options
| author | Joey Lu <a0987203069@gmail.com> | 2026-05-21 09:42:20 +0800 |
|---|---|---|
| committer | Brian Masney <bmasney@redhat.com> | 2026-06-29 13:27:28 -0400 |
| commit | e1311954cb600d5f95cd9e2fe9a7376edc2ac3c5 (patch) | |
| tree | 560da1371c0370d0d663833cf97e194bbdd928c0 /include/linux/raid | |
| parent | 26de5aed72d80bd8aec2583134aca3597c64fda9 (diff) | |
clk: nuvoton: ma35d1: fix ma35d1_clk_pll_determine_rate logic
ma35d1_clk_pll_determine_rate() called ma35d1_pll_find_closest()
unconditionally before the switch statement, and then every case
branch overwrote pll_freq by reading the current hardware registers.
For CAPLL and DDRPLL this means find_closest() ran unnecessarily
(and incorrectly, since those PLLs are read-only) and its result
was silently discarded.
Fix by moving the find_closest() call inside the APLL/EPLL/VPLL
branch where it belongs. Group CAPLL and DDRPLL together as
read-only PLLs that simply report their current rate; handle them
with an explicit if/else to keep the CAPLL (SMIC design) and DDRPLL
(standard design) paths distinct.
Fixes: 691521a367cf ("clk: nuvoton: Add clock driver for ma35d1 clock controller")
Signed-off-by: Joey Lu <a0987203069@gmail.com>
Signed-off-by: Brian Masney <bmasney@redhat.com>
Diffstat (limited to 'include/linux/raid')
0 files changed, 0 insertions, 0 deletions
