From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 8E8E31A1E57 for ; Tue, 25 Oct 2016 10:16:09 -0700 (PDT) Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 0C6D1332F99; Tue, 25 Oct 2016 17:16:09 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-71.phx2.redhat.com [10.3.116.71]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u9PHG78P024995; Tue, 25 Oct 2016 13:16:08 -0400 To: Ard Biesheuvel , edk2-devel@ml01.01.org, leif.lindholm@linaro.org References: <1477413440-680-1-git-send-email-ard.biesheuvel@linaro.org> From: Laszlo Ersek Message-ID: <6920fe25-3549-eab3-9e9e-01ddf9180706@redhat.com> Date: Tue, 25 Oct 2016 19:16:07 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <1477413440-680-1-git-send-email-ard.biesheuvel@linaro.org> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Tue, 25 Oct 2016 17:16:09 +0000 (UTC) Subject: Re: [PATCH 0/3] BeagleBoardPkg: eliminate calls to deprecated functions X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 25 Oct 2016 17:16:09 -0000 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit On 10/25/16 18:37, Ard Biesheuvel wrote: > Get rid of unused modules and switch to the generic BDS, so we are only > left with code that does not use any deprecated functionality, allowing > us to define DISABLE_NEW_DEPRECATED_INTERFACES > > Ard Biesheuvel (3): > BeagleBoardPkg: remove unused modules > BeagleBoardPkg: move to generic BDS > BeagleBoardPkg: enable -DDISABLE_NEW_DEPRECATED_INTERFACES > > BeagleBoardPkg/Bds/Bds.inf | 65 ----- > BeagleBoardPkg/Bds/BdsEntry.c | 242 ---------------- > BeagleBoardPkg/Bds/BdsEntry.h | 66 ----- > BeagleBoardPkg/Bds/FirmwareVolume.c | 150 ---------- > BeagleBoardPkg/BeagleBoardPkg.dsc | 64 ++++- > BeagleBoardPkg/BeagleBoardPkg.fdf | 7 +- > BeagleBoardPkg/Library/EblCmdLib/EblCmdLib.c | 299 -------------------- > BeagleBoardPkg/Library/EblCmdLib/EblCmdLib.inf | 53 ---- > BeagleBoardPkg/Library/GdbSerialLib/GdbSerialLib.c | 103 ------- > BeagleBoardPkg/Library/GdbSerialLib/GdbSerialLib.inf | 41 --- > BeagleBoardPkg/Sec/Arm/ModuleEntryPoint.S | 85 ------ > BeagleBoardPkg/Sec/Arm/ModuleEntryPoint.asm | 89 ------ > BeagleBoardPkg/Sec/Cache.c | 79 ------ > BeagleBoardPkg/Sec/Clock.c | 70 ----- > BeagleBoardPkg/Sec/LzmaDecompress.h | 103 ------- > BeagleBoardPkg/Sec/PadConfiguration.c | 282 ------------------ > BeagleBoardPkg/Sec/Sec.c | 186 ------------ > BeagleBoardPkg/Sec/Sec.inf | 73 ----- > 18 files changed, 58 insertions(+), 1999 deletions(-) > delete mode 100644 BeagleBoardPkg/Bds/Bds.inf > delete mode 100644 BeagleBoardPkg/Bds/BdsEntry.c > delete mode 100644 BeagleBoardPkg/Bds/BdsEntry.h > delete mode 100644 BeagleBoardPkg/Bds/FirmwareVolume.c > delete mode 100644 BeagleBoardPkg/Library/EblCmdLib/EblCmdLib.c > delete mode 100644 BeagleBoardPkg/Library/EblCmdLib/EblCmdLib.inf > delete mode 100644 BeagleBoardPkg/Library/GdbSerialLib/GdbSerialLib.c > delete mode 100644 BeagleBoardPkg/Library/GdbSerialLib/GdbSerialLib.inf > delete mode 100644 BeagleBoardPkg/Sec/Arm/ModuleEntryPoint.S > delete mode 100644 BeagleBoardPkg/Sec/Arm/ModuleEntryPoint.asm > delete mode 100644 BeagleBoardPkg/Sec/Cache.c > delete mode 100644 BeagleBoardPkg/Sec/Clock.c > delete mode 100644 BeagleBoardPkg/Sec/LzmaDecompress.h > delete mode 100644 BeagleBoardPkg/Sec/PadConfiguration.c > delete mode 100644 BeagleBoardPkg/Sec/Sec.c > delete mode 100644 BeagleBoardPkg/Sec/Sec.inf > I think I'll leave this to Leif -- I'm happy to review C code changes related to DISABLE_NEW_DEPRECATED_INTERFACES, but this series solves that question for BeagleBoardPkg by deleting a bunch of modules, and by moving over to some others. :) Thanks Laszlo