[U-Boot] [PATCH 7/9] at91rm9200: fix DCC support

Wolfgang Denk
2009-03-09T11:19:49Z

Dear Jean-Christophe PLAGNIOL-VILLARD,

In message <20090309083649.GE24237@game.jcrosoft.org> you wrote:
>
> > In message <1236089265-24598-7-git-send-email-plagnioj@jcrosoft.com> you wrote:
> > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@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: wd at denx.de
"There is such a fine line between genius and stupidity."
- David St. Hubbins, "Spinal Tap"