Adding Statuc Code Definitions from PI Spec v1.8

[edk2-devel] [PATCH v1 0/3] Adding Statuc Code Definitions from PI Spec v1.8

Posted by Kun Qin 11 months ago

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3966

This is an official patch series to add status codes into PiStatusCode,
continued from 2 previously sent code-first patch series:
https://edk2.groups.io/g/devel/message/91330
https://edk2.groups.io/g/devel/message/92093

Now the macro definitions have landed in PI spec v1.8, this patch is
to add them in Tianocore.

Patch v1 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v1

Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Qi Zhang <qi1.zhang@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>

Kun Qin (3):
  MdePkg: PiStatusCode: Add new Host Software class Error Code to MdePkg
  MdePkg: PiStatusCode: Add TPM subclass definition to MdePkg
  SecurityPkg: SubClassTpm: Updated default value

 MdePkg/Include/Pi/PiStatusCode.h | 42 ++++++++++----------
 SecurityPkg/SecurityPkg.dec      |  6 +--
 SecurityPkg/SecurityPkg.uni      |  2 +-
 3 files changed, 26 insertions(+), 24 deletions(-)

-- 
2.40.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105599): https://edk2.groups.io/g/devel/message/105599
Mute This Topic: https://groups.io/mt/99271145/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

[edk2-devel] 回复: [PATCH v1 0/3] Adding Statuc Code Definitions from PI Spec v1.8

Posted by gaoliming via groups.io 11 months ago

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: Kun Qin <kuqin12@gmail.com>
> 发送时间: 2023年6月2日 2:30
> 收件人: devel@edk2.groups.io
> 抄送: Jiewen Yao <jiewen.yao@intel.com>; Jian J Wang
> <jian.j.wang@intel.com>; Qi Zhang <qi1.zhang@intel.com>; Rahul Kumar
> <rahul1.kumar@intel.com>; Michael D Kinney <michael.d.kinney@intel.com>;
> Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu
> <zhiguang.liu@intel.com>
> 主题: [PATCH v1 0/3] Adding Statuc Code Definitions from PI Spec v1.8
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3966
> 
> This is an official patch series to add status codes into PiStatusCode,
> continued from 2 previously sent code-first patch series:
> https://edk2.groups.io/g/devel/message/91330
> https://edk2.groups.io/g/devel/message/92093
> 
> Now the macro definitions have landed in PI spec v1.8, this patch is
> to add them in Tianocore.
> 
> Patch v1 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v1
> 
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Qi Zhang <qi1.zhang@intel.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> 
> Kun Qin (3):
>   MdePkg: PiStatusCode: Add new Host Software class Error Code to MdePkg
>   MdePkg: PiStatusCode: Add TPM subclass definition to MdePkg
>   SecurityPkg: SubClassTpm: Updated default value
> 
>  MdePkg/Include/Pi/PiStatusCode.h | 42 ++++++++++----------
>  SecurityPkg/SecurityPkg.dec      |  6 +--
>  SecurityPkg/SecurityPkg.uni      |  2 +-
>  3 files changed, 26 insertions(+), 24 deletions(-)
> 
> --
> 2.40.1.windows.1





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105609): https://edk2.groups.io/g/devel/message/105609
Mute This Topic: https://groups.io/mt/99278049/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] 回复: [PATCH v1 0/3] Adding Statuc Code Definitions from PI Spec v1.8

Posted by Kun Qin 11 months ago

Hi Liming/Jiewen,

Do you mind helping merging this patch series? It has collected necessary
reviewed-by tags already:
https://edk2.groups.io/g/devel/message/105602
https://edk2.groups.io/g/devel/message/105609

Please let me know if it is preferable for me to add the garner and send 
a v2.

Thanks,
Kun

On 6/1/2023 6:18 PM, gaoliming via groups.io wrote:
> Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
>
>> -----邮件原件-----
>> 发件人: Kun Qin <kuqin12@gmail.com>
>> 发送时间: 2023年6月2日 2:30
>> 收件人: devel@edk2.groups.io
>> 抄送: Jiewen Yao <jiewen.yao@intel.com>; Jian J Wang
>> <jian.j.wang@intel.com>; Qi Zhang <qi1.zhang@intel.com>; Rahul Kumar
>> <rahul1.kumar@intel.com>; Michael D Kinney <michael.d.kinney@intel.com>;
>> Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu
>> <zhiguang.liu@intel.com>
>> 主题: [PATCH v1 0/3] Adding Statuc Code Definitions from PI Spec v1.8
>>
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3966
>>
>> This is an official patch series to add status codes into PiStatusCode,
>> continued from 2 previously sent code-first patch series:
>> https://edk2.groups.io/g/devel/message/91330
>> https://edk2.groups.io/g/devel/message/92093
>>
>> Now the macro definitions have landed in PI spec v1.8, this patch is
>> to add them in Tianocore.
>>
>> Patch v1 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v1
>>
>> Cc: Jiewen Yao <jiewen.yao@intel.com>
>> Cc: Jian J Wang <jian.j.wang@intel.com>
>> Cc: Qi Zhang <qi1.zhang@intel.com>
>> Cc: Rahul Kumar <rahul1.kumar@intel.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
>>
>> Kun Qin (3):
>>    MdePkg: PiStatusCode: Add new Host Software class Error Code to MdePkg
>>    MdePkg: PiStatusCode: Add TPM subclass definition to MdePkg
>>    SecurityPkg: SubClassTpm: Updated default value
>>
>>   MdePkg/Include/Pi/PiStatusCode.h | 42 ++++++++++----------
>>   SecurityPkg/SecurityPkg.dec      |  6 +--
>>   SecurityPkg/SecurityPkg.uni      |  2 +-
>>   3 files changed, 26 insertions(+), 24 deletions(-)
>>
>> --
>> 2.40.1.windows.1
>
>
>
>
> 
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105830): https://edk2.groups.io/g/devel/message/105830
Mute This Topic: https://groups.io/mt/99278049/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-