From patchwork Thu Apr 6 10:29:33 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ard Biesheuvel X-Patchwork-Id: 96919 Delivered-To: patch@linaro.org Received: by 10.140.89.233 with SMTP id v96csp663671qgd; Thu, 6 Apr 2017 03:30:33 -0700 (PDT) X-Received: by 10.55.69.79 with SMTP id s76mr34389993qka.23.1491474633388; Thu, 06 Apr 2017 03:30:33 -0700 (PDT) Return-Path: Received: from lists.linaro.org (lists.linaro.org. [54.225.227.206]) by mx.google.com with ESMTP id v41si1025871qtc.81.2017.04.06.03.30.32; Thu, 06 Apr 2017 03:30:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linaro-uefi-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) client-ip=54.225.227.206; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linaro-uefi-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) smtp.mailfrom=linaro-uefi-bounces@lists.linaro.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id B11BC6435D; Thu, 6 Apr 2017 10:30:32 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on ip-10-142-244-252 X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, URIBL_BLOCKED autolearn=disabled version=3.4.0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by lists.linaro.org (Postfix) with ESMTP id 4957264336; Thu, 6 Apr 2017 10:30:22 +0000 (UTC) X-Original-To: linaro-uefi@lists.linaro.org Delivered-To: linaro-uefi@lists.linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id 4F6646433A; Thu, 6 Apr 2017 10:29:55 +0000 (UTC) Received: from mail-wr0-f178.google.com (mail-wr0-f178.google.com [209.85.128.178]) by lists.linaro.org (Postfix) with ESMTPS id 76FB564334 for ; Thu, 6 Apr 2017 10:29:53 +0000 (UTC) Received: by mail-wr0-f178.google.com with SMTP id w11so51190204wrc.3 for ; Thu, 06 Apr 2017 03:29:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=EzyRuOXxsGmvbelQ2iZx9a+eApn4Nfj8FqDteHXoZ74=; b=WSRZY+Mwv0n/8eHNJ+Ulc113QLNOoAR13x3CMs1ng2cX06yv3bJ4KRb2yyOUIwrEQ2 VQVRkOuIf5d32MDKidmehEUhdUgjyLDzFLv+rZd91iYAKaU00QpXjPX0BSQoNmayAPuQ fWWlbqrJFS2KTVE0egCn8WQ+noCtmxPzcSo9Quh6wrqCs5HraeQZpksVi8YgE8eFEgh/ 9zJtXVq4Ujl2oQgFJsmVwnsD99C8ikYmHOAWN/7zXMkqjgvzvJoF70Wv/ZSLAC7TPuEF RDjuioZA/CeaVz/ERtwPiPEvqd4pVWY2Bp2vLQM2ZpCeWAYMnOl0c9vAEOjutIUOb5C2 d1oA== X-Gm-Message-State: AFeK/H15OMROcjWf9jPSHtmCqnhYdv6AOlXuvyBYb8H1tW6VT4zjHXpVxsK6KKqOVQfK8zsCqow= X-Received: by 10.28.227.213 with SMTP id a204mr21990437wmh.120.1491474591873; Thu, 06 Apr 2017 03:29:51 -0700 (PDT) Received: from localhost.localdomain ([160.163.145.113]) by smtp.gmail.com with ESMTPSA id s27sm1562822wra.69.2017.04.06.03.29.50 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 06 Apr 2017 03:29:51 -0700 (PDT) From: Ard Biesheuvel To: linaro-uefi@lists.linaro.org Date: Thu, 6 Apr 2017 11:29:33 +0100 Message-Id: <20170406102941.14802-1-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.9.3 Cc: alan@softiron.co.uk Subject: [Linaro-uefi] [PATCH 0/8] Platforms/AMD/Overdrive: switch to DtPlatformDxe X-BeenThere: linaro-uefi@lists.linaro.org X-Mailman-Version: 2.1.16 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: linaro-uefi-bounces@lists.linaro.org Sender: "Linaro-uefi" The FdtDxe implementation that lives in Platforms/AMD/Styx is a rather unhealthy mix of fossilized pieces of LinuxLoader combined with other dependencies on the deprecated ARM BdsLib. Since it does lots of things we don't need (involving memory DT nodes and other bits we don't use), and doesn't do what we do need (select between ACPI and DT but not enable both at the same time), it is time to get rid of it. So implement a DtPlatformDtbLoaderLib based on the pieces we want to keep, and switch Overdrive and Overdrive 1000 to use it, and finally remove FdtDxe altogether. Ard Biesheuvel (8): Platforms/AMD: remove /pmu node from Overdrive 1000 DTS Platforms/AMD: clean up whitespace in Overdrive 1000 DTS file Platforms/AMD: remove /pmu node from Overdrive DTS Platforms/AMD: clean up whitespace in Overdrive DTS file Platforms/AMD: implement DtPlatformDtbLoaderLib Platforms/AMD: switch Overdrive to DtPlatformDxe Platforms/AMD: switch Overdrive 1000 to DtPlatformDxe Platform/AMD: remove FdtDxe driver Platforms/AMD/Styx/Drivers/FdtDxe/BdsLinuxFdt.c | 760 ----------------- Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.c | 274 ------ Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.h | 54 -- Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.inf | 76 -- Platforms/AMD/Styx/Drivers/FdtDxe/LinuxLoader.h | 173 ---- Platforms/AMD/Styx/Drivers/FdtDxe/LinuxLoaderHelper.c | 200 ----- Platforms/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c | 473 +++++++++++ Platforms/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf | 64 ++ Platforms/AMD/Styx/Overdrive1000Board/FdtBlob/styx-overdrive1000.dtb | Bin 8117 -> 7969 bytes Platforms/AMD/Styx/Overdrive1000Board/FdtBlob/styx-overdrive1000.dts | 72 +- Platforms/AMD/Styx/Overdrive1000Board/Overdrive1000Board.dsc | 10 +- Platforms/AMD/Styx/Overdrive1000Board/Overdrive1000Board.fdf | 4 +- Platforms/AMD/Styx/OverdriveBoard/FdtBlob/styx-overdrive.dtb | Bin 8089 -> 7973 bytes Platforms/AMD/Styx/OverdriveBoard/FdtBlob/styx-overdrive.dts | 878 ++++++++++---------- Platforms/AMD/Styx/OverdriveBoard/OverdriveBoard.dsc | 11 +- Platforms/AMD/Styx/OverdriveBoard/OverdriveBoard.fdf | 4 +- 16 files changed, 1016 insertions(+), 2037 deletions(-) delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/BdsLinuxFdt.c delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.c delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.h delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/FdtDxe.inf delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/LinuxLoader.h delete mode 100644 Platforms/AMD/Styx/Drivers/FdtDxe/LinuxLoaderHelper.c create mode 100644 Platforms/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c create mode 100644 Platforms/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf