diff options
author | wang xiaofeng <winggundum82@163.com> | 2016-05-13 11:20:28 +0800 |
---|---|---|
committer | Star Zeng <star.zeng@intel.com> | 2016-08-15 10:01:09 +0800 |
commit | 69f75ff0701e3907af7294f875a1a88528245dba (patch) | |
tree | fdbee695d67f0bc4c559e9b7a93af8ff1ea6b2fe | |
parent | 94eb6722a0adfa9f5a23288b6350e53b73571d4b (diff) | |
download | edk2-69f75ff0701e3907af7294f875a1a88528245dba.zip edk2-69f75ff0701e3907af7294f875a1a88528245dba.tar.gz edk2-69f75ff0701e3907af7294f875a1a88528245dba.tar.bz2 |
PcAtChipsetPkg AcpiTimerLib: Fix a logic error
if ((PciRead8 (PCI_LIB_ADDRESS (Bus, Device, Function, EnableRegister) &
EnableMask) != EnableMask)) {
The bracket place is not right, I think it should be
if ((PciRead8 (PCI_LIB_ADDRESS (Bus, Device, Function, EnableRegister)) &
EnableMask) != EnableMask)
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: wang xiaofeng <winggundum82@163.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
(cherry picked from commit dde4aedc35ee3c06b67c1a5c4f392e7a0b0f8254)
-rw-r--r-- | PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c b/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c index 5a0782c..e6fea23 100644 --- a/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c +++ b/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c @@ -89,7 +89,7 @@ AcpiTimerLibConstructor ( //
// If ACPI I/O space is not enabled yet, program ACPI I/O base address and enable it.
//
- if ((PciRead8 (PCI_LIB_ADDRESS (Bus, Device, Function, EnableRegister) & EnableMask) != EnableMask)) {
+ if ((PciRead8 (PCI_LIB_ADDRESS (Bus, Device, Function, EnableRegister)) & EnableMask) != EnableMask) {
PciWrite16 (
PCI_LIB_ADDRESS (Bus, Device, Function, PcdGet16 (PcdAcpiIoPciBarRegisterOffset)),
PcdGet16 (PcdAcpiIoPortBaseAddress)
|