Re: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in MpWakeupStructure

["duntan" <dun.tan@intel.com>]

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 <gaoliming@byosoft.com.cn> 
Sent: Friday, March 10, 2023 10:16 AM
To: Tan, Dun <dun.tan@intel.com>; devel@edk2.groups.io
Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>
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 <dun.tan@intel.com>
> 发送时间: 2023年3月9日 11:40
> 收件人: devel@edk2.groups.io
> 抄送: Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao 
> <gaoliming@byosoft.com.cn>; Zhiguang Liu <zhiguang.liu@intel.com>
> 主题: [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 <dun.tan@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
>  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 {
>    UINT16    Command;
>    UINT16    Reserved;
> -  UINT32    AcpiId;
> +  UINT32    ApicId;
>    UINT64    WakeupVector;
>    UINT8     ReservedForOs[2032];
>    UINT8     ReservedForFirmware[2048];
> --
> 2.31.1.windows.1