* [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-06 13:19 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Ezequiel Garcia
Here's the seventh round for IMG PDC Watchdog driver.
The series is based on v3.19-rc3.
Thanks!
Changes from v6:
* Reworked set_timeout function to get rid of the min_delay field.
* Addressed a few comments made by Guenter Roeck.
* Added a description of the clocks to the devicetree binding doc.
Changes from v5:
* Removed unused struct pdc_wdt_dev variables.
* Removed out_wdt label in probe function.
Changes from v4:
* Added warning message if initial timeout value exceeds maximum timeout.
Changes from v3:
* Removed PDC_WD_MAX_DELAY.
* Removed spinlock.
* Removed timeout check inside pdc_wdt_set_timeout function.
* Changed identity to IMG PDC Watchdog.
* Removed pdc_wdt_set_timeout call from the probe function.
* Removed pdc_wdt_shutdown function call from pdc_wdt_remove function
and added pdc_wdt_stop function call instead.
Changes from v2:
* Removed interrupt.h header file.
Changes from v1:
* Arranged all include files in alphabetical order.
* Arranged register field masks/shifts defines next to their
corresponding register
* Registered clock rate-change notifier callback.
* Removed all references to 32 kHz fixed clock rate.
Now the clock rate is read dynamically using clk_get_rate().
* Defined a macro PDC_WD_CONFIG_ENABLE.
* Added code for getting system clock.
* Moved module_param stuffs next to the corresponding variable defines.
* Added code in the function pdc_wdt_remove for stopping watchdog.
* Removed MODULE_ALIAS.
* Removed few unwanted spaces and unwanted blank lines.
Naidu Tellapati (2):
watchdog: ImgTec PDC Watchdog Timer Driver
DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
.../devicetree/bindings/watchdog/imgpdc-wdt.txt | 19 ++
drivers/watchdog/Kconfig | 11 +
drivers/watchdog/Makefile | 1 +
drivers/watchdog/imgpdc_wdt.c | 289 +++++++++++++++++++++
4 files changed, 320 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
create mode 100644 drivers/watchdog/imgpdc_wdt.c
--
2.2.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-06 13:19 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree, Ezequiel Garcia
Here's the seventh round for IMG PDC Watchdog driver.
The series is based on v3.19-rc3.
Thanks!
Changes from v6:
* Reworked set_timeout function to get rid of the min_delay field.
* Addressed a few comments made by Guenter Roeck.
* Added a description of the clocks to the devicetree binding doc.
Changes from v5:
* Removed unused struct pdc_wdt_dev variables.
* Removed out_wdt label in probe function.
Changes from v4:
* Added warning message if initial timeout value exceeds maximum timeout.
Changes from v3:
* Removed PDC_WD_MAX_DELAY.
* Removed spinlock.
* Removed timeout check inside pdc_wdt_set_timeout function.
* Changed identity to IMG PDC Watchdog.
* Removed pdc_wdt_set_timeout call from the probe function.
* Removed pdc_wdt_shutdown function call from pdc_wdt_remove function
and added pdc_wdt_stop function call instead.
Changes from v2:
* Removed interrupt.h header file.
Changes from v1:
* Arranged all include files in alphabetical order.
* Arranged register field masks/shifts defines next to their
corresponding register
* Registered clock rate-change notifier callback.
* Removed all references to 32 kHz fixed clock rate.
Now the clock rate is read dynamically using clk_get_rate().
* Defined a macro PDC_WD_CONFIG_ENABLE.
* Added code for getting system clock.
* Moved module_param stuffs next to the corresponding variable defines.
* Added code in the function pdc_wdt_remove for stopping watchdog.
* Removed MODULE_ALIAS.
* Removed few unwanted spaces and unwanted blank lines.
Naidu Tellapati (2):
watchdog: ImgTec PDC Watchdog Timer Driver
DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
.../devicetree/bindings/watchdog/imgpdc-wdt.txt | 19 ++
drivers/watchdog/Kconfig | 11 +
drivers/watchdog/Makefile | 1 +
drivers/watchdog/imgpdc_wdt.c | 289 +++++++++++++++++++++
4 files changed, 320 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
create mode 100644 drivers/watchdog/imgpdc_wdt.c
--
2.2.1
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v7 1/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-06 13:19 ` Ezequiel Garcia
@ 2015-01-06 13:19 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Naidu Tellapati,
Naidu Tellapati, Jude Abraham, Ezequiel Garcia
From: Naidu Tellapati <naidu.tellapati-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
This commit adds support for ImgTec PowerDown Controller Watchdog Timer.
Reviewed-by: Andrew Bresticker <abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Signed-off-by: Naidu Tellapati <Naidu.Tellapati-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Signed-off-by: Jude Abraham <Jude.Abraham-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
[ezequiel: Minor style fixes]
Signed-off-by: Ezequiel Garcia <ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
---
drivers/watchdog/Kconfig | 11 ++
drivers/watchdog/Makefile | 1 +
drivers/watchdog/imgpdc_wdt.c | 289 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 301 insertions(+)
create mode 100644 drivers/watchdog/imgpdc_wdt.c
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index 08f41ad..0a1396b 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -1235,6 +1235,17 @@ config BCM_KONA_WDT_DEBUG
If in doubt, say 'N'.
+config IMGPDC_WDT
+ tristate "Imagination Technologies PDC Watchdog Timer"
+ depends on HAS_IOMEM
+ depends on METAG || MIPS || COMPILE_TEST
+ help
+ Driver for Imagination Technologies PowerDown Controller
+ Watchdog Timer.
+
+ To compile this driver as a loadable module, choose M here.
+ The module will be called imgpdc_wdt.
+
config LANTIQ_WDT
tristate "Lantiq SoC watchdog"
depends on LANTIQ
diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
index c569ec8..d4dfbb4 100644
--- a/drivers/watchdog/Makefile
+++ b/drivers/watchdog/Makefile
@@ -142,6 +142,7 @@ obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o
octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o
obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o
obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o
+obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o
# PARISC Architecture
diff --git a/drivers/watchdog/imgpdc_wdt.c b/drivers/watchdog/imgpdc_wdt.c
new file mode 100644
index 0000000..c8def68
--- /dev/null
+++ b/drivers/watchdog/imgpdc_wdt.c
@@ -0,0 +1,289 @@
+/*
+ * Imagination Technologies PowerDown Controller Watchdog Timer.
+ *
+ * Copyright (c) 2014 Imagination Technologies Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * Based on drivers/watchdog/sunxi_wdt.c Copyright (c) 2013 Carlo Caione
+ * 2012 Henrik Nordstrom
+ */
+
+#include <linux/clk.h>
+#include <linux/io.h>
+#include <linux/log2.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/slab.h>
+#include <linux/watchdog.h>
+
+/* registers */
+#define PDC_WDT_SOFT_RESET 0x00
+#define PDC_WDT_CONFIG 0x04
+ #define PDC_WDT_CONFIG_ENABLE BIT(31)
+ #define PDC_WDT_CONFIG_DELAY_MASK 0x1f
+
+#define PDC_WDT_TICKLE1 0x08
+#define PDC_WDT_TICKLE1_MAGIC 0xabcd1234
+#define PDC_WDT_TICKLE2 0x0c
+#define PDC_WDT_TICKLE2_MAGIC 0x4321dcba
+
+#define PDC_WDT_TICKLE_STATUS_MASK 0x7
+#define PDC_WDT_TICKLE_STATUS_SHIFT 0
+#define PDC_WDT_TICKLE_STATUS_HRESET 0x0 /* Hard reset */
+#define PDC_WDT_TICKLE_STATUS_TIMEOUT 0x1 /* Timeout */
+#define PDC_WDT_TICKLE_STATUS_TICKLE 0x2 /* Tickled incorrectly */
+#define PDC_WDT_TICKLE_STATUS_SRESET 0x3 /* Soft reset */
+#define PDC_WDT_TICKLE_STATUS_USER 0x4 /* User reset */
+
+/* Timeout values are in seconds */
+#define PDC_WDT_MIN_TIMEOUT 1
+#define PDC_WDT_DEF_TIMEOUT 64
+
+static int heartbeat;
+module_param(heartbeat, int, 0);
+MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. "
+ "(default = " __MODULE_STRING(PDC_WDT_DEF_TIMEOUT) ")");
+
+static bool nowayout = WATCHDOG_NOWAYOUT;
+module_param(nowayout, bool, 0);
+MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started "
+ "(default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")");
+
+struct pdc_wdt_dev {
+ struct watchdog_device wdt_dev;
+ struct clk *wdt_clk;
+ struct clk *sys_clk;
+ void __iomem *base;
+};
+
+static int pdc_wdt_keepalive(struct watchdog_device *wdt_dev)
+{
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ writel(PDC_WDT_TICKLE1_MAGIC, wdt->base + PDC_WDT_TICKLE1);
+ writel(PDC_WDT_TICKLE2_MAGIC, wdt->base + PDC_WDT_TICKLE2);
+
+ return 0;
+}
+
+static int pdc_wdt_stop(struct watchdog_device *wdt_dev)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ val = readl(wdt->base + PDC_WDT_CONFIG);
+ val &= ~PDC_WDT_CONFIG_ENABLE;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ /* Must tickle to finish the stop */
+ pdc_wdt_keepalive(wdt_dev);
+
+ return 0;
+}
+
+static int pdc_wdt_set_timeout(struct watchdog_device *wdt_dev,
+ unsigned int new_timeout)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+ unsigned long clk_rate = clk_get_rate(wdt->wdt_clk);
+
+ wdt->wdt_dev.timeout = new_timeout;
+
+ val = readl(wdt->base + PDC_WDT_CONFIG) & ~PDC_WDT_CONFIG_DELAY_MASK;
+ val |= order_base_2(new_timeout * clk_rate) - 1;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ return 0;
+}
+
+/* Start the watchdog timer (delay should already be set) */
+static int pdc_wdt_start(struct watchdog_device *wdt_dev)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ val = readl(wdt->base + PDC_WDT_CONFIG);
+ val |= PDC_WDT_CONFIG_ENABLE;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ return 0;
+}
+
+static struct watchdog_info pdc_wdt_info = {
+ .identity = "IMG PDC Watchdog",
+ .options = WDIOF_SETTIMEOUT |
+ WDIOF_KEEPALIVEPING |
+ WDIOF_MAGICCLOSE,
+};
+
+static const struct watchdog_ops pdc_wdt_ops = {
+ .owner = THIS_MODULE,
+ .start = pdc_wdt_start,
+ .stop = pdc_wdt_stop,
+ .ping = pdc_wdt_keepalive,
+ .set_timeout = pdc_wdt_set_timeout,
+};
+
+static int pdc_wdt_probe(struct platform_device *pdev)
+{
+ int ret, val;
+ unsigned long clk_rate;
+ struct resource *res;
+ struct pdc_wdt_dev *pdc_wdt;
+
+ pdc_wdt = devm_kzalloc(&pdev->dev, sizeof(*pdc_wdt), GFP_KERNEL);
+ if (!pdc_wdt)
+ return -ENOMEM;
+
+ res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ pdc_wdt->base = devm_ioremap_resource(&pdev->dev, res);
+ if (IS_ERR(pdc_wdt->base))
+ return PTR_ERR(pdc_wdt->base);
+
+ pdc_wdt->sys_clk = devm_clk_get(&pdev->dev, "sys");
+ if (IS_ERR(pdc_wdt->sys_clk)) {
+ dev_err(&pdev->dev, "failed to get the sys clock\n");
+ return PTR_ERR(pdc_wdt->sys_clk);
+ }
+
+ pdc_wdt->wdt_clk = devm_clk_get(&pdev->dev, "wdt");
+ if (IS_ERR(pdc_wdt->wdt_clk)) {
+ dev_err(&pdev->dev, "failed to get the wdt clock\n");
+ return PTR_ERR(pdc_wdt->wdt_clk);
+ }
+
+ ret = clk_prepare_enable(pdc_wdt->sys_clk);
+ if (ret) {
+ dev_err(&pdev->dev, "could not prepare or enable sys clock\n");
+ return ret;
+ }
+
+ ret = clk_prepare_enable(pdc_wdt->wdt_clk);
+ if (ret) {
+ dev_err(&pdev->dev, "could not prepare or enable wdt clock\n");
+ goto disable_sys_clk;
+ }
+
+ /* We use the clock rate to calculate the max timeout */
+ clk_rate = clk_get_rate(pdc_wdt->wdt_clk);
+ if (clk_rate == 0) {
+ dev_err(&pdev->dev, "failed to get clock rate\n");
+ ret = -EINVAL;
+ goto disable_wdt_clk;
+ }
+
+ if (order_base_2(clk_rate) > PDC_WDT_CONFIG_DELAY_MASK + 1) {
+ dev_err(&pdev->dev, "invalid clock rate\n");
+ ret = -EINVAL;
+ goto disable_wdt_clk;
+ }
+
+ if (order_base_2(clk_rate) == 0)
+ pdc_wdt->wdt_dev.min_timeout = PDC_WDT_MIN_TIMEOUT + 1;
+ else
+ pdc_wdt->wdt_dev.min_timeout = PDC_WDT_MIN_TIMEOUT;
+
+ pdc_wdt->wdt_dev.info = &pdc_wdt_info;
+ pdc_wdt->wdt_dev.ops = &pdc_wdt_ops;
+ pdc_wdt->wdt_dev.max_timeout = 1 << PDC_WDT_CONFIG_DELAY_MASK;
+ pdc_wdt->wdt_dev.parent = &pdev->dev;
+
+ ret = watchdog_init_timeout(&pdc_wdt->wdt_dev, heartbeat, &pdev->dev);
+ if (ret < 0) {
+ pdc_wdt->wdt_dev.timeout = pdc_wdt->wdt_dev.max_timeout;
+ dev_warn(&pdev->dev,
+ "Initial timeout out of range! setting max timeout\n");
+ }
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+
+ /* Find what caused the last reset */
+ val = readl(pdc_wdt->base + PDC_WDT_TICKLE1);
+ val = (val & PDC_WDT_TICKLE_STATUS_MASK) >> PDC_WDT_TICKLE_STATUS_SHIFT;
+ switch (val) {
+ case PDC_WDT_TICKLE_STATUS_TICKLE:
+ case PDC_WDT_TICKLE_STATUS_TIMEOUT:
+ pdc_wdt->wdt_dev.bootstatus |= WDIOF_CARDRESET;
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to timeout\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_HRESET:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to hard reset\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_SRESET:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to soft reset\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_USER:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to user reset\n");
+ break;
+ default:
+ dev_info(&pdev->dev,
+ "contains an illegal status code (%08x)\n", val);
+ break;
+ }
+
+ watchdog_set_nowayout(&pdc_wdt->wdt_dev, nowayout);
+
+ platform_set_drvdata(pdev, pdc_wdt);
+ watchdog_set_drvdata(&pdc_wdt->wdt_dev, pdc_wdt);
+
+ ret = watchdog_register_device(&pdc_wdt->wdt_dev);
+ if (ret)
+ goto disable_wdt_clk;
+
+ return 0;
+
+disable_wdt_clk:
+ clk_disable_unprepare(pdc_wdt->wdt_clk);
+disable_sys_clk:
+ clk_disable_unprepare(pdc_wdt->sys_clk);
+ return ret;
+}
+
+static void pdc_wdt_shutdown(struct platform_device *pdev)
+{
+ struct pdc_wdt_dev *pdc_wdt = platform_get_drvdata(pdev);
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+}
+
+static int pdc_wdt_remove(struct platform_device *pdev)
+{
+ struct pdc_wdt_dev *pdc_wdt = platform_get_drvdata(pdev);
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+ watchdog_unregister_device(&pdc_wdt->wdt_dev);
+ clk_disable_unprepare(pdc_wdt->wdt_clk);
+ clk_disable_unprepare(pdc_wdt->sys_clk);
+
+ return 0;
+}
+
+static const struct of_device_id pdc_wdt_match[] = {
+ { .compatible = "img,pdc-wdt" },
+ {}
+};
+MODULE_DEVICE_TABLE(of, pdc_wdt_match);
+
+static struct platform_driver pdc_wdt_driver = {
+ .driver = {
+ .name = "imgpdc-wdt",
+ .of_match_table = pdc_wdt_match,
+ },
+ .probe = pdc_wdt_probe,
+ .remove = pdc_wdt_remove,
+ .shutdown = pdc_wdt_shutdown,
+};
+module_platform_driver(pdc_wdt_driver);
+
+MODULE_AUTHOR("Jude Abraham <Jude.Abraham-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>");
+MODULE_AUTHOR("Naidu Tellapati <Naidu.Tellapati-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>");
+MODULE_DESCRIPTION("Imagination Technologies PDC Watchdog Timer Driver");
+MODULE_LICENSE("GPL v2");
--
2.2.1
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v7 1/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-06 13:19 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree, Naidu Tellapati, Naidu Tellapati,
Jude Abraham, Ezequiel Garcia
From: Naidu Tellapati <naidu.tellapati@gmail.com>
This commit adds support for ImgTec PowerDown Controller Watchdog Timer.
Reviewed-by: Andrew Bresticker <abrestic@chromium.org>
Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com>
Signed-off-by: Jude Abraham <Jude.Abraham@imgtec.com>
[ezequiel: Minor style fixes]
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
---
drivers/watchdog/Kconfig | 11 ++
drivers/watchdog/Makefile | 1 +
drivers/watchdog/imgpdc_wdt.c | 289 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 301 insertions(+)
create mode 100644 drivers/watchdog/imgpdc_wdt.c
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index 08f41ad..0a1396b 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -1235,6 +1235,17 @@ config BCM_KONA_WDT_DEBUG
If in doubt, say 'N'.
+config IMGPDC_WDT
+ tristate "Imagination Technologies PDC Watchdog Timer"
+ depends on HAS_IOMEM
+ depends on METAG || MIPS || COMPILE_TEST
+ help
+ Driver for Imagination Technologies PowerDown Controller
+ Watchdog Timer.
+
+ To compile this driver as a loadable module, choose M here.
+ The module will be called imgpdc_wdt.
+
config LANTIQ_WDT
tristate "Lantiq SoC watchdog"
depends on LANTIQ
diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
index c569ec8..d4dfbb4 100644
--- a/drivers/watchdog/Makefile
+++ b/drivers/watchdog/Makefile
@@ -142,6 +142,7 @@ obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o
octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o
obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o
obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o
+obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o
# PARISC Architecture
diff --git a/drivers/watchdog/imgpdc_wdt.c b/drivers/watchdog/imgpdc_wdt.c
new file mode 100644
index 0000000..c8def68
--- /dev/null
+++ b/drivers/watchdog/imgpdc_wdt.c
@@ -0,0 +1,289 @@
+/*
+ * Imagination Technologies PowerDown Controller Watchdog Timer.
+ *
+ * Copyright (c) 2014 Imagination Technologies Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * Based on drivers/watchdog/sunxi_wdt.c Copyright (c) 2013 Carlo Caione
+ * 2012 Henrik Nordstrom
+ */
+
+#include <linux/clk.h>
+#include <linux/io.h>
+#include <linux/log2.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/slab.h>
+#include <linux/watchdog.h>
+
+/* registers */
+#define PDC_WDT_SOFT_RESET 0x00
+#define PDC_WDT_CONFIG 0x04
+ #define PDC_WDT_CONFIG_ENABLE BIT(31)
+ #define PDC_WDT_CONFIG_DELAY_MASK 0x1f
+
+#define PDC_WDT_TICKLE1 0x08
+#define PDC_WDT_TICKLE1_MAGIC 0xabcd1234
+#define PDC_WDT_TICKLE2 0x0c
+#define PDC_WDT_TICKLE2_MAGIC 0x4321dcba
+
+#define PDC_WDT_TICKLE_STATUS_MASK 0x7
+#define PDC_WDT_TICKLE_STATUS_SHIFT 0
+#define PDC_WDT_TICKLE_STATUS_HRESET 0x0 /* Hard reset */
+#define PDC_WDT_TICKLE_STATUS_TIMEOUT 0x1 /* Timeout */
+#define PDC_WDT_TICKLE_STATUS_TICKLE 0x2 /* Tickled incorrectly */
+#define PDC_WDT_TICKLE_STATUS_SRESET 0x3 /* Soft reset */
+#define PDC_WDT_TICKLE_STATUS_USER 0x4 /* User reset */
+
+/* Timeout values are in seconds */
+#define PDC_WDT_MIN_TIMEOUT 1
+#define PDC_WDT_DEF_TIMEOUT 64
+
+static int heartbeat;
+module_param(heartbeat, int, 0);
+MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. "
+ "(default = " __MODULE_STRING(PDC_WDT_DEF_TIMEOUT) ")");
+
+static bool nowayout = WATCHDOG_NOWAYOUT;
+module_param(nowayout, bool, 0);
+MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started "
+ "(default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")");
+
+struct pdc_wdt_dev {
+ struct watchdog_device wdt_dev;
+ struct clk *wdt_clk;
+ struct clk *sys_clk;
+ void __iomem *base;
+};
+
+static int pdc_wdt_keepalive(struct watchdog_device *wdt_dev)
+{
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ writel(PDC_WDT_TICKLE1_MAGIC, wdt->base + PDC_WDT_TICKLE1);
+ writel(PDC_WDT_TICKLE2_MAGIC, wdt->base + PDC_WDT_TICKLE2);
+
+ return 0;
+}
+
+static int pdc_wdt_stop(struct watchdog_device *wdt_dev)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ val = readl(wdt->base + PDC_WDT_CONFIG);
+ val &= ~PDC_WDT_CONFIG_ENABLE;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ /* Must tickle to finish the stop */
+ pdc_wdt_keepalive(wdt_dev);
+
+ return 0;
+}
+
+static int pdc_wdt_set_timeout(struct watchdog_device *wdt_dev,
+ unsigned int new_timeout)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+ unsigned long clk_rate = clk_get_rate(wdt->wdt_clk);
+
+ wdt->wdt_dev.timeout = new_timeout;
+
+ val = readl(wdt->base + PDC_WDT_CONFIG) & ~PDC_WDT_CONFIG_DELAY_MASK;
+ val |= order_base_2(new_timeout * clk_rate) - 1;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ return 0;
+}
+
+/* Start the watchdog timer (delay should already be set) */
+static int pdc_wdt_start(struct watchdog_device *wdt_dev)
+{
+ unsigned int val;
+ struct pdc_wdt_dev *wdt = watchdog_get_drvdata(wdt_dev);
+
+ val = readl(wdt->base + PDC_WDT_CONFIG);
+ val |= PDC_WDT_CONFIG_ENABLE;
+ writel(val, wdt->base + PDC_WDT_CONFIG);
+
+ return 0;
+}
+
+static struct watchdog_info pdc_wdt_info = {
+ .identity = "IMG PDC Watchdog",
+ .options = WDIOF_SETTIMEOUT |
+ WDIOF_KEEPALIVEPING |
+ WDIOF_MAGICCLOSE,
+};
+
+static const struct watchdog_ops pdc_wdt_ops = {
+ .owner = THIS_MODULE,
+ .start = pdc_wdt_start,
+ .stop = pdc_wdt_stop,
+ .ping = pdc_wdt_keepalive,
+ .set_timeout = pdc_wdt_set_timeout,
+};
+
+static int pdc_wdt_probe(struct platform_device *pdev)
+{
+ int ret, val;
+ unsigned long clk_rate;
+ struct resource *res;
+ struct pdc_wdt_dev *pdc_wdt;
+
+ pdc_wdt = devm_kzalloc(&pdev->dev, sizeof(*pdc_wdt), GFP_KERNEL);
+ if (!pdc_wdt)
+ return -ENOMEM;
+
+ res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ pdc_wdt->base = devm_ioremap_resource(&pdev->dev, res);
+ if (IS_ERR(pdc_wdt->base))
+ return PTR_ERR(pdc_wdt->base);
+
+ pdc_wdt->sys_clk = devm_clk_get(&pdev->dev, "sys");
+ if (IS_ERR(pdc_wdt->sys_clk)) {
+ dev_err(&pdev->dev, "failed to get the sys clock\n");
+ return PTR_ERR(pdc_wdt->sys_clk);
+ }
+
+ pdc_wdt->wdt_clk = devm_clk_get(&pdev->dev, "wdt");
+ if (IS_ERR(pdc_wdt->wdt_clk)) {
+ dev_err(&pdev->dev, "failed to get the wdt clock\n");
+ return PTR_ERR(pdc_wdt->wdt_clk);
+ }
+
+ ret = clk_prepare_enable(pdc_wdt->sys_clk);
+ if (ret) {
+ dev_err(&pdev->dev, "could not prepare or enable sys clock\n");
+ return ret;
+ }
+
+ ret = clk_prepare_enable(pdc_wdt->wdt_clk);
+ if (ret) {
+ dev_err(&pdev->dev, "could not prepare or enable wdt clock\n");
+ goto disable_sys_clk;
+ }
+
+ /* We use the clock rate to calculate the max timeout */
+ clk_rate = clk_get_rate(pdc_wdt->wdt_clk);
+ if (clk_rate == 0) {
+ dev_err(&pdev->dev, "failed to get clock rate\n");
+ ret = -EINVAL;
+ goto disable_wdt_clk;
+ }
+
+ if (order_base_2(clk_rate) > PDC_WDT_CONFIG_DELAY_MASK + 1) {
+ dev_err(&pdev->dev, "invalid clock rate\n");
+ ret = -EINVAL;
+ goto disable_wdt_clk;
+ }
+
+ if (order_base_2(clk_rate) == 0)
+ pdc_wdt->wdt_dev.min_timeout = PDC_WDT_MIN_TIMEOUT + 1;
+ else
+ pdc_wdt->wdt_dev.min_timeout = PDC_WDT_MIN_TIMEOUT;
+
+ pdc_wdt->wdt_dev.info = &pdc_wdt_info;
+ pdc_wdt->wdt_dev.ops = &pdc_wdt_ops;
+ pdc_wdt->wdt_dev.max_timeout = 1 << PDC_WDT_CONFIG_DELAY_MASK;
+ pdc_wdt->wdt_dev.parent = &pdev->dev;
+
+ ret = watchdog_init_timeout(&pdc_wdt->wdt_dev, heartbeat, &pdev->dev);
+ if (ret < 0) {
+ pdc_wdt->wdt_dev.timeout = pdc_wdt->wdt_dev.max_timeout;
+ dev_warn(&pdev->dev,
+ "Initial timeout out of range! setting max timeout\n");
+ }
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+
+ /* Find what caused the last reset */
+ val = readl(pdc_wdt->base + PDC_WDT_TICKLE1);
+ val = (val & PDC_WDT_TICKLE_STATUS_MASK) >> PDC_WDT_TICKLE_STATUS_SHIFT;
+ switch (val) {
+ case PDC_WDT_TICKLE_STATUS_TICKLE:
+ case PDC_WDT_TICKLE_STATUS_TIMEOUT:
+ pdc_wdt->wdt_dev.bootstatus |= WDIOF_CARDRESET;
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to timeout\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_HRESET:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to hard reset\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_SRESET:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to soft reset\n");
+ break;
+ case PDC_WDT_TICKLE_STATUS_USER:
+ dev_info(&pdev->dev,
+ "watchdog module last reset due to user reset\n");
+ break;
+ default:
+ dev_info(&pdev->dev,
+ "contains an illegal status code (%08x)\n", val);
+ break;
+ }
+
+ watchdog_set_nowayout(&pdc_wdt->wdt_dev, nowayout);
+
+ platform_set_drvdata(pdev, pdc_wdt);
+ watchdog_set_drvdata(&pdc_wdt->wdt_dev, pdc_wdt);
+
+ ret = watchdog_register_device(&pdc_wdt->wdt_dev);
+ if (ret)
+ goto disable_wdt_clk;
+
+ return 0;
+
+disable_wdt_clk:
+ clk_disable_unprepare(pdc_wdt->wdt_clk);
+disable_sys_clk:
+ clk_disable_unprepare(pdc_wdt->sys_clk);
+ return ret;
+}
+
+static void pdc_wdt_shutdown(struct platform_device *pdev)
+{
+ struct pdc_wdt_dev *pdc_wdt = platform_get_drvdata(pdev);
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+}
+
+static int pdc_wdt_remove(struct platform_device *pdev)
+{
+ struct pdc_wdt_dev *pdc_wdt = platform_get_drvdata(pdev);
+
+ pdc_wdt_stop(&pdc_wdt->wdt_dev);
+ watchdog_unregister_device(&pdc_wdt->wdt_dev);
+ clk_disable_unprepare(pdc_wdt->wdt_clk);
+ clk_disable_unprepare(pdc_wdt->sys_clk);
+
+ return 0;
+}
+
+static const struct of_device_id pdc_wdt_match[] = {
+ { .compatible = "img,pdc-wdt" },
+ {}
+};
+MODULE_DEVICE_TABLE(of, pdc_wdt_match);
+
+static struct platform_driver pdc_wdt_driver = {
+ .driver = {
+ .name = "imgpdc-wdt",
+ .of_match_table = pdc_wdt_match,
+ },
+ .probe = pdc_wdt_probe,
+ .remove = pdc_wdt_remove,
+ .shutdown = pdc_wdt_shutdown,
+};
+module_platform_driver(pdc_wdt_driver);
+
+MODULE_AUTHOR("Jude Abraham <Jude.Abraham@imgtec.com>");
+MODULE_AUTHOR("Naidu Tellapati <Naidu.Tellapati@imgtec.com>");
+MODULE_DESCRIPTION("Imagination Technologies PDC Watchdog Timer Driver");
+MODULE_LICENSE("GPL v2");
--
2.2.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v7 2/2] DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
2015-01-06 13:19 ` Ezequiel Garcia
@ 2015-01-06 13:19 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Naidu Tellapati,
Naidu Tellapati, Jude Abraham, Ezequiel Garcia
From: Naidu Tellapati <naidu.tellapati-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Add the devicetree binding document for ImgTec PDC Watchdog Timer.
Reviewed-by: Andrew Bresticker <abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Signed-off-by: Naidu Tellapati <Naidu.Tellapati-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Signed-off-by: Jude Abraham <Jude.Abraham-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
---
.../devicetree/bindings/watchdog/imgpdc-wdt.txt | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
diff --git a/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
new file mode 100644
index 0000000..b2fa11f
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
@@ -0,0 +1,19 @@
+*ImgTec PowerDown Controller (PDC) Watchdog Timer (WDT)
+
+Required properties:
+- compatible : Should be "img,pdc-wdt"
+- reg : Should contain WDT registers location and length
+- clocks: Must contain an entry for each entry in clock-names.
+- clock-names: Should contain "wdt" and "sys"; the watchdog counter
+ clock and register interface clock respectively.
+- interrupts : Should contain WDT interrupt
+
+Examples:
+
+watchdog@18102100 {
+ compatible = "img,pdc-wdt";
+ reg = <0x18102100 0x100>;
+ clocks = <&pdc_wdt_clk>, <&sys_clk>;
+ clock-names = "wdt", "sys";
+ interrupts = <0 52 IRQ_TYPE_LEVEL_HIGH>;
+};
--
2.2.1
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v7 2/2] DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
@ 2015-01-06 13:19 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-06 13:19 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree, Naidu Tellapati, Naidu Tellapati,
Jude Abraham, Ezequiel Garcia
From: Naidu Tellapati <naidu.tellapati@gmail.com>
Add the devicetree binding document for ImgTec PDC Watchdog Timer.
Reviewed-by: Andrew Bresticker <abrestic@chromium.org>
Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com>
Signed-off-by: Jude Abraham <Jude.Abraham@gmail.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
---
.../devicetree/bindings/watchdog/imgpdc-wdt.txt | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
diff --git a/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
new file mode 100644
index 0000000..b2fa11f
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt
@@ -0,0 +1,19 @@
+*ImgTec PowerDown Controller (PDC) Watchdog Timer (WDT)
+
+Required properties:
+- compatible : Should be "img,pdc-wdt"
+- reg : Should contain WDT registers location and length
+- clocks: Must contain an entry for each entry in clock-names.
+- clock-names: Should contain "wdt" and "sys"; the watchdog counter
+ clock and register interface clock respectively.
+- interrupts : Should contain WDT interrupt
+
+Examples:
+
+watchdog@18102100 {
+ compatible = "img,pdc-wdt";
+ reg = <0x18102100 0x100>;
+ clocks = <&pdc_wdt_clk>, <&sys_clk>;
+ clock-names = "wdt", "sys";
+ interrupts = <0 52 IRQ_TYPE_LEVEL_HIGH>;
+};
--
2.2.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v7 1/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-06 13:19 ` Ezequiel Garcia
@ 2015-01-13 12:08 ` Guenter Roeck
-1 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-13 12:08 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Naidu Tellapati,
Naidu Tellapati, Jude Abraham
On 01/06/2015 05:19 AM, Ezequiel Garcia wrote:
> From: Naidu Tellapati <naidu.tellapati-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
> This commit adds support for ImgTec PowerDown Controller Watchdog Timer.
>
> Reviewed-by: Andrew Bresticker <abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
> Signed-off-by: Naidu Tellapati <Naidu.Tellapati-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Jude Abraham <Jude.Abraham-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
> [ezequiel: Minor style fixes]
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 1/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-13 12:08 ` Guenter Roeck
0 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-13 12:08 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree, Naidu Tellapati, Naidu Tellapati,
Jude Abraham
On 01/06/2015 05:19 AM, Ezequiel Garcia wrote:
> From: Naidu Tellapati <naidu.tellapati@gmail.com>
>
> This commit adds support for ImgTec PowerDown Controller Watchdog Timer.
>
> Reviewed-by: Andrew Bresticker <abrestic@chromium.org>
> Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com>
> Signed-off-by: Jude Abraham <Jude.Abraham@imgtec.com>
> [ezequiel: Minor style fixes]
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 2/2] DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
2015-01-06 13:19 ` Ezequiel Garcia
@ 2015-01-13 12:08 ` Guenter Roeck
-1 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-13 12:08 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Naidu Tellapati,
Naidu Tellapati, Jude Abraham
On 01/06/2015 05:19 AM, Ezequiel Garcia wrote:
> From: Naidu Tellapati <naidu.tellapati-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
> Add the devicetree binding document for ImgTec PDC Watchdog Timer.
>
> Reviewed-by: Andrew Bresticker <abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
> Signed-off-by: Naidu Tellapati <Naidu.Tellapati-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Jude Abraham <Jude.Abraham-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 2/2] DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation
@ 2015-01-13 12:08 ` Guenter Roeck
0 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-13 12:08 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree, Naidu Tellapati, Naidu Tellapati,
Jude Abraham
On 01/06/2015 05:19 AM, Ezequiel Garcia wrote:
> From: Naidu Tellapati <naidu.tellapati@gmail.com>
>
> Add the devicetree binding document for ImgTec PDC Watchdog Timer.
>
> Reviewed-by: Andrew Bresticker <abrestic@chromium.org>
> Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com>
> Signed-off-by: Jude Abraham <Jude.Abraham@gmail.com>
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-06 13:19 ` Ezequiel Garcia
@ 2015-01-20 11:13 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-20 11:13 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA
Hi Wim,
On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> Here's the seventh round for IMG PDC Watchdog driver.
> The series is based on v3.19-rc3.
>
Guenter has reviewed both patches. Do you think you can review this
soonishly and let me know if you have more comments?
We are near the merge window and I wouldn't like to miss it this time.
Thanks a lot!
--
Ezequiel
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-20 11:13 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-20 11:13 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree
Hi Wim,
On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> Here's the seventh round for IMG PDC Watchdog driver.
> The series is based on v3.19-rc3.
>
Guenter has reviewed both patches. Do you think you can review this
soonishly and let me know if you have more comments?
We are near the merge window and I wouldn't like to miss it this time.
Thanks a lot!
--
Ezequiel
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-20 13:59 ` Guenter Roeck
@ 2015-01-20 13:59 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-20 13:59 UTC (permalink / raw)
To: Guenter Roeck, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA
On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>> Hi Wim,
>>
>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>> Here's the seventh round for IMG PDC Watchdog driver.
>>> The series is based on v3.19-rc3.
>>>
>>
>> Guenter has reviewed both patches. Do you think you can review this
>> soonishly and let me know if you have more comments?
>>
>
> I'll send a pull request to Wim either this or next week.
>
Great. Let me know if you need anything from me.
Thanks a lot,
--
Ezequiel
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-20 13:59 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-20 13:59 UTC (permalink / raw)
To: Guenter Roeck, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree
On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>> Hi Wim,
>>
>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>> Here's the seventh round for IMG PDC Watchdog driver.
>>> The series is based on v3.19-rc3.
>>>
>>
>> Guenter has reviewed both patches. Do you think you can review this
>> soonishly and let me know if you have more comments?
>>
>
> I'll send a pull request to Wim either this or next week.
>
Great. Let me know if you need anything from me.
Thanks a lot,
--
Ezequiel
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-20 11:13 ` Ezequiel Garcia
@ 2015-01-20 13:59 ` Guenter Roeck
-1 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-20 13:59 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA
On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
> Hi Wim,
>
> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>> Here's the seventh round for IMG PDC Watchdog driver.
>> The series is based on v3.19-rc3.
>>
>
> Guenter has reviewed both patches. Do you think you can review this
> soonishly and let me know if you have more comments?
>
I'll send a pull request to Wim either this or next week.
Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-20 13:59 ` Guenter Roeck
0 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-20 13:59 UTC (permalink / raw)
To: Ezequiel Garcia, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree
On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
> Hi Wim,
>
> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>> Here's the seventh round for IMG PDC Watchdog driver.
>> The series is based on v3.19-rc3.
>>
>
> Guenter has reviewed both patches. Do you think you can review this
> soonishly and let me know if you have more comments?
>
I'll send a pull request to Wim either this or next week.
Guenter
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-20 13:59 ` Guenter Roeck
@ 2015-01-28 18:39 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-28 18:39 UTC (permalink / raw)
To: Guenter Roeck, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA
On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>> Hi Wim,
>>
>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>> Here's the seventh round for IMG PDC Watchdog driver.
>>> The series is based on v3.19-rc3.
>>>
>>
>> Guenter has reviewed both patches. Do you think you can review this
>> soonishly and let me know if you have more comments?
>>
>
> I'll send a pull request to Wim either this or next week.
>
Guenter,
I hate to be so spammy here. Do you still plan to send a pull request?
Next week is -rc7, so I'm wondering if there's any way we can be on time
for v3.20.
--
Ezequiel
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-28 18:39 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-28 18:39 UTC (permalink / raw)
To: Guenter Roeck, Wim Van Sebroeck, James Hartley,
Andrew Bresticker, James Hogan
Cc: linux-watchdog, devicetree
On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>> Hi Wim,
>>
>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>> Here's the seventh round for IMG PDC Watchdog driver.
>>> The series is based on v3.19-rc3.
>>>
>>
>> Guenter has reviewed both patches. Do you think you can review this
>> soonishly and let me know if you have more comments?
>>
>
> I'll send a pull request to Wim either this or next week.
>
Guenter,
I hate to be so spammy here. Do you still plan to send a pull request?
Next week is -rc7, so I'm wondering if there's any way we can be on time
for v3.20.
--
Ezequiel
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-28 18:57 ` Guenter Roeck
@ 2015-01-28 18:56 ` Ezequiel Garcia
-1 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-28 18:56 UTC (permalink / raw)
To: Guenter Roeck
Cc: Wim Van Sebroeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 01/28/2015 03:57 PM, Guenter Roeck wrote:
> On Wed, Jan 28, 2015 at 03:39:22PM -0300, Ezequiel Garcia wrote:
>> On 01/20/2015 10:59 AM, Guenter Roeck wrote:
>>> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>>>> Hi Wim,
>>>>
>>>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>>>> Here's the seventh round for IMG PDC Watchdog driver.
>>>>> The series is based on v3.19-rc3.
>>>>>
>>>>
>>>> Guenter has reviewed both patches. Do you think you can review this
>>>> soonishly and let me know if you have more comments?
>>>>
>>>
>>> I'll send a pull request to Wim either this or next week.
>>>
>>
>> Guenter,
>>
>> I hate to be so spammy here. Do you still plan to send a pull request?
>>
> Yes. Sorry for the delay.
>
OK, no problem, was just checking.
Thanks a lot for taking care of this!
--
Ezequiel
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-28 18:56 ` Ezequiel Garcia
0 siblings, 0 replies; 24+ messages in thread
From: Ezequiel Garcia @ 2015-01-28 18:56 UTC (permalink / raw)
To: Guenter Roeck
Cc: Wim Van Sebroeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog, devicetree
On 01/28/2015 03:57 PM, Guenter Roeck wrote:
> On Wed, Jan 28, 2015 at 03:39:22PM -0300, Ezequiel Garcia wrote:
>> On 01/20/2015 10:59 AM, Guenter Roeck wrote:
>>> On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
>>>> Hi Wim,
>>>>
>>>> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
>>>>> Here's the seventh round for IMG PDC Watchdog driver.
>>>>> The series is based on v3.19-rc3.
>>>>>
>>>>
>>>> Guenter has reviewed both patches. Do you think you can review this
>>>> soonishly and let me know if you have more comments?
>>>>
>>>
>>> I'll send a pull request to Wim either this or next week.
>>>
>>
>> Guenter,
>>
>> I hate to be so spammy here. Do you still plan to send a pull request?
>>
> Yes. Sorry for the delay.
>
OK, no problem, was just checking.
Thanks a lot for taking care of this!
--
Ezequiel
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-28 18:39 ` Ezequiel Garcia
@ 2015-01-28 18:57 ` Guenter Roeck
-1 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-28 18:57 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Wim Van Sebroeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Wed, Jan 28, 2015 at 03:39:22PM -0300, Ezequiel Garcia wrote:
> On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> > On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
> >> Hi Wim,
> >>
> >> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> >>> Here's the seventh round for IMG PDC Watchdog driver.
> >>> The series is based on v3.19-rc3.
> >>>
> >>
> >> Guenter has reviewed both patches. Do you think you can review this
> >> soonishly and let me know if you have more comments?
> >>
> >
> > I'll send a pull request to Wim either this or next week.
> >
>
> Guenter,
>
> I hate to be so spammy here. Do you still plan to send a pull request?
>
Yes. Sorry for the delay.
Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-01-28 18:57 ` Guenter Roeck
0 siblings, 0 replies; 24+ messages in thread
From: Guenter Roeck @ 2015-01-28 18:57 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Wim Van Sebroeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog, devicetree
On Wed, Jan 28, 2015 at 03:39:22PM -0300, Ezequiel Garcia wrote:
> On 01/20/2015 10:59 AM, Guenter Roeck wrote:
> > On 01/20/2015 03:13 AM, Ezequiel Garcia wrote:
> >> Hi Wim,
> >>
> >> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> >>> Here's the seventh round for IMG PDC Watchdog driver.
> >>> The series is based on v3.19-rc3.
> >>>
> >>
> >> Guenter has reviewed both patches. Do you think you can review this
> >> soonishly and let me know if you have more comments?
> >>
> >
> > I'll send a pull request to Wim either this or next week.
> >
>
> Guenter,
>
> I hate to be so spammy here. Do you still plan to send a pull request?
>
Yes. Sorry for the delay.
Guenter
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
2015-01-28 18:39 ` Ezequiel Garcia
@ 2015-02-02 22:37 ` Wim Van Sebroeck
-1 siblings, 0 replies; 24+ messages in thread
From: Wim Van Sebroeck @ 2015-02-02 22:37 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Guenter Roeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA
Hi Ezequiel,
> >> Hi Wim,
> >>
> >> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> >>> Here's the seventh round for IMG PDC Watchdog driver.
> >>> The series is based on v3.19-rc3.
> >>>
> >>
> >> Guenter has reviewed both patches. Do you think you can review this
> >> soonishly and let me know if you have more comments?
> >>
> >
> > I'll send a pull request to Wim either this or next week.
> >
>
> Guenter,
>
> I hate to be so spammy here. Do you still plan to send a pull request?
>
> Next week is -rc7, so I'm wondering if there's any way we can be on time
> for v3.20.
It has been added to linux-watchdog-next and will thus be on time for v3.20.
Kind regards,
Wim.
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver
@ 2015-02-02 22:37 ` Wim Van Sebroeck
0 siblings, 0 replies; 24+ messages in thread
From: Wim Van Sebroeck @ 2015-02-02 22:37 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Guenter Roeck, James Hartley, Andrew Bresticker, James Hogan,
linux-watchdog, devicetree
Hi Ezequiel,
> >> Hi Wim,
> >>
> >> On 01/06/2015 10:19 AM, Ezequiel Garcia wrote:
> >>> Here's the seventh round for IMG PDC Watchdog driver.
> >>> The series is based on v3.19-rc3.
> >>>
> >>
> >> Guenter has reviewed both patches. Do you think you can review this
> >> soonishly and let me know if you have more comments?
> >>
> >
> > I'll send a pull request to Wim either this or next week.
> >
>
> Guenter,
>
> I hate to be so spammy here. Do you still plan to send a pull request?
>
> Next week is -rc7, so I'm wondering if there's any way we can be on time
> for v3.20.
It has been added to linux-watchdog-next and will thus be on time for v3.20.
Kind regards,
Wim.
^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2015-02-02 22:37 UTC | newest]
Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-06 13:19 [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver Ezequiel Garcia
2015-01-06 13:19 ` Ezequiel Garcia
[not found] ` <1420550375-22867-1-git-send-email-ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
2015-01-06 13:19 ` [PATCH v7 1/2] " Ezequiel Garcia
2015-01-06 13:19 ` Ezequiel Garcia
[not found] ` <1420550375-22867-2-git-send-email-ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
2015-01-13 12:08 ` Guenter Roeck
2015-01-13 12:08 ` Guenter Roeck
2015-01-06 13:19 ` [PATCH v7 2/2] DT: watchdog: Add ImgTec PDC Watchdog Timer binding documentation Ezequiel Garcia
2015-01-06 13:19 ` Ezequiel Garcia
[not found] ` <1420550375-22867-3-git-send-email-ezequiel.garcia-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
2015-01-13 12:08 ` Guenter Roeck
2015-01-13 12:08 ` Guenter Roeck
2015-01-20 11:13 ` [PATCH v7 0/2] watchdog: ImgTec PDC Watchdog Timer Driver Ezequiel Garcia
2015-01-20 11:13 ` Ezequiel Garcia
[not found] ` <54BE383F.6050209-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
2015-01-20 13:59 ` Guenter Roeck
2015-01-20 13:59 ` Guenter Roeck
[not found] ` <54BE5F50.9050900-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2015-01-20 13:59 ` Ezequiel Garcia
2015-01-20 13:59 ` Ezequiel Garcia
2015-01-28 18:39 ` Ezequiel Garcia
2015-01-28 18:39 ` Ezequiel Garcia
[not found] ` <54C92CDA.8030500-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
2015-01-28 18:57 ` Guenter Roeck
2015-01-28 18:57 ` Guenter Roeck
[not found] ` <20150128185711.GA6730-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2015-01-28 18:56 ` Ezequiel Garcia
2015-01-28 18:56 ` Ezequiel Garcia
2015-02-02 22:37 ` Wim Van Sebroeck
2015-02-02 22:37 ` Wim Van Sebroeck
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.