On 00:31 Mon 09 Mar     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> 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.
> 
> > diff --git a/cpu/arm920t/at91rm9200/interrupts.c 
> > b/cpu/arm920t/at91rm9200/interrupts.c
> > index 5f0703c..3e35f68 100644
> > --- a/cpu/arm920t/at91rm9200/interrupts.c
> > +++ b/cpu/arm920t/at91rm9200/interrupts.c
> > @@ -179,8 +179,10 @@ void reset_cpu (ulong ignored)
> >     AT91PS_PIO pio = AT91C_BASE_PIOA;
> >  #endif
> >  
> > +#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

Best Regards,
J.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to