[PATCH] Watchdog: Adding support for Watchdog found with ARM CortexA9

[srinidhi kasagar]

Viresh,

Recently I have sent two patches which are already queued in Russell's
tracker

6125/1      ARM TWD: move TWD registers to common header
6126/1      ARM mpcore_wdt: fix build failure and other fixes

which addresses the following:
1) fix build failure
2) movement of the MPCore TWD registers.

And currently, Ux500 machine uses this.

What I feel is to patch this existing driver if you think it is missing some
functionality.

regards,
Srinidhi



On Wed, May 19, 2010 at 9:32 AM, Viresh KUMAR <viresh.kumar at st.com> wrote:

> 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.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20100519/0438244b/attachment.htm>