mbox series

[edk2,0/3] Arm*Pkg : remove ArmCpuLib

Message ID 20170331085302.5450-1-ard.biesheuvel@linaro.org
Headers show
Series Arm*Pkg : remove ArmCpuLib | expand

Message

Ard Biesheuvel March 31, 2017, 8:52 a.m. UTC
As pointed out by Michael, ArmCpuLib is not used, and doesn't contain
anything particularly interesting or appropriate for doing in non-secure
EL2/1. So let's remove it before someone decides to use it.

Ard Biesheuvel (3):
  ArmVirtPkg: remove ArmCpuLib references
  BeagleBoardPkg: remove ArmCpuLib reference
  ArmPkg: remove ArmCpuLib header and implementations

 ArmPkg/ArmPkg.dsc                                                     |  7 --
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c            | 67 ----------------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf          | 35 --------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S | 27 -------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c            | 84 --------------------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf          | 38 ---------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c              | 41 ----------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf            | 27 -------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S           | 23 ------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm         | 29 -------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c              | 76 ------------------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf            | 37 ---------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c        | 39 ---------
 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf      | 34 --------
 ArmPkg/Include/Library/ArmCpuLib.h                                    | 28 -------
 ArmVirtPkg/ArmVirtQemu.dsc                                            |  6 --
 ArmVirtPkg/ArmVirtQemuKernel.dsc                                      |  6 --
 ArmVirtPkg/ArmVirtXen.dsc                                             |  7 --
 BeagleBoardPkg/BeagleBoardPkg.dsc                                     |  1 -
 19 files changed, 612 deletions(-)
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c
 delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf
 delete mode 100644 ArmPkg/Include/Library/ArmCpuLib.h

-- 
2.9.3

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Comments

Laszlo Ersek March 31, 2017, 9:01 a.m. UTC | #1
On 03/31/17 10:52, Ard Biesheuvel wrote:
> As pointed out by Michael, ArmCpuLib is not used, and doesn't contain

> anything particularly interesting or appropriate for doing in non-secure

> EL2/1. So let's remove it before someone decides to use it.

> 

> Ard Biesheuvel (3):

>   ArmVirtPkg: remove ArmCpuLib references

>   BeagleBoardPkg: remove ArmCpuLib reference

>   ArmPkg: remove ArmCpuLib header and implementations

> 

>  ArmPkg/ArmPkg.dsc                                                     |  7 --

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c            | 67 ----------------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf          | 35 --------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S | 27 -------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c            | 84 --------------------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf          | 38 ---------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c              | 41 ----------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf            | 27 -------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S           | 23 ------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm         | 29 -------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c              | 76 ------------------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf            | 37 ---------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c        | 39 ---------

>  ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf      | 34 --------

>  ArmPkg/Include/Library/ArmCpuLib.h                                    | 28 -------

>  ArmVirtPkg/ArmVirtQemu.dsc                                            |  6 --

>  ArmVirtPkg/ArmVirtQemuKernel.dsc                                      |  6 --

>  ArmVirtPkg/ArmVirtXen.dsc                                             |  7 --

>  BeagleBoardPkg/BeagleBoardPkg.dsc                                     |  1 -

>  19 files changed, 612 deletions(-)

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c

>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf

>  delete mode 100644 ArmPkg/Include/Library/ArmCpuLib.h

> 


series
Reviewed-by: Laszlo Ersek <lersek@redhat.com>

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel March 31, 2017, 9:44 a.m. UTC | #2
On 31 March 2017 at 10:01, Laszlo Ersek <lersek@redhat.com> wrote:
> On 03/31/17 10:52, Ard Biesheuvel wrote:

>> As pointed out by Michael, ArmCpuLib is not used, and doesn't contain

>> anything particularly interesting or appropriate for doing in non-secure

>> EL2/1. So let's remove it before someone decides to use it.

>>

>> Ard Biesheuvel (3):

>>   ArmVirtPkg: remove ArmCpuLib references

>>   BeagleBoardPkg: remove ArmCpuLib reference

>>   ArmPkg: remove ArmCpuLib header and implementations

>>

>>  ArmPkg/ArmPkg.dsc                                                     |  7 --

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c            | 67 ----------------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf          | 35 --------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S | 27 -------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c            | 84 --------------------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf          | 38 ---------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c              | 41 ----------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf            | 27 -------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S           | 23 ------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm         | 29 -------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c              | 76 ------------------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf            | 37 ---------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c        | 39 ---------

>>  ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf      | 34 --------

>>  ArmPkg/Include/Library/ArmCpuLib.h                                    | 28 -------

>>  ArmVirtPkg/ArmVirtQemu.dsc                                            |  6 --

>>  ArmVirtPkg/ArmVirtQemuKernel.dsc                                      |  6 --

>>  ArmVirtPkg/ArmVirtXen.dsc                                             |  7 --

>>  BeagleBoardPkg/BeagleBoardPkg.dsc                                     |  1 -

>>  19 files changed, 612 deletions(-)

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.c

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA15Lib/ArmCortexA15Lib.inf

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/AArch64/ArmCortexA5xHelper.S

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.c

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA5xLib/ArmCortexA5xLib.inf

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.c

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA8Lib/ArmCortexA8Lib.inf

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.S

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Helper.asm

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.c

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexA9Lib/ArmCortexA9Lib.inf

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.c

>>  delete mode 100644 ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf

>>  delete mode 100644 ArmPkg/Include/Library/ArmCpuLib.h

>>

>

> series

> Reviewed-by: Laszlo Ersek <lersek@redhat.com>


1/3 and 2/3 pushed as

28a5191cd142 ArmVirtPkg: remove ArmCpuLib references
e6c9036b2c4b BeagleBoardPkg: remove ArmCpuLib reference

I will merge 3/3 once the OpenPlatformPkg patch removing the remaining
reference is merged

Thanks,
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel