Merge pull request #694 from AubrCool/fix-serial

[DeviceDrivers] Fix serial
This commit is contained in:
Bernard Xiong 2017-01-19 22:14:29 +08:00 committed by GitHub
commit ad02d293f6
2 changed files with 46 additions and 17 deletions

View File

@ -14,6 +14,7 @@
* 2013-05-13 aozima update for kehong-lingtai. * 2013-05-13 aozima update for kehong-lingtai.
* 2015-01-31 armink make sure the serial transmit complete in putc() * 2015-01-31 armink make sure the serial transmit complete in putc()
* 2016-05-13 armink add DMA Rx mode * 2016-05-13 armink add DMA Rx mode
* 2017-01-19 aubr.cool add interrupt Tx mode
*/ */
#include "stm32f10x.h" #include "stm32f10x.h"
@ -144,8 +145,21 @@ static int stm32_putc(struct rt_serial_device *serial, char c)
RT_ASSERT(serial != RT_NULL); RT_ASSERT(serial != RT_NULL);
uart = (struct stm32_uart *)serial->parent.user_data; uart = (struct stm32_uart *)serial->parent.user_data;
uart->uart_device->DR = c; if(serial->parent.open_flag & RT_DEVICE_FLAG_INT_TX)
while (!(uart->uart_device->SR & USART_FLAG_TC)); {
if(!(uart->uart_device->SR & USART_FLAG_TXE))
{
USART_ITConfig(uart->uart_device, USART_IT_TC, ENABLE);
return -1;
}
uart->uart_device->DR = c;
USART_ITConfig(uart->uart_device, USART_IT_TC, ENABLE);
}
else
{
uart->uart_device->DR = c;
while (!(uart->uart_device->SR & USART_FLAG_TC));
}
return 1; return 1;
} }
@ -244,6 +258,11 @@ static void uart_isr(struct rt_serial_device *serial) {
if (USART_GetITStatus(uart->uart_device, USART_IT_TC) != RESET) if (USART_GetITStatus(uart->uart_device, USART_IT_TC) != RESET)
{ {
/* clear interrupt */ /* clear interrupt */
if(serial->parent.open_flag & RT_DEVICE_FLAG_INT_TX)
{
rt_hw_serial_isr(serial, RT_SERIAL_EVENT_TX_DONE);
}
USART_ITConfig(uart->uart_device, USART_IT_RXNE, DISABLE);
USART_ClearITPendingBit(uart->uart_device, USART_IT_TC); USART_ClearITPendingBit(uart->uart_device, USART_IT_TC);
} }
if (USART_GetFlagStatus(uart->uart_device, USART_FLAG_ORE) == SET) if (USART_GetFlagStatus(uart->uart_device, USART_FLAG_ORE) == SET)
@ -561,7 +580,8 @@ void rt_hw_usart_init(void)
/* register UART1 device */ /* register UART1 device */
rt_hw_serial_register(&serial1, "uart1", rt_hw_serial_register(&serial1, "uart1",
RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX | RT_DEVICE_FLAG_DMA_RX, RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX |
RT_DEVICE_FLAG_INT_TX | RT_DEVICE_FLAG_DMA_RX,
uart); uart);
#endif /* RT_USING_UART1 */ #endif /* RT_USING_UART1 */
@ -576,7 +596,8 @@ void rt_hw_usart_init(void)
/* register UART2 device */ /* register UART2 device */
rt_hw_serial_register(&serial2, "uart2", rt_hw_serial_register(&serial2, "uart2",
RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX | RT_DEVICE_FLAG_DMA_RX, RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX |
RT_DEVICE_FLAG_INT_TX | RT_DEVICE_FLAG_DMA_RX,
uart); uart);
#endif /* RT_USING_UART2 */ #endif /* RT_USING_UART2 */
@ -592,7 +613,8 @@ void rt_hw_usart_init(void)
/* register UART3 device */ /* register UART3 device */
rt_hw_serial_register(&serial3, "uart3", rt_hw_serial_register(&serial3, "uart3",
RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX | RT_DEVICE_FLAG_DMA_RX, RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX |
RT_DEVICE_FLAG_INT_TX | RT_DEVICE_FLAG_DMA_RX,
uart); uart);
#endif /* RT_USING_UART3 */ #endif /* RT_USING_UART3 */
@ -608,7 +630,8 @@ void rt_hw_usart_init(void)
/* register UART4 device */ /* register UART4 device */
rt_hw_serial_register(&serial4, "uart4", rt_hw_serial_register(&serial4, "uart4",
RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX | RT_DEVICE_FLAG_DMA_RX, RT_DEVICE_FLAG_RDWR | RT_DEVICE_FLAG_INT_RX |
RT_DEVICE_FLAG_INT_TX | RT_DEVICE_FLAG_DMA_RX,
uart); uart);
#endif /* RT_USING_UART4 */ #endif /* RT_USING_UART4 */
} }

View File

@ -31,6 +31,7 @@
* in open function. * in open function.
* 2015-11-10 bernard fix the poll rx issue when there is no data. * 2015-11-10 bernard fix the poll rx issue when there is no data.
* 2016-05-10 armink add fifo mode to DMA rx when serial->config.bufsz != 0. * 2016-05-10 armink add fifo mode to DMA rx when serial->config.bufsz != 0.
* 2017-01-19 aubr.cool prevent change serial rx bufsz when serial is opened.
*/ */
#include <rthw.h> #include <rthw.h>
@ -603,17 +604,22 @@ static rt_err_t rt_serial_control(struct rt_device *dev,
break; break;
case RT_DEVICE_CTRL_CONFIG: case RT_DEVICE_CTRL_CONFIG:
if (args) if (args)
{ {
/* set serial configure */ struct serial_configure *pconfig = (struct serial_configure *) args;
serial->config = *(struct serial_configure *)args; if(pconfig->bufsz != serial->config.bufsz && serial->parent.ref_count)
{
if (dev->ref_count) /*can not change buffer size*/
{ return RT_EBUSY;
/* serial device has been opened, to configure it */ }
serial->ops->configure(serial, (struct serial_configure *)args); /* set serial configure */
} serial->config = *pconfig;
} if(serial->parent.ref_count)
{
/* serial device has been opened, to configure it */
serial->ops->configure(serial, (struct serial_configure *)args);
}
}
break; break;