Hmmm .. yes, reading the description again, it doesn't really describe well+ data->wdt_dev.max_hw_heartbeat_ms = GXBB_WDT_TCNT_SETUP_MASK;
+ data->wdt_dev.min_hw_heartbeat_ms = 1;
Does the device require a minimum time between heartbeats ?
Just asking, because you violate it yourself below.
If you want to set the minimum timeout, that would be min_timeout.
Ok, these values are not very clear actually.
+ data->wdt_dev.timeout = DEFAULT_TIMEOUT;
+ watchdog_set_drvdata(&data->wdt_dev, data);
+
+ watchdog_init_timeout(&data->wdt_dev, DEFAULT_TIMEOUT, &pdev->dev);
+
DEFAULT_TIMEOUT is unnecessary here. Since the default timeout is already set,
it makes the call useless.
+ ret = watchdog_register_device(&data->wdt_dev);
+ if (ret)
+ return ret;
+
+ /* Setup with 1ms timebase */
+ writel(((clk_get_rate(data->clk) / 1000) & GXBB_WDT_CTRL_DIV_MASK) |
+ GXBB_WDT_CTRL_EE_RESET |
+ GXBB_WDT_CTRL_CLK_EN |
+ GXBB_WDT_CTRL_CLKDIV_EN,
+ data->reg_base + GXBB_WDT_CTRL_REG);
+ meson_gxbb_wdt_set_timeout(&data->wdt_dev, data->wdt_dev.timeout);
set_timeout already calls the ping function. Also, the functions should be called
prior to watchdog registration to avoid race conditions.
+ meson_gxbb_wdt_ping(&data->wdt_dev);
This is unusual. If the watchdog can be already running, it might make sense
to tell the core about it (set WDOG_HW_RUNNING in the status field), so it
can send heartbeats until user space opens the device.
Yes, since meson_gxbb_wdt_set_timeout() already ping, this is useless.