[U-Boot] [PATCH 1/6] am33xx: Enable UART{1,2,4,5} clocks

Tom Rini trini at ti.com
Sun Oct 21 16:54:32 CEST 2012


On Sat, Oct 20, 2012 at 11:57 AM, Marek Vasut <marex at denx.de> wrote:
> Dear Tom Rini,
[snip]
>> Lets just match the rest of the SoC code and have the empty loop.  We
>> can have the discussion about what kind of delay or macro makes most
>> sense another time.
>
> WATCHDOG_RESET() shall obviously be called, just enable watchdog and your uboot
> will keep restarting in such loops.

The reason I want to have this discussion outside of Andrew's changes
is that if the SoC isn't setting these flags when asked, you should
just throw the board out, something fundamental is broken.  The only
real question is, would it make sense, or not, to enforce a real delay
or just keep reading until it is set.  I'll poke around a little
tomorrow.

-- 
Tom


More information about the U-Boot mailing list