Merge commit 'collin80-uart-fix' into ide-1.5.x

Author: cmaglie

build/shared/revisions.txt

@@ -6,6 +6,9 @@ The following changes are included also in the Arduino IDE 1.0.7:
 [ide]
 * Mitigated Serial Monitor resource exhaustion when the connected device sends a lot of data (Paul Stoffregen)
 
+[core]
+* sam: HardwareSerial now performs ISR based buffered transmission (Collin Kidder)
+
 ARDUINO 1.6.0rc1
 
 * IDE internals have been refactored and sorted out. (Claudio Indellicati)

hardware/arduino/sam/cores/arduino/RingBuffer.cpp

@@ -21,7 +21,7 @@
 
 RingBuffer::RingBuffer( void )
 {
-    memset( _aucBuffer, 0, SERIAL_BUFFER_SIZE ) ;
+    memset( (void *)_aucBuffer, 0, SERIAL_BUFFER_SIZE ) ;
     _iHead=0 ;
     _iTail=0 ;
 }

hardware/arduino/sam/cores/arduino/RingBuffer.h

@@ -25,14 +25,14 @@
 // using a ring buffer (I think), in which head is the index of the location
 // to which to write the next incoming character and tail is the index of the
 // location from which to read.
-#define SERIAL_BUFFER_SIZE 64
+#define SERIAL_BUFFER_SIZE 128
 
 class RingBuffer
 {
   public:
-    uint8_t _aucBuffer[SERIAL_BUFFER_SIZE] ;
-    int _iHead ;
-    int _iTail ;
+    volatile uint8_t _aucBuffer[SERIAL_BUFFER_SIZE] ;
+    volatile int _iHead ;
+    volatile int _iTail ;
 
   public:
     RingBuffer( void ) ;

hardware/arduino/sam/cores/arduino/UARTClass.cpp

@@ -23,33 +23,39 @@
 
 // Constructors ////////////////////////////////////////////////////////////////
 
-UARTClass::UARTClass( Uart* pUart, IRQn_Type dwIrq, uint32_t dwId, RingBuffer* pRx_buffer )
+UARTClass::UARTClass( Uart *pUart, IRQn_Type dwIrq, uint32_t dwId, RingBuffer *pRx_buffer, RingBuffer *pTx_buffer )
 {
-  _rx_buffer = pRx_buffer ;
+  _rx_buffer = pRx_buffer;
+  _tx_buffer = pTx_buffer;
 
-  _pUart=pUart ;
-  _dwIrq=dwIrq ;
-  _dwId=dwId ;
+  _pUart=pUart;
+  _dwIrq=dwIrq;
+  _dwId=dwId;
 }
 
 // Public Methods //////////////////////////////////////////////////////////////
 
 void UARTClass::begin( const uint32_t dwBaudRate )
+{
+  begin( dwBaudRate, UART_MR_PAR_NO | UART_MR_CHMODE_NORMAL );
+}
+
+void UARTClass::begin( const uint32_t dwBaudRate, const uint32_t config )
 {
   // Configure PMC
-  pmc_enable_periph_clk( _dwId ) ;
+  pmc_enable_periph_clk( _dwId );
 
   // Disable PDC channel
-  _pUart->UART_PTCR = UART_PTCR_RXTDIS | UART_PTCR_TXTDIS ;
+  _pUart->UART_PTCR = UART_PTCR_RXTDIS | UART_PTCR_TXTDIS;
 
   // Reset and disable receiver and transmitter
-  _pUart->UART_CR = UART_CR_RSTRX | UART_CR_RSTTX | UART_CR_RXDIS | UART_CR_TXDIS ;
+  _pUart->UART_CR = UART_CR_RSTRX | UART_CR_RSTTX | UART_CR_RXDIS | UART_CR_TXDIS;
 
   // Configure mode
-  _pUart->UART_MR = UART_MR_PAR_NO | UART_MR_CHMODE_NORMAL ;
+  _pUart->UART_MR = config;
 
   // Configure baudrate (asynchronous, no oversampling)
-  _pUart->UART_BRGR = (SystemCoreClock / dwBaudRate) >> 4 ;
+  _pUart->UART_BRGR = (SystemCoreClock / dwBaudRate) >> 4;
 
   // Configure interrupts
   _pUart->UART_IDR = 0xFFFFFFFF;
@@ -58,79 +64,128 @@ void UARTClass::begin( const uint32_t dwBaudRate )
   // Enable UART interrupt in NVIC
   NVIC_EnableIRQ(_dwIrq);
 
+  // Make sure both ring buffers are initialized back to empty.
+  _rx_buffer->_iHead = _rx_buffer->_iTail = 0;
+  _tx_buffer->_iHead = _tx_buffer->_iTail = 0;
+
   // Enable receiver and transmitter
-  _pUart->UART_CR = UART_CR_RXEN | UART_CR_TXEN ;
+  _pUart->UART_CR = UART_CR_RXEN | UART_CR_TXEN;
 }
 
 void UARTClass::end( void )
 {
-  // clear any received data
-  _rx_buffer->_iHead = _rx_buffer->_iTail ;
-
-  // Disable UART interrupt in NVIC
-  NVIC_DisableIRQ( _dwIrq ) ;
+  // Clear any received data
+  _rx_buffer->_iHead = _rx_buffer->_iTail;
 
   // Wait for any outstanding data to be sent
   flush();
 
-  pmc_disable_periph_clk( _dwId ) ;
+  // Disable UART interrupt in NVIC
+  NVIC_DisableIRQ( _dwIrq );
+
+  pmc_disable_periph_clk( _dwId );
+}
+
+void UARTClass::setInterruptPriority(uint32_t priority)
+{
+  NVIC_SetPriority(_dwIrq, priority & 0x0F);
+}
+
+uint32_t UARTClass::getInterruptPriority()
+{
+  return NVIC_GetPriority(_dwIrq);
 }
 
 int UARTClass::available( void )
 {
-  return (uint32_t)(SERIAL_BUFFER_SIZE + _rx_buffer->_iHead - _rx_buffer->_iTail) % SERIAL_BUFFER_SIZE ;
+  return (uint32_t)(SERIAL_BUFFER_SIZE + _rx_buffer->_iHead - _rx_buffer->_iTail) % SERIAL_BUFFER_SIZE;
+}
+
+int UARTClass::availableForWrite(void)
+{
+  int head = _tx_buffer->_iHead;
+  int tail = _tx_buffer->_iTail;
+  if (head >= tail) return SERIAL_BUFFER_SIZE - 1 - head + tail;
+  return tail - head - 1;
 }
 
 int UARTClass::peek( void )
 {
   if ( _rx_buffer->_iHead == _rx_buffer->_iTail )
-    return -1 ;
+    return -1;
 
-  return _rx_buffer->_aucBuffer[_rx_buffer->_iTail] ;
+  return _rx_buffer->_aucBuffer[_rx_buffer->_iTail];
 }
 
 int UARTClass::read( void )
 {
   // if the head isn't ahead of the tail, we don't have any characters
   if ( _rx_buffer->_iHead == _rx_buffer->_iTail )
-    return -1 ;
+    return -1;
 
-  uint8_t uc = _rx_buffer->_aucBuffer[_rx_buffer->_iTail] ;
-  _rx_buffer->_iTail = (unsigned int)(_rx_buffer->_iTail + 1) % SERIAL_BUFFER_SIZE ;
-  return uc ;
+  uint8_t uc = _rx_buffer->_aucBuffer[_rx_buffer->_iTail];
+  _rx_buffer->_iTail = (unsigned int)(_rx_buffer->_iTail + 1) % SERIAL_BUFFER_SIZE;
+  return uc;
 }
 
 void UARTClass::flush( void )
 {
+  while (_tx_buffer->_iHead != _tx_buffer->_iTail); //wait for transmit data to be sent
   // Wait for transmission to complete
   while ((_pUart->UART_SR & UART_SR_TXRDY) != UART_SR_TXRDY)
-    ;
+   ;
 }
 
 size_t UARTClass::write( const uint8_t uc_data )
 {
-  // Check if the transmitter is ready
-  while ((_pUart->UART_SR & UART_SR_TXRDY) != UART_SR_TXRDY)
-    ;
-
-  // Send character
-  _pUart->UART_THR = uc_data;
+  // Is the hardware currently busy?
+  if (((_pUart->UART_SR & UART_SR_TXRDY) != UART_SR_TXRDY) |
+      (_tx_buffer->_iTail != _tx_buffer->_iHead))
+  {
+    // If busy we buffer
+    unsigned int l = (_tx_buffer->_iHead + 1) % SERIAL_BUFFER_SIZE;
+    while (_tx_buffer->_iTail == l)
+      ; // Spin locks if we're about to overwrite the buffer. This continues once the data is sent
+
+    _tx_buffer->_aucBuffer[_tx_buffer->_iHead] = uc_data;
+    _tx_buffer->_iHead = l;
+    // Make sure TX interrupt is enabled
+    _pUart->UART_IER = UART_IER_TXRDY;
+  }
+  else 
+  {
+     // Bypass buffering and send character directly
+     _pUart->UART_THR = uc_data;
+  }
   return 1;
 }
 
 void UARTClass::IrqHandler( void )
 {
   uint32_t status = _pUart->UART_SR;
 
-  // Did we receive data ?
+  // Did we receive data?
   if ((status & UART_SR_RXRDY) == UART_SR_RXRDY)
     _rx_buffer->store_char(_pUart->UART_RHR);
 
+  // Do we need to keep sending data?
+  if ((status & UART_SR_TXRDY) == UART_SR_TXRDY) 
+  {
+    if (_tx_buffer->_iTail != _tx_buffer->_iHead) {
+      _pUart->UART_THR = _tx_buffer->_aucBuffer[_tx_buffer->_iTail];
+      _tx_buffer->_iTail = (unsigned int)(_tx_buffer->_iTail + 1) % SERIAL_BUFFER_SIZE;
+    }
+    else
+    {
+      // Mask off transmit interrupt so we don't get it anymore
+      _pUart->UART_IDR = UART_IDR_TXRDY;
+    }
+  }
+
   // Acknowledge errors
-  if ((status & UART_SR_OVRE) == UART_SR_OVRE ||
-		  (status & UART_SR_FRAME) == UART_SR_FRAME)
+  if ((status & UART_SR_OVRE) == UART_SR_OVRE || (status & UART_SR_FRAME) == UART_SR_FRAME)
   {
-	// TODO: error reporting outside ISR
+    // TODO: error reporting outside ISR
     _pUart->UART_CR |= UART_CR_RSTSTA;
   }
 }

hardware/arduino/sam/cores/arduino/UARTClass.h

@@ -28,32 +28,32 @@
 class UARTClass : public HardwareSerial
 {
   protected:
-    RingBuffer *_rx_buffer ;
+    RingBuffer *_rx_buffer;
+    RingBuffer *_tx_buffer;
 
   protected:
-    Uart* _pUart ;
-    IRQn_Type _dwIrq ;
-    uint32_t _dwId ;
+    Uart* _pUart;
+    IRQn_Type _dwIrq;
+    uint32_t _dwId;
 
   public:
-    UARTClass( Uart* pUart, IRQn_Type dwIrq, uint32_t dwId, RingBuffer* pRx_buffer ) ;
-
-    void begin( const uint32_t dwBaudRate ) ;
-    void end( void ) ;
-    int available( void ) ;
-    int peek( void ) ;
-    int read( void ) ;
-    void flush( void ) ;
-    size_t write( const uint8_t c ) ;
-
-    void IrqHandler( void ) ;
-
-#if defined __GNUC__ /* GCC CS3 */
-    using Print::write ; // pull in write(str) and write(buf, size) from Print
-#elif defined __ICCARM__ /* IAR Ewarm 5.41+ */
-//    virtual void write( const char *str ) ;
-//    virtual void write( const uint8_t *buffer, size_t size ) ;
-#endif
+    UARTClass(Uart* pUart, IRQn_Type dwIrq, uint32_t dwId, RingBuffer* pRx_buffer, RingBuffer* pTx_buffer);
+
+    void begin(const uint32_t dwBaudRate);
+    void begin(const uint32_t dwBaudRate, const uint32_t config);
+    void end(void);
+    int available(void);
+    int availableForWrite(void);
+    int peek(void);
+    int read(void);
+    void flush(void);
+    size_t write(const uint8_t c);
+    using Print::write; // pull in write(str) and write(buf, size) from Print
+
+    void setInterruptPriority(uint32_t priority);
+    uint32_t getInterruptPriority();
+
+    void IrqHandler(void);
 
     operator bool() { return true; }; // UART always active
 };