[Top] [All Lists]

Re: [PATCH 05/10] MIPS: lantiq: add watchdog support

To: Jamie Iles <>
Subject: Re: [PATCH 05/10] MIPS: lantiq: add watchdog support
From: John Crispin <>
Date: Thu, 06 Jan 2011 10:51:10 +0100
Cc: Ralf Baechle <>, Ralph Hempel <>, Wim Van Sebroeck <>,,
In-reply-to: <20110105234910.GD2112@gallagher>
Original-recipient: rfc822;
References: <> <> <20110105234910.GD2112@gallagher>
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv: Gecko/20100913 Icedove/3.0.7
On 06/01/11 00:49, Jamie Iles wrote:
> I think you need a clk_put() here too to balance the clk_get() in the 
> probe method so you'll need to keep a reference to the clk.

Hi Jamie,

i will fold your suggestions into the series.

the clk.c/h implementation on the lantiq target is very simple. it only
allows to read the static rates of the 3 clocks. clk_put is implemented
as follows

clk_put(struct clk *clk)
    /* not used */

so in theory you are right and we should call that function, however as
it is only a stub and the driver is only used by the lantiq target i
think it is save to leave out the clk_put(); call. we could however put
a commet in the code to make this clear (same as with the clk_enable()
not being needed as the clocks are always running)


<Prev in Thread] Current Thread [Next in Thread>