Pushed as eb3950b842..6198c1c497 -----Original Message----- From: [email protected] <[email protected]> On Behalf Of Isaac Oram Sent: Wednesday, January 4, 2023 4:30 PM To: S, Ashraf Ali <[email protected]>; [email protected] Cc: Ni, Ray <[email protected]>; Chaganty, Rangasai V <[email protected]>; Chiu, Chasel <[email protected]>; Desimone, Nathaniel L <[email protected]>; Gao, Liming <[email protected]>; Dong, Eric <[email protected]> Subject: Re: [edk2-devel] [PATCH v2] FIX MinPlatformPkg PCIE Base typecasting error.
Reviewed-by: Isaac Oram <[email protected]> Note I had to fix your author format from S, Ashraf Ali to match convention/use. I believe that you can fix this in your groups.io settings since your signoff is correct implies git is correct. -----Original Message----- From: S, Ashraf Ali <[email protected]> Sent: Sunday, December 25, 2022 10:06 PM To: [email protected] Cc: S, Ashraf Ali <[email protected]>; Ni, Ray <[email protected]>; Chaganty, Rangasai V <[email protected]>; Oram, Isaac W <[email protected]>; Chiu, Chasel <[email protected]>; Desimone, Nathaniel L <[email protected]>; Gao, Liming <[email protected]>; Dong, Eric <[email protected]> Subject: [PATCH v2] FIX MinPlatformPkg PCIE Base typecasting error. PCIE Base Address is 64bit PCD and the Mem Limit UINT64. so typecasting to 32bit is not needed. REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4068 Signed-off-by: Ashraf Ali S <[email protected]> Cc: Ray Ni <[email protected]> Cc: Rangasai V Chaganty <[email protected]> Cc: Isaac Oram <[email protected]> Cc: Chasel Chiu <[email protected]> Cc: Nate DeSimone <[email protected]> Cc: Isaac Oram <[email protected]> Cc: Liming Gao <[email protected]> Cc: Eric Dong <[email protected]> --- .../Pci/Library/PciHostBridgeLibSimple/PciHostBridgeLibSimple.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/Intel/MinPlatformPkg/Pci/Library/PciHostBridgeLibSimple/PciHostBridgeLibSimple.c b/Platform/Intel/MinPlatformPkg/Pci/Library/PciHostBridgeLibSimple/PciHostBridgeLibSimple.c index 0e3fee28b5..e38975eee5 100644 --- a/Platform/Intel/MinPlatformPkg/Pci/Library/PciHostBridgeLibSimple/PciHostBridgeLibSimple.c +++ b/Platform/Intel/MinPlatformPkg/Pci/Library/PciHostBridgeLibSimple/PciHostBridgeLibSimple.c @@ -90,7 +90,7 @@ PciHostBridgeGetRootBridges ( if (PcdGet32(PcdPciReservedMemLimit) != 0) { mRootBridgeTemplate.Mem.Limit = PcdGet32 (PcdPciReservedMemLimit); } else { - mRootBridgeTemplate.Mem.Limit = (UINT32) PcdGet64 (PcdPciExpressBaseAddress) - 1; + mRootBridgeTemplate.Mem.Limit = PcdGet64 (PcdPciExpressBaseAddress) - 1; } mRootBridgeTemplate.MemAbove4G.Base = PcdGet64 (PcdPciReservedMemAbove4GBBase); -- 2.33.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#97959): https://edk2.groups.io/g/devel/message/97959 Mute This Topic: https://groups.io/mt/95884079/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
