Re: [PATCH v4 0/8] watchdog/at91sam9_wdt: Convert to use the new framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Nicolas,

> > The purpose of the patch series is to convert at91sam9_wdt to use the watchdog framework,
> >  
> > Because the Watchdog Mode Register(WDT_MR) which WDDIS bit to disable or disable the watchdog
> > can be written only once after reset. so if you want to use the watchdog in the kernel, 
> > you must be sure the watchdog doesn't be disabled in the at91bootstrap.
> > 
> > It is tested on at91sam9m10g45ek both DT and non-DT support, at91sam9g25ek with DT support
> > by building Documentation/watchdog/src/watchdog-simple.c.
> > 
> > It is based on v3.8-rc6 + Arnd Bergmann's at91_wdt_dt_ids cannot be __init
> > 			+ Fabio Porcedda's add timeout-sec property binding
> 
> On my side, I give my acknowledgement to the whole patch series:
> 
> Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxx>
> 
> I hope that once Fabio Porcedda's series is merged, this series can be
> applied on top of it seamlessly and quickly.
> 
> Wim,
> Tell us if you need that we resend this series or rebase it on top of
> another git tree.
> 
> Thanks for your work and for your reviews.

This is how I'm going to do it:
1) We are going to add my reworked version of Fabio's v7 patch of the core timeout-sec part.
2) then we are going to do the conversion of the at91sam9 driver but without doing the other changes that Wenyou proposed.
3) Then we add the driver related part of Fabio's patches (I don't want non converted drivers to have the watchdog_core select, hence we do the conversion of at91sam9 first in step 2).
4) then we can see what is still needed of Wenyou's patches.
-> all will go through the linuw-watchdog trees.

Note: I know we have another patch in Greg's driver tree that changes the module_platform_driver_probe, but that should not interfere with these patches.

Kind regards,
Wim.
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux