diff mbox

[RFC,12/16] ARM: watchdog: make mpcore_wdt independant from asm/smp_twd.h

Message ID 1308251204-16719-13-git-send-email-marc.zyngier@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Marc Zyngier June 16, 2011, 7:06 p.m. UTC
As the life expectancy of asm/smp_twd.h seems rather limited,
move the few defines concerning mpcore_wdt to the driver itself.

Change the dependency to ARM_SMP_TWD as well.

Cc: Wim Van Sebroeck <wim@iguana.be>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
---
 drivers/watchdog/Kconfig      |    2 +-
 drivers/watchdog/mpcore_wdt.c |    7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

Comments

Wim Van Sebroeck June 17, 2011, 7:08 a.m. UTC | #1
Hi Marc,

> As the life expectancy of asm/smp_twd.h seems rather limited,
> move the few defines concerning mpcore_wdt to the driver itself.
> 
> Change the dependency to ARM_SMP_TWD as well.
> 
> Cc: Wim Van Sebroeck <wim@iguana.be>
> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>

Signed-off-by: Wim Van Sebroeck <wim@iguana.be>

I presume you Will keep this patch together with the other patches?

Kind regards,
Wim.
Marc Zyngier June 17, 2011, 8:04 a.m. UTC | #2
On 17/06/11 08:08, Wim Van Sebroeck wrote:
> Hi Marc,
> 
>> As the life expectancy of asm/smp_twd.h seems rather limited,
>> move the few defines concerning mpcore_wdt to the driver itself.
>>
>> Change the dependency to ARM_SMP_TWD as well.
>>
>> Cc: Wim Van Sebroeck <wim@iguana.be>
>> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> 
> Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
> 
> I presume you Will keep this patch together with the other patches?

Most probably, though it is fairly independent from the whole series.

Thanks,

	M.
diff mbox

Patch

diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index c70f82b..55647ac 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -197,7 +197,7 @@  config DW_WATCHDOG
 
 config MPCORE_WATCHDOG
 	tristate "MPcore watchdog"
-	depends on HAVE_ARM_TWD
+	depends on ARM_SMP_TWD
 	help
 	  Watchdog timer embedded into the MPcore system.
 
diff --git a/drivers/watchdog/mpcore_wdt.c b/drivers/watchdog/mpcore_wdt.c
index 2b4af22..f638206 100644
--- a/drivers/watchdog/mpcore_wdt.c
+++ b/drivers/watchdog/mpcore_wdt.c
@@ -33,7 +33,12 @@ 
 #include <linux/slab.h>
 #include <linux/io.h>
 
-#include <asm/smp_twd.h>
+#define TWD_WDOG_LOAD			0x20
+#define TWD_WDOG_COUNTER		0x24
+#define TWD_WDOG_CONTROL		0x28
+#define TWD_WDOG_INTSTAT		0x2C
+#define TWD_WDOG_RESETSTAT		0x30
+#define TWD_WDOG_DISABLE		0x34
 
 struct mpcore_wdt {
 	unsigned long	timer_alive;