[U-Boot] [PATCH 1/2] watchdog: imx: Add DT ext-reset handling
Robert Hancock
hancock at sedsystems.ca
Tue Aug 6 18:49:43 UTC 2019
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 at 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.
--
Robert Hancock
Senior Software Developer
SED Systems, a division of Calian Ltd.
Email: hancock at sedsystems.ca
More information about the U-Boot
mailing list