On Tue, Feb 22, 2011 at 12:56 PM, Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> wrote: > On Mon, Feb 21, 2011 at 06:39:50PM -0800, Colin Cross wrote: >> + 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. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html