diff --git a/components/finsh/msh_parse.c b/components/finsh/msh_parse.c
index a0f5f5e96d9..e9deb3adb54 100644
--- a/components/finsh/msh_parse.c
+++ b/components/finsh/msh_parse.c
@@ -79,7 +79,7 @@ rt_bool_t msh_ishex(char *strvalue)
  *
  * @param strvalue string
  *
- * @return true or false
+ * @return integer transformed from string
  */
 int msh_strtohex(char *strvalue)
 {
diff --git a/src/idle.c b/src/idle.c
index 4903ed3fe5f..500b16c7799 100644
--- a/src/idle.c
+++ b/src/idle.c
@@ -140,6 +140,8 @@ rt_err_t rt_thread_idle_delhook(void (*hook)(void))
 /**
  * @brief Enqueue a thread to defunct queue.
  *
+ * @param thread the thread to be enqueued.
+ *
  * @note It must be called between rt_hw_interrupt_disable and rt_hw_interrupt_enable
  */
 void rt_thread_defunct_enqueue(rt_thread_t thread)
diff --git a/src/slab.c b/src/slab.c
index f48179afafc..e883a9de9b5 100644
--- a/src/slab.c
+++ b/src/slab.c
@@ -204,7 +204,7 @@ struct rt_slab
 /**
  * @brief Alloc memory size by page.
  *
- * @param slab the slab memory management object.
+ * @param m the slab memory management object.
  *
  * @param npages the number of pages.
  */
@@ -243,7 +243,7 @@ void *rt_slab_page_alloc(rt_slab_t m, rt_size_t npages)
 /**
  * @brief Free memory by page.
  *
- * @param slab the slab memory management object.
+ * @param m the slab memory management object.
  *
  * @param addr is the head address of first page.
  *
@@ -308,8 +308,6 @@ static void rt_slab_page_init(struct rt_slab *slab, void *addr, rt_size_t npages
 /**
  * @brief This function will init slab memory management algorithm
  *
- * @param slab the slab memory management object.
- *
  * @param name is the name of the slab memory management object.
  *
  * @param begin_addr the beginning address of system page.