On 2/15/2023 10:13 PM, Rebecca Cran via groups.io wrote:

+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
+  ## Support rate limiting
+  gUsbNetworkPkgTokenSpaceGuid.EnableRateLimiting|FALSE|BOOLEAN|0x00010001
+
+  ## The rate limiting Credit value is check in rate limiter event.
+  #  It is to control the RateLimitingCreditCount max value.
+  gUsbNetworkPkgTokenSpaceGuid.RateLimitingCredit|10|UINT32|0x00010002
+
Is that valid to have all those PCD types in a single section?


I found this style in MdeModulePkg, and thanks to this Platform code may 
specify this PCD as the type they want to use

https://github.com/tianocore/edk2/blob/5c551d6d912967ada3084033acea8acf37256043/MdeModulePkg/MdeModulePkg.dec#L1637

But I have no idea whether that is valid or not.

oop it reminds me of my issue.

Recently you add ArmPlatformPkg/PrePeiCore: Print the firmware version early in 
boot · tianocore/edk2@56035d1 
(github.com)<https://github.com/tianocore/edk2/commit/56035d1c8b255b33b34cd4625c26bc7d38504e30>

this causes a problem when using 
gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwareVersionString as a dynamic PCD 
(PrePei doesn't support pcdlib);

e.g., we need to add information to PcdFirmwareVersionString (that can come 
from arm-tf or optionRom ...).

Thanks,

Tinh


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#100290): https://edk2.groups.io/g/devel/message/100290
Mute This Topic: https://groups.io/mt/96977783/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to