On Sat, Aug 12, 2017 at 08:43:15PM +0200, Linus Walleij wrote:
> These two drivers is for the same hardware, the only difference
> is that Gemini uses the EXTCLK @5MHz by default (it also works
> fine using PCLK) while the MOXA ART is using the PCLK to clock
> the watchdog.
>
> Delete the old MOXA ART driver and augment the FTWDT010 to
> probe on this platform too, fetching the operating frequency
> from the PCLK. Select the driver by default for ARCH_MOXART
> to make a smooth transition of the platform.
>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> drivers/watchdog/Kconfig | 11 +--
> drivers/watchdog/Makefile | 1 -
> drivers/watchdog/ftwdt010_wdt.c | 1 +
> drivers/watchdog/moxart_wdt.c | 178 ----------------------------------------
> 4 files changed, 2 insertions(+), 189 deletions(-)
> delete mode 100644 drivers/watchdog/moxart_wdt.c
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index fd44a542036a..beef6bb5c6d9 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -326,6 +326,7 @@ config FTWDT010_WATCHDOG
> depends on ARM || COMPILE_TEST
> select WATCHDOG_CORE
> default ARCH_GEMINI
> + default ARCH_MOXART
Can one have more than one default statement ? What is the semantics ?
Just making sure that it isn't "last one wins".
> help
> Say Y here if to include support for the Faraday Technology
> FTWDT010 watchdog timer embedded in the Cortina Systems Gemini
> @@ -609,16 +610,6 @@ config RETU_WATCHDOG
> To compile this driver as a module, choose M here: the
> module will be called retu_wdt.
>
> -config MOXART_WDT
> - tristate "MOXART watchdog"
> - depends on ARCH_MOXART || COMPILE_TEST
> - help
> - Say Y here to include Watchdog timer support for the watchdog
> - existing on the MOXA ART SoC series platforms.
> -
> - To compile this driver as a module, choose M here: the
> - module will be called moxart_wdt.
> -
> config SIRFSOC_WATCHDOG
> tristate "SiRFSOC watchdog"
> depends on ARCH_SIRF || COMPILE_TEST
> diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
> index df91d7db821c..fcab71f0a1c7 100644
> --- a/drivers/watchdog/Makefile
> +++ b/drivers/watchdog/Makefile
> @@ -68,7 +68,6 @@ obj-$(CONFIG_IMX2_WDT) += imx2_wdt.o
> obj-$(CONFIG_UX500_WATCHDOG) += ux500_wdt.o
> obj-$(CONFIG_RETU_WATCHDOG) += retu_wdt.o
> obj-$(CONFIG_BCM2835_WDT) += bcm2835_wdt.o
> -obj-$(CONFIG_MOXART_WDT) += moxart_wdt.o
> obj-$(CONFIG_SIRFSOC_WATCHDOG) += sirfsoc_wdt.o
> obj-$(CONFIG_ST_LPC_WATCHDOG) += st_lpc_wdt.o
> obj-$(CONFIG_QCOM_WDT) += qcom-wdt.o
> diff --git a/drivers/watchdog/ftwdt010_wdt.c b/drivers/watchdog/ftwdt010_wdt.c
> index 7fc865eea28d..912b55e67110 100644
> --- a/drivers/watchdog/ftwdt010_wdt.c
> +++ b/drivers/watchdog/ftwdt010_wdt.c
> @@ -298,6 +298,7 @@ static const struct dev_pm_ops ftwdt010_wdt_dev_pm_ops = {
> static const struct of_device_id ftwdt010_wdt_match[] = {
> { .compatible = "faraday,ftwdt010" },
> { .compatible = "cortina,gemini-watchdog" },
> + { .compatible = "moxa,moxart-watchdog" },
> {},
> };
> MODULE_DEVICE_TABLE(of, ftwdt010_wdt_match);
> diff --git a/drivers/watchdog/moxart_wdt.c b/drivers/watchdog/moxart_wdt.c
> deleted file mode 100644
> index 2c4a73d1e214..000000000000
> --- a/drivers/watchdog/moxart_wdt.c
> +++ /dev/null
> @@ -1,178 +0,0 @@
> -/*
> - * MOXA ART SoCs watchdog driver.
> - *
> - * Copyright (C) 2013 Jonas Jensen
> - *
> - * Jonas Jensen <jonas.jensen@xxxxxxxxx>
> - *
> - * This file is licensed under the terms of the GNU General Public
> - * License version 2. This program is licensed "as is" without any
> - * warranty of any kind, whether express or implied.
> - */
> -
> -#include <linux/clk.h>
> -#include <linux/io.h>
> -#include <linux/module.h>
> -#include <linux/err.h>
> -#include <linux/kernel.h>
> -#include <linux/platform_device.h>
> -#include <linux/watchdog.h>
> -#include <linux/moduleparam.h>
> -
> -#define REG_COUNT 0x4
> -#define REG_MODE 0x8
> -#define REG_ENABLE 0xC
> -
> -struct moxart_wdt_dev {
> - struct watchdog_device dev;
> - void __iomem *base;
> - unsigned int clock_frequency;
> -};
> -
> -static int heartbeat;
> -
> -static int moxart_wdt_restart(struct watchdog_device *wdt_dev,
> - unsigned long action, void *data)
> -{
> - struct moxart_wdt_dev *moxart_wdt = watchdog_get_drvdata(wdt_dev);
> -
> - writel(1, moxart_wdt->base + REG_COUNT);
> - writel(0x5ab9, moxart_wdt->base + REG_MODE);
> - writel(0x03, moxart_wdt->base + REG_ENABLE);
> -
> - return 0;
> -}
> -
> -static int moxart_wdt_stop(struct watchdog_device *wdt_dev)
> -{
> - struct moxart_wdt_dev *moxart_wdt = watchdog_get_drvdata(wdt_dev);
> -
> - writel(0, moxart_wdt->base + REG_ENABLE);
> -
> - return 0;
> -}
> -
> -static int moxart_wdt_start(struct watchdog_device *wdt_dev)
> -{
> - struct moxart_wdt_dev *moxart_wdt = watchdog_get_drvdata(wdt_dev);
> -
> - writel(moxart_wdt->clock_frequency * wdt_dev->timeout,
> - moxart_wdt->base + REG_COUNT);
> - writel(0x5ab9, moxart_wdt->base + REG_MODE);
> - writel(0x03, moxart_wdt->base + REG_ENABLE);
> -
> - return 0;
> -}
> -
> -static int moxart_wdt_set_timeout(struct watchdog_device *wdt_dev,
> - unsigned int timeout)
> -{
> - wdt_dev->timeout = timeout;
> -
> - return 0;
> -}
> -
> -static const struct watchdog_info moxart_wdt_info = {
> - .identity = "moxart-wdt",
> - .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING |
> - WDIOF_MAGICCLOSE,
> -};
> -
> -static const struct watchdog_ops moxart_wdt_ops = {
> - .owner = THIS_MODULE,
> - .start = moxart_wdt_start,
> - .stop = moxart_wdt_stop,
> - .set_timeout = moxart_wdt_set_timeout,
> - .restart = moxart_wdt_restart,
> -};
> -
> -static int moxart_wdt_probe(struct platform_device *pdev)
> -{
> - struct moxart_wdt_dev *moxart_wdt;
> - struct device *dev = &pdev->dev;
> - struct device_node *node = dev->of_node;
> - struct resource *res;
> - struct clk *clk;
> - int err;
> - unsigned int max_timeout;
> - bool nowayout = WATCHDOG_NOWAYOUT;
> -
> - moxart_wdt = devm_kzalloc(dev, sizeof(*moxart_wdt), GFP_KERNEL);
> - if (!moxart_wdt)
> - return -ENOMEM;
> -
> - platform_set_drvdata(pdev, moxart_wdt);
> -
> - res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> - moxart_wdt->base = devm_ioremap_resource(dev, res);
> - if (IS_ERR(moxart_wdt->base))
> - return PTR_ERR(moxart_wdt->base);
> -
> - clk = of_clk_get(node, 0);
> - if (IS_ERR(clk)) {
> - pr_err("%s: of_clk_get failed\n", __func__);
> - return PTR_ERR(clk);
> - }
> -
> - moxart_wdt->clock_frequency = clk_get_rate(clk);
> - if (moxart_wdt->clock_frequency == 0) {
> - pr_err("%s: incorrect clock frequency\n", __func__);
> - return -EINVAL;
> - }
> -
> - max_timeout = UINT_MAX / moxart_wdt->clock_frequency;
> -
> - moxart_wdt->dev.info = &moxart_wdt_info;
> - moxart_wdt->dev.ops = &moxart_wdt_ops;
> - moxart_wdt->dev.timeout = max_timeout;
> - moxart_wdt->dev.min_timeout = 1;
> - moxart_wdt->dev.max_timeout = max_timeout;
> - moxart_wdt->dev.parent = dev;
> -
> - watchdog_init_timeout(&moxart_wdt->dev, heartbeat, dev);
> - watchdog_set_nowayout(&moxart_wdt->dev, nowayout);
> - watchdog_set_restart_priority(&moxart_wdt->dev, 128);
> -
> - watchdog_set_drvdata(&moxart_wdt->dev, moxart_wdt);
> -
> - err = watchdog_register_device(&moxart_wdt->dev);
> - if (err)
> - return err;
> -
> - dev_dbg(dev, "Watchdog enabled (heartbeat=%d sec, nowayout=%d)\n",
> - moxart_wdt->dev.timeout, nowayout);
> -
> - return 0;
> -}
> -
> -static int moxart_wdt_remove(struct platform_device *pdev)
> -{
> - struct moxart_wdt_dev *moxart_wdt = platform_get_drvdata(pdev);
> -
> - moxart_wdt_stop(&moxart_wdt->dev);
> -
> - return 0;
> -}
> -
> -static const struct of_device_id moxart_watchdog_match[] = {
> - { .compatible = "moxa,moxart-watchdog" },
> - { },
> -};
> -MODULE_DEVICE_TABLE(of, moxart_watchdog_match);
> -
> -static struct platform_driver moxart_wdt_driver = {
> - .probe = moxart_wdt_probe,
> - .remove = moxart_wdt_remove,
> - .driver = {
> - .name = "moxart-watchdog",
> - .of_match_table = moxart_watchdog_match,
> - },
> -};
> -module_platform_driver(moxart_wdt_driver);
> -
> -module_param(heartbeat, int, 0);
> -MODULE_PARM_DESC(heartbeat, "Watchdog heartbeat in seconds");
> -
> -MODULE_DESCRIPTION("MOXART watchdog driver");
> -MODULE_LICENSE("GPL");
> -MODULE_AUTHOR("Jonas Jensen <jonas.jensen@xxxxxxxxx>");
> --
> 2.13.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
