Removed multiple return statement in CheckChipID function

Author: Arbi Gunbardhi

src/st25r3916.cpp

@@ -483,19 +483,12 @@ bool RfalRfST25R3916Class::st25r3916CheckChipID(uint8_t *rev)
 
   const uint8_t chipId = id & ST25R3916_REG_IC_IDENTITY_ic_type_mask;
 
-  const bool isChipIdValid =
-    chipId == ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916 ||
-    chipId == ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916b;
-
-  if (!isChipIdValid) {
-    return false;
-  }
-
   if (rev) {
     *rev = (id & ST25R3916_REG_IC_IDENTITY_ic_rev_mask);
   }
 
-  return true;
+  return (chipId == ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916 ||
+          chipId == ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916b);
 }
 
 /*******************************************************************************/

src/st25r3916_com.h

@@ -974,6 +974,7 @@
 #define ST25R3916_REG_IC_IDENTITY_ic_type1                    (1U<<4)
 #define ST25R3916_REG_IC_IDENTITY_ic_type0                    (1U<<3)
 #define ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916           (5U<<3)
+#define ST25R3916_REG_IC_IDENTITY_ic_type_st25r3916b          (6U<<3)
 #define ST25R3916_REG_IC_IDENTITY_ic_type_mask                (0x1fU<<3)
 #define ST25R3916_REG_IC_IDENTITY_ic_type_shift               (3U)
 #define ST25R3916_REG_IC_IDENTITY_ic_rev2                     (1U<<2)