Dear Jean-Christophe PLAGNIOL-VILLARD, In message <20090309083649.ge24...@game.jcrosoft.org> you wrote: > > > In message <1236089265-24598-7-git-send-email-plagn...@jcrosoft.com> you > > wrote: > > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagn...@jcrosoft.com> > > > --- > > > cpu/arm920t/at91rm9200/interrupts.c | 2 ++ > > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > Please explain which problem you are trying to fix.
----^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Which problem is there with the current code? > > > +#if defined(CONFIG_DBGU) || defined(CONFIG_USART0) || > > > defined(CONFIG_USART1) > > > /*shutdown the console to avoid strange chars during reset */ > > > us->US_CR = (AT91C_US_RSTRX | AT91C_US_RSTTX); > > > +#endif > > > > Why the #ifdef? Shutting down the console seems a good thing to me. > when using DCC we will not activate the at91 uart at all so we do not have to > shut it down The #ifdef above does not seem to depend on DCC at all? And what does it hurt if we always shut down the console? To me that looks clearer than an additional #ifdef. Finally, what if somebody wants to use DCC and an UART simultaneously? I guess that should be possible, too. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de "There is such a fine line between genius and stupidity." - David St. Hubbins, "Spinal Tap" _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
