'$' no longer causes bus pirate to reset to bootloader

[Google Code Exporter]

What steps will reproduce the problem?
1. Over a serial terminal type '$' <enter> then when asked "are you sure" press 
'y'

What is the expected output? What do you see instead?
I expect to see:
HiZ>$
Are you sure? y
BOOTLOADER
and the `mode` LED come on.
I see the same text but with the `mode` LED off.
I can no longer enter any text over the serial port.

What version of the product are you using? On what operating system?
Built firmware from latest source (Firmware v6.3-beta1 r2088)
Talking via teraterm on win 7 x86

Please provide any additional information below.
Using a jumper between PGC and PGD still works as expected.
This was not broken in last stable version (6.1)

Original issue reported on code.google.com by [e-mail] on 29 May 2013 at 11:21

[Google Code Exporter]

I traced this back to revision r1955. In that revision, the "wait for TX done" 
line that is ran after $ is changed. It used to read:

    while (UART1TXRdy == 0); //wait untill TX finishes

But since that takes the address of the UART1TXRdy function (which is always 
non-zero) this comparison would always be false and this loop would never 
block. After r1955, the line reads:

    while (0 == UART1TXRdy()); //wait untill TX finishes

So now the function is actually called. I presume that for some reason that 
loop never finishes. Perhaps the TX never completes because it doesn't use 
interrupts on the BPv3 (that's just a guess, though).

In any case, if I comment out this while loop, the $ command works as expected 
again.

Original comment by [e-mail] on 20 Oct 2013 at 8:58

[Google Code Exporter]

Cool, thanks :)


On 20 October 2013 21:58, <[e-mail]

Original comment by [e-mail] on 20 Oct 2013 at 11:31

[Google Code Exporter]

There is more related discussion here: 
http://dangerousprototypes.com/forum/viewtopic.php?f=4&t=5052&p=54931#p54923

Original comment by [e-mail] on 21 Oct 2013 at 8:29

[Google Code Exporter]

I think I found a solution for the issue:
Change 
while (0 == UART1TXRdy()); //wait untill TX finishes
To
while (U1STAbits.TRMT == 0); //wait untill TX finishes

This way we wait to empty the WHOLE TX buffer before jumping to the bootloader. 

Original comment by [e-mail] on 31 May 2014 at 8:43