[edk2] [PATCH v3 0/3] ArmPlatformPkg: FVP spring cleaning -- EDK2 edition

Ard Biesheuvel posted 3 patches 7 years, 6 months ago
Failed in applying to current master (apply log)
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c | 113 ----------------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c         |  84 ------------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.c                | 137 +-------------------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.inf              |  42 ------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.c                 |  43 +-----
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.inf               |   3 -
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c        |  48 -------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h      |  99 --------------
ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec                        |  28 ----
9 files changed, 6 insertions(+), 591 deletions(-)
delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c
delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c
delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c
delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h
[edk2] [PATCH v3 0/3] ArmPlatformPkg: FVP spring cleaning -- EDK2 edition
Posted by Ard Biesheuvel 7 years, 6 months ago
This v3 of the ArmPlatformPkg now only contains FVP changes. TC2 and Juno
will receive a similar treatment once we have sorted out how the loading of
the DTB image from the NOR flash will be implemented.

So what remains here is two somewhat unrelated cleanup patches, and the
meat in #3 which removes the functionality that interoperates with the
FdtPlatformDxe that we are trying to get rid of.

Ard Biesheuvel (3):
  ArmPlatformPkg/ArmVExpressDxe: remove ARM support
  ArmPlatformPkg/ArmVExpressDxe: remove unused cruft from ArmHwDxe
  ArmPlatformPkg/ArmVExpressDxe: remove FDT handling from ArmFvpDxe

 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c | 113 ----------------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c         |  84 ------------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.c                | 137 +-------------------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.inf              |  42 ------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.c                 |  43 +-----
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.inf               |   3 -
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c        |  48 -------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h      |  99 --------------
 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec                        |  28 ----
 9 files changed, 6 insertions(+), 591 deletions(-)
 delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c
 delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c
 delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c
 delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h

-- 
2.9.3

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v3 0/3] ArmPlatformPkg: FVP spring cleaning -- EDK2 edition
Posted by Leif Lindholm 7 years, 6 months ago
On Mon, Apr 10, 2017 at 05:21:49PM +0100, Ard Biesheuvel wrote:
> This v3 of the ArmPlatformPkg now only contains FVP changes. TC2 and Juno
> will receive a similar treatment once we have sorted out how the loading of
> the DTB image from the NOR flash will be implemented.
> 
> So what remains here is two somewhat unrelated cleanup patches, and the
> meat in #3 which removes the functionality that interoperates with the
> FdtPlatformDxe that we are trying to get rid of.
> 

For the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

> Ard Biesheuvel (3):
>   ArmPlatformPkg/ArmVExpressDxe: remove ARM support
>   ArmPlatformPkg/ArmVExpressDxe: remove unused cruft from ArmHwDxe
>   ArmPlatformPkg/ArmVExpressDxe: remove FDT handling from ArmFvpDxe
> 
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c | 113 ----------------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c         |  84 ------------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.c                | 137 +-------------------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.inf              |  42 ------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.c                 |  43 +-----
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.inf               |   3 -
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c        |  48 -------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h      |  99 --------------
>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec                        |  28 ----
>  9 files changed, 6 insertions(+), 591 deletions(-)
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h
> 
> -- 
> 2.9.3
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v3 0/3] ArmPlatformPkg: FVP spring cleaning -- EDK2 edition
Posted by Ryan Harkin 7 years, 6 months ago
On 10 April 2017 at 17:33, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Mon, Apr 10, 2017 at 05:21:49PM +0100, Ard Biesheuvel wrote:
>> This v3 of the ArmPlatformPkg now only contains FVP changes. TC2 and Juno
>> will receive a similar treatment once we have sorted out how the loading of
>> the DTB image from the NOR flash will be implemented.
>>
>> So what remains here is two somewhat unrelated cleanup patches, and the
>> meat in #3 which removes the functionality that interoperates with the
>> FdtPlatformDxe that we are trying to get rid of.
>>
>
> For the series:
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>

This deletes some seriously nasty code!

Tested on FVP Foundation & AEMv8, TC2 and Juno R0/1/2.

Tested-by: Ryan Harkin <ryan.harkin@linaro.org>


>> Ard Biesheuvel (3):
>>   ArmPlatformPkg/ArmVExpressDxe: remove ARM support
>>   ArmPlatformPkg/ArmVExpressDxe: remove unused cruft from ArmHwDxe
>>   ArmPlatformPkg/ArmVExpressDxe: remove FDT handling from ArmFvpDxe
>>
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c | 113 ----------------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c         |  84 ------------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.c                | 137 +-------------------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmFvpDxe.inf              |  42 ------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.c                 |  43 +-----
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmHwDxe.inf               |   3 -
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c        |  48 -------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h      |  99 --------------
>>  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec                        |  28 ----
>>  9 files changed, 6 insertions(+), 591 deletions(-)
>>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/AArch64/ArmFvpDxeAArch64.c
>>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/Arm/ArmFvpDxeArm.c
>>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressCommon.c
>>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/ArmVExpressDxe/ArmVExpressInternal.h
>>
>> --
>> 2.9.3
>>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel