1 From 546bac0479e51024027f8c8820f912573643b101 Mon Sep 17 00:00:00 2001
2 From: Eric Anholt <eric@anholt.net>
3 Date: Wed, 18 Jan 2017 07:31:57 +1100
4 Subject: [PATCH] clk: bcm2835: Add leaf clock measurement support, disabled by
7 This proved incredibly useful during debugging of the DSI driver, to
8 see if our clocks were running at rate we requested. Let's leave it
9 here for the next person interacting with clocks on the platform (and
10 so that hopefully we can just hook it up to debugfs some day).
12 Signed-off-by: Eric Anholt <eric@anholt.net>
13 Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
14 (cherry picked from commit 3f9195811d8d829556c4cd88d3f9e56a80d5ba60)
16 drivers/clk/bcm/clk-bcm2835.c | 144 ++++++++++++++++++++++++++++++++++--------
17 1 file changed, 119 insertions(+), 25 deletions(-)
19 --- a/drivers/clk/bcm/clk-bcm2835.c
20 +++ b/drivers/clk/bcm/clk-bcm2835.c
22 #include <linux/clk.h>
23 #include <linux/clk/bcm2835.h>
24 #include <linux/debugfs.h>
25 +#include <linux/delay.h>
26 #include <linux/module.h>
28 #include <linux/platform_device.h>
30 #define CM_SMIDIV 0x0b4
31 /* no definition for 0x0b8 and 0x0bc */
32 #define CM_TCNTCTL 0x0c0
33 -#define CM_TCNTDIV 0x0c4
34 +# define CM_TCNT_SRC1_SHIFT 12
35 +#define CM_TCNTCNT 0x0c4
36 #define CM_TECCTL 0x0c8
37 #define CM_TECDIV 0x0cc
38 #define CM_TD0CTL 0x0d0
39 @@ -338,6 +340,61 @@ static inline u32 cprman_read(struct bcm
40 return readl(cprman->regs + reg);
43 +/* Does a cycle of measuring a clock through the TCNT clock, which may
44 + * source from many other clocks in the system.
46 +static unsigned long bcm2835_measure_tcnt_mux(struct bcm2835_cprman *cprman,
49 + u32 osccount = 19200; /* 1ms */
53 + spin_lock(&cprman->regs_lock);
55 + cprman_write(cprman, CM_TCNTCTL, CM_KILL);
57 + cprman_write(cprman, CM_TCNTCTL,
58 + (tcnt_mux & CM_SRC_MASK) |
59 + (tcnt_mux >> CM_SRC_BITS) << CM_TCNT_SRC1_SHIFT);
61 + cprman_write(cprman, CM_OSCCOUNT, osccount);
63 + /* do a kind delay at the start */
66 + /* Finish off whatever is left of OSCCOUNT */
67 + timeout = ktime_add_ns(ktime_get(), LOCK_TIMEOUT_NS);
68 + while (cprman_read(cprman, CM_OSCCOUNT)) {
69 + if (ktime_after(ktime_get(), timeout)) {
70 + dev_err(cprman->dev, "timeout waiting for OSCCOUNT\n");
77 + /* Wait for BUSY to clear. */
78 + timeout = ktime_add_ns(ktime_get(), LOCK_TIMEOUT_NS);
79 + while (cprman_read(cprman, CM_TCNTCTL) & CM_BUSY) {
80 + if (ktime_after(ktime_get(), timeout)) {
81 + dev_err(cprman->dev, "timeout waiting for !BUSY\n");
88 + count = cprman_read(cprman, CM_TCNTCNT);
90 + cprman_write(cprman, CM_TCNTCTL, 0);
93 + spin_unlock(&cprman->regs_lock);
95 + return count * 1000;
98 static int bcm2835_debugfs_regset(struct bcm2835_cprman *cprman, u32 base,
99 struct debugfs_reg32 *regs, size_t nregs,
100 struct dentry *dentry)
101 @@ -473,6 +530,8 @@ struct bcm2835_clock_data {
109 struct bcm2835_gate_data {
110 @@ -1012,6 +1071,17 @@ static int bcm2835_clock_on(struct clk_h
112 spin_unlock(&cprman->regs_lock);
114 + /* Debug code to measure the clock once it's turned on to see
115 + * if it's ticking at the rate we expect.
117 + if (data->tcnt_mux && false) {
118 + dev_info(cprman->dev,
119 + "clk %s: rate %ld, measure %ld\n",
121 + clk_hw_get_rate(hw),
122 + bcm2835_measure_tcnt_mux(cprman, data->tcnt_mux));
128 @@ -1778,7 +1848,8 @@ static const struct bcm2835_clk_desc clk
129 .ctl_reg = CM_OTPCTL,
130 .div_reg = CM_OTPDIV,
136 * Used for a 1Mhz clock for the system clocksource, and also used
137 * bythe watchdog timer and the camera pulse generator.
138 @@ -1812,13 +1883,15 @@ static const struct bcm2835_clk_desc clk
139 .ctl_reg = CM_H264CTL,
140 .div_reg = CM_H264DIV,
145 [BCM2835_CLOCK_ISP] = REGISTER_VPU_CLK(
147 .ctl_reg = CM_ISPCTL,
148 .div_reg = CM_ISPDIV,
155 * Secondary SDRAM clock. Used for low-voltage modes when the PLL
156 @@ -1829,13 +1902,15 @@ static const struct bcm2835_clk_desc clk
157 .ctl_reg = CM_SDCCTL,
158 .div_reg = CM_SDCDIV,
163 [BCM2835_CLOCK_V3D] = REGISTER_VPU_CLK(
165 .ctl_reg = CM_V3DCTL,
166 .div_reg = CM_V3DDIV,
172 * VPU clock. This doesn't have an enable bit, since it drives
173 * the bus for everything else, and is special so it doesn't need
174 @@ -1849,7 +1924,8 @@ static const struct bcm2835_clk_desc clk
177 .flags = CLK_IS_CRITICAL,
178 - .is_vpu_clock = true),
179 + .is_vpu_clock = true,
182 /* clocks with per parent mux */
183 [BCM2835_CLOCK_AVEO] = REGISTER_PER_CLK(
184 @@ -1857,19 +1933,22 @@ static const struct bcm2835_clk_desc clk
185 .ctl_reg = CM_AVEOCTL,
186 .div_reg = CM_AVEODIV,
191 [BCM2835_CLOCK_CAM0] = REGISTER_PER_CLK(
193 .ctl_reg = CM_CAM0CTL,
194 .div_reg = CM_CAM0DIV,
199 [BCM2835_CLOCK_CAM1] = REGISTER_PER_CLK(
201 .ctl_reg = CM_CAM1CTL,
202 .div_reg = CM_CAM1DIV,
207 [BCM2835_CLOCK_DFT] = REGISTER_PER_CLK(
209 .ctl_reg = CM_DFTCTL,
210 @@ -1881,7 +1960,8 @@ static const struct bcm2835_clk_desc clk
211 .ctl_reg = CM_DPICTL,
212 .div_reg = CM_DPIDIV,
218 /* Arasan EMMC clock */
219 [BCM2835_CLOCK_EMMC] = REGISTER_PER_CLK(
220 @@ -1889,7 +1969,8 @@ static const struct bcm2835_clk_desc clk
221 .ctl_reg = CM_EMMCCTL,
222 .div_reg = CM_EMMCDIV,
228 /* General purpose (GPIO) clocks */
229 [BCM2835_CLOCK_GP0] = REGISTER_PER_CLK(
230 @@ -1898,7 +1979,8 @@ static const struct bcm2835_clk_desc clk
231 .div_reg = CM_GP0DIV,
234 - .is_mash_clock = true),
235 + .is_mash_clock = true,
237 [BCM2835_CLOCK_GP1] = REGISTER_PER_CLK(
239 .ctl_reg = CM_GP1CTL,
240 @@ -1906,7 +1988,8 @@ static const struct bcm2835_clk_desc clk
243 .flags = CLK_IS_CRITICAL,
244 - .is_mash_clock = true),
245 + .is_mash_clock = true,
247 [BCM2835_CLOCK_GP2] = REGISTER_PER_CLK(
249 .ctl_reg = CM_GP2CTL,
250 @@ -1921,40 +2004,46 @@ static const struct bcm2835_clk_desc clk
251 .ctl_reg = CM_HSMCTL,
252 .div_reg = CM_HSMDIV,
257 [BCM2835_CLOCK_PCM] = REGISTER_PER_CLK(
259 .ctl_reg = CM_PCMCTL,
260 .div_reg = CM_PCMDIV,
263 - .is_mash_clock = true),
264 + .is_mash_clock = true,
266 [BCM2835_CLOCK_PWM] = REGISTER_PER_CLK(
268 .ctl_reg = CM_PWMCTL,
269 .div_reg = CM_PWMDIV,
272 - .is_mash_clock = true),
273 + .is_mash_clock = true,
275 [BCM2835_CLOCK_SLIM] = REGISTER_PER_CLK(
277 .ctl_reg = CM_SLIMCTL,
278 .div_reg = CM_SLIMDIV,
281 - .is_mash_clock = true),
282 + .is_mash_clock = true,
284 [BCM2835_CLOCK_SMI] = REGISTER_PER_CLK(
286 .ctl_reg = CM_SMICTL,
287 .div_reg = CM_SMIDIV,
292 [BCM2835_CLOCK_UART] = REGISTER_PER_CLK(
294 .ctl_reg = CM_UARTCTL,
295 .div_reg = CM_UARTDIV,
301 /* TV encoder clock. Only operating frequency is 108Mhz. */
302 [BCM2835_CLOCK_VEC] = REGISTER_PER_CLK(
303 @@ -1967,7 +2056,8 @@ static const struct bcm2835_clk_desc clk
304 * Allow rate change propagation only on PLLH_AUX which is
305 * assigned index 7 in the parent array.
307 - .set_rate_parent = BIT(7)),
308 + .set_rate_parent = BIT(7),
312 [BCM2835_CLOCK_DSI0E] = REGISTER_PER_CLK(
313 @@ -1975,25 +2065,29 @@ static const struct bcm2835_clk_desc clk
314 .ctl_reg = CM_DSI0ECTL,
315 .div_reg = CM_DSI0EDIV,
320 [BCM2835_CLOCK_DSI1E] = REGISTER_PER_CLK(
322 .ctl_reg = CM_DSI1ECTL,
323 .div_reg = CM_DSI1EDIV,
328 [BCM2835_CLOCK_DSI0P] = REGISTER_DSI0_CLK(
330 .ctl_reg = CM_DSI0PCTL,
331 .div_reg = CM_DSI0PDIV,
336 [BCM2835_CLOCK_DSI1P] = REGISTER_DSI1_CLK(
338 .ctl_reg = CM_DSI1PCTL,
339 .div_reg = CM_DSI1PDIV,