[PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[samer]

Set the default value of PcdAcpiExposedTableVersions for Aarch64
platforms to 0x20. Previously, the default was set to 0x3E for all
platforms. The new value removes ACPI 1.0b compatability, which forces
the use of XSDT 64-bit pointer, as required by Arm SBBR specification.
This also resolves an error reported by acpiview command, as seen on
the RPi (see https://github.com/pftf/RPi4/issues/25).

Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>

Signed-off-by: Samer El-Haj-Mahmoud <samer@elhajmahmoud.com>
---
 MdeModulePkg/MdeModulePkg.dec | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec
index 91a3c608231c..027f444903ef 100644
--- a/MdeModulePkg/MdeModulePkg.dec
+++ b/MdeModulePkg/MdeModulePkg.dec
@@ -1549,6 +1549,9 @@ [PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
   # @Prompt Console Output Row of Text Setup
   gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow|25|UINT32|0x4000000e
 
+[PcdsFixedAtBuild.AARCH64, PcdsPatchableInModule.AARCH64]
+  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20|UINT32|0x0001004c
+
 [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
   ## UART clock frequency is for the baud rate configuration.
   # @Prompt Serial Port Clock Rate.
-- 
2.17.1

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Laszlo Ersek]

On 03/22/20 17:08, Samer El-Haj-Mahmoud wrote:
> Set the default value of PcdAcpiExposedTableVersions for Aarch64
> platforms to 0x20. Previously, the default was set to 0x3E for all
> platforms. The new value removes ACPI 1.0b compatability, which forces
> the use of XSDT 64-bit pointer, as required by Arm SBBR specification.
> This also resolves an error reported by acpiview command, as seen on
> the RPi (see https://github.com/pftf/RPi4/issues/25).
> 
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> 
> Signed-off-by: Samer El-Haj-Mahmoud <samer@elhajmahmoud.com>
> ---
>  MdeModulePkg/MdeModulePkg.dec | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec
> index 91a3c608231c..027f444903ef 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -1549,6 +1549,9 @@ [PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    # @Prompt Console Output Row of Text Setup
>    gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow|25|UINT32|0x4000000e
>  
> +[PcdsFixedAtBuild.AARCH64, PcdsPatchableInModule.AARCH64]
> +  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20|UINT32|0x0001004c
> +
>  [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    ## UART clock frequency is for the baud rate configuration.
>    # @Prompt Serial Port Clock Rate.
> 

Please consider appending another patch to this series: once the new DEC
default is in place, the same PCD setting is no longer necessary in, and
should arguably be removed from, "ArmVirtPkg/ArmVirtQemu.dsc":

  # ACPI predates the AARCH64 architecture by 5 versions, so
  # we only target OSes that support ACPI v5.0 or later
  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20

Basically: "git revert e0692789058e".

Thanks!
Laszlo

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Samer El-Haj-Mahmoud]

Thanks Laszlo

I agree. But shouldn't we remove the PCD settings from all Arm64 platforms that have this override value of 0x20?

Namely:

edk2/ArmVirtPkg/ArmVirtQemu.dsc
edk2-platforms/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc
edk2-platforms/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
edk2-platforms/Platform/Comcast/RDKQemu/RDKQemu.dsc
edk2-platforms/Platform/Hisilicon/D06/D06.dsc
edk2-platforms/Platform/LeMaker/CelloBoard/CelloBoard.dsc
edk2-platforms/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
edk2-platforms/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
edk2-platforms/Silicon/Socionext/SynQuacer/Acpi.dsc.inc







-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo Ersek via Groups.Io
Sent: Monday, March 23, 2020 8:31 AM
To: devel@edk2.groups.io; samer@elhajmahmoud.com
Cc: Dandan Bi <dandan.bi@intel.com>; Liming Gao <liming.gao@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

On 03/22/20 17:08, Samer El-Haj-Mahmoud wrote:
> Set the default value of PcdAcpiExposedTableVersions for Aarch64
> platforms to 0x20. Previously, the default was set to 0x3E for all
> platforms. The new value removes ACPI 1.0b compatability, which forces
> the use of XSDT 64-bit pointer, as required by Arm SBBR specification.
> This also resolves an error reported by acpiview command, as seen on
> the RPi (see https://github.com/pftf/RPi4/issues/25).
>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> Signed-off-by: Samer El-Haj-Mahmoud <samer@elhajmahmoud.com>
> ---
>  MdeModulePkg/MdeModulePkg.dec | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/MdeModulePkg/MdeModulePkg.dec
> b/MdeModulePkg/MdeModulePkg.dec index 91a3c608231c..027f444903ef
> 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -1549,6 +1549,9 @@ [PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    # @Prompt Console Output Row of Text Setup
>
> gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow|25|UINT32|0x4000000e
>
> +[PcdsFixedAtBuild.AARCH64, PcdsPatchableInModule.AARCH64]
> +
> +gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20|UINT3
> +2|0x0001004c
> +
>  [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    ## UART clock frequency is for the baud rate configuration.
>    # @Prompt Serial Port Clock Rate.
>

Please consider appending another patch to this series: once the new DEC default is in place, the same PCD setting is no longer necessary in, and should arguably be removed from, "ArmVirtPkg/ArmVirtQemu.dsc":

  # ACPI predates the AARCH64 architecture by 5 versions, so
  # we only target OSes that support ACPI v5.0 or later
  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20

Basically: "git revert e0692789058e".

Thanks!
Laszlo




IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Laszlo Ersek]

On 03/23/20 13:53, Samer El-Haj-Mahmoud wrote:
> Thanks Laszlo
> 
> I agree. But shouldn't we remove the PCD settings from all Arm64 platforms that have this override value of 0x20?
> 
> Namely:
> 
> edk2/ArmVirtPkg/ArmVirtQemu.dsc
> edk2-platforms/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc
> edk2-platforms/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
> edk2-platforms/Platform/Comcast/RDKQemu/RDKQemu.dsc
> edk2-platforms/Platform/Hisilicon/D06/D06.dsc
> edk2-platforms/Platform/LeMaker/CelloBoard/CelloBoard.dsc
> edk2-platforms/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
> edk2-platforms/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> edk2-platforms/Silicon/Socionext/SynQuacer/Acpi.dsc.inc

If you can order the edk2-platforms changes safely after the edk2 DEC
file change, then yes.

Otherwise, I can't say.

AFAICT, edk2-platforms does not consume edk2 via a git submodule, so the
ordering looks risky. But I generally don't follow edk2-platforms (which
is also why I didn't grep its DSC files for the PCD in question), so I
really don't know how such ordering questions have been dealt with in
the past.

Thanks
Laszlo

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Samer El-Haj-Mahmoud]

Thanks Laszlo,

I will wait for the maintainers to accept the patch, then submit an amended version for ArmVirtQemu.

-----Original Message-----
From: Laszlo Ersek <lersek@redhat.com>
Sent: Tuesday, March 24, 2020 7:03 PM
To: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; devel@edk2.groups.io; samer@elhajmahmoud.com
Cc: Dandan Bi <dandan.bi@intel.com>; Liming Gao <liming.gao@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

On 03/23/20 13:53, Samer El-Haj-Mahmoud wrote:
> Thanks Laszlo
>
> I agree. But shouldn't we remove the PCD settings from all Arm64 platforms that have this override value of 0x20?
>
> Namely:
>
> edk2/ArmVirtPkg/ArmVirtQemu.dsc
> edk2-platforms/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc
> edk2-platforms/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
> edk2-platforms/Platform/Comcast/RDKQemu/RDKQemu.dsc
> edk2-platforms/Platform/Hisilicon/D06/D06.dsc
> edk2-platforms/Platform/LeMaker/CelloBoard/CelloBoard.dsc
> edk2-platforms/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board
> .dsc edk2-platforms/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> edk2-platforms/Silicon/Socionext/SynQuacer/Acpi.dsc.inc

If you can order the edk2-platforms changes safely after the edk2 DEC file change, then yes.

Otherwise, I can't say.

AFAICT, edk2-platforms does not consume edk2 via a git submodule, so the ordering looks risky. But I generally don't follow edk2-platforms (which is also why I didn't grep its DSC files for the PCD in question), so I really don't know how such ordering questions have been dealt with in the past.

Thanks
Laszlo

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Ard Biesheuvel]

On Wed, 25 Mar 2020 at 03:04, Samer El-Haj-Mahmoud
<Samer.El-Haj-Mahmoud@arm.com> wrote:
>
> Thanks Laszlo,
>
> I will wait for the maintainers to accept the patch, then submit an amended version for ArmVirtQemu.
>

For this patch

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

For any patch that removes this PCD override from AArch64 platforms in
edk2 or edk2-platforms:

Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>


> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Tuesday, March 24, 2020 7:03 PM
> To: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; devel@edk2.groups.io; samer@elhajmahmoud.com
> Cc: Dandan Bi <dandan.bi@intel.com>; Liming Gao <liming.gao@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Subject: Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms
>
> On 03/23/20 13:53, Samer El-Haj-Mahmoud wrote:
> > Thanks Laszlo
> >
> > I agree. But shouldn't we remove the PCD settings from all Arm64 platforms that have this override value of 0x20?
> >
> > Namely:
> >
> > edk2/ArmVirtPkg/ArmVirtQemu.dsc
> > edk2-platforms/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> > edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc
> > edk2-platforms/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
> > edk2-platforms/Platform/Comcast/RDKQemu/RDKQemu.dsc
> > edk2-platforms/Platform/Hisilicon/D06/D06.dsc
> > edk2-platforms/Platform/LeMaker/CelloBoard/CelloBoard.dsc
> > edk2-platforms/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board
> > .dsc edk2-platforms/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> > edk2-platforms/Silicon/Socionext/SynQuacer/Acpi.dsc.inc
>
> If you can order the edk2-platforms changes safely after the edk2 DEC file change, then yes.
>
> Otherwise, I can't say.
>
> AFAICT, edk2-platforms does not consume edk2 via a git submodule, so the ordering looks risky. But I generally don't follow edk2-platforms (which is also why I didn't grep its DSC files for the PCD in question), so I really don't know how such ordering questions have been dealt with in the past.
>
> Thanks
> Laszlo
>
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Samer El-Haj-Mahmoud]

Thanks Ard and Laszlo.

Leif, Dandan, Liming, Hao, any objection in merging this patch to edk2?



> -----Original Message-----
> From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Sent: Wednesday, March 25, 2020 5:16 AM
> To: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
> Cc: Laszlo Ersek <lersek@redhat.com>; devel@edk2.groups.io;
> samer@elhajmahmoud.com; Dandan Bi <dandan.bi@intel.com>; Liming Gao
> <liming.gao@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Leif Lindholm
> <leif@nuviainc.com>
> Subject: Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg:
> PcdAcpiExposedTableVersions default for Arm64 platforms
>
> On Wed, 25 Mar 2020 at 03:04, Samer El-Haj-Mahmoud <Samer.El-Haj-
> Mahmoud@arm.com> wrote:
> >
> > Thanks Laszlo,
> >
> > I will wait for the maintainers to accept the patch, then submit an amended
> version for ArmVirtQemu.
> >
>
> For this patch
>
> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> For any patch that removes this PCD override from AArch64 platforms in
> edk2 or edk2-platforms:
>
> Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
>
> > -----Original Message-----
> > From: Laszlo Ersek <lersek@redhat.com>
> > Sent: Tuesday, March 24, 2020 7:03 PM
> > To: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>;
> > devel@edk2.groups.io; samer@elhajmahmoud.com
> > Cc: Dandan Bi <dandan.bi@intel.com>; Liming Gao
> > <liming.gao@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Leif Lindholm
> > <leif@nuviainc.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Subject: Re: [edk2-devel] [PATCH v1 1/1] MdeModulePkg:
> > PcdAcpiExposedTableVersions default for Arm64 platforms
> >
> > On 03/23/20 13:53, Samer El-Haj-Mahmoud wrote:
> > > Thanks Laszlo
> > >
> > > I agree. But shouldn't we remove the PCD settings from all Arm64
> platforms that have this override value of 0x20?
> > >
> > > Namely:
> > >
> > > edk2/ArmVirtPkg/ArmVirtQemu.dsc
> > > edk2-platforms/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> > > edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc
> > > edk2-platforms/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
> > > edk2-platforms/Platform/Comcast/RDKQemu/RDKQemu.dsc
> > > edk2-platforms/Platform/Hisilicon/D06/D06.dsc
> > > edk2-platforms/Platform/LeMaker/CelloBoard/CelloBoard.dsc
> > > edk2-platforms/Platform/SoftIron/Overdrive1000Board/Overdrive1000Boa
> > > rd .dsc edk2-platforms/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> > > edk2-platforms/Silicon/Socionext/SynQuacer/Acpi.dsc.inc
> >
> > If you can order the edk2-platforms changes safely after the edk2 DEC file
> change, then yes.
> >
> > Otherwise, I can't say.
> >
> > AFAICT, edk2-platforms does not consume edk2 via a git submodule, so the
> ordering looks risky. But I generally don't follow edk2-platforms (which is also
> why I didn't grep its DSC files for the PCD in question), so I really don't know
> how such ordering questions have been dealt with in the past.
> >
> > Thanks
> > Laszlo
> >
> > IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended recipient,
> please notify the sender immediately and do not disclose the contents to any
> other person, use it for any purpose, or store or copy the information in any
> medium. Thank you.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions default for Arm64 platforms

[Dandan Bi]

Reviewed-by: Dandan Bi <dandan.bi@intel.com>


Thanks,
Dandan
> -----Original Message-----
> From: Samer El-Haj-Mahmoud [mailto:samer@elhajmahmoud.com]
> Sent: Monday, March 23, 2020 12:08 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan <dandan.bi@intel.com>; Gao, Liming <liming.gao@intel.com>;
> Wu, Hao A <hao.a.wu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> Biesheuvel <ard.biesheuvel@linaro.org>
> Subject: [PATCH v1 1/1] MdeModulePkg: PcdAcpiExposedTableVersions
> default for Arm64 platforms
> 
> Set the default value of PcdAcpiExposedTableVersions for Aarch64 platforms
> to 0x20. Previously, the default was set to 0x3E for all platforms. The new
> value removes ACPI 1.0b compatability, which forces the use of XSDT 64-bit
> pointer, as required by Arm SBBR specification.
> This also resolves an error reported by acpiview command, as seen on the
> RPi (see https://github.com/pftf/RPi4/issues/25).
> 
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> 
> Signed-off-by: Samer El-Haj-Mahmoud <samer@elhajmahmoud.com>
> ---
>  MdeModulePkg/MdeModulePkg.dec | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/MdeModulePkg/MdeModulePkg.dec
> b/MdeModulePkg/MdeModulePkg.dec index 91a3c608231c..027f444903ef
> 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -1549,6 +1549,9 @@ [PcdsPatchableInModule, PcdsDynamic,
> PcdsDynamicEx]
>    # @Prompt Console Output Row of Text Setup
> 
> gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow|25|UINT32|0x40
> 00000e
> 
> +[PcdsFixedAtBuild.AARCH64, PcdsPatchableInModule.AARCH64]
> +
> +gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20|
> UINT32|
> +0x0001004c
> +
>  [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    ## UART clock frequency is for the baud rate configuration.
>    # @Prompt Serial Port Clock Rate.
> --
> 2.17.1