Fix a few places where there was a semicolon following the 'if' condition, makeing the following logic unconditional.

arch/arm/src/imx6/imx_ecspi.c

@@ -1058,7 +1058,7 @@ static uint8_t spi_status(FAR struct spi_dev_s *dev, uint32_t devid)
 
   DEBUGASSERT(priv != NULL);
 
-  if (priv->status != NULL);
+  if (priv->status != NULL)
     {
       ret = priv->select(dev, devid);
     }
@@ -1099,7 +1099,7 @@ static int spi_cmddata(FAR struct spi_dev_s *dev, uint32_t devid,
 
   DEBUGASSERT(priv != NULL);
 
-  if (priv->cmddata != NULL);
+  if (priv->cmddata != NULL)
     {
       ret = priv->cmddata(dev, devid, cmd);
     }

arch/avr/src/at90usb/at90usb_usbdev.c

@@ -2459,7 +2459,7 @@ static int avr_epsubmit(FAR struct usbdev_ep_s *ep,
 
           /* If there is something avaible in the fifo now, then go get it */
 
-          if (avr_fifoready(AVR_TIMEOUT_NONE) == OK);
+          if (avr_fifoready(AVR_TIMEOUT_NONE) == OK)
             {
               ret = avr_epOUTqueue(privep);
             }
@@ -2968,3 +2968,4 @@ void avr_pollvbus(void)
   leave_critical_section(flags);
 }
 #endif
+

crypto/testmngr.c

@@ -124,8 +124,12 @@ static int test_aes(void)
 int crypto_test(void)
 {
 #if defined(CONFIG_CRYPTO_AES)
-  if (test_aes()) return -1;
+  if (test_aes())
+    {
+      return -1;
+    }
 #endif
+
   return OK;
 }
 

drivers/usbdev/cdcacm.c

@@ -570,7 +570,7 @@ static int cdcacm_recvpacket(FAR struct cdcacm_dev_s *priv,
        * processing. This allows proper utilization of hardware flow control.
        */
 
-      if (nexthead == rxbuf->tail);
+      if (nexthead == rxbuf->tail)
         {
           if (cdcuart_rxflowcontrol(&priv->serdev, recv->size, true))
             {

sched/mqueue/mq_send.c

@@ -94,7 +94,7 @@ int nxmq_send(mqd_t mqdes, FAR const char *msg, size_t msglen, int prio)
    */
 
   ret = nxmq_verify_send(mqdes, msg, msglen, prio);
-  if (ret < 0);
+  if (ret < 0)
     {
       return ret;
     }