Hi Fabio, On 28/11/2019 16.43, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 12:31 PM Claudius Heine <c...@denx.de> wrote: > >> Sorry, but we are probably misunderstanding each other here. So I try to >> go step by step to show how I think about this. Maybe that way we figure >> out where our understanding differs. Please bear with me, its a bit >> verbose :) > > What I don't understand is why do we need "wdt-reboot" for i.MX in U-Boot? > > The i.MX watchdog driver is capable of resetting via internal watchdog > or via a WDOG_B pin when the fsl,ext-reset-output property is passed.
Ok, right. Well my patches fixes the 'reset' command in u-boot and for that we need a sysreset driver in u-boot currently. The watchdog itself works fine IIRC. > >> My patch adds the 'wdt-reboot' device node which is needed by u-boot to >> reset the device but doesn't exist in the linux kernel (since there is > > Right. This is the point I don't understand: why do we need a U-Boot > wdt-reboot implementation in the first place? > > The imx watchdog works in the kernel with DT. Why can't you just use > the same binding in U-Boot? > > Thanks > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot