Wolfgang Denk wrote:
Dear Tom,
In message 4A60EB7C.4010108@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@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