Remove byte sent at 9600 baud on init on NRF51-DK

[c1728p9]

Remove the dummy byte sent on init and instead keep track of if the
the TX line is idle. This fixes the junk data seen on the serial port
when it is initialized.

The code to send a dummy byte was added in the following commit:
91a77d7

[0xc0170]

What's the function of idle variable? It's 1 only after init, once something is sent, it becomes 0 and never returns to 1.

@sg- review please

[sg-]

I think a more simple solution would be to wait baud/clk cycles and remove the flag.

[sg-]

Looks good!

[0xc0170]

+1, will merge once I push the release, will happen soon

[c1728p9]

Rebased onto master and added support for 56000 baud.

[leibin2014]

Hi guys,
Looks like there is a small mistake on adding support for 56000 baud. After adding this baud, the array size also need to be added by 1. It will be 18, not 17 as now.

static const int acceptedSpeeds[17][2] = { // Here should be [18][2]
{1200, UART_BAUDRATE_BAUDRATE_Baud1200},
{2400, UART_BAUDRATE_BAUDRATE_Baud2400},
{4800, UART_BAUDRATE_BAUDRATE_Baud4800},
{9600, UART_BAUDRATE_BAUDRATE_Baud9600},
{14400, UART_BAUDRATE_BAUDRATE_Baud14400},
{19200, UART_BAUDRATE_BAUDRATE_Baud19200},
{28800, UART_BAUDRATE_BAUDRATE_Baud28800},
{31250, (0x00800000UL) /* 31250 baud /},
{38400, UART_BAUDRATE_BAUDRATE_Baud38400},
{56000, (0x00E51000UL) / 56000 baud */},
{57600, UART_BAUDRATE_BAUDRATE_Baud57600},
{76800, UART_BAUDRATE_BAUDRATE_Baud76800},
{115200, UART_BAUDRATE_BAUDRATE_Baud115200},
{230400, UART_BAUDRATE_BAUDRATE_Baud230400},
{250000, UART_BAUDRATE_BAUDRATE_Baud250000},
{460800, UART_BAUDRATE_BAUDRATE_Baud460800},
{921600, UART_BAUDRATE_BAUDRATE_Baud921600},
{1000000, UART_BAUDRATE_BAUDRATE_Baud1M}
};

[c1728p9]

Hi @leibin2014 , good catch. I pushed up #1479 to fix this.