clk: ingenic: Fix bugs with divided dividers
authorPaul Cercueil <paul@crapouillou.net>
Fri, 1 Oct 2021 17:20:33 +0000 (18:20 +0100)
committerStephen Boyd <sboyd@kernel.org>
Tue, 2 Nov 2021 21:27:22 +0000 (14:27 -0700)
commited84ef1cd7eddf933d4ffce2caa8161d6f947245
treeaaee25e6da992b00cc711ba2ad1576af376beb78
parente2ceaa867d266472b31f3e03ba16f3120aefc152
clk: ingenic: Fix bugs with divided dividers

Two fixes in one:

- In the "impose hardware constraints" block, the "logical" divider
  value (aka. not translated to the hardware) was clamped to fit in the
  register area, but this totally ignored the fact that the divider
  value can itself have a fixed divider.

- The code that made sure that the divider value returned by the
  function was a multiple of its own fixed divider could result in a
  wrong value being calculated, because it was rounded down instead of
  rounded up.

Fixes: 4afe2d1a6ed5 ("clk: ingenic: Allow divider value to be divided")
Co-developed-by: Artur Rojek <contact@artur-rojek.eu>
Signed-off-by: Artur Rojek <contact@artur-rojek.eu>
Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20211001172033.122329-1-paul@crapouillou.net
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/ingenic/cgu.c