* [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
@ 2017-02-24 18:40 Ard Biesheuvel
2017-02-24 23:18 ` Laszlo Ersek
2017-02-27 15:44 ` Leif Lindholm
0 siblings, 2 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-02-24 18:40 UTC (permalink / raw)
To: edk2-devel, leif.lindholm; +Cc: lersek, Ard Biesheuvel
This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
any remaining references to it in various platform .DSC files. It is
no longer used now that we removed the virtual uncached pages protocol
and the associated DebugUncachedMemoryAllocationLib library instance.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
ArmPkg/ArmPkg.dec | 1 -
ArmVirtPkg/ArmVirtQemu.dsc | 5 -----
ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -----
BeagleBoardPkg/BeagleBoardPkg.dsc | 5 -----
Omap35xxPkg/Omap35xxPkg.dsc | 5 -----
5 files changed, 21 deletions(-)
diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
index 4fd7a5be5158..c4b4da2f95bb 100644
--- a/ArmPkg/ArmPkg.dec
+++ b/ArmPkg/ArmPkg.dec
@@ -84,7 +84,6 @@ [PcdsFixedAtBuild.common]
# Using a FeaturePcd make a '(BOOLEAN) casting for its value which is not understood by the preprocessor.
gArmTokenSpaceGuid.PcdVFPEnabled|0|UINT32|0x00000024
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000080000000|UINT64|0x00000002
# This PCD will free the unallocated buffers if their size reach this threshold.
# We set the default value to 512MB.
gArmTokenSpaceGuid.PcdArmFreeUncachedMemorySizeThreshold|0x20000000|UINT64|0x00000003
diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
index 8fe3c3816961..00cdd7c20201 100644
--- a/ArmVirtPkg/ArmVirtQemu.dsc
+++ b/ArmVirtPkg/ArmVirtQemu.dsc
@@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
# Size of the region used by UEFI in permanent memory (Reserved 64MB)
gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
- #
- # ARM Pcds
- #
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
-
## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
index aa40374745af..8354b13ddb1a 100644
--- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
+++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
@@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
# Size of the region used by UEFI in permanent memory (Reserved 64MB)
gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
- #
- # ARM Pcds
- #
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
-
## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc
index a71a01ac7723..87879c33fcd9 100644
--- a/BeagleBoardPkg/BeagleBoardPkg.dsc
+++ b/BeagleBoardPkg/BeagleBoardPkg.dsc
@@ -368,11 +368,6 @@ [PcdsFixedAtBuild.common]
# Shell.
gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
- #
- # ARM Pcds
- #
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
-
gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10
# GUID of the UEFI Shell
diff --git a/Omap35xxPkg/Omap35xxPkg.dsc b/Omap35xxPkg/Omap35xxPkg.dsc
index ad7d9898c330..fc94ca7ed129 100644
--- a/Omap35xxPkg/Omap35xxPkg.dsc
+++ b/Omap35xxPkg/Omap35xxPkg.dsc
@@ -162,11 +162,6 @@ [PcdsFixedAtBuild.common]
# OMAP Interrupt Controller
gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000
- #
- # ARM Pcds
- #
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
-
################################################################################
#
# Components Section - list of all EDK II Modules needed by this Platform
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
2017-02-24 18:40 [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD Ard Biesheuvel
@ 2017-02-24 23:18 ` Laszlo Ersek
2017-02-27 15:44 ` Leif Lindholm
1 sibling, 0 replies; 6+ messages in thread
From: Laszlo Ersek @ 2017-02-24 23:18 UTC (permalink / raw)
To: Ard Biesheuvel, edk2-devel, leif.lindholm
On 02/24/17 19:40, Ard Biesheuvel wrote:
> This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
> any remaining references to it in various platform .DSC files. It is
> no longer used now that we removed the virtual uncached pages protocol
> and the associated DebugUncachedMemoryAllocationLib library instance.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
> ArmPkg/ArmPkg.dec | 1 -
> ArmVirtPkg/ArmVirtQemu.dsc | 5 -----
> ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -----
> BeagleBoardPkg/BeagleBoardPkg.dsc | 5 -----
> Omap35xxPkg/Omap35xxPkg.dsc | 5 -----
> 5 files changed, 21 deletions(-)
>
> diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
> index 4fd7a5be5158..c4b4da2f95bb 100644
> --- a/ArmPkg/ArmPkg.dec
> +++ b/ArmPkg/ArmPkg.dec
> @@ -84,7 +84,6 @@ [PcdsFixedAtBuild.common]
> # Using a FeaturePcd make a '(BOOLEAN) casting for its value which is not understood by the preprocessor.
> gArmTokenSpaceGuid.PcdVFPEnabled|0|UINT32|0x00000024
>
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000080000000|UINT64|0x00000002
> # This PCD will free the unallocated buffers if their size reach this threshold.
> # We set the default value to 512MB.
> gArmTokenSpaceGuid.PcdArmFreeUncachedMemorySizeThreshold|0x20000000|UINT64|0x00000003
> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
> index 8fe3c3816961..00cdd7c20201 100644
> --- a/ArmVirtPkg/ArmVirtQemu.dsc
> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>
> diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
> index aa40374745af..8354b13ddb1a 100644
> --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
> +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>
> diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc
> index a71a01ac7723..87879c33fcd9 100644
> --- a/BeagleBoardPkg/BeagleBoardPkg.dsc
> +++ b/BeagleBoardPkg/BeagleBoardPkg.dsc
> @@ -368,11 +368,6 @@ [PcdsFixedAtBuild.common]
> # Shell.
> gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10
>
> # GUID of the UEFI Shell
> diff --git a/Omap35xxPkg/Omap35xxPkg.dsc b/Omap35xxPkg/Omap35xxPkg.dsc
> index ad7d9898c330..fc94ca7ed129 100644
> --- a/Omap35xxPkg/Omap35xxPkg.dsc
> +++ b/Omap35xxPkg/Omap35xxPkg.dsc
> @@ -162,11 +162,6 @@ [PcdsFixedAtBuild.common]
> # OMAP Interrupt Controller
> gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ################################################################################
> #
> # Components Section - list of all EDK II Modules needed by this Platform
>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
2017-02-24 18:40 [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD Ard Biesheuvel
2017-02-24 23:18 ` Laszlo Ersek
@ 2017-02-27 15:44 ` Leif Lindholm
2017-02-27 16:18 ` Ard Biesheuvel
1 sibling, 1 reply; 6+ messages in thread
From: Leif Lindholm @ 2017-02-27 15:44 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel, lersek
On Fri, Feb 24, 2017 at 06:40:25PM +0000, Ard Biesheuvel wrote:
> This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
> any remaining references to it in various platform .DSC files. It is
> no longer used now that we removed the virtual uncached pages protocol
> and the associated DebugUncachedMemoryAllocationLib library instance.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
> ArmPkg/ArmPkg.dec | 1 -
> ArmVirtPkg/ArmVirtQemu.dsc | 5 -----
> ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -----
> BeagleBoardPkg/BeagleBoardPkg.dsc | 5 -----
> Omap35xxPkg/Omap35xxPkg.dsc | 5 -----
> 5 files changed, 21 deletions(-)
>
> diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
> index 4fd7a5be5158..c4b4da2f95bb 100644
> --- a/ArmPkg/ArmPkg.dec
> +++ b/ArmPkg/ArmPkg.dec
> @@ -84,7 +84,6 @@ [PcdsFixedAtBuild.common]
> # Using a FeaturePcd make a '(BOOLEAN) casting for its value which is not understood by the preprocessor.
> gArmTokenSpaceGuid.PcdVFPEnabled|0|UINT32|0x00000024
>
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000080000000|UINT64|0x00000002
> # This PCD will free the unallocated buffers if their size reach this threshold.
> # We set the default value to 512MB.
> gArmTokenSpaceGuid.PcdArmFreeUncachedMemorySizeThreshold|0x20000000|UINT64|0x00000003
> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
> index 8fe3c3816961..00cdd7c20201 100644
> --- a/ArmVirtPkg/ArmVirtQemu.dsc
> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>
> diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
> index aa40374745af..8354b13ddb1a 100644
> --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
> +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>
> diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc
> index a71a01ac7723..87879c33fcd9 100644
> --- a/BeagleBoardPkg/BeagleBoardPkg.dsc
> +++ b/BeagleBoardPkg/BeagleBoardPkg.dsc
> @@ -368,11 +368,6 @@ [PcdsFixedAtBuild.common]
> # Shell.
> gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10
>
> # GUID of the UEFI Shell
> diff --git a/Omap35xxPkg/Omap35xxPkg.dsc b/Omap35xxPkg/Omap35xxPkg.dsc
> index ad7d9898c330..fc94ca7ed129 100644
> --- a/Omap35xxPkg/Omap35xxPkg.dsc
> +++ b/Omap35xxPkg/Omap35xxPkg.dsc
> @@ -162,11 +162,6 @@ [PcdsFixedAtBuild.common]
> # OMAP Interrupt Controller
> gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
> -
> ################################################################################
> #
> # Components Section - list of all EDK II Modules needed by this Platform
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
2017-02-27 15:44 ` Leif Lindholm
@ 2017-02-27 16:18 ` Ard Biesheuvel
2017-02-27 16:25 ` Laszlo Ersek
0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2017-02-27 16:18 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel@lists.01.org, Laszlo Ersek
On 27 February 2017 at 15:44, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Fri, Feb 24, 2017 at 06:40:25PM +0000, Ard Biesheuvel wrote:
>> This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
>> any remaining references to it in various platform .DSC files. It is
>> no longer used now that we removed the virtual uncached pages protocol
>> and the associated DebugUncachedMemoryAllocationLib library instance.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>
Thank you both
I missed this one
diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc
index cc09d38910a2..d40ea69b072d 100644
--- a/ArmVirtPkg/ArmVirt.dsc.inc
+++ b/ArmVirtPkg/ArmVirt.dsc.inc
@@ -360,11 +360,6 @@ [PcdsFixedAtBuild.common]
gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderCode|20
gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderData|0
- #
- # ARM Pcds
- #
- gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000000000000
-
!if $(SECURE_BOOT_ENABLE) == TRUE
# override the default values from SecurityPkg to ensure images
from all sources are verified in secure boot
gEfiSecurityPkgTokenSpaceGuid.PcdOptionRomImageVerificationPolicy|0x04
so I will fold that in, if there are no objections.
Thanks,
Ard.
>> ---
>> ArmPkg/ArmPkg.dec | 1 -
>> ArmVirtPkg/ArmVirtQemu.dsc | 5 -----
>> ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -----
>> BeagleBoardPkg/BeagleBoardPkg.dsc | 5 -----
>> Omap35xxPkg/Omap35xxPkg.dsc | 5 -----
>> 5 files changed, 21 deletions(-)
>>
>> diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
>> index 4fd7a5be5158..c4b4da2f95bb 100644
>> --- a/ArmPkg/ArmPkg.dec
>> +++ b/ArmPkg/ArmPkg.dec
>> @@ -84,7 +84,6 @@ [PcdsFixedAtBuild.common]
>> # Using a FeaturePcd make a '(BOOLEAN) casting for its value which is not understood by the preprocessor.
>> gArmTokenSpaceGuid.PcdVFPEnabled|0|UINT32|0x00000024
>>
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000080000000|UINT64|0x00000002
>> # This PCD will free the unallocated buffers if their size reach this threshold.
>> # We set the default value to 512MB.
>> gArmTokenSpaceGuid.PcdArmFreeUncachedMemorySizeThreshold|0x20000000|UINT64|0x00000003
>> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
>> index 8fe3c3816961..00cdd7c20201 100644
>> --- a/ArmVirtPkg/ArmVirtQemu.dsc
>> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
>> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
>> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
>> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>>
>> - #
>> - # ARM Pcds
>> - #
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>> -
>> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
>> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>>
>> diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
>> index aa40374745af..8354b13ddb1a 100644
>> --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
>> +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
>> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
>> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
>> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>>
>> - #
>> - # ARM Pcds
>> - #
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>> -
>> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
>> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>>
>> diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc
>> index a71a01ac7723..87879c33fcd9 100644
>> --- a/BeagleBoardPkg/BeagleBoardPkg.dsc
>> +++ b/BeagleBoardPkg/BeagleBoardPkg.dsc
>> @@ -368,11 +368,6 @@ [PcdsFixedAtBuild.common]
>> # Shell.
>> gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>>
>> - #
>> - # ARM Pcds
>> - #
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>> -
>> gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10
>>
>> # GUID of the UEFI Shell
>> diff --git a/Omap35xxPkg/Omap35xxPkg.dsc b/Omap35xxPkg/Omap35xxPkg.dsc
>> index ad7d9898c330..fc94ca7ed129 100644
>> --- a/Omap35xxPkg/Omap35xxPkg.dsc
>> +++ b/Omap35xxPkg/Omap35xxPkg.dsc
>> @@ -162,11 +162,6 @@ [PcdsFixedAtBuild.common]
>> # OMAP Interrupt Controller
>> gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000
>>
>> - #
>> - # ARM Pcds
>> - #
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>> -
>> ################################################################################
>> #
>> # Components Section - list of all EDK II Modules needed by this Platform
>> --
>> 2.7.4
>>
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
2017-02-27 16:18 ` Ard Biesheuvel
@ 2017-02-27 16:25 ` Laszlo Ersek
2017-02-27 16:39 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: Laszlo Ersek @ 2017-02-27 16:25 UTC (permalink / raw)
To: Ard Biesheuvel, Leif Lindholm; +Cc: edk2-devel@lists.01.org
On 02/27/17 17:18, Ard Biesheuvel wrote:
> On 27 February 2017 at 15:44, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>> On Fri, Feb 24, 2017 at 06:40:25PM +0000, Ard Biesheuvel wrote:
>>> This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
>>> any remaining references to it in various platform .DSC files. It is
>>> no longer used now that we removed the virtual uncached pages protocol
>>> and the associated DebugUncachedMemoryAllocationLib library instance.
>>>
>>> Contributed-under: TianoCore Contribution Agreement 1.0
>>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>>
>> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>>
>
> Thank you both
>
> I missed this one
>
> diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc
> index cc09d38910a2..d40ea69b072d 100644
> --- a/ArmVirtPkg/ArmVirt.dsc.inc
> +++ b/ArmVirtPkg/ArmVirt.dsc.inc
> @@ -360,11 +360,6 @@ [PcdsFixedAtBuild.common]
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderCode|20
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderData|0
>
> - #
> - # ARM Pcds
> - #
> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000000000000
> -
> !if $(SECURE_BOOT_ENABLE) == TRUE
> # override the default values from SecurityPkg to ensure images
> from all sources are verified in secure boot
> gEfiSecurityPkgTokenSpaceGuid.PcdOptionRomImageVerificationPolicy|0x04
>
> so I will fold that in, if there are no objections.
sure, go ahead
>
> Thanks,
> Ard.
>
>
>
>
>>> ---
>>> ArmPkg/ArmPkg.dec | 1 -
>>> ArmVirtPkg/ArmVirtQemu.dsc | 5 -----
>>> ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -----
>>> BeagleBoardPkg/BeagleBoardPkg.dsc | 5 -----
>>> Omap35xxPkg/Omap35xxPkg.dsc | 5 -----
>>> 5 files changed, 21 deletions(-)
>>>
>>> diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
>>> index 4fd7a5be5158..c4b4da2f95bb 100644
>>> --- a/ArmPkg/ArmPkg.dec
>>> +++ b/ArmPkg/ArmPkg.dec
>>> @@ -84,7 +84,6 @@ [PcdsFixedAtBuild.common]
>>> # Using a FeaturePcd make a '(BOOLEAN) casting for its value which is not understood by the preprocessor.
>>> gArmTokenSpaceGuid.PcdVFPEnabled|0|UINT32|0x00000024
>>>
>>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000080000000|UINT64|0x00000002
>>> # This PCD will free the unallocated buffers if their size reach this threshold.
>>> # We set the default value to 512MB.
>>> gArmTokenSpaceGuid.PcdArmFreeUncachedMemorySizeThreshold|0x20000000|UINT64|0x00000003
>>> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
>>> index 8fe3c3816961..00cdd7c20201 100644
>>> --- a/ArmVirtPkg/ArmVirtQemu.dsc
>>> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
>>> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
>>> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
>>> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>>>
>>> - #
>>> - # ARM Pcds
>>> - #
>>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>>> -
>>> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
>>> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>>>
>>> diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
>>> index aa40374745af..8354b13ddb1a 100644
>>> --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
>>> +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
>>> @@ -113,11 +113,6 @@ [PcdsFixedAtBuild.common]
>>> # Size of the region used by UEFI in permanent memory (Reserved 64MB)
>>> gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
>>>
>>> - #
>>> - # ARM Pcds
>>> - #
>>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>>> -
>>> ## Trustzone enable (to make the transition from EL3 to EL2 in ArmPlatformPkg/Sec)
>>> gArmTokenSpaceGuid.PcdTrustzoneSupport|FALSE
>>>
>>> diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc
>>> index a71a01ac7723..87879c33fcd9 100644
>>> --- a/BeagleBoardPkg/BeagleBoardPkg.dsc
>>> +++ b/BeagleBoardPkg/BeagleBoardPkg.dsc
>>> @@ -368,11 +368,6 @@ [PcdsFixedAtBuild.common]
>>> # Shell.
>>> gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>>>
>>> - #
>>> - # ARM Pcds
>>> - #
>>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>>> -
>>> gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10
>>>
>>> # GUID of the UEFI Shell
>>> diff --git a/Omap35xxPkg/Omap35xxPkg.dsc b/Omap35xxPkg/Omap35xxPkg.dsc
>>> index ad7d9898c330..fc94ca7ed129 100644
>>> --- a/Omap35xxPkg/Omap35xxPkg.dsc
>>> +++ b/Omap35xxPkg/Omap35xxPkg.dsc
>>> @@ -162,11 +162,6 @@ [PcdsFixedAtBuild.common]
>>> # OMAP Interrupt Controller
>>> gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000
>>>
>>> - #
>>> - # ARM Pcds
>>> - #
>>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000040000000
>>> -
>>> ################################################################################
>>> #
>>> # Components Section - list of all EDK II Modules needed by this Platform
>>> --
>>> 2.7.4
>>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD
2017-02-27 16:25 ` Laszlo Ersek
@ 2017-02-27 16:39 ` Ard Biesheuvel
0 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-02-27 16:39 UTC (permalink / raw)
To: Laszlo Ersek; +Cc: Leif Lindholm, edk2-devel@lists.01.org
On 27 February 2017 at 16:25, Laszlo Ersek <lersek@redhat.com> wrote:
> On 02/27/17 17:18, Ard Biesheuvel wrote:
>> On 27 February 2017 at 15:44, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>>> On Fri, Feb 24, 2017 at 06:40:25PM +0000, Ard Biesheuvel wrote:
>>>> This removes the PCD PcdArmUncachedMemoryMask from ArmPkg, along with
>>>> any remaining references to it in various platform .DSC files. It is
>>>> no longer used now that we removed the virtual uncached pages protocol
>>>> and the associated DebugUncachedMemoryAllocationLib library instance.
>>>>
>>>> Contributed-under: TianoCore Contribution Agreement 1.0
>>>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>>>
>>> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>>>
>>
>> Thank you both
>>
>> I missed this one
>>
>> diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc
>> index cc09d38910a2..d40ea69b072d 100644
>> --- a/ArmVirtPkg/ArmVirt.dsc.inc
>> +++ b/ArmVirtPkg/ArmVirt.dsc.inc
>> @@ -360,11 +360,6 @@ [PcdsFixedAtBuild.common]
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderCode|20
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderData|0
>>
>> - #
>> - # ARM Pcds
>> - #
>> - gArmTokenSpaceGuid.PcdArmUncachedMemoryMask|0x0000000000000000
>> -
>> !if $(SECURE_BOOT_ENABLE) == TRUE
>> # override the default values from SecurityPkg to ensure images
>> from all sources are verified in secure boot
>> gEfiSecurityPkgTokenSpaceGuid.PcdOptionRomImageVerificationPolicy|0x04
>>
>> so I will fold that in, if there are no objections.
>
> sure, go ahead
>
Pushed, thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-02-27 16:39 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-24 18:40 [PATCH] ArmPkg: remove unused PcdArmUncachedMemoryMask PCD Ard Biesheuvel
2017-02-24 23:18 ` Laszlo Ersek
2017-02-27 15:44 ` Leif Lindholm
2017-02-27 16:18 ` Ard Biesheuvel
2017-02-27 16:25 ` Laszlo Ersek
2017-02-27 16:39 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox