2015-04-16 16:38:03

by Vivien Didelot

[permalink] [raw]
Subject: Re: [PATCH 0/4] watchdog: MAX63xx cleanup and platform settings

Hi Wim, all,

> The purpose of this patchset is to add a platform data structure for the
> max63xx_wdt driver in order to setup the device in a platform code. This is
> especially handy if the driver is built-in and/or the device registers aren't
> memory mapped.
>
> First, fix the Kconfig entry to allow building the support for architectures
> other than ARM. Then clean up the driver to help distinguish different device
> connections and to ease the introduction of new features: support for GPIO
> wired MAX63xx devices and heartbeat as a platform setting.
>
> Tested with a GPIO wired MAX6373 on an Atom platform, with a built-in driver.
>
> Vivien Didelot (4):
> watchdog: MAX63XX_WATCHDOG does not depend on ARM
> watchdog: max63xx: cleanup
> watchdog: max63xx: add GPIO support
> watchdog: max63xx: add heartbeat to platform data

Did you get any chance to look into merging this patchset?
(It still applies well on the latest watchdog-next/master tree.)

Best,
-v