Hello Liming,
My take is that the consumers of the ACPI header file will always use the
latest version.
Is my understanding correct? Or should I update all the header files relating
to the affected versions (5.1~6.4) of the ACPI spec?
Best Regards,
Hao Wu
> -----Original Message-----
> From: gaoliming <gaolim...@byosoft.com.cn>
> Sent: Tuesday, March 8, 2022 9:42 AM
> To: Wu, Hao A <hao.a...@intel.com>; devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Liu, Zhiguang
> <zhiguang....@intel.com>
> Subject: 回复: [PATCH] MdePkg/Acpi64: Update Error Severity type for
> Generic Error Status Block
>
> Hao:
> Can you let me know why you update Acpi64.h only?
>
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Hao A Wu <hao.a...@intel.com>
> > 发送时间: 2022年3月7日 16:35
> > 收件人: devel@edk2.groups.io
> > 抄送: Hao A Wu <hao.a...@intel.com>; Michael D Kinney
> > <michael.d.kin...@intel.com>; Liming Gao <gaolim...@byosoft.com.cn>;
> > Zhiguang Liu <zhiguang....@intel.com>
> > 主题: [PATCH] MdePkg/Acpi64: Update Error Severity type for Generic
> > Error Status Block
> >
> > Starting from ACPI Specification Version 5.1 Errata B, the term
> > 'Correctable' is no longer being used as an error severity of the
> > reported error in Chapter 18 APEI.
> >
> > This commit adds a new macro to reflect this and also keeps the origin
> > definition (but add comments to make it deprecated) for compatibility
> > consideration.
> >
> > Please also note that this commit only updates Acpi64.h and does NOT
> > update below headers:
> > Acpi51.h
> > Acpi60.h
> > Acpi61.h
> > Acpi62.h
> > Acpi63.h
> >
> > Cc: Michael D Kinney <michael.d.kin...@intel.com>
> > Cc: Liming Gao <gaolim...@byosoft.com.cn>
> > Cc: Zhiguang Liu <zhiguang....@intel.com>
> > Signed-off-by: Hao A Wu <hao.a...@intel.com>
> > ---
> > MdePkg/Include/IndustryStandard/Acpi64.h | 10 ++++++++--
> > 1 file changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/MdePkg/Include/IndustryStandard/Acpi64.h
> > b/MdePkg/Include/IndustryStandard/Acpi64.h
> > index a79b7f2eaa..232697f228 100644
> > --- a/MdePkg/Include/IndustryStandard/Acpi64.h
> > +++ b/MdePkg/Include/IndustryStandard/Acpi64.h
> > @@ -1,7 +1,7 @@
> > /** @file
> >
> > ACPI 6.4 definitions from the ACPI Specification Revision 6.4 Jan,
> 2021.
> >
> >
> >
> > - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
> >
> > + Copyright (c) 2017 - 2022, Intel Corporation. All rights
> > + reserved.<BR>
> >
> > Copyright (c) 2019 - 2021, ARM Ltd. All rights reserved.<BR>
> >
> >
> >
> > SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > @@ -1783,10 +1783,16 @@ typedef struct { //
> >
> > // Boot Error Severity types
> >
> > //
> >
> > -#define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTABLE 0x00
> >
> > +#define EFI_ACPI_6_4_ERROR_SEVERITY_RECOVERABLE 0x00
> >
> > #define EFI_ACPI_6_4_ERROR_SEVERITY_FATAL 0x01
> >
> > #define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTED 0x02
> >
> > #define EFI_ACPI_6_4_ERROR_SEVERITY_NONE 0x03
> >
> > +//
> >
> > +// The term 'Correctable' is no longer being used as an error
> > +severity of
> the
> >
> > +// reported error since ACPI Specification Version 5.1 Errata B.
> >
> > +// The below macro is considered as deprecated and should no longer
> > +be
> > used.
> >
> > +//
> >
> > +#define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTABLE 0x00
> >
> >
> >
> > ///
> >
> > /// Generic Error Data Entry Definition
> >
> > --
> > 2.27.0.windows.1
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#87322): https://edk2.groups.io/g/devel/message/87322
Mute This Topic: https://groups.io/mt/89628719/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
