Re: [U-Boot] [PATCH 1/3] common/cmd_rsmode.c: add imx reset mode command

Dear Eric Nelson,

In message 4FC4DC4E.9030007@boundarydevices.com you wrote:

Writing a 4 to the WDOG_BASE register enables the watchdog with a 1/2s delay. I think there should be a delay or while(1) loop afterwards though. It appears that the reset_cpu() routine will return.

I recommend NOT messing with the watchdog for any unrelated functions.

The function of the watchdog should be reserved for auch systems who need it to implement security related functions; messing with it in unrelated places (like setting it into unexpected states and or timeouts) may cause undefined behaviour on such systems.

Don't do it.

Best regards,

Wolfgang Denk