Viresh,<br><br>Recently I have sent two patches which are already queued in Russell's tracker<br><br>6125/1 ARM TWD: move TWD registers to common header <br>6126/1 ARM mpcore_wdt: fix build failure and other fixes<br>
<br>which addresses the following:<br>1) fix build failure<br>2) movement of the MPCore TWD registers. <br><br>And currently, Ux500 machine uses this.<br><br>What I feel is to patch this existing driver if you think it is missing some functionality.<br>
<br>regards,<br>Srinidhi<br><br><br><br><div class="gmail_quote">On Wed, May 19, 2010 at 9:32 AM, Viresh KUMAR <span dir="ltr"><<a href="mailto:viresh.kumar@st.com">viresh.kumar@st.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 5/18/2010 6:50 PM, George G. Davis wrote:<br>
> Hi,<br>
><br>
> On Tue, May 18, 2010 at 02:55:21PM +0530, srinidhi kasagar wrote:<br>
>> I do not understand why a new driver is required while having similar one<br>
>> for MPCore watchdog.<br>
>> Please have a look at drivers/watchdog/mpcore_wdt.c<br>
><br>
> The same thought occurred to me. Based on a cursory glance at the Cortex-A9<br>
> MPCore and ARM11 MPCore, the per-CPU watchdogs do appear to be the same<br>
> and could use the same driver.<br>
><br>
> Any reason for creating a new driver rather than reusing the ARM11<br>
> MPCore driver?<br>
><br>
<br>
</div>Sorry Guys!!! Earlier I tried to find existing driver, but couldn't found one.<br>
So sent a new one.<br>
<br>
I went through this driver and found some shortcomings in it:<br>
- It looks it is not used by anyone now, as it required<br>
asm/hardware/arm_twd.h and it was removed in v2.6.31.<br>
So it would never compile.<br>
- It doesn't support clock framework, and works on fixed rate.<br>
- It works on fixed prescale value (256).<br>
- It doesn't support WDIOC_GETBOOTSTATUS.<br>
<br>
So, please suggest how to proceed as my patch contains all these features.<br>
<br>
regards,<br>
<font color="#888888">viresh kumar.<br>
</font></blockquote></div><br>