Re: [PATCH 2/4] watchdog: coh901327_wdt: Keep clock enabled after loading driver

From: Linus Walleij
Date: Mon Jan 09 2017 - 14:39:09 EST


On Wed, Jan 4, 2017 at 4:25 AM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:

> Enabling the clock before accessing chip registers and disabling it
> afterwards does not really make sense and only adds complexity to
> the driver. In addition to that, a comment int the driver suggests
> that it does not serve a useful purpose either.
>
> "The watchdog block is of course always clocked, the
> clk_enable()/clk_disable() calls are mainly for performing reference
> counting higher up in the clock hierarchy."
>
> Just keep the clock enabled instead.
>
> Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>

Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij