Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-29 Thread Wim Van Sebroeck
Hi Adrian,

> I might be blind, but I'm not seeing what should be removed.

[EMAIL PROTECTED] linux-2.6]$ grep -i tco Documentation/watchdog/*
Documentation/watchdog/watchdog-api.txt:i810-tco.c -- Intel 810 chipset
Documentation/watchdog/watchdog.txt:The i810 TCO watchdog modules can be 
configured with the "i810_margin"
Documentation/watchdog/watchdog.txt:The i810 TCO watchdog driver also 
implements the WDIOC_GETSTATUS and
Documentation/watchdog/watchdog.txt:and WDIOC_GETBOOTSTATUS returns the value 
of TCO2 Status Register (see Intel's
Documentation/watchdog/watchdog.txt:WDT501P WDT500P 
SoftwareBerkshire   i810 TCOSA1100WD

But I need to clean this up anyway. So no real need to put this in the same 
patch yet.
I'll create a new patch for this that deals with the complete watchdog 
Documentation.

Greetings,
Wim.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-29 Thread Wim Van Sebroeck
Hi Adrian,

 I might be blind, but I'm not seeing what should be removed.

[EMAIL PROTECTED] linux-2.6]$ grep -i tco Documentation/watchdog/*
Documentation/watchdog/watchdog-api.txt:i810-tco.c -- Intel 810 chipset
Documentation/watchdog/watchdog.txt:The i810 TCO watchdog modules can be 
configured with the i810_margin
Documentation/watchdog/watchdog.txt:The i810 TCO watchdog driver also 
implements the WDIOC_GETSTATUS and
Documentation/watchdog/watchdog.txt:and WDIOC_GETBOOTSTATUS returns the value 
of TCO2 Status Register (see Intel's
Documentation/watchdog/watchdog.txt:WDT501P WDT500P 
SoftwareBerkshire   i810 TCOSA1100WD

But I need to clean this up anyway. So no real need to put this in the same 
patch yet.
I'll create a new patch for this that deals with the complete watchdog 
Documentation.

Greetings,
Wim.

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-27 Thread Adrian Bunk
On Sun, Apr 15, 2007 at 09:40:07PM +0200, Wim Van Sebroeck wrote:
> Hello Adrian,
> 
> > This patch contains the scheduled removal of the i8xx_tco watchdog 
> > driver.
> > 
> > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
> 
> Signed-off-by me also.
> 
> We probably also need to remove some stuff in the Documentation/watchdog 
> files.

I might be blind, but I'm not seeing what should be removed.

> Greetings,
> Wim.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-27 Thread Adrian Bunk
On Sun, Apr 15, 2007 at 09:40:07PM +0200, Wim Van Sebroeck wrote:
 Hello Adrian,
 
  This patch contains the scheduled removal of the i8xx_tco watchdog 
  driver.
  
  Signed-off-by: Adrian Bunk [EMAIL PROTECTED]
 
 Signed-off-by me also.
 
 We probably also need to remove some stuff in the Documentation/watchdog 
 files.

I might be blind, but I'm not seeing what should be removed.

 Greetings,
 Wim.

cu
Adrian

-- 

   Is there not promise of rain? Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   Only a promise, Lao Er said.
   Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-15 Thread Wim Van Sebroeck
Hello Adrian,

> This patch contains the scheduled removal of the i8xx_tco watchdog 
> driver.
> 
> Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>

Signed-off-by me also.

We probably also need to remove some stuff in the Documentation/watchdog files.

Greetings,
Wim.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-15 Thread Wim Van Sebroeck
Hello Adrian,

 This patch contains the scheduled removal of the i8xx_tco watchdog 
 driver.
 
 Signed-off-by: Adrian Bunk [EMAIL PROTECTED]

Signed-off-by me also.

We probably also need to remove some stuff in the Documentation/watchdog files.

Greetings,
Wim.

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-06 Thread Adrian Bunk
This patch contains the scheduled removal of the i8xx_tco watchdog 
driver.

Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>

---

 Documentation/feature-removal-schedule.txt |8 
 MAINTAINERS|6 
 drivers/char/watchdog/Kconfig  |   22 
 drivers/char/watchdog/Makefile |1 
 drivers/char/watchdog/i8xx_tco.c   |  571 -
 drivers/char/watchdog/i8xx_tco.h   |   42 -
 6 files changed, 650 deletions(-)

--- linux-2.6.21-rc5-mm4/Documentation/feature-removal-schedule.txt.old 
2007-04-05 21:22:19.0 +0200
+++ linux-2.6.21-rc5-mm4/Documentation/feature-removal-schedule.txt 
2007-04-05 21:22:59.0 +0200
@@ -235,14 +235,6 @@ Who:   Richard Purdie <[EMAIL PROTECTED]>
 
 ---
 
-What:  i8xx_tco watchdog driver
-When:  in 2.6.22
-Why:   the i8xx_tco watchdog driver has been replaced by the iTCO_wdt
-   watchdog driver.
-Who:   Wim Van Sebroeck <[EMAIL PROTECTED]>
-

-
 What:  i2c-rpx driver
 When:  June 2007
 Why:   Broken. It depends on i2c-algo-8xx which has never been in the
--- linux-2.6.21-rc5-mm4/MAINTAINERS.old2007-04-05 21:28:18.0 
+0200
+++ linux-2.6.21-rc5-mm4/MAINTAINERS2007-04-05 21:35:03.0 +0200
@@ -1690,12 +1690,6 @@ P:   H. Peter Anvin
 M: [EMAIL PROTECTED]
 S: Maintained
 
-i810 TCO TIMER WATCHDOG
-P: Nils Faerber
-M: [EMAIL PROTECTED]
-W: http://www.kernelconcepts.de/
-S: Maintained
-
 IA64 (Itanium) PLATFORM
 P: Tony Luck
 M: [EMAIL PROTECTED]
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/Kconfig.old  2007-04-05 
21:23:11.0 +0200
+++ linux-2.6.21-rc5-mm4/drivers/char/watchdog/Kconfig  2007-04-05 
21:23:21.0 +0200
@@ -298,28 +298,6 @@ config I6300ESB_WDT
  To compile this driver as a module, choose M here: the
  module will be called i6300esb.
 
-config I8XX_TCO
-   tristate "Intel i8xx TCO Timer/Watchdog"
-   depends on WATCHDOG && (X86 || IA64) && PCI
-   default n
-   ---help---
- Hardware driver for the TCO timer built into the Intel 82801
- I/O Controller Hub family.  The TCO (Total Cost of Ownership)
- timer is a watchdog timer that will reboot the machine after
- its second expiration. The expiration time can be configured
- with the "heartbeat" parameter.
-
- On some motherboards the driver may fail to reset the chipset's
- NO_REBOOT flag which prevents the watchdog from rebooting the
- machine. If this is the case you will get a kernel message like
- "failed to reset NO_REBOOT flag, reboot disabled by hardware".
-
- To compile this driver as a module, choose M here: the
- module will be called i8xx_tco.
-
- Note: This driver will be removed in the near future. Please
- use the Intel TCO Timer/Watchdog driver.
-
 config ITCO_WDT
tristate "Intel TCO Timer/Watchdog"
depends on WATCHDOG && (X86 || IA64) && PCI
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/Makefile.old 2007-04-05 
21:23:45.0 +0200
+++ linux-2.6.21-rc5-mm4/drivers/char/watchdog/Makefile 2007-04-05 
21:23:51.0 +0200
@@ -46,7 +46,6 @@ obj-$(CONFIG_IB700_WDT) += ib700wdt.o
 obj-$(CONFIG_IBMASR) += ibmasr.o
 obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
 obj-$(CONFIG_I6300ESB_WDT) += i6300esb.o
-obj-$(CONFIG_I8XX_TCO) += i8xx_tco.o
 obj-$(CONFIG_ITCO_WDT) += iTCO_wdt.o iTCO_vendor_support.o
 obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
 obj-$(CONFIG_SCx200_WDT) += scx200_wdt.o
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/i8xx_tco.h   2007-02-04 
19:44:54.0 +0100
+++ /dev/null   2006-09-19 00:45:31.0 +0200
@@ -1,42 +0,0 @@
-/*
- * i8xx_tco:   TCO timer driver for i8xx chipsets
- *
- * (c) Copyright 2000 kernel concepts <[EMAIL PROTECTED]>, All Rights 
Reserved.
- * http://www.kernelconcepts.de
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version
- * 2 of the License, or (at your option) any later version.
- *
- * Neither kernel concepts nor Nils Faerber admit liability nor provide
- * warranty for any of this software. This material is provided
- * "AS-IS" and at no charge.
- *
- * (c) Copyright 2000  kernel concepts <[EMAIL PROTECTED]>
- * developed for
- *  Jentro AG, Haar/Munich (Germany)
- *
- * TCO timer driver for i8xx chipsets
- * based on softdog.c by Alan Cox <[EMAIL PROTECTED]>
- *
- * For history and the complete list of supported I/O Controller Hub's
- * see i8xx_tco.c
- */
-
-
-/*
- * Some address definitions for the TCO
- */
-
-#defineTCOBASE ACPIBASE + 0x60 /* TCO base address 

[2.6.22 patch] the scheduled removal of the i8xx_tco watchdog driver

2007-04-06 Thread Adrian Bunk
This patch contains the scheduled removal of the i8xx_tco watchdog 
driver.

Signed-off-by: Adrian Bunk [EMAIL PROTECTED]

---

 Documentation/feature-removal-schedule.txt |8 
 MAINTAINERS|6 
 drivers/char/watchdog/Kconfig  |   22 
 drivers/char/watchdog/Makefile |1 
 drivers/char/watchdog/i8xx_tco.c   |  571 -
 drivers/char/watchdog/i8xx_tco.h   |   42 -
 6 files changed, 650 deletions(-)

--- linux-2.6.21-rc5-mm4/Documentation/feature-removal-schedule.txt.old 
2007-04-05 21:22:19.0 +0200
+++ linux-2.6.21-rc5-mm4/Documentation/feature-removal-schedule.txt 
2007-04-05 21:22:59.0 +0200
@@ -235,14 +235,6 @@ Who:   Richard Purdie [EMAIL PROTECTED]
 
 ---
 
-What:  i8xx_tco watchdog driver
-When:  in 2.6.22
-Why:   the i8xx_tco watchdog driver has been replaced by the iTCO_wdt
-   watchdog driver.
-Who:   Wim Van Sebroeck [EMAIL PROTECTED]
-

-
 What:  i2c-rpx driver
 When:  June 2007
 Why:   Broken. It depends on i2c-algo-8xx which has never been in the
--- linux-2.6.21-rc5-mm4/MAINTAINERS.old2007-04-05 21:28:18.0 
+0200
+++ linux-2.6.21-rc5-mm4/MAINTAINERS2007-04-05 21:35:03.0 +0200
@@ -1690,12 +1690,6 @@ P:   H. Peter Anvin
 M: [EMAIL PROTECTED]
 S: Maintained
 
-i810 TCO TIMER WATCHDOG
-P: Nils Faerber
-M: [EMAIL PROTECTED]
-W: http://www.kernelconcepts.de/
-S: Maintained
-
 IA64 (Itanium) PLATFORM
 P: Tony Luck
 M: [EMAIL PROTECTED]
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/Kconfig.old  2007-04-05 
21:23:11.0 +0200
+++ linux-2.6.21-rc5-mm4/drivers/char/watchdog/Kconfig  2007-04-05 
21:23:21.0 +0200
@@ -298,28 +298,6 @@ config I6300ESB_WDT
  To compile this driver as a module, choose M here: the
  module will be called i6300esb.
 
-config I8XX_TCO
-   tristate Intel i8xx TCO Timer/Watchdog
-   depends on WATCHDOG  (X86 || IA64)  PCI
-   default n
-   ---help---
- Hardware driver for the TCO timer built into the Intel 82801
- I/O Controller Hub family.  The TCO (Total Cost of Ownership)
- timer is a watchdog timer that will reboot the machine after
- its second expiration. The expiration time can be configured
- with the heartbeat parameter.
-
- On some motherboards the driver may fail to reset the chipset's
- NO_REBOOT flag which prevents the watchdog from rebooting the
- machine. If this is the case you will get a kernel message like
- failed to reset NO_REBOOT flag, reboot disabled by hardware.
-
- To compile this driver as a module, choose M here: the
- module will be called i8xx_tco.
-
- Note: This driver will be removed in the near future. Please
- use the Intel TCO Timer/Watchdog driver.
-
 config ITCO_WDT
tristate Intel TCO Timer/Watchdog
depends on WATCHDOG  (X86 || IA64)  PCI
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/Makefile.old 2007-04-05 
21:23:45.0 +0200
+++ linux-2.6.21-rc5-mm4/drivers/char/watchdog/Makefile 2007-04-05 
21:23:51.0 +0200
@@ -46,7 +46,6 @@ obj-$(CONFIG_IB700_WDT) += ib700wdt.o
 obj-$(CONFIG_IBMASR) += ibmasr.o
 obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
 obj-$(CONFIG_I6300ESB_WDT) += i6300esb.o
-obj-$(CONFIG_I8XX_TCO) += i8xx_tco.o
 obj-$(CONFIG_ITCO_WDT) += iTCO_wdt.o iTCO_vendor_support.o
 obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
 obj-$(CONFIG_SCx200_WDT) += scx200_wdt.o
--- linux-2.6.21-rc5-mm4/drivers/char/watchdog/i8xx_tco.h   2007-02-04 
19:44:54.0 +0100
+++ /dev/null   2006-09-19 00:45:31.0 +0200
@@ -1,42 +0,0 @@
-/*
- * i8xx_tco:   TCO timer driver for i8xx chipsets
- *
- * (c) Copyright 2000 kernel concepts [EMAIL PROTECTED], All Rights 
Reserved.
- * http://www.kernelconcepts.de
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version
- * 2 of the License, or (at your option) any later version.
- *
- * Neither kernel concepts nor Nils Faerber admit liability nor provide
- * warranty for any of this software. This material is provided
- * AS-IS and at no charge.
- *
- * (c) Copyright 2000  kernel concepts [EMAIL PROTECTED]
- * developed for
- *  Jentro AG, Haar/Munich (Germany)
- *
- * TCO timer driver for i8xx chipsets
- * based on softdog.c by Alan Cox [EMAIL PROTECTED]
- *
- * For history and the complete list of supported I/O Controller Hub's
- * see i8xx_tco.c
- */
-
-
-/*
- * Some address definitions for the TCO
- */
-
-#defineTCOBASE ACPIBASE + 0x60 /* TCO base address 
*/
-#define TCO1_RLD