[edk2-devel] [edk2-staging][PATCH v1 0/7] Add TPM subclass definition

[Kun Qin]

Hi Mike,

The intention was to encourage the usage of new definition in .H file 
once the definition
is updated. The PCD is left as is only to serve as backwards 
compatibility purpose.

Do you think only updating the DEC and .H file is more ideal? I could 
drop the other changes
if so desired.

Thanks,
Kun

On 7/13/2022 5:30 PM, Michael D Kinney wrote:
> Hi Kun,
>
> Why was the PCD usage not preserved and only the value updated in the DEC file and .H file?
>
> Mike
>
>> -----Original Message-----
>> From: Kun Qin <kuqin12 at gmail.com>
>> Sent: Wednesday, July 13, 2022 5:21 PM
>> To: devel at edk2.groups.io
>> Cc: Yao, Jiewen <jiewen.yao at intel.com>; Wang, Jian J <jian.j.wang at intel.com>; Zhang, Qi1 <qi1.zhang at intel.com>; Kumar, Rahul1
>> <rahul1.kumar at intel.com>; Andrew Fish <afish at apple.com>; Leif Lindholm <leif at nuviainc.com>; Kinney, Michael D
>> <michael.d.kinney at intel.com>; Gao, Liming <gaoliming at byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu at intel.com>
>> Subject: Re: [edk2-devel] [edk2-staging][PATCH v1 0/7] Add TPM subclass definition
>>
>> Hi SecurityPkg maintainers & EDK2 stewards,
>>
>> I sent out this patch series intending to update/fix the PI spec through
>> code first process.
>>
>> Could you please shed some light on it and let me know if any feedback?
>>
>> Thanks in advance!
>>
>> Regards,
>> Kun
>>
>> On 7/5/2022 8:38 PM, Kun Qin via groups.io wrote:
>>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3966
>>>
>>>   From PI Specification v1.7 Errata A, EFI_PERIPHERAL_DOCKING is defined as
>>> 0xD0000 (as well as included in PiStatusCode.h).
>>>
>>> However, subclass employed as PCD for TPM peripheral in SecurityPkg is
>>> also defined as 0xD0000. The TPM subclass code was used in TcgPei.c when
>>> reporting error codes.
>>>
>>> The collision of subclass definition could cause the parsing of reported
>>> errors being ambiguous.
>>>
>>> This patch series add EFI_PERIPHERAL_TPM as a spec-defined value and
>>> removed potential usages in the SecurityPkg.
>>>
>>> Patch v1 branch: https://github.com/kuqin12/edk2/tree/BZ3966-add_tpm_subclass
>>>
>>> Cc: Jiewen Yao <jiewen.yao at intel.com>
>>> Cc: Jian J Wang <jian.j.wang at intel.com>
>>> Cc: Qi Zhang <qi1.zhang at intel.com>
>>> Cc: Rahul Kumar <rahul1.kumar at intel.com>
>>> Cc: Andrew Fish <afish at apple.com>
>>> Cc: Leif Lindholm <leif at nuviainc.com>
>>> Cc: Michael D Kinney <michael.d.kinney at intel.com>
>>> Cc: Liming Gao <gaoliming at byosoft.com.cn>
>>> Cc: Zhiguang Liu <zhiguang.liu at intel.com>
>>>
>>> Kun Qin (7):
>>>     EDK2 Code First: PI Specification: New peripheral subclass for TPM
>>>     MdePkg: MmCommunication: Add TPM subclass definition to MdePkg
>>>     SecurityPkg: Tcg2Dxe: Replace PcdStatusCodeSubClassTpmDevice
>>>     SecurityPkg: Tcg2Pei: Replace PcdStatusCodeSubClassTpmDevice
>>>     SecurityPkg: TcgDxe: Replace PcdStatusCodeSubClassTpmDevice
>>>     SecurityPkg: TcgPei: Replace PcdStatusCodeSubClassTpmDevice
>>>     SecurityPkg: SubClassTpm: Updated default value
>>>
>>>    SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c   |  4 +-
>>>    SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c   |  4 +-
>>>    SecurityPkg/Tcg/TcgDxe/TcgDxe.c     |  2 +-
>>>    SecurityPkg/Tcg/TcgPei/TcgPei.c     |  4 +-
>>>    CodeFirst/BZ3966-SpecChange.md      | 60 ++++++++++++++++++++
>>>    MdePkg/Include/Pi/PiStatusCode.h    |  1 +
>>>    SecurityPkg/SecurityPkg.dec         |  6 +-
>>>    SecurityPkg/SecurityPkg.uni         |  2 +-
>>>    SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.inf |  1 -
>>>    SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.inf |  1 -
>>>    SecurityPkg/Tcg/TcgDxe/TcgDxe.inf   |  1 -
>>>    SecurityPkg/Tcg/TcgPei/TcgPei.inf   |  1 -
>>>    12 files changed, 72 insertions(+), 15 deletions(-)
>>>    create mode 100644 CodeFirst/BZ3966-SpecChange.md
>>>
>
> 
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91327): https://edk2.groups.io/g/devel/message/91327
Mute This Topic: https://groups.io/mt/92200211/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-