clk: rockchip: Remove 48 MHz PLL rate from rk3288
authorDouglas Anderson <dianders@chromium.org>
Tue, 4 Jun 2019 22:31:59 +0000 (15:31 -0700)
committerHeiko Stuebner <heiko@sntech.de>
Thu, 6 Jun 2019 10:49:30 +0000 (12:49 +0200)
commit7d2129310b8c1f22850d56b1725447aef88d8b1f
tree3fac9b6c058a2d9f45b062d71eed9919a5018e9c
parentf14b3c91ec5f013a8be337541fd34a42a31fb074
clk: rockchip: Remove 48 MHz PLL rate from rk3288

The 48 MHz PLL rate is not present in the downstream chromeos-3.14
tree.  Looking at history, it was originally removed in
<https://crrev.com/c/265810> ("CHROMIUM: clk: rockchip: expand more
clocks support") with no explanation.  Much of that patch was later
reverted in <https://crrev.com/c/284595> ("CHROMIUM: clk: rockchip:
Revert more questionable PLL rates"), but that patch left in the
removal of 48 MHz.  What I wrote in that patch:

> Note that the original change also removed the rate (48000000, 1,
> 64, 32) from the table.  I have no idea why that was squashed in
> there, but that rate was invalid anyway (it appears to have an out
> of bounds NO).  I'm not putting that rate in.

Reading the TRM I see that NO is defined as
- NO: 1, 2-16 (even only)
...and furthermore only 4 bits are assigned for NO-1, which means that
the highest NO we could even represent is 16.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
drivers/clk/rockchip/clk-rk3288.c