| Message ID | 20200123190505.18809-1-walter.lozano@collabora.com |
|---|---|
| State | Accepted |
| Commit | ed666fb12910f46cff73a3869058ea59aef4261b |
| Delegated to: | Stefan Roese |
| Headers | show |
| Series | watchdog: Use dev_read only if OF_PLATDATA is not enabled | expand |
On Thu, 23 Jan 2020 at 12:05, Walter Lozano <walter.lozano@collabora.com> wrote: > > Currently watchdog tries to use dev_read_u32_default to get timeout > configuration in case OF_CONTROL is enabled. However, if SPL is > built with OF_PLATDATA this has no sense as there is no device tree. > > This patch fixes this issue by only use dev_read_u32_default if OF_CONTROL > is enabled but OF_PLATDATA is not. > > Signed-off-by: Walter Lozano <walter.lozano@collabora.com> > --- > include/wdt.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > Reviewed-by: Simon Glass <sjg@chromium.org>
On 23.01.20 20:05, Walter Lozano wrote: > Currently watchdog tries to use dev_read_u32_default to get timeout > configuration in case OF_CONTROL is enabled. However, if SPL is > built with OF_PLATDATA this has no sense as there is no device tree. > > This patch fixes this issue by only use dev_read_u32_default if OF_CONTROL > is enabled but OF_PLATDATA is not. > > Signed-off-by: Walter Lozano <walter.lozano@collabora.com> > --- Applied to u-boot-marvell/master. Thanks, Stefan
diff --git a/include/wdt.h b/include/wdt.h index 5bcff24ab3..dd83dfdd32 100644 --- a/include/wdt.h +++ b/include/wdt.h @@ -130,11 +130,10 @@ static inline int initr_watchdog(void) } } - if (CONFIG_IS_ENABLED(OF_CONTROL)) { + if (CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)) { timeout = dev_read_u32_default(gd->watchdog_dev, "timeout-sec", WATCHDOG_TIMEOUT_SECS); } - wdt_start(gd->watchdog_dev, timeout * 1000, 0); gd->flags |= GD_FLG_WDT_READY; printf("WDT: Started with%s servicing (%ds timeout)\n",
Currently watchdog tries to use dev_read_u32_default to get timeout configuration in case OF_CONTROL is enabled. However, if SPL is built with OF_PLATDATA this has no sense as there is no device tree. This patch fixes this issue by only use dev_read_u32_default if OF_CONTROL is enabled but OF_PLATDATA is not. Signed-off-by: Walter Lozano <walter.lozano@collabora.com> --- include/wdt.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)