Skip to content

Commit ea56ad6

Browse files
lkundrakbebarino
authored andcommitted
clk: mmp2: Stop pretending PLL outputs are constant
The hardcoded values for PLL1 and PLL2 are wrong. PLL1 is slightly off -- it defaults to 797.33 MHz, not 800 MHz. PLL2 is disabled by default, but also configurable. Tested on a MMP2-based OLPC XO-1.75 laptop, with PLL1=797.33 and various values of PLL2 set via set-pll2-520mhz, set-pll2-910mhz and set-pll2-988mhz Open Firmware words. Signed-off-by: Lubomir Rintel <[email protected]> Link: https://lkml.kernel.org/r/[email protected] Signed-off-by: Stephen Boyd <[email protected]>
1 parent 5d34d0b commit ea56ad6

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

drivers/clk/mmp/clk-of-mmp2.c

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
*
44
* Copyright (C) 2012 Marvell
55
* Chao Xie <[email protected]>
6+
* Copyright (C) 2020 Lubomir Rintel <[email protected]>
67
*
78
* This file is licensed under the terms of the GNU General Public
89
* License version 2. This program is licensed "as is" without any
@@ -55,7 +56,11 @@
5556
#define APMU_CCIC1 0xf4
5657
#define APMU_USBHSIC0 0xf8
5758
#define APMU_USBHSIC1 0xfc
59+
60+
#define MPMU_FCCR 0x8
61+
#define MPMU_POSR 0x10
5862
#define MPMU_UART_PLL 0x14
63+
#define MPMU_PLL2_CR 0x34
5964

6065
struct mmp2_clk_unit {
6166
struct mmp_clk_unit unit;
@@ -67,11 +72,14 @@ struct mmp2_clk_unit {
6772
static struct mmp_param_fixed_rate_clk fixed_rate_clks[] = {
6873
{MMP2_CLK_CLK32, "clk32", NULL, 0, 32768},
6974
{MMP2_CLK_VCTCXO, "vctcxo", NULL, 0, 26000000},
70-
{MMP2_CLK_PLL1, "pll1", NULL, 0, 800000000},
71-
{MMP2_CLK_PLL2, "pll2", NULL, 0, 960000000},
7275
{MMP2_CLK_USB_PLL, "usb_pll", NULL, 0, 480000000},
7376
};
7477

78+
static struct mmp_param_pll_clk pll_clks[] = {
79+
{MMP2_CLK_PLL1, "pll1", 797330000, MPMU_FCCR, 0x4000, MPMU_POSR, 0},
80+
{MMP2_CLK_PLL2, "pll2", 0, MPMU_PLL2_CR, 0x0300, MPMU_PLL2_CR, 10},
81+
};
82+
7583
static struct mmp_param_fixed_factor_clk fixed_factor_clks[] = {
7684
{MMP2_CLK_PLL1_2, "pll1_2", "pll1", 1, 2, 0},
7785
{MMP2_CLK_PLL1_4, "pll1_4", "pll1_2", 1, 2, 0},
@@ -113,6 +121,10 @@ static void mmp2_pll_init(struct mmp2_clk_unit *pxa_unit)
113121
mmp_register_fixed_rate_clks(unit, fixed_rate_clks,
114122
ARRAY_SIZE(fixed_rate_clks));
115123

124+
mmp_register_pll_clks(unit, pll_clks,
125+
pxa_unit->mpmu_base,
126+
ARRAY_SIZE(pll_clks));
127+
116128
mmp_register_fixed_factor_clks(unit, fixed_factor_clks,
117129
ARRAY_SIZE(fixed_factor_clks));
118130

0 commit comments

Comments
 (0)