public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Leif Lindholm <leif.lindholm@linaro.org>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: edk2-devel@lists.01.org
Subject: Re: [PATCH 0/7] ArmPlatformPkg cleanup - phase II
Date: Thu, 30 Nov 2017 15:55:47 +0000	[thread overview]
Message-ID: <20171130155547.g5aezeo3vgnxu4cg@bivouac.eciton.net> (raw)
In-Reply-To: <20171130141509.27230-1-ard.biesheuvel@linaro.org>

On Thu, Nov 30, 2017 at 02:15:02PM +0000, Ard Biesheuvel wrote:
> Proceed with removing junk from ArmPlatformPkg, and tidying up the bits
> we want to keep.
> 
> While we make up our minds how to proceed with the migration of
> ArmVExpressPkg, and the dependent BootMonFs and ShellCmdRunAxf modules,
> this series removes the outdated documentation and CTA9x4 code, moves
> some header files out of Include/Drivers, and removes some unused stuff
> from ArmPlatformLib.

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

> Ard Biesheuvel (7):
>   ArmPlatformPkg/Documentation: remove outdated porting manual
>   ArmPlatformPkg: remove unused ArmVExpressLibCTA9x4 code
>   ArmPlatformPkg/ArmPlatformLib: remove
>     ARM_SYSTEM_MEMORY_REGION_DESCRIPTOR
>   ArmPlatformPkg: remove ArmPlatformInitializeSystemMemory
>   ArmPlatformPkg: move internal PL061 header into driver directory
>   ArmPlatformPkg: move internal SP805 header into driver directory
>   ArmPlatformPkg: move internal PL031 header into driver directory
> 
>  ArmPlatformPkg/ArmPlatformPkg.dec                                                      |   1 -
>  ArmPlatformPkg/ArmVExpressPkg/Include/Platform/CTA9x4/ArmPlatform.h                    | 121 ------------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA15-A7/CTA15-A7.c                |  13 --
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/ArmVExpressLib.inf          |  57 ------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/ArmVExpressLibSec.inf       |  54 ------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4.c                    | 198 --------------------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Helper.S              |  49 -----
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Helper.asm            |  63 -------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Mem.c                 | 119 ------------
>  ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibRTSM/RTSM.c                        |  14 --
>  ArmPlatformPkg/Documentation/ARM-EDK2-Overview.png                                     | Bin 54500 -> 0 bytes
>  ArmPlatformPkg/Documentation/ArmPlatformLib-2nd-Stage.png                              | Bin 56506 -> 0 bytes
>  ArmPlatformPkg/Documentation/ArmPlatformLib-Full-Boot.png                              | Bin 79676 -> 0 bytes
>  ArmPlatformPkg/Documentation/ArmPlatformPkg.txt                                        |  74 --------
>  ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c                                        |   3 +-
>  ArmPlatformPkg/{Include/Drivers => Drivers/PL061GpioDxe}/PL061Gpio.h                   |   0
>  ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805Watchdog.c                                |   3 +-
>  ArmPlatformPkg/{Include/Drivers => Drivers/SP805WatchdogDxe}/SP805Watchdog.h           |   0
>  ArmPlatformPkg/Include/Library/ArmPlatformLib.h                                        |  24 ---
>  ArmPlatformPkg/Library/ArmPlatformLibNull/ArmPlatformLibNull.c                         |  14 --
>  ArmPlatformPkg/{Include/Drivers => Library/PL031RealTimeClockLib}/PL031RealTimeClock.h |   0
>  ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c                   |  19 +-
>  ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.c                                          |   8 -
>  ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.inf                                        |   1 -
>  BeagleBoardPkg/Library/BeagleBoardLib/BeagleBoard.c                                    |  14 --
>  25 files changed, 13 insertions(+), 836 deletions(-)
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Include/Platform/CTA9x4/ArmPlatform.h
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/ArmVExpressLib.inf
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/ArmVExpressLibSec.inf
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4.c
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Helper.S
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Helper.asm
>  delete mode 100644 ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA9x4/CTA9x4Mem.c
>  delete mode 100644 ArmPlatformPkg/Documentation/ARM-EDK2-Overview.png
>  delete mode 100644 ArmPlatformPkg/Documentation/ArmPlatformLib-2nd-Stage.png
>  delete mode 100644 ArmPlatformPkg/Documentation/ArmPlatformLib-Full-Boot.png
>  delete mode 100644 ArmPlatformPkg/Documentation/ArmPlatformPkg.txt
>  rename ArmPlatformPkg/{Include/Drivers => Drivers/PL061GpioDxe}/PL061Gpio.h (100%)
>  rename ArmPlatformPkg/{Include/Drivers => Drivers/SP805WatchdogDxe}/SP805Watchdog.h (100%)
>  rename ArmPlatformPkg/{Include/Drivers => Library/PL031RealTimeClockLib}/PL031RealTimeClock.h (100%)
> 
> -- 
> 2.11.0
> 


  parent reply	other threads:[~2017-11-30 15:51 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-30 14:15 [PATCH 0/7] ArmPlatformPkg cleanup - phase II Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 1/7] ArmPlatformPkg/Documentation: remove outdated porting manual Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 2/7] ArmPlatformPkg: remove unused ArmVExpressLibCTA9x4 code Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 3/7] ArmPlatformPkg/ArmPlatformLib: remove ARM_SYSTEM_MEMORY_REGION_DESCRIPTOR Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 4/7] ArmPlatformPkg: remove ArmPlatformInitializeSystemMemory Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 5/7] ArmPlatformPkg: move internal PL061 header into driver directory Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 6/7] ArmPlatformPkg: move internal SP805 " Ard Biesheuvel
2017-11-30 14:15 ` [PATCH 7/7] ArmPlatformPkg: move internal PL031 " Ard Biesheuvel
2017-11-30 15:55 ` Leif Lindholm [this message]
2017-11-30 16:10   ` [PATCH 0/7] ArmPlatformPkg cleanup - phase II Ard Biesheuvel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171130155547.g5aezeo3vgnxu4cg@bivouac.eciton.net \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox