Re: [PATCH] watchdog: add support for w83697hg chip

[Pádraig Brady]

Samuel Tardieu wrote:
>>>>>>"Sam" == Samuel Tardieu <sam@xxxxxxxxxxx> writes:
> 
> 
> Sam> I suggest that you use the following patch instead. [...] I
> Sam> disable the watchdog until it is first used
> 
> Oops, disabling will work best if done at module initialization time :)
> Revised patch attached.

Personally I don't agree with disabling the watchdog on load.
If it is already started from the BIOS or GRUB for example,
there will be a large window of time/logic that is not protected.

Pádraig.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/