Skip to content

clk: meson: MPLL fixes for Meson8b

Jerome Brunet linux-amlogic linux-arm-kernel linux-clk 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 0 repliesNew
  2. v12 patches 0 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2clk: meson: mpll: fix division by zero in rate_from_paramsdrivers/clk/meson/clk-mpll.c+15 -111 reply
2/2clk: meson: mpll: use 64bit math in rate_from_paramsdrivers/clk/meson/clk-mpll.c+1 -11 reply

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by.

PatchNeil Armstrong
1/2R-b
2/2R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/2] clk: meson: MPLL fixes for Meson8b

MPLL clocks have been recently added to the Meson8b clock driver: [0]

On meson8b boards this unfortunately causes a division by zero error which is fixed by patch #1 in this series.

While investigating this I found that there also seems to be a 32bit overflow in the calculation in rate_from_params(), which is fixed by patch #2 in this series.

Changes since v1: [1]
- Return an error code when the mpll parameters are out of the specified range.
- As agreed with martin, use DIV_ROUND_UP_ULL instead of mul_u64_u32_div.

[0] http://lists.infradead.org/pipermail/linux-amlogic/2017-March/002757.html
[1] https://lkml.kernel.org/r/20170401130225.8811-1-martin.blumenstingl at googlemail.com

Martin Blumenstingl (2):
clk: meson: mpll: fix division by zero in rate_from_params
clk: meson: mpll: use 64bit math in rate_from_params

 drivers/clk/meson/clk-mpll.c | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)
Signature · 1 line
2.9.3