Re: [edk2-devel] [PATCH] edk II C Coding Standard: Remove section 5.4.2.2 STATIC

2022-11-25 Thread Chang, Abner via groups.io
[AMD Official Use Only - General] Hi Ray and Mike, Besides removing 5.4.2.2.1 and 5.4.2.2.2, the "static" should be also removed from 5.6.1.2 section title. I just went back to check the comment of BZ1766 and yes we can add the recommendation of using static. Just having it in 5.4.2.2. Will

[edk2-devel] [edk2-platforms][PATCH V1 1/1] Silicon/ARM/NeoverseN1Soc: Update CCIX PNP ID

2022-11-25 Thread sahil
There is no need for a separate ID for CCIX host bridge, therefore reusing PCIe PNP ID for CCIX. Also, updating the file's license to resolve error during ECC checks. Signed-off-by: sahil --- Silicon/ARM/NeoverseN1Soc/Library/PciHostBridgeLib/PciHostBridgeLib.c | 17 + 1 file ch

Re: [edk2-devel] [PATCH] Multiple Coverity issues were found from EDK2 ShellPkg

2022-11-25 Thread kalaivanip via groups.io
Please ignore this path. Sorry for the inconvenience, will send version 2 patch. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96629): https://edk2.groups.io/g/devel/message/96629 Mute This Topic: https://groups.io/mt/95197753/21656 Grou

[edk2-devel] [PATCH EDK2 v1 1/1] CryptoPkg/BaseCryptLib:time overflow

2022-11-25 Thread wenyi,xie via groups.io
From: Zihong Yi REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4167 In CrtLibSupport.h, time_t is defined as INT32, and its maximum value is 2147483647. That is, the corresponding maximum timestamp is 2038-01-19 11:14:07. Therefore, overflow occurs when the test time exceeds 2038-01-19 11:1

[edk2-devel] [PATCH EDK2 v1 0/1] CryptoPkg/BaseCryptLib:time overflow

2022-11-25 Thread wenyi,xie via groups.io
Main Changes : 1.Change type of time_t to INT64. 2.Using function DivS64x64Remainder to deal with INT64 divide. Zihong Yi (1): CryptoPkg/BaseCryptLib:time overflow CryptoPkg/Library/Include/CrtLibSupport.h | 2 +- CryptoPkg/Library/BaseCryptLib/SysCall/TimerWrapper.c | 51