From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-x235.google.com (mail-qk0-x235.google.com [IPv6:2607:f8b0:400d:c09::235]) (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 6DDAF20D76483 for ; Tue, 11 Apr 2017 10:04:25 -0700 (PDT) Received: by mail-qk0-x235.google.com with SMTP id p68so2989689qke.1 for ; Tue, 11 Apr 2017 10:04:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=a2c63IVYSweOj4pAJiuZNbw+C5oh0p3dX37I8jw+pQg=; b=U44g6g7v2UGX4vJ+/4oEB6EJQU3jMeGMH2AcBpYHTh7YrvygvwIy5E+emqQNjG5nYE Z1BC8DzDZGNirVKd7dKlcvYVkAD40GjpqIsRrJXmOH5XPtktYNP2hKTFEURrAzc4RFLT voQEdp2aQKVyIghZz86DJgET5iZeNo0NJWET0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=a2c63IVYSweOj4pAJiuZNbw+C5oh0p3dX37I8jw+pQg=; b=GNo2kH6X0Kwqn1jsnr56r1yUc7z0yCqemkChfTwgVJxjsvB8CEnCRcySNlEIqjKqYc Y6ucqB7kk+Tez/7U12lk2X33norNKJ3uzPUf/xwp/UJz0eDvhSZKIxhXoCxI/BkFN7Mn lcltPCOyajPuTAqKTrZrQUQEyDvYhOM5mY47clq79jQOxwO0U4dyfavCq3TO/BLfQbe5 uZaaF1W+z28SFKjmpSQZjZ2XugGWbwy4S1rm9NhXaCC4FmVeZ3sATGSMA16YEK2ZdPFs Apnx0mPs1STdrXYh6N4hJXzRzgAEKjAYJ8p5lrIR4fZjW8EV1frajoEOSKKr5jTComum nkIA== X-Gm-Message-State: AN3rC/6BWjRxLV4XDlCmgSwijv/N+HyoCfevrmf4C59vCcHP+FQXIf41+G91sWmrQCfh3WLuBsGt4aC+kqF51q15 X-Received: by 10.55.165.4 with SMTP id o4mr43629353qke.95.1491930264586; Tue, 11 Apr 2017 10:04:24 -0700 (PDT) MIME-Version: 1.0 Received: by 10.140.36.138 with HTTP; Tue, 11 Apr 2017 10:04:24 -0700 (PDT) In-Reply-To: <20170410163321.GQ1657@bivouac.eciton.net> References: <20170410162152.20555-1-ard.biesheuvel@linaro.org> <20170410163321.GQ1657@bivouac.eciton.net> From: Ryan Harkin Date: Tue, 11 Apr 2017 18:04:24 +0100 Message-ID: To: Leif Lindholm Cc: Ard Biesheuvel , "edk2-devel@lists.01.org" Subject: Re: [PATCH v3 0/3] ArmPlatformPkg: FVP spring cleaning -- EDK2 edition 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: Tue, 11 Apr 2017 17:04:25 -0000 Content-Type: text/plain; charset=UTF-8 On 10 April 2017 at 17:33, Leif Lindholm 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 > This deletes some seriously nasty code! Tested on FVP Foundation & AEMv8, TC2 and Juno R0/1/2. Tested-by: Ryan Harkin >> 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 >>