Re: [edk2-devel] 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in MpWakeupStructure

2023-03-24 Thread duntan 
Thanks Liming. Sorry for missing the email.
Here is the PR for the two patches: https://github.com/tianocore/edk2/pull/4169

Thanks,
Dun
-Original Message-
From: devel@edk2.groups.io  On Behalf Of gaoliming via 
groups.io
Sent: Thursday, March 23, 2023 9:17 AM
To: Tan, Dun ; devel@edk2.groups.io
Cc: Kinney, Michael D ; Liu, Zhiguang 

Subject: [edk2-devel] 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 
'ApicId' in MpWakeupStructure

Dun:
  OK. Reviewed-by: Liming Gao  

  Can you create PR for this patch set? I will merge this PR. 

Thanks
Liming
> -邮件原件-
> 发件人: Tan, Dun 
> 发送时间: 2023年3月10日 14:17
> 收件人: Gao, Liming ; devel@edk2.groups.io
> 抄送: Kinney, Michael D ; Liu, Zhiguang 
> 
> 主题: RE: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' 
> in MpWakeupStructure
> 
> Hi Liming,
> 
> I've checked edk2 and edk2-platforms repo. This code change doesn't 
> impacted any code.
> The MultiprocessorWakeupMailbox structure has not been used yet.
> 
> Thanks,
> Dun
> 
> -Original Message-
> From: gaoliming 
> Sent: Friday, March 10, 2023 10:16 AM
> To: Tan, Dun ; devel@edk2.groups.io
> Cc: Kinney, Michael D ; Liu, Zhiguang 
> 
> Subject: 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId'
> in MpWakeupStructure
> 
> Dun:
>   Is there any code impact with this change?
> 
> Thanks
> Liming
> > -邮件原件-
> > 发件人: Dun Tan 
> > 发送时间: 2023年3月9日 11:40
> > 收件人: devel@edk2.groups.io
> > 抄送: Michael D Kinney ; Liming Gao 
> > ; Zhiguang Liu 
> > 主题: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in 
> > MpWakeupStructure
> >
> > modify the wrong 'AcpiId' to 'ApicId' of MpWakeupStructure 
> > defination in Acpi64.h.
> >
> > Signed-off-by: Dun Tan 
> > Cc: Michael D Kinney 
> > Cc: Liming Gao 
> > Reviewed-by: Zhiguang Liu 
> > ---
> >  MdePkg/Include/IndustryStandard/Acpi64.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/MdePkg/Include/IndustryStandard/Acpi64.h
> > b/MdePkg/Include/IndustryStandard/Acpi64.h
> > index fe5ebfac2b..bfd022b6b6 100644
> > --- a/MdePkg/Include/IndustryStandard/Acpi64.h
> > +++ b/MdePkg/Include/IndustryStandard/Acpi64.h
> > @@ -607,7 +607,7 @@ typedef struct {  typedef struct {
> >UINT16Command;
> >UINT16Reserved;
> > -  UINT32AcpiId;
> > +  UINT32ApicId;
> >UINT64WakeupVector;
> >UINT8 ReservedForOs[2032];
> >UINT8 ReservedForFirmware[2048];
> > --
> > 2.31.1.windows.1
> 
> 










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

[edk2-devel] 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in MpWakeupStructure

2023-03-22 Thread gaoliming via groups.io 
Dun:
  OK. Reviewed-by: Liming Gao  

  Can you create PR for this patch set? I will merge this PR. 

Thanks
Liming
> -邮件原件-
> 发件人: Tan, Dun 
> 发送时间: 2023年3月10日 14:17
> 收件人: Gao, Liming ; devel@edk2.groups.io
> 抄送: Kinney, Michael D ; Liu, Zhiguang
> 
> 主题: RE: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in
> MpWakeupStructure
> 
> Hi Liming,
> 
> I've checked edk2 and edk2-platforms repo. This code change doesn't
> impacted any code.
> The MultiprocessorWakeupMailbox structure has not been used yet.
> 
> Thanks,
> Dun
> 
> -Original Message-
> From: gaoliming 
> Sent: Friday, March 10, 2023 10:16 AM
> To: Tan, Dun ; devel@edk2.groups.io
> Cc: Kinney, Michael D ; Liu, Zhiguang
> 
> Subject: 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId'
> in MpWakeupStructure
> 
> Dun:
>   Is there any code impact with this change?
> 
> Thanks
> Liming
> > -邮件原件-
> > 发件人: Dun Tan 
> > 发送时间: 2023年3月9日 11:40
> > 收件人: devel@edk2.groups.io
> > 抄送: Michael D Kinney ; Liming Gao
> > ; Zhiguang Liu 
> > 主题: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in
> > MpWakeupStructure
> >
> > modify the wrong 'AcpiId' to 'ApicId' of MpWakeupStructure defination
> > in Acpi64.h.
> >
> > Signed-off-by: Dun Tan 
> > Cc: Michael D Kinney 
> > Cc: Liming Gao 
> > Reviewed-by: Zhiguang Liu 
> > ---
> >  MdePkg/Include/IndustryStandard/Acpi64.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/MdePkg/Include/IndustryStandard/Acpi64.h
> > b/MdePkg/Include/IndustryStandard/Acpi64.h
> > index fe5ebfac2b..bfd022b6b6 100644
> > --- a/MdePkg/Include/IndustryStandard/Acpi64.h
> > +++ b/MdePkg/Include/IndustryStandard/Acpi64.h
> > @@ -607,7 +607,7 @@ typedef struct {
> >  typedef struct {
> >UINT16Command;
> >UINT16Reserved;
> > -  UINT32AcpiId;
> > +  UINT32ApicId;
> >UINT64WakeupVector;
> >UINT8 ReservedForOs[2032];
> >UINT8 ReservedForFirmware[2048];
> > --
> > 2.31.1.windows.1
> 
> 





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

[edk2-devel] 回复: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in MpWakeupStructure

2023-03-09 Thread gaoliming via groups.io 
Dun:
  Is there any code impact with this change?

Thanks
Liming
> -邮件原件-
> 发件人: Dun Tan 
> 发送时间: 2023年3月9日 11:40
> 收件人: devel@edk2.groups.io
> 抄送: Michael D Kinney ; Liming Gao
> ; Zhiguang Liu 
> 主题: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in
> MpWakeupStructure
> 
> modify the wrong 'AcpiId' to 'ApicId' of MpWakeupStructure defination
> in Acpi64.h.
> 
> Signed-off-by: Dun Tan 
> Cc: Michael D Kinney 
> Cc: Liming Gao 
> Reviewed-by: Zhiguang Liu 
> ---
>  MdePkg/Include/IndustryStandard/Acpi64.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MdePkg/Include/IndustryStandard/Acpi64.h
> b/MdePkg/Include/IndustryStandard/Acpi64.h
> index fe5ebfac2b..bfd022b6b6 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi64.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi64.h
> @@ -607,7 +607,7 @@ typedef struct {
>  typedef struct {
>UINT16Command;
>UINT16Reserved;
> -  UINT32AcpiId;
> +  UINT32ApicId;
>UINT64WakeupVector;
>UINT8 ReservedForOs[2032];
>UINT8 ReservedForFirmware[2048];
> --
> 2.31.1.windows.1





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