Re: [PATCH 0/3] improved watchdog driver for STMP3xyz/imx23/imx28

[Wolfram Sang <w.sang@pengutronix.de>]

[-- Attachment #1: Type: text/plain, Size: 2650 bytes --]

On Fri, Jan 04, 2013 at 02:58:46PM +0100, Wolfram Sang wrote:
> Geez, this should have been sent a year ago but it dropped off somehow.
> Introduce a new user of the watchdog framework. This is a rewritten driver
> for STMP and imx23/28: Quoting Patch 2/3 for the motivation of a rewrite:
> 
> ===
> 
> Replace the existing STMP3xxx driver because it has enough drawbacks
> that a rewrite is apropriate. The new driver is designed to use the
> watchdog framework which makes it a lot smaller and avoids open coding
> the watchdog API again. It also uses now an explicitly exported function
> from the RTC driver to set up its registers (the old driver silently
> reused the hopefully(!) already remapped RTC registers). Also, this
> driver is mach independent, while the old one depends on a mach replaced
> by another one a year ago. Since the user interface is still the
> standard watchdog API, users don't need to adapt.
> 
> ===
> 
> The first patch adds an accessor routine to the RTC-driver which can hopefully
> go via the watchdog-tree as well (asking for Andrew's ack here). It is needed
> because the old driver silently assumed its IO area to be already remapped by
> the RTC driver. The second patch adds the new driver, the third removes the old
> one. The old driver was broken (= not compilable, no arch for it) anyhow, so
> there cannot be any regression. Based on 3.8-rc2 and tested on various MX28
> boards.
> 
> Thanks,
> 
>    Wolfram

Ping.

I rebased and tested the series against v3.8-rc5, added Andrew's ack and
pushed it out to

	git://git.pengutronix.de/git/wsa/linux.git watchdog_mxs

Would be awesome to have this (simple) driver in 3.9.

> 
> 
> Wolfram Sang (3):
>   rtc: stmp3xxx: add wdt-accessor function
>   watchdog: add new driver for STMP3xxx and i.MX23/28
>   watchdog: remove old STMP3xxx driver
> 
>  drivers/rtc/rtc-stmp3xxx.c          |   64 ++++++++
>  drivers/watchdog/Kconfig            |   13 +-
>  drivers/watchdog/Makefile           |    2 +-
>  drivers/watchdog/stmp3xxx_rtc_wdt.c |  111 ++++++++++++++
>  drivers/watchdog/stmp3xxx_wdt.c     |  288 -----------------------------------
>  include/linux/stmp3xxx_rtc_wdt.h    |   15 ++
>  6 files changed, 198 insertions(+), 295 deletions(-)
>  create mode 100644 drivers/watchdog/stmp3xxx_rtc_wdt.c
>  delete mode 100644 drivers/watchdog/stmp3xxx_wdt.c
>  create mode 100644 include/linux/stmp3xxx_rtc_wdt.h
> 
> -- 
> 1.7.10.4
> 

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]