Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
Hi Fabio, On 28/11/2019 16.43, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 12:31 PM Claudius Heine 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 ou

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Robert Hancock
On 2019-11-28 9:43 a.m., Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 12:31 PM Claudius Heine 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 under

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Fabio Estevam
Hi Robert, On Thu, Nov 28, 2019 at 1:22 PM Robert Hancock wrote: > I ended up needing to add this node for our board as well to be able to > reset from U-Boot using DM. The watchdog itself is set up just from its > own device tree entry, but there's nothing to tie the sysreset code into > using

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Fabio Estevam
Hi Claudius, On Thu, Nov 28, 2019 at 12:31 PM Claudius Heine 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 > verbos

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
On 28/11/2019 14.55, Fabio Estevam wrote: > On Thu, Nov 28, 2019 at 10:42 AM Claudius Heine wrote: >> >> On 28/11/2019 14.18, Fabio Estevam wrote: >>> Hi Claudius, >>> >>> On Thu, Nov 28, 2019 at 10:15 AM Claudius Heine wrote: >>> That is the sysreset device node, not the imx2_wdt one. (I wi

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Fabio Estevam
On Thu, Nov 28, 2019 at 10:42 AM Claudius Heine wrote: > > On 28/11/2019 14.18, Fabio Estevam wrote: > > Hi Claudius, > > > > On Thu, Nov 28, 2019 at 10:15 AM Claudius Heine wrote: > > > >> That is the sysreset device node, not the imx2_wdt one. (I will move > >> that into a '*-u-boot.dtsi' in v2

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
On 28/11/2019 14.18, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 10:15 AM Claudius Heine wrote: > >> That is the sysreset device node, not the imx2_wdt one. (I will move >> that into a '*-u-boot.dtsi' in v2) >> >> Or am I misunderstanding you? > > What I am asking is: why do

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Fabio Estevam
Hi Claudius, On Thu, Nov 28, 2019 at 10:15 AM Claudius Heine wrote: > That is the sysreset device node, not the imx2_wdt one. (I will move > that into a '*-u-boot.dtsi' in v2) > > Or am I misunderstanding you? What I am asking is: why do we need a specific sysreset node for U-Boot? Can't we ju

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
Hi Fabio, On 28/11/2019 13.49, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 9:07 AM Claudius Heine wrote: >> >> Signed-off-by: Claudius Heine >> --- >> arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + >> 1 file changed, 5 insertions(+) >> >> diff --git a/arch/arm/dts/imx6qdl-

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Harald Seiler
Hello Claudius, Fabio, On Thu, 2019-11-28 at 09:49 -0300, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 9:07 AM Claudius Heine wrote: > > Signed-off-by: Claudius Heine > > --- > > arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + > > 1 file changed, 5 insertions(+) > > > > dif

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Fabio Estevam
Hi Claudius, On Thu, Nov 28, 2019 at 9:07 AM Claudius Heine wrote: > > Signed-off-by: Claudius Heine > --- > arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi > b/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi > index

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
On 28/11/2019 13.14, Marek Vasut wrote: > On 11/28/19 1:06 PM, Claudius Heine wrote: >> Signed-off-by: Claudius Heine >> --- >> arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + >> 1 file changed, 5 insertions(+) >> >> diff --git a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi >> b/arch/arm/dts/imx6qdl-dhc

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Marek Vasut
On 11/28/19 1:06 PM, Claudius Heine wrote: > Signed-off-by: Claudius Heine > --- > arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi > b/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi > index af4719aaeb..572bcbf8f0 1006

[U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Claudius Heine
Signed-off-by: Claudius Heine --- arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi b/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi index af4719aaeb..572bcbf8f0 100644 --- a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi +++ b/arch/ar