clk: tegra: divider: Check UART's divider enable-bit state on rate's recalculation
authorDmitry Osipenko <digetx@gmail.com>
Wed, 18 Dec 2019 18:44:05 +0000 (21:44 +0300)
committerThierry Reding <treding@nvidia.com>
Fri, 10 Jan 2020 14:50:05 +0000 (15:50 +0100)
UART clock is divided using divisor values from DLM/DLL registers when
enable-bit is unset in clk register and clk's divider configuration isn't
taken onto account in this case. This doesn't cause any problems, but
let's add a check for the divider's enable-bit state, for consistency.

Acked-by: Peter De Schrijver <pdeschrijver@nvidia.com>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/clk/tegra/clk-divider.c

index ca0de5f..38daf48 100644 (file)
@@ -40,8 +40,13 @@ static unsigned long clk_frac_div_recalc_rate(struct clk_hw *hw,
        int div, mul;
        u64 rate = parent_rate;
 
-       reg = readl_relaxed(divider->reg) >> divider->shift;
-       div = reg & div_mask(divider);
+       reg = readl_relaxed(divider->reg);
+
+       if ((divider->flags & TEGRA_DIVIDER_UART) &&
+           !(reg & PERIPH_CLK_UART_DIV_ENB))
+               return rate;
+
+       div = (reg >> divider->shift) & div_mask(divider);
 
        mul = get_mul(divider);
        div += mul;