From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qt0-x234.google.com (mail-qt0-x234.google.com [IPv6:2607:f8b0:400d:c0d::234]) (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 6720021939313 for ; Tue, 4 Apr 2017 08:47:30 -0700 (PDT) Received: by mail-qt0-x234.google.com with SMTP id n21so142397326qta.1 for ; Tue, 04 Apr 2017 08:47:30 -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=pPseMXEaJIDGscjoi6CwmvVUSuTMRUfdZNXIoSECvnE=; b=dBDPDS2w5fUsWWhby3jeIBqOpNmlRkyipXcJYbu9RhGkt8HrLfBlu78oCtNnXyhtZQ OfPWvpU0dbGPLVC6PZD2CiESjynlu/5lYSyWwg505BKq6bAxmUvABK0bYmITFnaJS8P+ ZUqzecmHYWb5jzD7yQrtn4AoS6ZBLtp7KkWGg= 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=pPseMXEaJIDGscjoi6CwmvVUSuTMRUfdZNXIoSECvnE=; b=rorrpWYrN/gy06Z6b75Vyk5zLRgdPHcgOESdpoVWpB/1WtMoTwLj8anLp6kcoCqzq8 +jddo8AblLY8dsCsnFzhIM/scxxQASVrR/TpMW+RaQVQE2at59i4KTXp8U8AJxh1d/a9 9y/bo/FBJK54/nXyR9f15ZujTDjE3fH79F96HP119k1Y7uPIF+ufsma4lay0OpfTmsFf dgqDYQmSunvmzLjwun2kvgGshlKSCTnC3OGKWXROKDpODcW3SBi8L3YW0uFquNzW7Hcu SSIeMsoN+ov0EIFqcPEHZ4jrvS4XiyVQa7CTIEkE2orIhOaSGx09tAgDLBYkYKt+krFU vtUA== X-Gm-Message-State: AFeK/H3uRXAhnkocEU0F4RUJJpzvKIcY8QX53cuOSkVsDEPY+7I/P+ROoEfQhQQRoFnpc5rHQH40vGSMAlQv9VHb X-Received: by 10.237.33.143 with SMTP id l15mr23947300qtc.222.1491320849258; Tue, 04 Apr 2017 08:47:29 -0700 (PDT) MIME-Version: 1.0 Received: by 10.140.36.138 with HTTP; Tue, 4 Apr 2017 08:47:28 -0700 (PDT) In-Reply-To: <20170404123010.11722-1-ard.biesheuvel@linaro.org> References: <20170404123010.11722-1-ard.biesheuvel@linaro.org> From: Ryan Harkin Date: Tue, 4 Apr 2017 16:47:28 +0100 Message-ID: To: Ard Biesheuvel Cc: "edk2-devel@lists.01.org" , Leif Lindholm Subject: Re: [PATCH v3 0/6] Juno cleanup -- 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, 04 Apr 2017 15:47:30 -0000 Content-Type: text/plain; charset=UTF-8 On 4 April 2017 at 13:30, Ard Biesheuvel wrote: > This is tagged as v3 because these patches have been isolated from a > ArmPlatformPkg cleanup series that is mostly about ACPI vs DT. > > This series is specific to Juno; it replaces the cargo culted and ancient > PCI 'emulation' code with calls into the new non-discoverable device API, > and removes the Juno specific PCI host bridge driver in favor of the generic > one. > > Ard Biesheuvel (6): > ArmPlatformPkg/ArmShellCmdRunAxf: remove BdsLib dependency > ArmPlatformPkg/ArmJunoDxe: remove BdsLib dependency > ArmPlatformPkg/ArmJunoDxe: use the generic non-discoverable device > support > ArmPlatformPkg/ArmJunoDxe: don't register OnEndOfDxe event on rev R0 > ArmPlatformPkg/ArmJunoPkg: remove PCI host bridge driver > ArmPlatformPkg/ArmJunoDxe: simplify ACPI table installation > > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 110 ++- > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf | 4 +- > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxeInternal.h | 5 - > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciEmulation.c | 596 ---------------- > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciEmulation.h | 284 -------- > ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciRootBridgeIo.c | 299 -------- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridge.c | 199 ------ > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridge.h | 324 --------- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridgeDxe.inf | 76 -- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridgeResourceAllocation.c | 642 ----------------- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciRootBridge.c | 748 -------------------- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/XPressRich3.c | 170 ----- > ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/XPressRich3.h | 111 --- > ArmPlatformPkg/Library/ArmShellCmdRunAxf/ArmShellCmdRunAxf.inf | 1 - > ArmPlatformPkg/Library/ArmShellCmdRunAxf/RunAxf.c | 58 +- > 15 files changed, 104 insertions(+), 3523 deletions(-) > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciEmulation.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciEmulation.h > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/PciRootBridgeIo.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridge.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridge.h > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridgeDxe.inf > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciHostBridgeResourceAllocation.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/PciRootBridge.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/XPressRich3.c > delete mode 100644 ArmPlatformPkg/ArmJunoPkg/Drivers/PciHostBridgeDxe/XPressRich3.h > > -- > 2.9.3 > I tested the whole series in conjunction with the OpenPlatformPkg series and it works fine on Juno R0/1/2, TC2 and FVP Foundation and AEMv8 models. I tested PCIe ethernet and SATA HDD on R1 and R2, and a USB memory stick on R0/1/2.