Re: Fwd: Patch for Fintek F71869 watchdoh

[Giel van Schijndel]

On Mon, Dec 06, 2010 at 11:14:05 +0100, Hans de Goede wrote:
>> -------- Original Message --------
>> Subject: Patch for Fintek F71869 watchdoh
>> Date: Mon, 6 Dec 2010 00:40:43 +0100
>> From: Michel Arboi <michel@xxxxxxxxxxxxxxx>
>> Organization: Compilo ergo sum
>> To: Hans de Goede <hdegoede@xxxxxxxxxx>
>> 
>> I'm not sure that you are the right person, but actually I do not know
>> who I should contact. I saw that you were working on the support of
>> the Fintek F71869 sensors.

These are the e-mail addresses to send patches for that driver to:
> ~/git/linux-2.6 $ scripts/get_maintainer.pl -f drivers/watchdog/f71808e_wdt.c
> Wim Van Sebroeck <wim@xxxxxxxxx>
> Giel van Schijndel <me@xxxxxxxxx>
> linux-watchdog@xxxxxxxxxxxxxxx
> linux-kernel@xxxxxxxxxxxxxxx

>> I have a Jetway NC9C-550-LF mobo, I quickly hacked this to support the
>> watchdog. It seems to work.
> 
> Here is a patch send to me to add support for the F71869 to the Fintek
> superio watchdog driver.
> 
> Michel, I've forwarded you're patch to Giel, who is the author and
> maintainer of the Fintek superio watchdog driver.

> --- ./drivers/watchdog/f71808e_wdt.c	2010-10-20 22:30:22.000000000 +0200
> +++ /tmp/f71808e_wdt.c	2010-12-05 01:19:48.819567802 +0100
> @@ -52,6 +52,8 @@
>  #define SIO_F71882_ID		0x0541	/* Chipset ID */
>  #define SIO_F71889_ID		0x0723	/* Chipset ID */
>  
> +#define SIO_F71869_ID		0x0814
> +
>  #define	F71882FG_REG_START		0x01
>  
>  #define F71808FG_REG_WDO_CONF		0xf0

Please insert that constant in the list of other Chipset ID's
alphabetically sorted (to keep that list consistent).

> @@ -98,7 +100,7 @@
>  MODULE_PARM_DESC(start_withtimeout, "Start watchdog timer on module load with"
>  	" given initial timeout. Zero (default) disables this feature.");
>  
> -enum chips { f71808fg, f71858fg, f71862fg, f71882fg, f71889fg };
> +enum chips { f71808fg, f71858fg, f71862fg, f71882fg, f71889fg, f71869 };
>  
>  static const char *f71808e_names[] = {
>  	"f71808fg",
> @@ -106,6 +108,7 @@
>  	"f71862fg",
>  	"f71882fg",
>  	"f71889fg",
> +	"f71869",
>  };
>  
>  /* Super-I/O Function prototypes */
> @@ -308,6 +311,10 @@
>  		superio_set_bit(watchdog.sioaddr, 0x29, 1);
>  		break;
>  
> +	case f71869:
> +		/* GPIO14 --> WDTRST# */
> +		superio_clear_bit(watchdog.sioaddr, 0x29, 4);
> +		break;
>  	default:
>  		/*
>  		 * 'default' label to shut up the compiler and catch
> @@ -708,6 +715,9 @@
>  	case SIO_F71882_ID:
>  		watchdog.type = f71882fg;
>  		break;
> +	case SIO_F71869_ID:
> +		watchdog.type = f71869;
> +		break;
>  	case SIO_F71862_ID:
>  	case SIO_F71889_ID:
>  		/* These have a watchdog, though it isn't implemented (yet). */

The rest of that patch looks fine.  So if you perform the above
modification you'll get my Ack.

-- 
Met vriendelijke groet,
With kind regards,
Giel van Schijndel
--
"Measuring programming progress by lines of code is like measuring
 aircraft building progress by weight."
  -- Bill Gates

Attachment: signature.asc
Description: Digital signature