[PATCH] Watchdog: Adding support for Watchdog found with ARM CortexA9
Viresh KUMAR
viresh.kumar at st.com
Wed May 19 00:02:57 EDT 2010
On 5/18/2010 6:50 PM, George G. Davis wrote:
> Hi,
>
> On Tue, May 18, 2010 at 02:55:21PM +0530, srinidhi kasagar wrote:
>> I do not understand why a new driver is required while having similar one
>> for MPCore watchdog.
>> Please have a look at drivers/watchdog/mpcore_wdt.c
>
> The same thought occurred to me. Based on a cursory glance at the Cortex-A9
> MPCore and ARM11 MPCore, the per-CPU watchdogs do appear to be the same
> and could use the same driver.
>
> Any reason for creating a new driver rather than reusing the ARM11
> MPCore driver?
>
Sorry Guys!!! Earlier I tried to find existing driver, but couldn't found one.
So sent a new one.
I went through this driver and found some shortcomings in it:
- It looks it is not used by anyone now, as it required
asm/hardware/arm_twd.h and it was removed in v2.6.31.
So it would never compile.
- It doesn't support clock framework, and works on fixed rate.
- It works on fixed prescale value (256).
- It doesn't support WDIOC_GETBOOTSTATUS.
So, please suggest how to proceed as my patch contains all these features.
regards,
viresh kumar.
More information about the linux-arm-kernel
mailing list