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

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

[Kun Qin]

Hi EDK2 maintainers,

This patch series was reviewed the week before and was not merged. Could
you please help merging this when you have a chance?

Thanks in advance.

Regards,
Kun

On 6/8/2023 11:25 AM, Kun Qin via groups.io wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3966
>
> This v2 series is a follow up of previously submitted patches:
> https://edk2.groups.io/g/devel/message/105599
>
> The v2 patches include `Reviewed-by` tags collected during the v1
> round and no other changes.
>
> Patch v2 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v2
>
> 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(-)
>

[PATCH v2 0/3] Adding Status Code Definitions from PI Spec v1.8

[Kun Qin]

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

This v2 series is a follow up of previously submitted patches:
https://edk2.groups.io/g/devel/message/105599

The v2 patches include `Reviewed-by` tags collected during the v1
round and no other changes.

Patch v2 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v2

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

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

[Michael D Kinney]

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kun Qin
> Sent: Thursday, June 8, 2023 11:25 AM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Zhang, Qi1 <qi1.zhang@intel.com>; Kumar, Rahul R
> <rahul.r.kumar@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>;
> Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> <zhiguang.liu@intel.com>
> Subject: [edk2-devel] [PATCH v2 0/3] Adding Status 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 v2 series is a follow up of previously submitted patches:
> https://edk2.groups.io/g/devel/message/105599
> 
> The v2 patches include `Reviewed-by` tags collected during the v1
> round and no other changes.
> 
> Patch v2 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v2
> 
> 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
> 
> 
> 
> 
> 

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

[Kun Qin]

Hi Mike,

Thank you for the rb tag. Could you please help merging the change if 
you have a chance?

If it is preferable to send a v3 with your extra tag, please let me know 
as well.

Thanks,
Kun

On 6/14/2023 3:31 PM, Michael D Kinney wrote:
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
>
> Mike
>
>> -----Original Message-----
>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kun Qin
>> Sent: Thursday, June 8, 2023 11:25 AM
>> To: devel@edk2.groups.io
>> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
>> <jian.j.wang@intel.com>; Zhang, Qi1 <qi1.zhang@intel.com>; Kumar, Rahul R
>> <rahul.r.kumar@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>;
>> Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
>> <zhiguang.liu@intel.com>
>> Subject: [edk2-devel] [PATCH v2 0/3] Adding Status 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 v2 series is a follow up of previously submitted patches:
>> https://edk2.groups.io/g/devel/message/105599
>>
>> The v2 patches include `Reviewed-by` tags collected during the v1
>> round and no other changes.
>>
>> Patch v2 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v2
>>
>> 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
>>
>>
>>
>>
>>
>
>
> 
>
>

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

[Kun Qin]

Hi package maintainers,

A gentle ping on this thread again. If has been a week without any movement
on this patch...

Could any package maintainer provide feedback on how to proceed? Either a
new round of patches with the new reviewed-by or helping me merging the 
changes?

Thanks...
Kun

On 6/16/2023 4:38 PM, Kun Qin via groups.io wrote:
> Hi Mike,
>
> Thank you for the rb tag. Could you please help merging the change if 
> you have a chance?
>
> If it is preferable to send a v3 with your extra tag, please let me 
> know as well.
>
> Thanks,
> Kun
>
> On 6/14/2023 3:31 PM, Michael D Kinney wrote:
>> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
>>
>> Mike
>>
>>> -----Original Message-----
>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kun Qin
>>> Sent: Thursday, June 8, 2023 11:25 AM
>>> To: devel@edk2.groups.io
>>> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
>>> <jian.j.wang@intel.com>; Zhang, Qi1 <qi1.zhang@intel.com>; Kumar, 
>>> Rahul R
>>> <rahul.r.kumar@intel.com>; Kinney, Michael D 
>>> <michael.d.kinney@intel.com>;
>>> Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
>>> <zhiguang.liu@intel.com>
>>> Subject: [edk2-devel] [PATCH v2 0/3] Adding Status 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 v2 series is a follow up of previously submitted patches:
>>> https://edk2.groups.io/g/devel/message/105599
>>>
>>> The v2 patches include `Reviewed-by` tags collected during the v1
>>> round and no other changes.
>>>
>>> Patch v2 branch: https://github.com/kuqin12/edk2/tree/pi_v1p8_v2
>>>
>>> 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
>>>
>>>
>>>
>>>
>>>
>>
>>
>>
>>
>>
>
>
> 
>
>