diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
index ab3b24b2bd..24c9342008 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
@@ -92,7 +92,7 @@ UnicodeToAscii (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CurBBSEntry     EDES_TODO: Add parameter description
   @param Index           EDES_TODO: Add parameter description
@@ -393,7 +393,7 @@ BdsCreateLegacyBootOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BootOptionVar   EDES_TODO: Add parameter description
   @param BbsEntry        EDES_TODO: Add parameter description
@@ -435,7 +435,7 @@ BdsIsLegacyBootOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param OptionNumber    EDES_TODO: Add parameter description
   @param BootOrder       EDES_TODO: Add parameter description
@@ -606,7 +606,7 @@ BdsDeleteAllInvalidLegacyBootOptions (
       );
   }
 
-  if (BootOrderSize) {
+  if (BootOrderSize != 0) {
     Status = gRT->SetVariable (
                     L"BootOrder",
                     &gEfiGlobalVariableGuid,
@@ -624,7 +624,7 @@ BdsDeleteAllInvalidLegacyBootOptions (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BootOrder       EDES_TODO: Add parameter description
   @param BootOptionNum   EDES_TODO: Add parameter description
@@ -694,7 +694,7 @@ BdsFindLegacyBootOptionByDevType (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BbsItem         EDES_TODO: Add parameter description
   @param Index           EDES_TODO: Add parameter description
@@ -858,7 +858,7 @@ BdsAddNonExistingLegacyBootOptions (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BbsTable        EDES_TODO: Add parameter description
   @param BbsType         EDES_TODO: Add parameter description
@@ -895,7 +895,7 @@ BdsFillDevOrderBuf (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BbsTable        EDES_TODO: Add parameter description
   @param BbsCount        EDES_TODO: Add parameter description
@@ -980,7 +980,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);
   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + FDCount * sizeof (UINT16));
   Ptr += sizeof (UINT16);
-  if (FDCount) {
+  if (FDCount != 0) {
     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_FLOPPY, BbsCount, (UINT16 *) Ptr);
   }
 
@@ -988,7 +988,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);
   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + HDCount * sizeof (UINT16));
   Ptr += sizeof (UINT16);
-  if (HDCount) {
+  if (HDCount != 0) {
     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_HARDDISK, BbsCount, (UINT16 *) Ptr);
   }
 
@@ -996,7 +996,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);
   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + CDCount * sizeof (UINT16));
   Ptr += sizeof (UINT16);
-  if (CDCount) {
+  if (CDCount != 0) {
     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_CDROM, BbsCount, (UINT16 *) Ptr);
   }
 
@@ -1004,7 +1004,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);
   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + NETCount * sizeof (UINT16));
   Ptr += sizeof (UINT16);
-  if (NETCount) {
+  if (NETCount != 0) {
     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_EMBED_NETWORK, BbsCount, (UINT16 *) Ptr);
   }
 
@@ -1012,7 +1012,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);
   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + BEVCount * sizeof (UINT16));
   Ptr += sizeof (UINT16);
-  if (BEVCount) {
+  if (BEVCount != 0) {
     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_BEV_DEVICE, BbsCount, (UINT16 *) Ptr);
   }
 
