In order to support other SoCs, it's needed to have a different enabled()
implementation for each SoC. This commit adds no functionality, and it
consists of preparation work.

Signed-off-by: Ezequiel Garcia <ezequiel.gar...@free-electrons.com>
---
 drivers/watchdog/orion_wdt.c | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/drivers/watchdog/orion_wdt.c b/drivers/watchdog/orion_wdt.c
index c3ee305..8fb8e65 100644
--- a/drivers/watchdog/orion_wdt.c
+++ b/drivers/watchdog/orion_wdt.c
@@ -58,6 +58,7 @@ struct orion_watchdog_data {
        int rstout_enable_bit;
        int (*clock_init)(struct platform_device *,
                          struct orion_watchdog *);
+       int (*enabled)(struct orion_watchdog *);
        int (*start)(struct watchdog_device *);
        int (*stop)(struct watchdog_device *);
 };
@@ -224,7 +225,7 @@ static int orion_wdt_stop(struct watchdog_device *wdt_dev)
        return dev->data->stop(wdt_dev);
 }
 
-static int orion_wdt_enabled(struct orion_watchdog *dev)
+static int orion_enabled(struct orion_watchdog *dev)
 {
        bool enabled, running;
 
@@ -234,6 +235,13 @@ static int orion_wdt_enabled(struct orion_watchdog *dev)
        return enabled && running;
 }
 
+static int orion_wdt_enabled(struct watchdog_device *wdt_dev)
+{
+       struct orion_watchdog *dev = watchdog_get_drvdata(wdt_dev);
+
+       return dev->data->enabled(dev);
+}
+
 static unsigned int orion_wdt_get_timeleft(struct watchdog_device *wdt_dev)
 {
        struct orion_watchdog *dev = watchdog_get_drvdata(wdt_dev);
@@ -295,6 +303,7 @@ static const struct orion_watchdog_data orion_data = {
        .wdt_enable_bit = BIT(4),
        .wdt_counter_offset = 0x24,
        .clock_init = orion_wdt_clock_init,
+       .enabled = orion_enabled,
        .start = orion_start,
        .stop = orion_stop,
 };
@@ -304,6 +313,7 @@ static const struct orion_watchdog_data armada370_data = {
        .wdt_enable_bit = BIT(8),
        .wdt_counter_offset = 0x34,
        .clock_init = armada370_wdt_clock_init,
+       .enabled = orion_enabled,
        .start = armada370_start,
        .stop = armada370_stop,
 };
@@ -313,6 +323,7 @@ static const struct orion_watchdog_data armadaxp_data = {
        .wdt_enable_bit = BIT(8),
        .wdt_counter_offset = 0x34,
        .clock_init = armadaxp_wdt_clock_init,
+       .enabled = orion_enabled,
        .start = armada370_start,
        .stop = armada370_stop,
 };
@@ -410,7 +421,7 @@ static int orion_wdt_probe(struct platform_device *pdev)
         * removed and re-insterted, or if the bootloader explicitly
         * set a running watchdog before booting the kernel.
         */
-       if (!orion_wdt_enabled(dev))
+       if (!orion_wdt_enabled(&dev->wdt))
                orion_wdt_stop(&dev->wdt);
 
        /* Request the IRQ only after the watchdog is disabled */
-- 
1.8.1.5

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to