On Wed, Oct 19, 2016 at 02:24:34PM +0100, Sudeep Holla wrote:
> Interestingly the build used to succeed even with missing leading '0x'
> prefix for a hex value before the commit 0613ccbd1357
> ("CorebootPayloadPkg/PciHostBridgeLib: Fix the wrong PCI resource limit")
> 
> With the above commit, luckily we get
>   ...
>   raise BadExpression(ERR_ARRAY_ELE % self._Token)
>   BadExpression: This must be HEX value for NList or Array: [e6].
> 
> This patch fixes the above error by adding the missing '0x' prefix to
> the hex value.
> 
> Cc: Yonghong Zhu <yonghong....@intel.com>
> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>
> Cc: Leif Lindholm <leif.lindh...@linaro.org>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com>

Reviewed-by: Leif Lindholm <leif.lindh...@linaro.org>
Pushed - many thanks!

> ---
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec 
> b/ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec
> index d375ed3b9772..c774d97541e1 100644
> --- a/ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec
> +++ b/ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec
> @@ -67,7 +67,7 @@
>    gArmVExpressTokenSpaceGuid.PcdFdtVExpressFvpA15x4|{ 0x72, 0x3b, 0x28, 
> 0x27, 0x90, 0x2f, 0xca, 0x4d, 0x9a, 0xb5, 0x98, 0x48, 0xfb, 0xc2, 0xd4, 0xed 
> }|VOID*|0x0000000A
>    # HW platforms
>    gArmVExpressTokenSpaceGuid.PcdFdtVExpressHwA9x4|{ 0xf6, 0x1c, 0xd2, 0x2f, 
> 0xe8, 0xe6, 0xf2, 0x4f, 0xa9, 0xca, 0x3b, 0x9f, 0x00, 0xe9, 0x28, 0x89 
> }|VOID*|0x0000000B
> -  gArmVExpressTokenSpaceGuid.PcdFdtVExpressHwA15x2A7x3|{ 0xeb, 0x06, e6, 
> 0xd5, 0xdf, 0x83, 0x90, 0x4e, 0x81, 0xe8, 0xc3, 0xdb, 0x2f, 0x77, 0x17, 0x9a 
> }|VOID*|0x0000000C
> +  gArmVExpressTokenSpaceGuid.PcdFdtVExpressHwA15x2A7x3|{ 0xeb, 0x06, 0xe6, 
> 0xd5, 0xdf, 0x83, 0x90, 0x4e, 0x81, 0xe8, 0xc3, 0xdb, 0x2f, 0x77, 0x17, 0x9a 
> }|VOID*|0x0000000C
>    gArmVExpressTokenSpaceGuid.PcdFdtVExpressHwA15|{ 0xc2, 0x47, 0x89, 0x6b, 
> 0x87, 0x42, 0x91, 0x4d, 0x8f, 0xe0, 0xa3, 0x81, 0xea, 0x5b, 0x56, 0x8f 
> }|VOID*|0x0000000D
>    gArmVExpressTokenSpaceGuid.PcdFdtVExpressHwA5|{ 0x63, 0x76, 0xcc, 0xa2, 
> 0x7c, 0x4d, 0x8a, 0x44, 0xaa, 0xb5, 0x4c, 0x03, 0x4b, 0x6f, 0xda, 0xb7 
> }|VOID*|0x0000000E
>  
> -- 
> 2.7.4
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to