@@ -1029,7 +1029,7 @@ BdsCreateDevOrder (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1334,7 +1334,7 @@ BdsUpdateLegacyDevOrder (
     // at this point we have copied those valid indexes to new buffer
     // and we should check if there is any new appeared boot device
     //
-    if (Idx) {
+    if (Idx != 0) {
       for (Index2 = 0; Index2 < *Idx; Index2++) {
         if ((NewDevPtr[Index2] & 0xFF) == (UINT16) Index) {
           break;
@@ -1353,7 +1353,7 @@ BdsUpdateLegacyDevOrder (
     }
   }
 
-  if (FDCount) {
+  if (FDCount != 0) {
     //
     // Just to make sure that disabled indexes are all at the end of the array
     //
@@ -1373,7 +1373,7 @@ BdsUpdateLegacyDevOrder (
     }
   }
 
-  if (HDCount) {
+  if (HDCount != 0) {
     //
     // Just to make sure that disabled indexes are all at the end of the array
     //
@@ -1393,7 +1393,7 @@ BdsUpdateLegacyDevOrder (
     }
   }
 
-  if (CDCount) {
+  if (CDCount != 0) {
     //
     // Just to make sure that disabled indexes are all at the end of the array
     //
@@ -1413,7 +1413,7 @@ BdsUpdateLegacyDevOrder (
     }
   }
 
-  if (NETCount) {
+  if (NETCount != 0) {
     //
     // Just to make sure that disabled indexes are all at the end of the array
     //
@@ -1433,7 +1433,7 @@ BdsUpdateLegacyDevOrder (
     }
   }
 
-  if (BEVCount) {
+  if (BEVCount!= 0) {
     //
     // Just to make sure that disabled indexes are all at the end of the array
     //
@@ -1468,7 +1468,7 @@ BdsUpdateLegacyDevOrder (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DeviceType      EDES_TODO: Add parameter description
   @param LocalBbsTable   EDES_TODO: Add parameter description
@@ -1538,7 +1538,7 @@ BdsSetBootPriority4SameTypeDev (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param LocalBbsTable   EDES_TODO: Add parameter description
 
@@ -1586,7 +1586,7 @@ PrintBbsTable (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Entry           EDES_TODO: Add parameter description
 
@@ -1671,7 +1671,7 @@ BdsRefreshBbsTableForBoot (
                           &gEfiGlobalVariableGuid,
                           &BootOrderSize
                           );
-  for (Index = 0; BootOrder && Index < BootOrderSize / sizeof (UINT16); Index++) {
+  for (Index = 0; ((BootOrder != NULL) && (Index < BootOrderSize / sizeof (UINT16))); Index++) {
     UnicodeSPrint (BootOption, sizeof (BootOption), L"Boot%04x", BootOrder[Index]);
     BootOptionVar = BdsLibGetVariableAndSize (
                       BootOption,
@@ -1714,7 +1714,7 @@ BdsRefreshBbsTableForBoot (
     }
   }
 
-  if (BootOrder) {
+  if (BootOrder != NULL) {
     SafeFreePool (BootOrder);
   }
   //
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
index 1b32084450..ce60cd9391 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
@@ -20,7 +20,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 // Bugbug: Candidate for a PCD entries
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CurBBSEntry     EDES_TODO: Add parameter description
   @param Index           EDES_TODO: Add parameter description
@@ -41,7 +41,7 @@ BdsBuildLegacyDevNameString (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -71,7 +71,7 @@ BdsAddNonExistingLegacyBootOptions (
 ;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -84,7 +84,7 @@ BdsUpdateLegacyDevOrder (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Entry           EDES_TODO: Add parameter description
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
index fbf2a0bfbe..d5c32c8ac1 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
@@ -145,7 +145,7 @@ EfiGrowBuffer (
   //
   // If this is an initial request, buffer will be null with a new buffer size
   //
-  if (!*Buffer && BufferSize) {
+  if ((*Buffer == NULL) && (BufferSize != 0)) {
     *Status = EFI_BUFFER_TOO_SMALL;
   }
   //
@@ -158,7 +158,7 @@ EfiGrowBuffer (
 
     *Buffer = EfiAllocateZeroPool (BufferSize);
 
-    if (*Buffer) {
+    if (*Buffer != NULL) {
       TryAgain = TRUE;
     } else {
       *Status = EFI_OUT_OF_RESOURCES;
@@ -167,7 +167,7 @@ EfiGrowBuffer (
   //
   // If there's an error, free the buffer
   //
-  if (!TryAgain && EFI_ERROR (*Status) && *Buffer) {
+  if (!TryAgain && EFI_ERROR (*Status) && (*Buffer != NULL)) {
     SafeFreePool (*Buffer);
     *Buffer = NULL;
   }
@@ -224,7 +224,7 @@ EfiLibDeleteVariable (
   VarBuf  = EfiLibGetVariable (VarName, VarGuid);
   Status  = EFI_NOT_FOUND;
 
-  if (VarBuf) {
+  if (VarBuf != NULL) {
     //
     // Delete variable from Storage
     //
@@ -296,7 +296,7 @@ EfiStrDuplicate (
   Size  = StrSize (Src);
   Dest  = EfiAllocateZeroPool (Size);
   ASSERT (Dest != NULL);
-  if (Dest) {
+  if (Dest != NULL) {
     CopyMem (Dest, Src, Size);
   }
 
@@ -393,12 +393,12 @@ EfiReallocatePool (
   VOID  *NewPool;
 
   NewPool = NULL;
-  if (NewSize) {
+  if (NewSize != 0) {
     NewPool = EfiAllocateZeroPool (NewSize);
   }
 
-  if (OldPool) {
-    if (NewPool) {
+  if (OldPool != NULL) {
+    if (NewPool != NULL) {
       CopyMem (NewPool, OldPool, OldSize < NewSize ? OldSize : NewSize);
     }
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
index 9fca22afbd..6eae9c18dd 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
@@ -38,7 +38,7 @@ InitAllMenu (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -723,7 +723,7 @@ Error:
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Private         EDES_TODO: Add parameter description
   @param CurrentFakeNVMap EDES_TODO: Add parameter description
@@ -812,7 +812,7 @@ InitializeBM (
   // Create LoadOption in BmmCallbackInfo for Driver Callback
   //
   Ptr = EfiAllocateZeroPool (sizeof (BM_LOAD_CONTEXT) + sizeof (BM_FILE_CONTEXT) + sizeof (BM_HANDLE_CONTEXT) + sizeof (BM_MENU_ENTRY));
-  if (!Ptr) {
+  if (Ptr == NULL) {
     SafeFreePool (BmmCallbackInfo);
     return EFI_OUT_OF_RESOURCES;
   }
@@ -1031,7 +1031,7 @@ InitializeBM (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1060,7 +1060,7 @@ InitAllMenu (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
index 2b8782944e..1ac2707c2f 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
@@ -470,7 +470,7 @@ typedef struct _STRING_DEPOSITORY {
 // For initializing File System menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -487,7 +487,7 @@ BOpt_FindFileSystem (
 // For cleaning up File System menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -504,7 +504,7 @@ BOpt_FreeFileSystem (
 // For initializing File Navigation menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param MenuEntry       EDES_TODO: Add parameter description
@@ -523,7 +523,7 @@ BOpt_FindFiles (
 // For cleaning up File Navigation menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -540,7 +540,7 @@ BOpt_FreeFiles (
 // For Initializing handle navigation menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -557,7 +557,7 @@ BOpt_FindDrivers (
 // For Cleaning up handle navigation menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -571,7 +571,7 @@ BOpt_FreeDrivers(VOID);
 // For initializing Boot Option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -587,7 +587,7 @@ BOpt_GetBootOptions (
 // For Initializing Driver option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -603,7 +603,7 @@ BOpt_GetDriverOptions (
 // For Cleaning up boot option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -617,7 +617,7 @@ BOpt_FreeBootOptions (VOID);
 // For cleaning up driver option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -631,7 +631,7 @@ BOpt_FreeDriverOptions(VOID);
 // For Initializing HD/FD/CD/NET/BEV option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -645,7 +645,7 @@ BOpt_GetLegacyOptions(VOID);
 // For cleaning up driver option menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -659,7 +659,7 @@ BOpt_FreeLegacyOptions(VOID);
 // this function is used to take place of all other free menu actions
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FreeMenu        EDES_TODO: Add parameter description
 
@@ -676,7 +676,7 @@ BOpt_FreeMenu (
 // Following are the helper functions used
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Str1            EDES_TODO: Add parameter description
   @param Str2            EDES_TODO: Add parameter description
@@ -691,7 +691,7 @@ BOpt_AppendFileName (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FileName        EDES_TODO: Add parameter description
 
@@ -704,7 +704,7 @@ BOpt_IsEfiImageName (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Dir             EDES_TODO: Add parameter description
   @param FileName        EDES_TODO: Add parameter description
@@ -722,7 +722,7 @@ BOpt_IsEfiApp (
 // Get current unused boot option number
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -736,7 +736,7 @@ BOpt_GetBootOptionNumber (VOID);
 // Get current unused driver option number
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -747,7 +747,7 @@ UINT16
 BOpt_GetDriverOptionNumber (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuType        EDES_TODO: Add parameter description
 
@@ -760,7 +760,7 @@ BOpt_CreateMenuEntry (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuEntry       EDES_TODO: Add parameter description
 
@@ -773,7 +773,7 @@ BOpt_DestroyMenuEntry (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuOption      EDES_TODO: Add parameter description
   @param MenuNumber      EDES_TODO: Add parameter description
@@ -791,7 +791,7 @@ BOpt_GetMenuEntry (
 // a helper function used to free pool type memory
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Buffer          EDES_TODO: Add parameter description
 
@@ -807,7 +807,7 @@ SafeFreePool (
 // Locate all serial io devices for console
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -821,7 +821,7 @@ LocateSerialIo (VOID);
 // Initializing Console menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -835,7 +835,7 @@ GetAllConsoles(VOID);
 // Get current mode information
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -851,7 +851,7 @@ GetConsoleOutMode (
 // Cleaning up console menu
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -862,7 +862,7 @@ EFI_STATUS
 FreeAllConsoles(VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
 
@@ -875,7 +875,7 @@ ChangeVariableDevicePath (
 );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
   @param ChangeTerminal  EDES_TODO: Add parameter description
@@ -892,7 +892,7 @@ ChangeTerminalDevicePath (
 // Variable operation by menu selection
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param NvRamMap        EDES_TODO: Add parameter description
@@ -907,7 +907,7 @@ Var_UpdateBootOption (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -918,7 +918,7 @@ EFI_STATUS
 Var_DelBootOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -929,7 +929,7 @@ EFI_STATUS
 Var_ChangeBootOrder (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param HiiHandle       EDES_TODO: Add parameter description
@@ -950,7 +950,7 @@ Var_UpdateDriverOption (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -961,7 +961,7 @@ EFI_STATUS
 Var_DelDriverOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -972,7 +972,7 @@ EFI_STATUS
 Var_ChangeDriverOrder (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -983,7 +983,7 @@ EFI_STATUS
 Var_UpdateConsoleInpOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -994,7 +994,7 @@ EFI_STATUS
 Var_UpdateConsoleOutOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1005,7 +1005,7 @@ EFI_STATUS
 Var_UpdateErrorOutOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1016,7 +1016,7 @@ VOID
 Var_UpdateAllConsoleOption (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1029,7 +1029,7 @@ Var_UpdateBootNext (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1042,7 +1042,7 @@ Var_UpdateBootOrder (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1055,7 +1055,7 @@ Var_UpdateDriverOrder (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1068,7 +1068,7 @@ Var_UpdateBBSOption (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1084,7 +1084,7 @@ Var_UpdateConMode (
 // Following are page create and refresh functions
 //
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1097,7 +1097,7 @@ RefreshUpdateData (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param LabelId         EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -1112,7 +1112,7 @@ CleanUpPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param UpdatingMenu    EDES_TODO: Add parameter description
@@ -1143,7 +1143,7 @@ UpdatePage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1156,7 +1156,7 @@ UpdateBootAddPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1169,7 +1169,7 @@ UpdateBootDelPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1182,7 +1182,7 @@ UpdateDrvAddFilePage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1195,7 +1195,7 @@ UpdateDrvAddHandlePage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1208,7 +1208,7 @@ UpdateDrvDelPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1221,7 +1221,7 @@ UpdateDriverAddHandleDescPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1234,7 +1234,7 @@ UpdateBootTimeOut (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1247,7 +1247,7 @@ UpdateConInPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1260,7 +1260,7 @@ UpdateConOutPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1273,7 +1273,7 @@ UpdateStdErrPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -1288,7 +1288,7 @@ UpdatePageBody (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1301,7 +1301,7 @@ UpdateCOM1Page (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1314,7 +1314,7 @@ UpdateCOM2Page (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1327,7 +1327,7 @@ UpdateBootOrderPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1340,7 +1340,7 @@ UpdateDriverOrderPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1353,7 +1353,7 @@ UpdateBootNextPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1366,7 +1366,7 @@ UpdateTimeOutPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1379,7 +1379,7 @@ UpdateTerminalPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1392,7 +1392,7 @@ UpdateConModePage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1405,7 +1405,7 @@ UpdateConCOMPage (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -1420,7 +1420,7 @@ UpdateSetLegacyDeviceOrderPage (
 );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param BbsType         EDES_TODO: Add parameter description
   @param BbsFlag         EDES_TODO: Add parameter description
@@ -1435,7 +1435,7 @@ BootLegacy (
 );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param TerminalNumber  EDES_TODO: Add parameter description
 
@@ -1448,7 +1448,7 @@ GetCurrentTerminal (
 );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DeviceHandle    EDES_TODO: Add parameter description
 
@@ -1461,7 +1461,7 @@ EfiLibOpenRoot (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FHand           EDES_TODO: Add parameter description
 
@@ -1474,7 +1474,7 @@ EfiLibFileSystemVolumeLabelInfo (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FHand           EDES_TODO: Add parameter description
 
@@ -1487,7 +1487,7 @@ EfiLibFileInfo (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UStr            EDES_TODO: Add parameter description
   @param Length          EDES_TODO: Add parameter description
@@ -1504,7 +1504,7 @@ UnicodeToAscii (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevPath         EDES_TODO: Add parameter description
 
@@ -1517,7 +1517,7 @@ DevicePathToStr (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Size            EDES_TODO: Add parameter description
 
@@ -1530,7 +1530,7 @@ EfiAllocateZeroPool (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param ProtocolGuid    EDES_TODO: Add parameter description
   @param Interface       EDES_TODO: Add parameter description
@@ -1545,7 +1545,7 @@ EfiLibLocateProtocol (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param OldPool         EDES_TODO: Add parameter description
   @param OldSize         EDES_TODO: Add parameter description
@@ -1562,7 +1562,7 @@ EfiReallocatePool (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevPath         EDES_TODO: Add parameter description
 
@@ -1575,7 +1575,7 @@ DevicePathToStr (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Name            EDES_TODO: Add parameter description
   @param VendorGuid      EDES_TODO: Add parameter description
@@ -1592,7 +1592,7 @@ BdsLibGetVariableAndSize (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VarName         EDES_TODO: Add parameter description
   @param VarGuid         EDES_TODO: Add parameter description
@@ -1607,7 +1607,7 @@ EfiLibDeleteVariable (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Src             EDES_TODO: Add parameter description
 
@@ -1620,7 +1620,7 @@ EfiStrDuplicate (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Multi           EDES_TODO: Add parameter description
   @param Single          EDES_TODO: Add parameter description
@@ -1635,7 +1635,7 @@ EfiLibMatchDevicePaths (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
 
@@ -1648,7 +1648,7 @@ EfiDevicePathInstanceCount (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param HiiHandle       EDES_TODO: Add parameter description
@@ -1665,7 +1665,7 @@ CreateMenuStringToken (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevPath         EDES_TODO: Add parameter description
 
@@ -1678,7 +1678,7 @@ EfiLibStrFromDatahub (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DeviceType      EDES_TODO: Add parameter description
   @param OptionIndex     EDES_TODO: Add parameter description
@@ -1695,7 +1695,7 @@ GetLegacyBootOptionVar (
  );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1708,7 +1708,7 @@ InitializeBM (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1721,7 +1721,7 @@ BdsStartBootMaint (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1732,7 +1732,7 @@ VOID
 InitializeStringDepository (VOID);
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param StringDepository EDES_TODO: Add parameter description
@@ -1747,7 +1747,7 @@ GetStringTokenFromDepository (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1760,7 +1760,7 @@ ReclaimStringDepository (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -1773,7 +1773,7 @@ CleanUpStringDepository (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Private         EDES_TODO: Add parameter description
   @param CurrentFakeNVMap EDES_TODO: Add parameter description
@@ -1790,7 +1790,7 @@ ApplyChangeHandler (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Private         EDES_TODO: Add parameter description
   @param CurrentFakeNVMap EDES_TODO: Add parameter description
@@ -1805,7 +1805,7 @@ DiscardChangeHandler (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Private         EDES_TODO: Add parameter description
   @param NewPageId       EDES_TODO: Add parameter description
@@ -1820,7 +1820,7 @@ UpdatePageId (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FileContext     EDES_TODO: Add parameter description
 
@@ -1833,7 +1833,7 @@ BootThisFile (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param KeyValue        EDES_TODO: Add parameter description
@@ -1848,7 +1848,7 @@ UpdateFileExplorer (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param This            EDES_TODO: Add parameter description
   @param Action          EDES_TODO: Add parameter description
@@ -1872,7 +1872,7 @@ FileExplorerCallback (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1885,7 +1885,7 @@ FormSetDispatcher (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Name            EDES_TODO: Add parameter description
   @param VendorGuid      EDES_TODO: Add parameter description
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
index ebaa48508e..89c8612d46 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
@@ -20,7 +20,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "BBSsupport.h"
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuType        Use this parameter to identify current
                          Menu type
@@ -92,7 +92,7 @@ BOpt_CreateMenuEntry (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuEntry       EDES_TODO: Add parameter description
 
@@ -173,7 +173,7 @@ BOpt_DestroyMenuEntry (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param MenuOption      EDES_TODO: Add parameter description
   @param MenuNumber      EDES_TODO: Add parameter description
@@ -206,7 +206,7 @@ BOpt_GetMenuEntry (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    BMM context data
 
@@ -338,7 +338,7 @@ BOpt_FindFileSystem (
       FileContext->Handle     = SimpleFsHandle[Index];
       MenuEntry->OptionNumber = Index;
       FileContext->FHandle    = EfiLibOpenRoot (FileContext->Handle);
-      if (!FileContext->FHandle) {
+      if (FileContext->FHandle == NULL) {
         BOpt_DestroyMenuEntry (MenuEntry);
         continue;
       }
@@ -583,7 +583,7 @@ Returns:
   }
 
   DirInfo = EfiLibFileInfo (NewDir);
-  if (!DirInfo) {
+  if (DirInfo == NULL) {
     return EFI_NOT_FOUND;
   }
 
@@ -598,7 +598,7 @@ Returns:
 
   DirBufferSize = sizeof (EFI_FILE_INFO) + 1024;
   DirInfo       = EfiAllocateZeroPool (DirBufferSize);
-  if (!DirInfo) {
+  if (DirInfo == NULL) {
     return EFI_OUT_OF_RESOURCES;
   }
   //
@@ -908,7 +908,7 @@ Returns:
               &BootNextSize
               );
 
-  if (BootNext) {
+  if (BootNext != NULL) {
     if (BootNextSize != sizeof (UINT16)) {
       SafeFreePool (BootNext);
       BootNext = NULL;
@@ -925,19 +925,19 @@ Returns:
                           &gEfiGlobalVariableGuid,
                           &BootOptionSize
                           );
-    if (!LoadOptionFromVar) {
+    if (LoadOptionFromVar == NULL) {
       continue;
     }
 
     LoadOption = EfiAllocateZeroPool (BootOptionSize);
-    if (!LoadOption) {
+    if (LoadOption == NULL) {
       continue;
     }
 
     CopyMem (LoadOption, LoadOptionFromVar, BootOptionSize);
     SafeFreePool (LoadOptionFromVar);
 
-    if (BootNext) {
+    if (BootNext != NULL) {
       BootNextFlag = (BOOLEAN) (*BootNext == BootOrderList[Index]);
     } else {
       BootNextFlag = FALSE;
@@ -1086,7 +1086,7 @@ BdsStrCpy (
   ASSERT (Destination != NULL);
 
   ReturnValue = Destination;
-  while (*Source) {
+  while (*Source != L'\0') {
     *(Destination++) = *(Source++);
   }
   *Destination = 0;
@@ -1180,7 +1180,7 @@ Returns:
   //
   // Search for ".efi" extension
   //
-  while (*FileName) {
+  while (*FileName != L'\0') {
     if (FileName[0] == '.') {
       if (FileName[1] == 'e' || FileName[1] == 'E') {
         if (FileName[2] == 'f' || FileName[2] == 'F') {
@@ -1406,7 +1406,7 @@ Returns:
                     &gEfiGlobalVariableGuid,
                     &BootOrderListSize
                     );
-  if (BootOrderList) {
+  if (BootOrderList != NULL) {
     //
     // already have Boot####
     //
@@ -1480,7 +1480,7 @@ Returns:
                       &gEfiGlobalVariableGuid,
                       &DriverOrderListSize
                       );
-  if (DriverOrderList) {
+  if (DriverOrderList != NULL) {
     //
     // already have Driver####
     //
@@ -1575,12 +1575,12 @@ Returns:
                           &gEfiGlobalVariableGuid,
                           &DriverOptionSize
                           );
-    if (!LoadOptionFromVar) {
+    if (LoadOptionFromVar == NULL) {
       continue;
     }
 
     LoadOption = EfiAllocateZeroPool (DriverOptionSize);
-    if (!LoadOption) {
+    if (LoadOption == NULL) {
       continue;
     }
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
index e3a22f26d0..270948633b 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
@@ -15,7 +15,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "BootMaint.h"
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevPath         EDES_TODO: Add parameter description
 
@@ -28,7 +28,7 @@ DevicePathInstanceDup (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
 
@@ -41,7 +41,7 @@ UpdateComAttributeFromVariable (
   );
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
   @param ChangeTerminal  EDES_TODO: Add parameter description
@@ -162,7 +162,7 @@ ChangeTerminalDevicePath (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
 
@@ -328,7 +328,7 @@ SortedUartHandle (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DevicePath      EDES_TODO: Add parameter description
   @param Termi           EDES_TODO: Add parameter description
@@ -420,7 +420,7 @@ LocateSerialIo (
 
     if (CompareMem (&Acpi->HID, &Match, sizeof (UINT32)) == 0) {
       NewMenuEntry = BOpt_CreateMenuEntry (BM_TERMINAL_CONTEXT_SELECT);
-      if (!NewMenuEntry) {
+      if (NewMenuEntry == NULL) {
         SafeFreePool (Handles);
         return EFI_OUT_OF_RESOURCES;
       }
@@ -482,15 +482,15 @@ LocateSerialIo (
   OutDevicePath = EfiLibGetVariable (L"ConOut", &gEfiGlobalVariableGuid);
   InpDevicePath = EfiLibGetVariable (L"ConIn", &gEfiGlobalVariableGuid);
   ErrDevicePath = EfiLibGetVariable (L"ErrOut", &gEfiGlobalVariableGuid);
-  if (OutDevicePath) {
+  if (OutDevicePath != NULL) {
     UpdateComAttributeFromVariable (OutDevicePath);
   }
 
-  if (InpDevicePath) {
+  if (InpDevicePath != NULL) {
     UpdateComAttributeFromVariable (InpDevicePath);
   }
 
-  if (ErrDevicePath) {
+  if (ErrDevicePath != NULL) {
     UpdateComAttributeFromVariable (ErrDevicePath);
   }
 
@@ -697,12 +697,12 @@ DevicePathInstanceDup (
   // Make a copy and set proper end type
   //
   NewDevPath = NULL;
-  if (Size) {
+  if (Size != 0) {
     NewDevPath = EfiAllocateZeroPool (Size);
     ASSERT (NewDevPath != NULL);
   }
 
-  if (NewDevPath) {
+  if (NewDevPath != NULL) {
     CopyMem (NewDevPath, DevicePathInst, Size);
     Ptr = (UINT8 *) NewDevPath;
     Ptr += Size - sizeof (EFI_DEVICE_PATH_PROTOCOL);
@@ -714,7 +714,7 @@ DevicePathInstanceDup (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param ConsoleMenuType EDES_TODO: Add parameter description
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
index be84fdb563..3dd5df7997 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
@@ -32,7 +32,7 @@ RefreshUpdateData (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -64,7 +64,7 @@ UpdatePageStart (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -121,7 +121,7 @@ UpdatePageEnd (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param LabelId         EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -151,7 +151,7 @@ CleanUpPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param FileContext     EDES_TODO: Add parameter description
 
@@ -192,7 +192,7 @@ BootThisFile (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -229,7 +229,7 @@ UpdateConCOMPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -276,7 +276,7 @@ UpdateBootDelPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -312,7 +312,7 @@ UpdateDrvAddHandlePage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -357,7 +357,7 @@ UpdateDrvDelPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -427,7 +427,7 @@ UpdateDriverAddHandleDescPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param ConsoleMenu     EDES_TODO: Add parameter description
@@ -482,9 +482,9 @@ UpdateConsolePage (
     NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, Index2);
     NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;
 
-    if ((NewTerminalContext->IsConIn && (UpdatePageId == FORM_CON_IN_ID)) ||
-        (NewTerminalContext->IsConOut && (UpdatePageId == FORM_CON_OUT_ID)) ||
-        (NewTerminalContext->IsStdErr && (UpdatePageId == FORM_CON_ERR_ID))
+    if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||
+        ((NewTerminalContext->IsConOut != 0) && (UpdatePageId == FORM_CON_OUT_ID)) ||
+        ((NewTerminalContext->IsStdErr != 0) && (UpdatePageId == FORM_CON_ERR_ID))
         ) {
       CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;
       CallbackData->BmmFakeNvData.ConsoleCheck[Index] = TRUE;
@@ -510,7 +510,7 @@ UpdateConsolePage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param OptionMenu      EDES_TODO: Add parameter description
@@ -580,7 +580,7 @@ UpdateOrderPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -653,7 +653,7 @@ UpdateBootNextPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -802,7 +802,7 @@ UpdateConModePage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -976,7 +976,7 @@ UpdateTerminalPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -1018,7 +1018,7 @@ UpdatePageBody (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param DeviceType      EDES_TODO: Add parameter description
   @param OptionIndex     EDES_TODO: Add parameter description
@@ -1100,7 +1100,7 @@ GetLegacyBootOptionVar (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param UpdatePageId    EDES_TODO: Add parameter description
   @param CallbackData    EDES_TODO: Add parameter description
@@ -1304,7 +1304,7 @@ UpdateSetLegacyDeviceOrderPage (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param Private         EDES_TODO: Add parameter description
   @param NewPageId       EDES_TODO: Add parameter description
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
index 498a40f722..6479cd0f17 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
@@ -121,7 +121,7 @@ Var_ChangeBootOrder (
   //
   // If exists, delete it to hold new BootOrder
   //
-  if (BootOrderList) {
+  if (BootOrderList != NULL) {
     EfiLibDeleteVariable (L"BootOrder", &gEfiGlobalVariableGuid);
     SafeFreePool (BootOrderList);
     BootOrderList = NULL;
@@ -271,7 +271,7 @@ Var_ChangeDriverOrder (
   //
   // If exists, delete it to hold new DriverOrder
   //
-  if (DriverOrderList) {
+  if (DriverOrderList != NULL) {
     EfiLibDeleteVariable (L"DriverOrder", &gEfiGlobalVariableGuid);
     SafeFreePool (DriverOrderList);
     DriverOrderList = NULL;
@@ -315,7 +315,7 @@ Var_ChangeDriverOrder (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -335,7 +335,7 @@ Var_UpdateAllConsoleOption (
   OutDevicePath = EfiLibGetVariable (L"ConOut", &gEfiGlobalVariableGuid);
   InpDevicePath = EfiLibGetVariable (L"ConIn", &gEfiGlobalVariableGuid);
   ErrDevicePath = EfiLibGetVariable (L"ErrOut", &gEfiGlobalVariableGuid);
-  if (OutDevicePath) {
+  if (OutDevicePath != NULL) {
     ChangeVariableDevicePath (OutDevicePath);
     Status = gRT->SetVariable (
                     L"ConOut",
@@ -347,7 +347,7 @@ Var_UpdateAllConsoleOption (
     ASSERT (!EFI_ERROR (Status));
   }
 
-  if (InpDevicePath) {
+  if (InpDevicePath != NULL) {
     ChangeVariableDevicePath (InpDevicePath);
     Status = gRT->SetVariable (
                     L"ConIn",
@@ -359,7 +359,7 @@ Var_UpdateAllConsoleOption (
     ASSERT (!EFI_ERROR (Status));
   }
 
-  if (ErrDevicePath) {
+  if (ErrDevicePath != NULL) {
     ChangeVariableDevicePath (ErrDevicePath);
     Status = gRT->SetVariable (
                     L"ErrOut",
@@ -373,7 +373,7 @@ Var_UpdateAllConsoleOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param ConsoleName     EDES_TODO: Add parameter description
   @param ConsoleMenu     EDES_TODO: Add parameter description
@@ -430,9 +430,9 @@ Var_UpdateConsoleOption (
     }
 
     NewTerminalContext = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;
-    if ((NewTerminalContext->IsConIn && (UpdatePageId == FORM_CON_IN_ID)) ||
-        (NewTerminalContext->IsConOut && (UpdatePageId == FORM_CON_OUT_ID)) ||
-        (NewTerminalContext->IsStdErr && (UpdatePageId == FORM_CON_ERR_ID))
+    if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||
+        ((NewTerminalContext->IsConOut != 0)  && (UpdatePageId == FORM_CON_OUT_ID)) ||
+        ((NewTerminalContext->IsStdErr  != 0) && (UpdatePageId == FORM_CON_ERR_ID))
         ) {
       Vendor.Header.Type    = MESSAGING_DEVICE_PATH;
       Vendor.Header.SubType = MSG_VENDOR_DP;
@@ -455,7 +455,7 @@ Var_UpdateConsoleOption (
     }
   }
 
-  if (ConDevicePath) {
+  if (ConDevicePath != NULL) {
     Status = gRT->SetVariable (
                     ConsoleName,
                     &gEfiGlobalVariableGuid,
@@ -473,7 +473,7 @@ Var_UpdateConsoleOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -489,7 +489,7 @@ Var_UpdateConsoleInpOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -505,7 +505,7 @@ Var_UpdateConsoleOutOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -521,7 +521,7 @@ Var_UpdateErrorOutOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param HiiHandle       EDES_TODO: Add parameter description
@@ -698,7 +698,7 @@ Var_UpdateDriverOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
   @param NvRamMap        EDES_TODO: Add parameter description
@@ -865,7 +865,7 @@ Var_UpdateBootOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -922,7 +922,7 @@ Var_UpdateBootNext (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -954,18 +954,18 @@ Var_UpdateBootOrder (
                     );
 
   NewBootOrderList = EfiAllocateZeroPool (BootOrderListSize);
-  if (!NewBootOrderList) {
+  if (NewBootOrderList == NULL) {
     return EFI_OUT_OF_RESOURCES;
   }
 
   Map = EfiAllocateZeroPool (BootOrderListSize / sizeof (UINT16));
-  if (!Map) {
+  if (Map == NULL) {
     return EFI_OUT_OF_RESOURCES;
   }
   //
   // If exists, delete it to hold new BootOrder
   //
-  if (BootOrderList) {
+  if (BootOrderList != NULL) {
     EfiLibDeleteVariable (L"BootOrder", &gEfiGlobalVariableGuid);
   }
 
@@ -995,7 +995,7 @@ Var_UpdateBootOrder (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1027,13 +1027,13 @@ Var_UpdateDriverOrder (
 
   NewDriverOrderList = EfiAllocateZeroPool (DriverOrderListSize);
 
-  if (!NewDriverOrderList) {
+  if (NewDriverOrderList == NULL) {
     return EFI_OUT_OF_RESOURCES;
   }
   //
   // If exists, delete it to hold new DriverOrder
   //
-  if (DriverOrderList) {
+  if (DriverOrderList != NULL) {
     EfiLibDeleteVariable (L"DriverOrder", &gEfiGlobalVariableGuid);
   }
 
@@ -1060,7 +1060,7 @@ Var_UpdateDriverOrder (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
@@ -1202,7 +1202,7 @@ Var_UpdateBBSOption (
     Tmp &= 0xFF;
     Pos = Tmp / 8;
     Bit = 7 - (Tmp % 8);
-    if (DisMap[Pos] & (1 << Bit)) {
+    if ((DisMap[Pos] & (1 << Bit)) != 0) {
       NewOrder[Index] = (UINT16) (0xFF00 | Tmp);
       Index++;
     }
@@ -1364,7 +1364,7 @@ Var_UpdateBBSOption (
 }
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param CallbackData    EDES_TODO: Add parameter description
 
diff --git a/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h b/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
index 5c83394fc5..3b40fa6d42 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
@@ -53,7 +53,7 @@ typedef struct {
 } BOOT_MANAGER_CALLBACK_DATA;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param This            EDES_TODO: Add parameter description
   @param Action          EDES_TODO: Add parameter description
@@ -78,7 +78,7 @@ BootManagerCallback (
 ;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -92,7 +92,7 @@ InitializeBootManager (
 ;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
diff --git a/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h b/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
index 3ced4c20c0..3b1564399e 100644
--- a/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
+++ b/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
@@ -72,7 +72,7 @@ typedef struct {
 } DEVICE_MANAGER_MENU_ITEM;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param This            EDES_TODO: Add parameter description
   @param Action          EDES_TODO: Add parameter description
@@ -97,7 +97,7 @@ DeviceManagerCallback (
 ;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description
 
@@ -111,7 +111,7 @@ InitializeDeviceManager (
 ;
 
 /**
-  EDES_TODO: Add function description
+  EDES_TODO: Add function description.
 
   @param VOID            EDES_TODO: Add parameter description