diff --git a/arch/arm/lib/findbit.S b/arch/arm/lib/findbit.S
index 30c8a1a..7fa93a5 100644
--- a/arch/arm/lib/findbit.S
+++ b/arch/arm/lib/findbit.S
@@ -1,7 +1,3 @@
-/**
- * @file
- * @brief common bitops
- */
 /*
  * Originally from Linux kernel
  *  arch/arm/lib/findbit.S
diff --git a/common/startup.c b/common/startup.c
index f0448e6..e40759d 100644
--- a/common/startup.c
+++ b/common/startup.c
@@ -25,6 +25,10 @@
  * MA 02111-1307 USA
  */
 
+/**
+ * @file
+ * @brief Main entry into the C part of U-Boot-v2
+ */
 #include <common.h>
 #include <init.h>
 #include <command.h>
diff --git a/drivers/serial/serial_ns16550.c b/drivers/serial/serial_ns16550.c
index 0dc389c..524190d 100644
--- a/drivers/serial/serial_ns16550.c
+++ b/drivers/serial/serial_ns16550.c
@@ -51,8 +51,8 @@
 /**
  * @brief ns16550_calc_divisor - compute the divisor for a baud rate
  *
- * @param[in] cdev - pointer to console device
- * @param[in] baudrate - baud rate
+ * @param[in] cdev pointer to console device
+ * @param[in] baudrate baud rate
  *
  * @return divisor to be set
  */
@@ -89,7 +89,7 @@
 /**
  * @brief ns16550_serial_init_port - initialize the device
  *
- * @param cdev[in] - pointer to console device
+ * @param[in] cdev pointer to console device
  */
 static void ns16550_serial_init_port(struct console_device *cdev)
 {
@@ -122,8 +122,8 @@
 /**
  * @brief ns16550_putc- put a character to the serial port
  *
- * @param cdev[in] - pointer to console device
- * @param c[in] - character to put
+ * @param[in] cdev pointer to console device
+ * @param[in] c character to put
  */
 static void ns16550_putc(struct console_device *cdev, char c)
 {
@@ -138,7 +138,7 @@
 /**
  * @brief ns16550_getc - retrieve a character from serial port
  *
- * @param cdev[in] - pointer to console device
+ * @param[in] cdev pointer to console device
  *
  * @return return the character read
  */
@@ -155,7 +155,7 @@
 /**
  * @brief ns16550_tstc - test if character is available
  *
- * @param cdev[in] -pointer to console device
+ * @param[in] cdev pointer to console device
  *
  * @return  - status based on data availability
  */
@@ -170,8 +170,8 @@
 /**
  * @brief ns16550_setbaudrate - set the baudrate for the uart port
  *
- * @param cdev[in] - console device
- * @param baud_rate[in] - baud rate to set
+ * @param[in] cdev  console device
+ * @param[in] baud_rate baud rate to set
  *
  * @return  0-implied to support the baudrate
  */
@@ -192,10 +192,9 @@
 }
 
 /**
- * @brief ns16550_probe - Probe entry point -called on the first
- *			  match for device
+ * @brief ns16550_probe - Probe entry point -called on the first match for device
  *
- * @param dev[in]  - matched device
+ * @param[in] dev matched device
  *
  * @return EINVAL if platform_data is not populated,
  *	   ENOMEM if calloc failed