On Tue, Feb 22, 2011 at 12:56 PM, Russell King - ARM Linux
[off-list ref] wrote:

On Mon, Feb 21, 2011 at 06:39:50PM -0800, Colin Cross wrote:

quoted

+ ? ? struct clk *clk;
? ? ? unsigned long rate = clk_measure_input_freq();
? ? ? int ret;

+ ? ? clk = clk_get_sys("timer", NULL);
+ ? ? BUG_ON(IS_ERR(clk));
+ ? ? clk_enable(clk);
+ ? ? clk_put(clk);
+
+ ? ? clk = clk_get_sys("rtc-tegra", NULL);
+ ? ? BUG_ON(IS_ERR(clk));
+ ? ? clk_enable(clk);
+ ? ? clk_put(clk);

It's probably better to leave them get'd if you're continuing to use
them and just forget the reference.

Good point, fixed.