diff --git a/bsp/efm32/board.h b/bsp/efm32/board.h
index a0c95e805eddf3f3f1699a53742c650b0d277bd0..6e91c290260520ba2242cc61e01860ff600751d1 100644
--- a/bsp/efm32/board.h
+++ b/bsp/efm32/board.h
@@ -122,7 +122,6 @@ extern volatile rt_uint32_t rt_system_status;
 #define IIC_RX_BUFFER_SIZE			(32)
 
 /* SECTION: ADC */
-
 #define ADC_CALI_REF				adcRef2V5
 #define ADC_CALI_CH 				adcSingleInpCh5
 #define ADC_CONVERT_FREQUENCY 		(7000000)
diff --git a/bsp/efm32/drv_ethernet.c b/bsp/efm32/drv_ethernet.c
index 6391db598643b4d01e9c08472fc9fd97ed387a6c..ca3a9236130fa2bd4dea002b5ce30c7e05d1f4c5 100644
--- a/bsp/efm32/drv_ethernet.c
+++ b/bsp/efm32/drv_ethernet.c
@@ -1021,9 +1021,9 @@ rt_err_t efm_hw_eth_init(void)
 	return -RT_ERROR;
 }
 
-/***************************************************************************//**
-* 	Export to FINSH
-******************************************************************************/
+/*******************************************************************************
+ * 	Export to FINSH
+ ******************************************************************************/
 #if	defined(EFM32_USING_ETH_UTILS)
 #ifdef RT_USING_FINSH
 #include <finsh.h>
diff --git a/bsp/efm32/drv_iic.c b/bsp/efm32/drv_iic.c
index c6eca696670a0567158c19e82f28bf1f520c457b..081871ca6aae6914408b01ed95710ccbd5783091 100644
--- a/bsp/efm32/drv_iic.c
+++ b/bsp/efm32/drv_iic.c
@@ -715,14 +715,12 @@ static struct efm32_iic_device_t *rt_hw_iic_unit_init(
 		switch (unitNumber)
 		{
 		case 0:
-			iic->lock		= &iic0_lock;
 			iic->iic_device	= I2C0;
 			iicClock 		= (CMU_Clock_TypeDef)cmuClock_I2C0;	
 			break;
 
 #if (I2C_COUNT > 1)
 		case 1:
-			iic->lock		= &iic1_lock;
 			iic->iic_device	= I2C1;
 			iicClock  		= (CMU_Clock_TypeDef)cmuClock_I2C1; 
 			break;
@@ -807,6 +805,7 @@ void rt_hw_iic_init(void)
 		}
 
 		/* Initialize lock for iic0 */
+		iic->lock = &iic0_lock;
 		if (rt_sem_init(iic->lock, RT_IIC0_NAME, 1, RT_IPC_FLAG_FIFO) != RT_EOK)
 		{
 			break;
@@ -825,6 +824,7 @@ void rt_hw_iic_init(void)
 		}
 
 		/* Initialize lock for iic1 */
+		iic->lock = &iic1_lock;
 		if (rt_sem_init(iic->lock, RT_IIC1_NAME, 1, RT_IPC_FLAG_FIFO) != RT_EOK)
 		{
 			break;
diff --git a/bsp/efm32/drv_sdcard.c b/bsp/efm32/drv_sdcard.c
index 4831acf2c760734d1f5a9a01218ffa67e7645656..993ca3087606dc9340410b09994ebec205fbc1dd 100644
--- a/bsp/efm32/drv_sdcard.c
+++ b/bsp/efm32/drv_sdcard.c
@@ -1264,9 +1264,9 @@ void efm_spiSd_deinit(void)
 	sdcard_debug("SPISD: Deinit OK\n");
 }
 
-/***************************************************************************//**
-* 	Export to FINSH
-******************************************************************************/
+/*******************************************************************************
+ * 	Export to FINSH
+ ******************************************************************************/
 #ifdef RT_USING_FINSH
 #include <finsh.h>
 
diff --git a/bsp/efm32/drv_usart.c b/bsp/efm32/drv_usart.c
index 0b6f491c817725422fa84b00fc224047358e8c76..81547d2c69080986d4bdcacb7d8bd5a5423fb244 100644
--- a/bsp/efm32/drv_usart.c
+++ b/bsp/efm32/drv_usart.c
@@ -499,7 +499,7 @@ static rt_size_t rt_usart_write (
 			true,
 			false,
 			(void *)&(usart->usart_device->TXDATA),
-			buffer,
+			(void *)buffer,
 			(rt_uint32_t)(size - 1));
 
 		/* Wait, otherwise the TX buffer is overwrite */
@@ -920,21 +920,18 @@ static struct efm32_usart_device_t *rt_hw_usart_unit_init(
 		switch (unitNumber)
 		{
 		case 0:
-			usart->lock			= &usart0_lock;
 			usart->usart_device	= USART0;
 			usartClock 			= (CMU_Clock_TypeDef)cmuClock_USART0;
 			txDmaSelect			= DMAREQ_USART0_TXBL;
 			break;
 			
 		case 1:
-			usart->lock			= &usart1_lock;
 			usart->usart_device	= USART1;
 			usartClock 			= (CMU_Clock_TypeDef)cmuClock_USART1;
 			txDmaSelect			= DMAREQ_USART1_TXBL;
 			break;
 			
 		case 2:
-			usart->lock			= &usart2_lock;
 			usart->usart_device	= USART2;
 			usartClock 			= (CMU_Clock_TypeDef)cmuClock_USART2;
 			txDmaSelect			= DMAREQ_USART2_TXBL;
@@ -1161,6 +1158,7 @@ void rt_hw_usart_init(void)
 			break;
 		}
 		/* Initialize lock for usart0 */
+		usart->lock = &usart0_lock;
 		if (rt_sem_init(usart->lock, RT_USART0_NAME, 1, RT_IPC_FLAG_FIFO) != RT_EOK)
 		{
 			break;
@@ -1215,6 +1213,7 @@ void rt_hw_usart_init(void)
 			break;
 		}
 		/* Initialize lock for usart1 */
+		usart->lock = &usart1_lock;
 		if (rt_sem_init(usart->lock, RT_USART1_NAME, 1, RT_IPC_FLAG_FIFO) != RT_EOK)
 		{
 			break;
@@ -1270,6 +1269,7 @@ void rt_hw_usart_init(void)
 			break;
 		}
 		/* Initialize lock for usart2 */
+		usart->lock = &usart2_lock;
 		if (rt_sem_init(usart->lock, RT_USART2_NAME, 1, RT_IPC_FLAG_FIFO) != RT_EOK)
 		{
 			break;