[U-Boot] [PATCH] AT91: revise serial lines pullup configuration,

Tom Tom.Rix at windriver.com
Thu Nov 26 15:14:39 CET 2009


Wolfgang Denk wrote:
> Dear Tom,
> 
> In message <4A60EB7C.4010108 at poczta.fm> Andrzej Wolski wrote:
>> Currently pullups are incorrectly set for all AT91 SoCs:
>> pullup for TXD line is enabled, what is not necessary, but
>> pullup for RXD line, what may be necessary for some boards,
>> is disabled. This patch fixes it: it enables pullup only
>> for RXD line.
>>
>> Signed-off-by: Andrzej Wolski <awolski at poczta.fm>
> 
> Could you please check the status of this patch:
> http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/64068
> 
> Best regards,
> 
> Wolfgang Denk
> 

Andrzej,
This patch looks similar to what was done recently for overo.
Please add a comment in the code to the setting of the pullup.
Please also add a comment to commit log explaining why this
needs to change, just saying it is wrong is not enough of
an explanation.  In the case of overo, the problem that was
fixed was random hangs when the serial wasn't connected.

Please also check is any new at91 boards were added since
you first submitted this patch.

Tom



More information about the U-Boot mailing list