public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Ni, Ruiyu" <ruiyu.ni@intel.com>
To: "Bi, Dandan" <dandan.bi@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [patch 4/8] FatPkg\EnhancedFatDxe: Make the variable name follow rule
Date: Fri, 9 Dec 2016 01:12:08 +0000	[thread overview]
Message-ID: <734D49CCEBEEF84792F5B80ED585239D58EAFB59@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <1481194467-75920-5-git-send-email-dandan.bi@intel.com>

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Regards,
Ray

>-----Original Message-----
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu <ruiyu.ni@intel.com>
>Subject: [patch 4/8] FatPkg\EnhancedFatDxe: Make the variable name follow rule
>
>Cc: Ruiyu Ni <ruiyu.ni@intel.com>
>Contributed-under: TianoCore Contribution Agreement 1.0
>Signed-off-by: Dandan Bi <dandan.bi@intel.com>
>---
> FatPkg/EnhancedFatDxe/DirectoryManage.c | 16 ++++-----
> FatPkg/EnhancedFatDxe/DiskCache.c       | 50 +++++++++++++--------------
> FatPkg/EnhancedFatDxe/Fat.h             | 28 +++++++--------
> FatPkg/EnhancedFatDxe/FileSpace.c       | 60 ++++++++++++++++-----------------
> FatPkg/EnhancedFatDxe/Flush.c           |  6 ++--
> FatPkg/EnhancedFatDxe/Init.c            | 16 ++++-----
> FatPkg/EnhancedFatDxe/Misc.c            |  4 +--
> FatPkg/EnhancedFatDxe/ReadWrite.c       | 20 +++++------
> 8 files changed, 100 insertions(+), 100 deletions(-)
>
>diff --git a/FatPkg/EnhancedFatDxe/DirectoryManage.c b/FatPkg/EnhancedFatDxe/DirectoryManage.c
>index 3328ae3..149119d 100644
>--- a/FatPkg/EnhancedFatDxe/DirectoryManage.c
>+++ b/FatPkg/EnhancedFatDxe/DirectoryManage.c
>@@ -58,11 +58,11 @@ Returns:
>   Position = EntryPos * sizeof (FAT_DIRECTORY_ENTRY);
>   if (Position >= Parent->FileSize) {
>     //
>     // End of directory
>     //
>-    ASSERT (IoMode == READ_DATA);
>+    ASSERT (IoMode == ReadData);
>     ((FAT_DIRECTORY_ENTRY *) Entry)->FileName[0] = EMPTY_ENTRY_MARK;
>     ((FAT_DIRECTORY_ENTRY *) Entry)->Attributes  = 0;
>     return EFI_SUCCESS;
>   }
>
>@@ -104,11 +104,11 @@ Returns:
>   EntryPos   = DirEnt->EntryPos;
>   EntryCount = DirEnt->EntryCount;
>   //
>   // Write directory entry
>   //
>-  Status = FatAccessEntry (OFile, WRITE_DATA, EntryPos, &DirEnt->Entry);
>+  Status = FatAccessEntry (OFile, WriteData, EntryPos, &DirEnt->Entry);
>   if (EFI_ERROR (Status)) {
>     return Status;
>   }
>
>   if (--EntryCount > 0) {
>@@ -145,11 +145,11 @@ Returns:
>       EntryPos--;
>       if (DirEnt->Invalid) {
>         LfnEntry.Ordinal = DELETE_ENTRY_MARK;
>       }
>
>-      Status = FatAccessEntry (OFile, WRITE_DATA, EntryPos, &LfnEntry);
>+      Status = FatAccessEntry (OFile, WriteData, EntryPos, &LfnEntry);
>       if (EFI_ERROR (Status)) {
>         return Status;
>       }
>     }
>   }
>@@ -320,11 +320,11 @@ Returns:
>       LfnBufferPointer = LfnBuffer;
>       break;
>     }
>
>     EntryPos--;
>-    Status = FatAccessEntry (Parent, READ_DATA, EntryPos, &LfnEntry);
>+    Status = FatAccessEntry (Parent, ReadData, EntryPos, &LfnEntry);
>     if (EFI_ERROR (Status) ||
>         LfnEntry.Attributes != FAT_ATTRIBUTE_LFN ||
>         LfnEntry.MustBeZero != 0 ||
>         LfnEntry.Checksum != LfnChecksum ||
>         (LfnEntry.Ordinal & (~FAT_LFN_LAST)) != LfnOrdinal ||
>@@ -440,11 +440,11 @@ Returns:
>
>   for (;;) {
>     //
>     // Read the next directory entry until we find a valid directory entry (excluding lfn entry)
>     //
>-    Status = FatAccessEntry (OFile, READ_DATA, ODir->CurrentEndPos, &Entry);
>+    Status = FatAccessEntry (OFile, ReadData, ODir->CurrentEndPos, &Entry);
>     if (EFI_ERROR (Status)) {
>       return Status;
>     }
>
>     if (((UINT8) Entry.FileName[0] != DELETE_ENTRY_MARK) && (Entry.Attributes & FAT_ATTRIBUTE_VOLUME_ID) == 0) {
>@@ -460,11 +460,11 @@ Returns:
>   if (Entry.FileName[0] != EMPTY_ENTRY_MARK) {
>     //
>     // Although FAT spec states this field is always 0 for FAT12 & FAT16, some applications
>     // might use it for some special usage, it is safer to zero it in memory for FAT12 & FAT16.
>     //
>-    if (OFile->Volume->FatType != FAT32) {
>+    if (OFile->Volume->FatType != Fat32) {
>       Entry.FileClusterHigh = 0;
>     }
>
>     //
>     // This is a valid directory entry
>@@ -855,11 +855,11 @@ Returns:
>
>   EntryPos        = 0;
>   Entry           = &DirEnt->Entry;
>   DirEnt->Invalid = TRUE;
>   do {
>-    Status = FatAccessEntry (Root, READ_DATA, EntryPos, Entry);
>+    Status = FatAccessEntry (Root, ReadData, EntryPos, Entry);
>     if (EFI_ERROR (Status)) {
>       return Status;
>     }
>
>     if (((UINT8) Entry->FileName[0] != DELETE_ENTRY_MARK) && (((Entry->Attributes) & (~FAT_ATTRIBUTE_ARCHIVE)) ==
>FAT_ATTRIBUTE_VOLUME_ID)) {
>@@ -1333,11 +1333,11 @@ Returns:
>       // The newly created OFile is root
>       //
>       Volume                = VOLUME_FROM_ROOT_DIRENT (DirEnt);
>       Volume->Root          = OFile;
>       OFile->FileCluster    = Volume->RootCluster;
>-      if (Volume->FatType  != FAT32) {
>+      if (Volume->FatType  != Fat32) {
>         OFile->IsFixedRootDir  = TRUE;
>       }
>     }
>
>     OFile->FileCurrentCluster  = OFile->FileCluster;
>diff --git a/FatPkg/EnhancedFatDxe/DiskCache.c b/FatPkg/EnhancedFatDxe/DiskCache.c
>index ddf99ed..7f1fcf4 100644
>--- a/FatPkg/EnhancedFatDxe/DiskCache.c
>+++ b/FatPkg/EnhancedFatDxe/DiskCache.c
>@@ -68,11 +68,11 @@ Returns:
>   UINT8       PageAlignment;
>   DISK_CACHE  *DiskCache;
>   CACHE_TAG   *CacheTag;
>   UINT8       *BaseAddress;
>
>-  DiskCache     = &Volume->DiskCache[CACHE_DATA];
>+  DiskCache     = &Volume->DiskCache[CacheData];
>   BaseAddress   = DiskCache->CacheBase;
>   GroupMask     = DiskCache->GroupMask;
>   PageAlignment = DiskCache->PageAlignment;
>   PageSize      = (UINTN)1 << PageAlignment;
>
>@@ -83,11 +83,11 @@ Returns:
>       //
>       // When reading data form disk directly, if some dirty data
>       // in cache is in this rang, this data in the Buffer need to
>       // be updated with the cache's dirty data.
>       //
>-      if (IoMode == READ_DISK) {
>+      if (IoMode == ReadDisk) {
>         if (CacheTag->Dirty) {
>           CopyMem (
>             Buffer + ((PageNo - StartPageNo) << PageAlignment),
>             BaseAddress + (GroupNo << PageAlignment),
>             PageSize
>@@ -148,21 +148,21 @@ Returns:
>   GroupNo       = PageNo & DiskCache->GroupMask;
>   PageAlignment = DiskCache->PageAlignment;
>   PageAddress   = DiskCache->CacheBase + (GroupNo << PageAlignment);
>   EntryPos      = DiskCache->BaseAddress + LShiftU64 (PageNo, PageAlignment);
>   RealSize      = CacheTag->RealSize;
>-  if (IoMode == READ_DISK) {
>+  if (IoMode == ReadDisk) {
>     RealSize  = (UINTN)1 << PageAlignment;
>     MaxSize   = DiskCache->LimitAddress - EntryPos;
>     if (MaxSize < RealSize) {
>       DEBUG ((EFI_D_INFO, "FatDiskIo: Cache Page OutBound occurred! \n"));
>       RealSize = (UINTN) MaxSize;
>     }
>   }
>
>   WriteCount = 1;
>-  if (DataType == CACHE_FAT && IoMode == WRITE_DISK) {
>+  if (DataType == CacheFat && IoMode == WriteDisk) {
>     WriteCount = Volume->NumFats;
>   }
>
>   do {
>     //
>@@ -222,20 +222,20 @@ Returns:
>
>   //
>   // Write dirty cache page back to disk
>   //
>   if (CacheTag->RealSize > 0 && CacheTag->Dirty) {
>-    Status = FatExchangeCachePage (Volume, CacheDataType, WRITE_DISK, CacheTag, NULL);
>+    Status = FatExchangeCachePage (Volume, CacheDataType, WriteDisk, CacheTag, NULL);
>     if (EFI_ERROR (Status)) {
>       return Status;
>     }
>   }
>   //
>   // Load new data from disk;
>   //
>   CacheTag->PageNo  = PageNo;
>-  Status            = FatExchangeCachePage (Volume, CacheDataType, READ_DISK, CacheTag, NULL);
>+  Status            = FatExchangeCachePage (Volume, CacheDataType, ReadDisk, CacheTag, NULL);
>
>   return Status;
> }
>
> STATIC
>@@ -284,11 +284,11 @@ Returns:
>   CacheTag  = &DiskCache->CacheTag[GroupNo];
>   Status    = FatGetCachePage (Volume, CacheDataType, PageNo, CacheTag);
>   if (!EFI_ERROR (Status)) {
>     Source      = DiskCache->CacheBase + (GroupNo << DiskCache->PageAlignment) + Offset;
>     Destination = Buffer;
>-    if (IoMode != READ_DISK) {
>+    if (IoMode != ReadDisk) {
>       CacheTag->Dirty   = TRUE;
>       DiskCache->Dirty  = TRUE;
>       Destination       = Source;
>       Source            = Buffer;
>     }
>@@ -389,11 +389,11 @@ Returns:
>   //
>   if (AlignedPageCount > 0) {
>     //
>     // Accessing fat table cannot have alignment data
>     //
>-    ASSERT (CacheDataType == CACHE_DATA);
>+    ASSERT (CacheDataType == CacheData);
>
>     EntryPos    = Volume->RootPos + LShiftU64 (PageNo, PageAlignment);
>     AlignedSize = AlignedPageCount << PageAlignment;
>     Status      = FatDiskIo (Volume, IoMode, EntryPos, AlignedSize, Buffer, Task);
>     if (EFI_ERROR (Status)) {
>@@ -448,11 +448,11 @@ Returns:
>   UINTN           GroupIndex;
>   UINTN           GroupMask;
>   DISK_CACHE      *DiskCache;
>   CACHE_TAG       *CacheTag;
>
>-  for (CacheDataType = (CACHE_DATA_TYPE) 0; CacheDataType < CACHE_MAX_TYPE; CacheDataType++) {
>+  for (CacheDataType = (CACHE_DATA_TYPE) 0; CacheDataType < CacheMaxType; CacheDataType++) {
>     DiskCache = &Volume->DiskCache[CacheDataType];
>     if (DiskCache->Dirty) {
>       //
>       // Data cache or fat cache is dirty, write the dirty data back
>       //
>@@ -461,11 +461,11 @@ Returns:
>         CacheTag = &DiskCache->CacheTag[GroupIndex];
>         if (CacheTag->RealSize > 0 && CacheTag->Dirty) {
>           //
>           // Write back all Dirty Data Cache Page to disk
>           //
>-          Status = FatExchangeCachePage (Volume, CacheDataType, WRITE_DISK, CacheTag, Task);
>+          Status = FatExchangeCachePage (Volume, CacheDataType, WriteDisk, CacheTag, Task);
>           if (EFI_ERROR (Status)) {
>             return Status;
>           }
>         }
>       }
>@@ -509,36 +509,36 @@ Returns:
>
>   DiskCache = Volume->DiskCache;
>   //
>   // Configure the parameters of disk cache
>   //
>-  if (Volume->FatType == FAT12) {
>+  if (Volume->FatType == Fat12) {
>     FatCacheGroupCount                  = FAT_FATCACHE_GROUP_MIN_COUNT;
>-    DiskCache[CACHE_FAT].PageAlignment  = FAT_FATCACHE_PAGE_MIN_ALIGNMENT;
>-    DiskCache[CACHE_DATA].PageAlignment = FAT_DATACACHE_PAGE_MIN_ALIGNMENT;
>+    DiskCache[CacheFat].PageAlignment  = FAT_FATCACHE_PAGE_MIN_ALIGNMENT;
>+    DiskCache[CacheData].PageAlignment = FAT_DATACACHE_PAGE_MIN_ALIGNMENT;
>   } else {
>     FatCacheGroupCount                  = FAT_FATCACHE_GROUP_MAX_COUNT;
>-    DiskCache[CACHE_FAT].PageAlignment  = FAT_FATCACHE_PAGE_MAX_ALIGNMENT;
>-    DiskCache[CACHE_DATA].PageAlignment = FAT_DATACACHE_PAGE_MAX_ALIGNMENT;
>+    DiskCache[CacheFat].PageAlignment  = FAT_FATCACHE_PAGE_MAX_ALIGNMENT;
>+    DiskCache[CacheData].PageAlignment = FAT_DATACACHE_PAGE_MAX_ALIGNMENT;
>   }
>
>-  DiskCache[CACHE_DATA].GroupMask     = FAT_DATACACHE_GROUP_COUNT - 1;
>-  DiskCache[CACHE_DATA].BaseAddress   = Volume->RootPos;
>-  DiskCache[CACHE_DATA].LimitAddress  = Volume->VolumeSize;
>-  DiskCache[CACHE_FAT].GroupMask      = FatCacheGroupCount - 1;
>-  DiskCache[CACHE_FAT].BaseAddress    = Volume->FatPos;
>-  DiskCache[CACHE_FAT].LimitAddress   = Volume->FatPos + Volume->FatSize;
>-  FatCacheSize                        = FatCacheGroupCount << DiskCache[CACHE_FAT].PageAlignment;
>-  DataCacheSize                       = FAT_DATACACHE_GROUP_COUNT <<
>DiskCache[CACHE_DATA].PageAlignment;
>+  DiskCache[CacheData].GroupMask     = FAT_DATACACHE_GROUP_COUNT - 1;
>+  DiskCache[CacheData].BaseAddress   = Volume->RootPos;
>+  DiskCache[CacheData].LimitAddress  = Volume->VolumeSize;
>+  DiskCache[CacheFat].GroupMask      = FatCacheGroupCount - 1;
>+  DiskCache[CacheFat].BaseAddress    = Volume->FatPos;
>+  DiskCache[CacheFat].LimitAddress   = Volume->FatPos + Volume->FatSize;
>+  FatCacheSize                        = FatCacheGroupCount << DiskCache[CacheFat].PageAlignment;
>+  DataCacheSize                       = FAT_DATACACHE_GROUP_COUNT <<
>DiskCache[CacheData].PageAlignment;
>   //
>   // Allocate the Fat Cache buffer
>   //
>   CacheBuffer = AllocateZeroPool (FatCacheSize + DataCacheSize);
>   if (CacheBuffer == NULL) {
>     return EFI_OUT_OF_RESOURCES;
>   }
>
>   Volume->CacheBuffer             = CacheBuffer;
>-  DiskCache[CACHE_FAT].CacheBase  = CacheBuffer;
>-  DiskCache[CACHE_DATA].CacheBase = CacheBuffer + FatCacheSize;
>+  DiskCache[CacheFat].CacheBase  = CacheBuffer;
>+  DiskCache[CacheData].CacheBase = CacheBuffer + FatCacheSize;
>   return EFI_SUCCESS;
> }
>diff --git a/FatPkg/EnhancedFatDxe/Fat.h b/FatPkg/EnhancedFatDxe/Fat.h
>index b73135c..42aa647 100644
>--- a/FatPkg/EnhancedFatDxe/Fat.h
>+++ b/FatPkg/EnhancedFatDxe/Fat.h
>@@ -126,32 +126,32 @@ typedef CHAR8                   LC_ISO_639_2;
>
> //
> // The fat types we support
> //
> typedef enum {
>-  FAT12,
>-  FAT16,
>-  FAT32,
>+  Fat12,
>+  Fat16,
>+  Fat32,
>   FatUndefined
> } FAT_VOLUME_TYPE;
>
> typedef enum {
>-  CACHE_FAT,
>-  CACHE_DATA,
>-  CACHE_MAX_TYPE
>+  CacheFat,
>+  CacheData,
>+  CacheMaxType
> } CACHE_DATA_TYPE;
>
> //
> // Used in FatDiskIo
> //
> typedef enum {
>-  READ_DISK     = 0,  // raw disk read
>-  WRITE_DISK    = 1,  // raw disk write
>-  READ_FAT      = 2,  // read fat cache
>-  WRITE_FAT     = 3,  // write fat cache
>-  READ_DATA     = 6,  // read data cache
>-  WRITE_DATA    = 7   // write data cache
>+  ReadDisk     = 0,  // raw disk read
>+  WriteDisk    = 1,  // raw disk write
>+  ReadFat      = 2,  // read fat cache
>+  WriteFat     = 3,  // write fat cache
>+  ReadData     = 6,  // read data cache
>+  WriteData    = 7   // write data cache
> } IO_MODE;
>
> #define CACHE_ENABLED(a)  ((a) >= 2)
> #define RAW_ACCESS(a)     ((IO_MODE)((a) & 0x1))
> #define CACHE_TYPE(a)     ((CACHE_DATA_TYPE)((a) >> 2))
>@@ -382,11 +382,11 @@ typedef struct _FAT_VOLUME {
>
>   //
>   // Disk Cache for this volume
>   //
>   VOID                            *CacheBuffer;
>-  DISK_CACHE                      DiskCache[CACHE_MAX_TYPE];
>+  DISK_CACHE                      DiskCache[CacheMaxType];
> } FAT_VOLUME;
>
> //
> // Function Prototypes
> //
>@@ -1098,11 +1098,11 @@ FatResetODirCursor (
>   IN FAT_OFILE          *OFile
>   );
>
> EFI_STATUS
> FatGetNextDirEnt (
>-  IN  FAT_OFILE         *OFILE,
>+  IN  FAT_OFILE         *OFile,
>   OUT FAT_DIRENT        **PtrDirEnt
>   );
>
> EFI_STATUS
> FatRemoveDirEnt (
>diff --git a/FatPkg/EnhancedFatDxe/FileSpace.c b/FatPkg/EnhancedFatDxe/FileSpace.c
>index ce7c067..db44a33 100644
>--- a/FatPkg/EnhancedFatDxe/FileSpace.c
>+++ b/FatPkg/EnhancedFatDxe/FileSpace.c
>@@ -57,15 +57,15 @@ Returns:
>   }
>   //
>   // Compute buffer position needed
>   //
>   switch (Volume->FatType) {
>-  case FAT12:
>+  case Fat12:
>     Pos = FAT_POS_FAT12 (Index);
>     break;
>
>-  case FAT16:
>+  case Fat16:
>     Pos = FAT_POS_FAT16 (Index);
>     break;
>
>   default:
>     Pos = FAT_POS_FAT32 (Index);
>@@ -74,11 +74,11 @@ Returns:
>   // Set the position and read the buffer
>   //
>   Volume->FatEntryPos = Volume->FatPos + Pos;
>   Status = FatDiskIo (
>              Volume,
>-             READ_FAT,
>+             ReadFat,
>              Volume->FatEntryPos,
>              Volume->FatEntrySize,
>              &Volume->FatEntryBuffer,
>              NULL
>              );
>@@ -111,38 +111,38 @@ Returns:
>   The value of the FAT entry.
>
> --*/
> {
>   VOID    *Pos;
>-  UINT8   *E12;
>-  UINT16  *E16;
>-  UINT32  *E32;
>+  UINT8   *En12;
>+  UINT16  *En16;
>+  UINT32  *En32;
>   UINTN   Accum;
>
>   Pos = FatLoadFatEntry (Volume, Index);
>
>   if (Index > (Volume->MaxCluster + 1)) {
>     return (UINTN) -1;
>   }
>
>   switch (Volume->FatType) {
>-  case FAT12:
>-    E12   = Pos;
>-    Accum = E12[0] | (E12[1] << 8);
>+  case Fat12:
>+    En12   = Pos;
>+    Accum = En12[0] | (En12[1] << 8);
>     Accum = FAT_ODD_CLUSTER_FAT12 (Index) ? (Accum >> 4) : (Accum & FAT_CLUSTER_MASK_FAT12);
>     Accum = Accum | ((Accum >= FAT_CLUSTER_SPECIAL_FAT12) ? FAT_CLUSTER_SPECIAL_EXT : 0);
>     break;
>
>-  case FAT16:
>-    E16   = Pos;
>-    Accum = *E16;
>+  case Fat16:
>+    En16   = Pos;
>+    Accum = *En16;
>     Accum = Accum | ((Accum >= FAT_CLUSTER_SPECIAL_FAT16) ? FAT_CLUSTER_SPECIAL_EXT : 0);
>     break;
>
>   default:
>-    E32   = Pos;
>-    Accum = *E32 & FAT_CLUSTER_MASK_FAT32;
>+    En32   = Pos;
>+    Accum = *En32 & FAT_CLUSTER_MASK_FAT32;
>     Accum = Accum | ((Accum >= FAT_CLUSTER_SPECIAL_FAT32) ? FAT_CLUSTER_SPECIAL_EXT : 0);
>   }
>
>   return Accum;
> }
>@@ -173,13 +173,13 @@ Returns:
>   other                 - An error occurred when operation the FAT entries.
>
> --*/
> {
>   VOID        *Pos;
>-  UINT8       *E12;
>-  UINT16      *E16;
>-  UINT32      *E32;
>+  UINT8       *En12;
>+  UINT16      *En16;
>+  UINT32      *En32;
>   UINTN       Accum;
>   EFI_STATUS  Status;
>   UINTN       OriginalVal;
>
>   if (Index < FAT_MIN_CLUSTER) {
>@@ -204,49 +204,49 @@ Returns:
>
>   //
>   // Update the value
>   //
>   switch (Volume->FatType) {
>-  case FAT12:
>-    E12   = Pos;
>-    Accum = E12[0] | (E12[1] << 8);
>+  case Fat12:
>+    En12   = Pos;
>+    Accum = En12[0] | (En12[1] << 8);
>     Value = Value & FAT_CLUSTER_MASK_FAT12;
>
>     if (FAT_ODD_CLUSTER_FAT12 (Index)) {
>       Accum = (Value << 4) | (Accum & 0xF);
>     } else {
>       Accum = Value | (Accum & FAT_CLUSTER_UNMASK_FAT12);
>     }
>
>-    E12[0]  = (UINT8) (Accum & 0xFF);
>-    E12[1]  = (UINT8) (Accum >> 8);
>+    En12[0]  = (UINT8) (Accum & 0xFF);
>+    En12[1]  = (UINT8) (Accum >> 8);
>     break;
>
>-  case FAT16:
>-    E16   = Pos;
>-    *E16  = (UINT16) Value;
>+  case Fat16:
>+    En16   = Pos;
>+    *En16  = (UINT16) Value;
>     break;
>
>   default:
>-    E32   = Pos;
>-    *E32  = (*E32 & FAT_CLUSTER_UNMASK_FAT32) | (UINT32) (Value & FAT_CLUSTER_MASK_FAT32);
>+    En32   = Pos;
>+    *En32  = (*En32 & FAT_CLUSTER_UNMASK_FAT32) | (UINT32) (Value & FAT_CLUSTER_MASK_FAT32);
>   }
>   //
>   // If the volume's dirty bit is not set, set it now
>   //
>-  if (!Volume->FatDirty && Volume->FatType != FAT12) {
>+  if (!Volume->FatDirty && Volume->FatType != Fat12) {
>     Volume->FatDirty = TRUE;
>-    FatAccessVolumeDirty (Volume, WRITE_FAT, &Volume->DirtyValue);
>+    FatAccessVolumeDirty (Volume, WriteFat, &Volume->DirtyValue);
>   }
>   //
>   // Write the updated fat entry value to the volume
>   // The fat is the first fat, and other fat will be in sync
>   // when the FAT cache flush back.
>   //
>   Status = FatDiskIo (
>              Volume,
>-             WRITE_FAT,
>+             WriteFat,
>              Volume->FatEntryPos,
>              Volume->FatEntrySize,
>              &Volume->FatEntryBuffer,
>              NULL
>              );
>diff --git a/FatPkg/EnhancedFatDxe/Flush.c b/FatPkg/EnhancedFatDxe/Flush.c
>index 2c960f3..172f202 100644
>--- a/FatPkg/EnhancedFatDxe/Flush.c
>+++ b/FatPkg/EnhancedFatDxe/Flush.c
>@@ -460,21 +460,21 @@ Returns:
>     //
>     // Update the free hint info. Volume->FreeInfoPos != 0
>     // indicates this a FAT32 volume
>     //
>     if (Volume->FreeInfoValid && Volume->FatDirty && Volume->FreeInfoPos) {
>-      Status = FatDiskIo (Volume, WRITE_DISK, Volume->FreeInfoPos, sizeof (FAT_INFO_SECTOR), &Volume->FatInfoSector,
>Task);
>+      Status = FatDiskIo (Volume, WriteDisk, Volume->FreeInfoPos, sizeof (FAT_INFO_SECTOR), &Volume->FatInfoSector,
>Task);
>       if (EFI_ERROR (Status)) {
>         return Status;
>       }
>     }
>     //
>     // Update that the volume is not dirty
>     //
>-    if (Volume->FatDirty && Volume->FatType != FAT12) {
>+    if (Volume->FatDirty && Volume->FatType != Fat12) {
>       Volume->FatDirty  = FALSE;
>-      Status            = FatAccessVolumeDirty (Volume, WRITE_FAT, &Volume->NotDirtyValue);
>+      Status            = FatAccessVolumeDirty (Volume, WriteFat, &Volume->NotDirtyValue);
>       if (EFI_ERROR (Status)) {
>         return Status;
>       }
>     }
>     //
>diff --git a/FatPkg/EnhancedFatDxe/Init.c b/FatPkg/EnhancedFatDxe/Init.c
>index 34a7250..d7cfa82 100644
>--- a/FatPkg/EnhancedFatDxe/Init.c
>+++ b/FatPkg/EnhancedFatDxe/Init.c
>@@ -263,11 +263,11 @@ Returns:
>   }
>
>   SectorsPerFat = FatBs.FatBsb.SectorsPerFat;
>   if (SectorsPerFat == 0) {
>     SectorsPerFat = FatBs.FatBse.Fat32Bse.LargeSectorsPerFat;
>-    FatType       = FAT32;
>+    FatType       = Fat32;
>   }
>   //
>   // Is boot sector a fat sector?
>   // (Note that so far we only know if the sector is FAT32 or not, we don't
>   // know if the sector is Fat16 or Fat12 until later when we can compute
>@@ -303,11 +303,11 @@ Returns:
>     return EFI_UNSUPPORTED;
>   }
>   //
>   // Initialize fields the volume information for this FatType
>   //
>-  if (FatType != FAT32) {
>+  if (FatType != Fat32) {
>     if (FatBs.FatBsb.RootEntries == 0) {
>       return EFI_UNSUPPORTED;
>     }
>     //
>     // Unpack fat12, fat16 info
>@@ -348,16 +348,16 @@ Returns:
>   Volume->ClusterSize       = (UINTN)1 << (Volume->ClusterAlignment);
>
>   //
>   // If this is not a fat32, determine if it's a fat16 or fat12
>   //
>-  if (FatType != FAT32) {
>+  if (FatType != Fat32) {
>     if (Volume->MaxCluster >= FAT_MAX_FAT16_CLUSTER) {
>       return EFI_VOLUME_CORRUPTED;
>     }
>
>-    FatType = Volume->MaxCluster < FAT_MAX_FAT12_CLUSTER ? FAT12 : FAT16;
>+    FatType = Volume->MaxCluster < FAT_MAX_FAT12_CLUSTER ? Fat12 : Fat16;
>     //
>     // fat12 & fat16 fat-entries are 2 bytes
>     //
>     Volume->FatEntrySize = sizeof (UINT16);
>     DirtyMask            = FAT16_DIRTY_MASK;
>@@ -374,25 +374,25 @@ Returns:
>   //
>   // Get the DirtyValue and NotDirtyValue
>   // We should keep the initial value as the NotDirtyValue
>   // in case the volume is dirty already
>   //
>-  if (FatType != FAT12) {
>-    Status = FatAccessVolumeDirty (Volume, READ_DISK, &Volume->NotDirtyValue);
>+  if (FatType != Fat12) {
>+    Status = FatAccessVolumeDirty (Volume, ReadDisk, &Volume->NotDirtyValue);
>     if (EFI_ERROR (Status)) {
>       return Status;
>     }
>
>     Volume->DirtyValue = Volume->NotDirtyValue & DirtyMask;
>   }
>   //
>   // If present, read the fat hint info
>   //
>-  if (FatType == FAT32) {
>+  if (FatType == Fat32) {
>     Volume->FreeInfoPos = FatBs.FatBse.Fat32Bse.FsInfoSector * BlockSize;
>     if (FatBs.FatBse.Fat32Bse.FsInfoSector != 0) {
>-      FatDiskIo (Volume, READ_DISK, Volume->FreeInfoPos, sizeof (FAT_INFO_SECTOR), &Volume->FatInfoSector, NULL);
>+      FatDiskIo (Volume, ReadDisk, Volume->FreeInfoPos, sizeof (FAT_INFO_SECTOR), &Volume->FatInfoSector, NULL);
>       if (Volume->FatInfoSector.Signature == FAT_INFO_SIGNATURE &&
>           Volume->FatInfoSector.InfoBeginSignature == FAT_INFO_BEGIN_SIGNATURE &&
>           Volume->FatInfoSector.InfoEndSignature == FAT_INFO_END_SIGNATURE &&
>           Volume->FatInfoSector.FreeInfo.ClusterCount <= Volume->MaxCluster
>           ) {
>diff --git a/FatPkg/EnhancedFatDxe/Misc.c b/FatPkg/EnhancedFatDxe/Misc.c
>index 6ad688c..48c99f9 100644
>--- a/FatPkg/EnhancedFatDxe/Misc.c
>+++ b/FatPkg/EnhancedFatDxe/Misc.c
>@@ -394,11 +394,11 @@ Returns:
>       if (Task == NULL) {
>         //
>         // Blocking access
>         //
>         DiskIo      = Volume->DiskIo;
>-        IoFunction  = (IoMode == READ_DISK) ? DiskIo->ReadDisk : DiskIo->WriteDisk;
>+        IoFunction  = (IoMode == ReadDisk) ? DiskIo->ReadDisk : DiskIo->WriteDisk;
>         Status      = IoFunction (DiskIo, Volume->MediaId, Offset, BufferSize, Buffer);
>       } else {
>         //
>         // Non-blocking access
>         //
>@@ -406,11 +406,11 @@ Returns:
>         if (Subtask == NULL) {
>           Status        = EFI_OUT_OF_RESOURCES;
>         } else {
>           Subtask->Signature  = FAT_SUBTASK_SIGNATURE;
>           Subtask->Task       = Task;
>-          Subtask->Write      = (BOOLEAN) (IoMode == WRITE_DISK);
>+          Subtask->Write      = (BOOLEAN) (IoMode == WriteDisk);
>           Subtask->Offset     = Offset;
>           Subtask->Buffer     = Buffer;
>           Subtask->BufferSize = BufferSize;
>           Status = gBS->CreateEvent (
>                           EVT_NOTIFY_SIGNAL,
>diff --git a/FatPkg/EnhancedFatDxe/ReadWrite.c b/FatPkg/EnhancedFatDxe/ReadWrite.c
>index 9afb6bf..81676cd 100644
>--- a/FatPkg/EnhancedFatDxe/ReadWrite.c
>+++ b/FatPkg/EnhancedFatDxe/ReadWrite.c
>@@ -251,19 +251,19 @@ Returns:
>   Task   = NULL;
>
>   //
>   // Write to a directory is unsupported
>   //
>-  if ((OFile->ODir != NULL) && (IoMode == WRITE_DATA)) {
>+  if ((OFile->ODir != NULL) && (IoMode == WriteData)) {
>     return EFI_UNSUPPORTED;
>   }
>
>   if (OFile->Error == EFI_NOT_FOUND) {
>     return EFI_DEVICE_ERROR;
>   }
>
>-  if (IoMode == READ_DATA) {
>+  if (IoMode == ReadData) {
>     //
>     // If position is at EOF, then return device error
>     //
>     if (IFile->Position > OFile->FileSize) {
>       return EFI_DEVICE_ERROR;
>@@ -303,11 +303,11 @@ Returns:
>   if (!EFI_ERROR (Status)) {
>     if (OFile->ODir != NULL) {
>       //
>       // Read a directory is supported
>       //
>-      ASSERT (IoMode == READ_DATA);
>+      ASSERT (IoMode == ReadData);
>       Status = FatIFileReadDir (IFile, BufferSize, Buffer);
>       OFile = NULL;
>     } else {
>       //
>       // Access a file
>@@ -315,11 +315,11 @@ Returns:
>       EndPosition = IFile->Position + *BufferSize;
>       if (EndPosition > OFile->FileSize) {
>         //
>         // The position goes beyond the end of file
>         //
>-        if (IoMode == READ_DATA) {
>+        if (IoMode == ReadData) {
>           //
>           // Adjust the actual size read
>           //
>           *BufferSize -= (UINTN) EndPosition - OFile->FileSize;
>         } else {
>@@ -399,11 +399,11 @@ Returns:
>   EFI_VOLUME_CORRUPTED  - The file type of open file is error.
>   other                 - An error occurred when operation the disk.
>
> --*/
> {
>-  return FatIFileAccess (FHand, READ_DATA, BufferSize, Buffer, NULL);
>+  return FatIFileAccess (FHand, ReadData, BufferSize, Buffer, NULL);
> }
>
> EFI_STATUS
> EFIAPI
> FatReadEx (
>@@ -428,11 +428,11 @@ Returns:
>   EFI_VOLUME_CORRUPTED  - The file type of open file is error.
>   other                 - An error occurred when operation the disk.
>
> --*/
> {
>-  return FatIFileAccess (FHand, READ_DATA, &Token->BufferSize, Token->Buffer, Token);
>+  return FatIFileAccess (FHand, ReadData, &Token->BufferSize, Token->Buffer, Token);
> }
>
> EFI_STATUS
> EFIAPI
> FatWrite (
>@@ -462,11 +462,11 @@ Returns:
>                         - The writing file size is larger than 4GB.
>   other                 - An error occurred when operation the disk.
>
> --*/
> {
>-  return FatIFileAccess (FHand, WRITE_DATA, BufferSize, Buffer, NULL);
>+  return FatIFileAccess (FHand, WriteData, BufferSize, Buffer, NULL);
> }
>
> EFI_STATUS
> EFIAPI
> FatWriteEx (
>@@ -491,11 +491,11 @@ Returns:
>   EFI_VOLUME_CORRUPTED  - The file type of open file is error.
>   other                 - An error occurred when operation the disk.
>
> --*/
> {
>-  return FatIFileAccess (FHand, WRITE_DATA, &Token->BufferSize, Token->Buffer, Token);
>+  return FatIFileAccess (FHand, WriteData, &Token->BufferSize, Token->Buffer, Token);
> }
>
> EFI_STATUS
> FatAccessOFile (
>   IN     FAT_OFILE      *OFile,
>@@ -561,11 +561,11 @@ Returns:
>     // Data was successfully accessed
>     //
>     Position   += Len;
>     UserBuffer += Len;
>     BufferSize -= Len;
>-    if (IoMode == WRITE_DATA) {
>+    if (IoMode == WriteData) {
>       OFile->Dirty    = TRUE;
>       OFile->Archive  = TRUE;
>     }
>     //
>     // Make sure no outbound occurred
>@@ -662,11 +662,11 @@ Returns:
>   }
>
>   do {
>     WriteSize     = AppendedSize > BufferSize ? BufferSize : (UINTN) AppendedSize;
>     AppendedSize -= WriteSize;
>-    Status = FatAccessOFile (OFile, WRITE_DATA, WritePos, &WriteSize, ZeroBuffer, NULL);
>+    Status = FatAccessOFile (OFile, WriteData, WritePos, &WriteSize, ZeroBuffer, NULL);
>     if (EFI_ERROR (Status)) {
>       break;
>     }
>
>     WritePos += WriteSize;
>--
>1.9.5.msysgit.1



  reply	other threads:[~2016-12-09  1:12 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-08 10:54 [patch 0/8] FatPkg: Fix coding style issues Dandan Bi
2016-12-08 10:54 ` [patch 1/8] FatPkg\EnhancedFatDxe: Avoid Non-Boolean type uses as Boolean Dandan Bi
2016-12-09  1:14   ` Ni, Ruiyu
2016-12-08 10:54 ` [patch 2/8] FatPkg\EnhancedFatDxe: Initialize variable after declaration Dandan Bi
2016-12-08 17:27   ` Kurt Kennett
2016-12-08 23:11     ` Andrew Fish
2016-12-20 23:26       ` Brian J. Johnson
2016-12-08 23:47     ` Yao, Jiewen
2016-12-09  0:26       ` Kurt Kennett
2016-12-09  0:36         ` Andrew Fish
2016-12-09  0:40           ` Yao, Jiewen
2016-12-09 16:32             ` Kurt Kennett
2016-12-09  0:40           ` Kurt Kennett
2016-12-09  0:57   ` Ni, Ruiyu
2016-12-09  1:18     ` Bi, Dandan
2016-12-08 10:54 ` [patch 3/8] FatPkg\EnhancedFatDxe: Make function prototype align with definition Dandan Bi
2016-12-09  1:13   ` Ni, Ruiyu
2016-12-08 10:54 ` [patch 4/8] FatPkg\EnhancedFatDxe: Make the variable name follow rule Dandan Bi
2016-12-09  1:12   ` Ni, Ruiyu [this message]
2016-12-08 10:54 ` [patch 5/8] FatPkg\EnhancedFatDxe: Use typedef for complex type Dandan Bi
2016-12-09  1:11   ` Ni, Ruiyu
2016-12-08 10:54 ` [patch 6/8] FatPkg\EnhancedFatDxe: Make the comments align with EDKII coding style Dandan Bi
2016-12-09  1:10   ` Ni, Ruiyu
2016-12-08 10:54 ` [patch 7/8] FatPkg\EnhancedFatDxe: Add comments for functions Dandan Bi
2016-12-09  1:10   ` Ni, Ruiyu
2016-12-08 10:54 ` [patch 8/8] FatPkg: Fix format issues in dec/inf/dsc files Dandan Bi
2016-12-09  1:09   ` Ni, Ruiyu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=734D49CCEBEEF84792F5B80ED585239D58EAFB59@SHSMSX104.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox