> -----Original Message----- > From: Marvin Häuser <mhaeu...@posteo.de> > Sent: Monday, August 16, 2021 4:12 AM > To: devel@edk2.groups.io > Cc: Wang, Jian J <jian.j.w...@intel.com>; Wu, Hao A <hao.a...@intel.com>; > Ni, Ray <ray...@intel.com> > Subject: [PATCH V2 1/3] MdeModulePkg: Rename IS_ALIGNED macros to > avoid name collisions > > This patch is a preparation for the patches that follow. The subsequent > patches will introduce and integrate new alignment-related macros, which > collide with existing definitions in MdeModulePkg. > Temporarily rename them to avoid build failure, till they can be substituted > with the new, shared definitions.
Reviewed-by: Hao A Wu <hao.a...@intel.com> Best Regards, Hao Wu > > Cc: Jian J Wang <jian.j.w...@intel.com> > Cc: Hao A Wu <hao.a...@intel.com> > Cc: Ray Ni <ray...@intel.com> > Signed-off-by: Marvin Häuser <mhaeu...@posteo.de> > --- > MdeModulePkg/Bus/Ata/AhciPei/AhciMode.c | 2 +- > MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c | 6 ++-- > MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.c | 12 +++---- > MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c | 2 +- > MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c | 4 +-- > MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 6 ++-- > MdeModulePkg/Universal/EbcDxe/EbcExecute.c | 36 ++++++++++--- > ------- > MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h | 2 +- > MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.h | 2 +- > MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h | 2 +- > MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h | 2 +- > MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.h | 2 +- > MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h | 2 +- > MdeModulePkg/Universal/EbcDxe/EbcExecute.h | 2 +- > 14 files changed, 41 insertions(+), 41 deletions(-) > > diff --git a/MdeModulePkg/Bus/Ata/AhciPei/AhciMode.c > b/MdeModulePkg/Bus/Ata/AhciPei/AhciMode.c > index 7636ad27c86c..cc32b5de4f98 100644 > --- a/MdeModulePkg/Bus/Ata/AhciPei/AhciMode.c > +++ b/MdeModulePkg/Bus/Ata/AhciPei/AhciMode.c > @@ -2099,7 +2099,7 @@ TrustTransferAtaDevice ( > // ATA PassThru PPI. // if ((AtaPassThru->Mode->IoAlign > 1) &&- > !IS_ALIGNED (Buffer, AtaPassThru->Mode->IoAlign)) > {+ !ADDRESS_IS_ALIGNED_ (Buffer, AtaPassThru->Mode->IoAlign)) > { NewBuffer = AllocateAlignedPages ( > EFI_SIZE_TO_PAGES > (TransferLength), AtaPassThru->Mode->IoAligndiff --git > a/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c > b/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c > index 191b78c88541..31ded8a31048 100644 > --- a/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c > +++ b/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c > @@ -193,15 +193,15 @@ AhciAtaPassThruPassThru ( > } IoAlign = This->Mode->IoAlign;- if ((IoAlign > 1) && !IS_ALIGNED > (Packet->InDataBuffer, IoAlign)) {+ if ((IoAlign > 1) > && !ADDRESS_IS_ALIGNED_ (Packet->InDataBuffer, IoAlign)) { return > EFI_INVALID_PARAMETER; } - if ((IoAlign > 1) && !IS_ALIGNED (Packet- > >OutDataBuffer, IoAlign)) {+ if ((IoAlign > 1) && !ADDRESS_IS_ALIGNED_ > (Packet->OutDataBuffer, IoAlign)) { return EFI_INVALID_PARAMETER; } - > if ((IoAlign > 1) && !IS_ALIGNED (Packet->Asb, IoAlign)) {+ if ((IoAlign > 1) > && !ADDRESS_IS_ALIGNED_ (Packet->Asb, IoAlign)) { return > EFI_INVALID_PARAMETER; } diff --git > a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.c > b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.c > index 86fe9d954fdb..cf98fcdaf344 100644 > --- a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.c > +++ b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.c > @@ -1281,15 +1281,15 @@ AtaPassThruPassThru ( > Instance = ATA_PASS_THRU_PRIVATE_DATA_FROM_THIS (This); - if > ((This->Mode->IoAlign > 1) && !IS_ALIGNED(Packet->InDataBuffer, This- > >Mode->IoAlign)) {+ if ((This->Mode->IoAlign > 1) > && !ADDRESS_IS_ALIGNED_(Packet->InDataBuffer, This->Mode->IoAlign)) > { return EFI_INVALID_PARAMETER; } - if ((This->Mode->IoAlign > 1) > && !IS_ALIGNED(Packet->OutDataBuffer, This->Mode->IoAlign)) {+ if ((This- > >Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->OutDataBuffer, > This->Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } - if ((This- > >Mode->IoAlign > 1) && !IS_ALIGNED(Packet->Asb, This->Mode->IoAlign)) > {+ if ((This->Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->Asb, > This->Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } @@ -2012,15 > +2012,15 @@ ExtScsiPassThruPassThru ( > return EFI_INVALID_PARAMETER; } - if ((This->Mode->IoAlign > 1) > && !IS_ALIGNED(Packet->InDataBuffer, This->Mode->IoAlign)) {+ if ((This- > >Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->InDataBuffer, > This->Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } - if ((This- > >Mode->IoAlign > 1) && !IS_ALIGNED(Packet->OutDataBuffer, This->Mode- > >IoAlign)) {+ if ((This->Mode->IoAlign > 1) > && !ADDRESS_IS_ALIGNED_(Packet->OutDataBuffer, This->Mode->IoAlign)) > { return EFI_INVALID_PARAMETER; } - if ((This->Mode->IoAlign > 1) > && !IS_ALIGNED(Packet->SenseData, This->Mode->IoAlign)) {+ if ((This- > >Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->SenseData, This- > >Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } diff --git > a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c > b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c > index 79026a4a957d..471fb6a7f440 100644 > --- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c > +++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c > @@ -1036,7 +1036,7 @@ TrustTransferAtaDevice ( > // Check the alignment of the incoming buffer prior to invoking > underlying > ATA PassThru // AtaPassThru = AtaDevice->AtaBusDriverData- > >AtaPassThru;- if ((AtaPassThru->Mode->IoAlign > 1) && !IS_ALIGNED > (Buffer, AtaPassThru->Mode->IoAlign)) {+ if ((AtaPassThru->Mode- > >IoAlign > 1) && !ADDRESS_IS_ALIGNED_ (Buffer, AtaPassThru->Mode- > >IoAlign)) { NewBuffer = AllocateAlignedBuffer (AtaDevice, > TransferLength); if (NewBuffer == NULL) { return > EFI_OUT_OF_RESOURCES;diff --git > a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > index c80e78fa8a6b..81c0fa217a0b 100644 > --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > @@ -1956,7 +1956,7 @@ ScsiDiskReceiveData ( > goto Done; } - if ((ScsiDiskDevice->ScsiIo->IoAlign > 1) > && !IS_ALIGNED (PayloadBuffer, ScsiDiskDevice->ScsiIo->IoAlign)) {+ if > ((ScsiDiskDevice->ScsiIo->IoAlign > 1) && !ADDRESS_IS_ALIGNED_ > (PayloadBuffer, ScsiDiskDevice->ScsiIo->IoAlign)) { AlignedBuffer = > AllocateAlignedBuffer (ScsiDiskDevice, PayloadBufferSize); if > (AlignedBuffer == NULL) { Status = EFI_OUT_OF_RESOURCES;@@ - > 2171,7 +2171,7 @@ ScsiDiskSendData ( > goto Done; } - if ((ScsiDiskDevice->ScsiIo->IoAlign > 1) > && !IS_ALIGNED (PayloadBuffer, ScsiDiskDevice->ScsiIo->IoAlign)) {+ if > ((ScsiDiskDevice->ScsiIo->IoAlign > 1) && !ADDRESS_IS_ALIGNED_ > (PayloadBuffer, ScsiDiskDevice->ScsiIo->IoAlign)) { AlignedBuffer = > AllocateAlignedBuffer (ScsiDiskDevice, PayloadBufferSize); if > (AlignedBuffer == NULL) { Status = EFI_OUT_OF_RESOURCES;diff --git > a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c > b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c > index 92ff958f161e..abfb422d1ea3 100644 > --- a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c > +++ b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c > @@ -170,15 +170,15 @@ UfsPassThruPassThru ( > return EFI_INVALID_PARAMETER; } - if ((This->Mode->IoAlign > 1) > && !IS_ALIGNED(Packet->InDataBuffer, This->Mode->IoAlign)) {+ if ((This- > >Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->InDataBuffer, > This->Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } - if ((This- > >Mode->IoAlign > 1) && !IS_ALIGNED(Packet->OutDataBuffer, This->Mode- > >IoAlign)) {+ if ((This->Mode->IoAlign > 1) > && !ADDRESS_IS_ALIGNED_(Packet->OutDataBuffer, This->Mode->IoAlign)) > { return EFI_INVALID_PARAMETER; } - if ((This->Mode->IoAlign > 1) > && !IS_ALIGNED(Packet->SenseData, This->Mode->IoAlign)) {+ if ((This- > >Mode->IoAlign > 1) && !ADDRESS_IS_ALIGNED_(Packet->SenseData, This- > >Mode->IoAlign)) { return EFI_INVALID_PARAMETER; } diff --git > a/MdeModulePkg/Universal/EbcDxe/EbcExecute.c > b/MdeModulePkg/Universal/EbcDxe/EbcExecute.c > index 1c4a4f5155c9..35f60cabdeb4 100644 > --- a/MdeModulePkg/Universal/EbcDxe/EbcExecute.c > +++ b/MdeModulePkg/Universal/EbcDxe/EbcExecute.c > @@ -2004,7 +2004,7 @@ ExecuteJMP ( > // check for alignment, and jump absolute. // Data64 = (UINT64) > VmReadImmed64 (VmPtr, 2);- if (!IS_ALIGNED ((UINTN) Data64, sizeof > (UINT16))) {+ if (!ADDRESS_IS_ALIGNED_ ((UINTN) Data64, sizeof (UINT16))) > { EbcDebugSignalException ( EXCEPT_EBC_ALIGNMENT_CHECK, > EXCEPTION_FLAG_FATAL,@@ -2059,7 +2059,7 @@ ExecuteJMP ( > // Form: JMP32 @Rx {Index32} // Addr = VmReadMemN (VmPtr, > (UINTN) Data64 + Index32);- if (!IS_ALIGNED ((UINTN) Addr, sizeof > (UINT16))) {+ if (!ADDRESS_IS_ALIGNED_ ((UINTN) Addr, sizeof (UINT16))) > { EbcDebugSignalException ( EXCEPT_EBC_ALIGNMENT_CHECK, > EXCEPTION_FLAG_FATAL,@@ -2082,7 +2082,7 @@ ExecuteJMP ( > // Form: JMP32 Rx {Immed32} // Addr = (UINTN) (Data64 + > Index32);- > if (!IS_ALIGNED ((UINTN) Addr, sizeof (UINT16))) {+ if > (!ADDRESS_IS_ALIGNED_ ((UINTN) Addr, sizeof (UINT16))) > { EbcDebugSignalException ( EXCEPT_EBC_ALIGNMENT_CHECK, > EXCEPTION_FLAG_FATAL,@@ -3128,7 +3128,7 @@ ExecuteRET ( > // Pull the return address off the VM app's stack and set the IP // > to it > //- if (!IS_ALIGNED ((UINTN) VmPtr->Gpr[0], sizeof (UINT16))) {+ if > (!ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Gpr[0], sizeof (UINT16))) > { EbcDebugSignalException ( EXCEPT_EBC_ALIGNMENT_CHECK, > EXCEPTION_FLAG_FATAL,@@ -4693,7 +4693,7 @@ VmWriteMem16 ( > // // Do a simple write if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT16))) {+ if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT16))) > { *(UINT16 *) Addr = Data; } else { //@@ -4756,7 +4756,7 @@ > VmWriteMem32 ( > // // Do a simple write if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT32))) {+ if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT32))) > { *(UINT32 *) Addr = Data; } else { //@@ -4819,7 +4819,7 @@ > VmWriteMem64 ( > // // Do a simple write if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT64))) {+ if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT64))) > { *(UINT64 *) Addr = Data; } else { //@@ -4885,7 +4885,7 @@ > VmWriteMemN ( > // // Do a simple write if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINTN))) > {+ if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINTN))) { *(UINTN *) Addr = > Data; } else { for (Index = 0; Index < sizeof (UINTN) / sizeof (UINT32); > Index++) {@@ -4949,7 +4949,7 @@ VmReadImmed16 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (INT16))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip + > Offset, sizeof (INT16))) { return * (INT16 *) (VmPtr->Ip + Offset); } > else > { //@@ -4993,7 +4993,7 @@ VmReadImmed32 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (UINT32))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip > + Offset, sizeof (UINT32))) { return * (INT32 *) (VmPtr->Ip + Offset); } > //@@ -5032,7 +5032,7 @@ VmReadImmed64 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (UINT64))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip > + Offset, sizeof (UINT64))) { return * (UINT64 *) (VmPtr->Ip + Offset); > } > //@@ -5069,7 +5069,7 @@ VmReadCode16 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (UINT16))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip > + Offset, sizeof (UINT16))) { return * (UINT16 *) (VmPtr->Ip + Offset); > } > else { //@@ -5110,7 +5110,7 @@ VmReadCode32 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (UINT32))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip > + Offset, sizeof (UINT32))) { return * (UINT32 *) (VmPtr->Ip + Offset); > } > //@@ -5147,7 +5147,7 @@ VmReadCode64 ( > // // Read direct if aligned //- if (IS_ALIGNED ((UINTN) VmPtr->Ip + > Offset, sizeof (UINT64))) {+ if (ADDRESS_IS_ALIGNED_ ((UINTN) VmPtr->Ip > + Offset, sizeof (UINT64))) { return * (UINT64 *) (VmPtr->Ip + Offset); > } > //@@ -5210,7 +5210,7 @@ VmReadMem16 ( > // // Read direct if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT16))) {+ > if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT16))) { return * (UINT16 *) > Addr; } //@@ -5243,7 +5243,7 @@ VmReadMem32 ( > // // Read direct if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT32))) {+ > if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT32))) { return * (UINT32 *) > Addr; } //@@ -5280,7 +5280,7 @@ VmReadMem64 ( > // // Read direct if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINT64))) {+ > if (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINT64))) { return * (UINT64 *) > Addr; } //@@ -5349,7 +5349,7 @@ VmReadMemN ( > // // Read direct if aligned //- if (IS_ALIGNED (Addr, sizeof > (UINTN))) {+ if > (ADDRESS_IS_ALIGNED_ (Addr, sizeof (UINTN))) { return * (UINTN *) > Addr; } //diff --git a/MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h > b/MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h > index 2be78076bee7..2a74c9984791 100644 > --- a/MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h > +++ b/MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h > @@ -145,7 +145,7 @@ typedef union { > #define AHCI_PORT_SERR 0x0030 #define AHCI_PORT_CI > 0x0038 -#define IS_ALIGNED(addr, size) (((UINTN) (addr) & > (size - 1)) > == 0)+#define ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) > (addr) & > (size - 1)) == 0) #define TIMER_PERIOD_SECONDS(Seconds) > MultU64x32((UINT64)(Seconds), 10000000) #pragma pack(1)diff --git > a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.h > b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.h > index 5f582b9b3e76..aa3472a71677 100644 > --- a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.h > +++ b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.h > @@ -147,7 +147,7 @@ struct _ATA_NONBLOCK_TASK { > #define ATA_ATAPI_TIMEOUT EFI_TIMER_PERIOD_SECONDS(3) > #define ATA_SPINUP_TIMEOUT EFI_TIMER_PERIOD_SECONDS(10) - > #define IS_ALIGNED(addr, size) (((UINTN) (addr) & (size - 1)) == > 0)+#define ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) (addr) & (size - > 1)) == 0) #define ATA_PASS_THRU_PRIVATE_DATA_FROM_THIS(a) \ CR (a, > \diff --git a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > index a5a865209942..13767a8a13c8 100644 > --- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > +++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > @@ -76,7 +76,7 @@ > #define ATA_TASK_SIGNATURE SIGNATURE_32 ('A', 'T', 'S', 'K') > #define ATA_DEVICE_SIGNATURE SIGNATURE_32 ('A', 'B', 'I', 'D') > #define ATA_SUB_TASK_SIGNATURE SIGNATURE_32 ('A', 'S', 'T', 'S')- > #define IS_ALIGNED(addr, size) (((UINTN) (addr) & (size - 1)) == > 0)+#define ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) (addr) & (size > - 1)) == 0) // // ATA bus data structure for ATA controllerdiff --git > a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > index ed9bbd6f8ba8..abe1d06a98da 100644 > --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > @@ -39,7 +39,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > #define IS_DEVICE_FIXED(a) (a)->FixedDevice ? 1 : 0 -#define > IS_ALIGNED(addr, size) (((UINTN) (addr) & (size - 1)) == 0)+#define > ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) (addr) & (size - 1)) == 0) > #define UFS_WLUN_RPMB 0xC4 diff --git > a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.h > b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.h > index 6e2305aa2bc2..4802fdc0ab9b 100644 > --- a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.h > +++ b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.h > @@ -133,7 +133,7 @@ typedef struct _UFS_PEIM_HC_PRIVATE_DATA { > #define ROUNDUP8(x) (((x) % 8 == 0) ? (x) : ((x) / 8 + 1) * 8) -#define > IS_ALIGNED(addr, size) (((UINTN) (addr) & (size - 1)) == 0)+#define > ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) (addr) & (size - 1)) == 0) > #define GET_UFS_PEIM_HC_PRIVATE_DATA_FROM_THIS(a) CR (a, > UFS_PEIM_HC_PRIVATE_DATA, BlkIoPpi, UFS_PEIM_HC_SIG) #define > GET_UFS_PEIM_HC_PRIVATE_DATA_FROM_THIS2(a) CR (a, > UFS_PEIM_HC_PRIVATE_DATA, BlkIo2Ppi, UFS_PEIM_HC_SIG)diff --git > a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h > b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h > index 79b86f7e6b3d..da3bf78bf9bc 100644 > --- a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h > +++ b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h > @@ -105,7 +105,7 @@ typedef struct { > #define ROUNDUP8(x) (((x) % 8 == 0) ? (x) : ((x) / 8 + 1) * 8) -#define > IS_ALIGNED(addr, size) (((UINTN) (addr) & (size - 1)) == 0)+#define > ADDRESS_IS_ALIGNED_(addr, size) (((UINTN) (addr) & (size - 1)) == 0) > #define UFS_PASS_THRU_PRIVATE_DATA_FROM_THIS(a) \ CR (a, \diff --git > a/MdeModulePkg/Universal/EbcDxe/EbcExecute.h > b/MdeModulePkg/Universal/EbcDxe/EbcExecute.h > index 1cb68bc5385a..f21c757a75b4 100644 > --- a/MdeModulePkg/Universal/EbcDxe/EbcExecute.h > +++ b/MdeModulePkg/Universal/EbcDxe/EbcExecute.h > @@ -15,7 +15,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > // Macros to check and set alignment // #define ASSERT_ALIGNED(addr, size) > ASSERT (!((UINT32) (addr) & (size - 1)))-#define IS_ALIGNED(addr, > size) !((UINT32) (addr) & (size - 1))+#define ADDRESS_IS_ALIGNED_(addr, > size) !((UINT32) (addr) & (size - 1)) // // Debug macro-- > 2.31.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#79618): https://edk2.groups.io/g/devel/message/79618 Mute This Topic: https://groups.io/mt/84909446/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-