From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c0c::241; helo=mail-wr0-x241.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr0-x241.google.com (mail-wr0-x241.google.com [IPv6:2a00:1450:400c:c0c::241]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D6A8F220C1C5B for ; Thu, 30 Nov 2017 07:51:26 -0800 (PST) Received: by mail-wr0-x241.google.com with SMTP id o2so7082563wro.5 for ; Thu, 30 Nov 2017 07:55:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=2w+JPTZD+lK/sbimtznOC+A9CDsaff+Qka7SqOp0YI8=; b=hk9Y7+HMc8EurMZbZW4PQ78mqn4J4e89JHVhmocLDJZjXsA9GKALqtm9uahG5Vm6i/ Wk6xc55SUPmVHBXowjx73nXGWinCStAShgLVa3Qzqen1MGkWXn2pQ/m94z/RYOsU5Rjj PDTaqBWlPORjwWDtoOhtpAgmbtmn9APraSmS4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=2w+JPTZD+lK/sbimtznOC+A9CDsaff+Qka7SqOp0YI8=; b=m0OLyzCZ3EhSqs/eclvOwCpzaIi7ICYPyovvua7gw3ncDk6kisHCN2wsAjQjoj2IYr Ujw94q8wyfN7a3NvuGsV8c1Pcxg2WONmEdjAER1YiXv95B2Abfvz8ja9eKHJxTR2v3aM yxSkjPOtkxpSKfxrLC1cabobnRUUA2YWcohmR62FF2TvIJMbuVYKOJWY3YhwO6JbMLdO 9rjiGqe/KeJCiucGJaxT5OWxhGc3amGuEeHnLzIjXQ6N9LaBHoRvCqXodXktMRw4+JA1 Il2kFqjdglV2YqPIJgSpBzJsB/4v1+ZKybqL4B8y+/kXP8SxXUCrPGSyfgccD+qZLOqO bz+g== X-Gm-Message-State: AJaThX6F5ZCEDkJYJ7J+V3rjCFBPBwKLyWv/S2JBg0i7mlJI+h4MBBMF 5nF15JKqR+UNC9vXcB/S4WBraMBLG6E= X-Google-Smtp-Source: AGs4zMYgwGaTChkNU7Vm9Y6wPNwhuYfllJ7dE8J87NpnFfqEnpSXCBylq+qLDlSQflSzKbCYlhnEmA== X-Received: by 10.223.161.158 with SMTP id u30mr2412876wru.96.1512057350688; Thu, 30 Nov 2017 07:55:50 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id w76sm8244029wrc.79.2017.11.30.07.55.49 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 30 Nov 2017 07:55:49 -0800 (PST) Date: Thu, 30 Nov 2017 15:55:47 +0000 From: Leif Lindholm To: Ard Biesheuvel Cc: edk2-devel@lists.01.org Message-ID: <20171130155547.g5aezeo3vgnxu4cg@bivouac.eciton.net> References: <20171130141509.27230-1-ard.biesheuvel@linaro.org> MIME-Version: 1.0 In-Reply-To: <20171130141509.27230-1-ard.biesheuvel@linaro.org> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH 0/7] ArmPlatformPkg cleanup - phase II X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Nov 2017 15:51:27 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline 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 > 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 >