On 8/6/19 8:49 PM, Robert Hancock wrote:
On 2019-08-06 11:11 a.m., Marek Vasut wrote:
On 8/6/19 7:05 PM, Robert Hancock wrote:
The Linux imx2_wdt driver uses a fsl,ext-reset-output boolean in the device tree to specify whether the board design should use the external reset instead of the internal reset. Use this boolean to determine which mode to use rather than using external reset unconditionally.
For the legacy non-DM mode, the external reset is always used in order to maintain the previous behavior.
Signed-off-by: Robert Hancock hancock@sedsystems.ca
[...]
@@ -124,6 +129,8 @@ static int imx_wdt_probe(struct udevice *dev) if (!priv->base) return -ENOENT;
- priv->ext_reset = dev_read_bool(dev, "fsl,ext-reset-output");
Do we need a vendor-specific, undocumented, DT property ?
It is documented in Linux in Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt:
- fsl,ext-reset-output: If present the watchdog device is configured to assert its external reset (WDOG_B) instead of issuing a software reset.
I'm not aware of anything non-vendor-specific defined for this.
Aha, then please ignore my comment, thanks for clarifying.