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
next prev parent 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