clk: qcom: Base rcg parent rate off plan frequency
authorEvan Green <evgreen@chromium.org>
Fri, 13 Apr 2018 20:33:36 +0000 (13:33 -0700)
committerStephen Boyd <sboyd@kernel.org>
Thu, 10 May 2018 18:45:32 +0000 (11:45 -0700)
_freq_tbl_determine_rate uses the pre_div found in the clock plan
multiplied by the requested rate from the caller to determine the
best parent rate to set. If the requested rate is not exactly equal
to the rate that was found in the clock plan, then using the requested
rate in parent rate calculations is incorrect. For instance, if 150MHz
was requested, but 200MHz was the match found, and that plan had a
pre_div of 3, then the parent should be set to 600MHz, not 450MHz.

Signed-off-by: Evan Green <evgreen@chromium.org>
Fixes: bcd61c0f535a ("clk: qcom: Add support for root clock generators (RCGs)")
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/qcom/clk-rcg2.c

index bbeaf9c09dbb4750cd6479ae16ab416626a19731..ec6cee8ff1bc3f059bd6a9b61b6bd65f6d117465 100644 (file)
@@ -211,6 +211,7 @@ static int _freq_tbl_determine_rate(struct clk_hw *hw, const struct freq_tbl *f,
        clk_flags = clk_hw_get_flags(hw);
        p = clk_hw_get_parent_by_index(hw, index);
        if (clk_flags & CLK_SET_RATE_PARENT) {
+               rate = f->freq;
                if (f->pre_div) {
                        rate /= 2;
                        rate *= f->pre_div + 1;