From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4001:c06::242; helo=mail-io0-x242.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-io0-x242.google.com (mail-io0-x242.google.com [IPv6:2607:f8b0:4001:c06::242]) (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 DF31420359E83 for ; Sun, 26 Nov 2017 03:00:01 -0800 (PST) Received: by mail-io0-x242.google.com with SMTP id 79so25850725ioi.3 for ; Sun, 26 Nov 2017 03:04:22 -0800 (PST) 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=cZ06/k2RhtRS9Tva+cziTLvffl3V/qD+f0LBYHWdLFo=; b=GZVWmNLc38f8V0hdcQWXBtRs9SSxhaz1R0l6gs409SG9pBwLI2H6+ngDI6h9mb+0WS u72iOJrNSY9JuOiH+LpuMJ9/SIZ4koE+i51ePkJnAHACWMzIAnFrtmjrDDgRYSa+HUdU sPe1oSgNx72ceq6ATGLenQqCTgWM+Io4Kzlxc= 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=cZ06/k2RhtRS9Tva+cziTLvffl3V/qD+f0LBYHWdLFo=; b=gE1CWjMEMiQYzbPU3Q1s8/UvHr1J5E3AsDN5SgiLNs+fUzFOH3QMV9SvsIU7zWGDKD TMuuSVfbWaJbOomLOafaV42vmQHtt76R4/52LIX04MofzuOoMDmiE1t2R/Shl3IttPzF poHuVkTjOug34OIZJN+maacxcerCTLL0Eh+UNfMJ+poErjR+t50SKbvlNrstjNr+15ot 5z7Y9YhwL2XJcF+BGMAnygH+sgJTHj0z62xdLHa1R8IyGf/n5NE0CGpuIDQ7RHys21kx 8vPvkqNymGCnF7w1WXOg9il1gyqQoX4aRdFYu5YniBoyTuskSK2Siw9adAQW1I1qA44H /zQg== X-Gm-Message-State: AJaThX5WyiEW/Vpq1Lmh8VFBCgp9pjZg4PsHwR5q5BjfMyNq2HeyjbR7 2J26pzrgMkisjfZ31RMqlZWuzpi3Z+mWjikCh4drmQ== X-Google-Smtp-Source: AGs4zMZERx30Ep7xgmmgRR9QjuYXeD09kzkbRe15h8Rmf+w0d3MxA0f+VjbQX/NPHlRlDrourw5rA/RY59JJXU5EarA= X-Received: by 10.107.151.142 with SMTP id z136mr1093947iod.248.1511694261538; Sun, 26 Nov 2017 03:04:21 -0800 (PST) MIME-Version: 1.0 Received: by 10.107.104.16 with HTTP; Sun, 26 Nov 2017 03:04:21 -0800 (PST) In-Reply-To: <20171125114739.bchqwtp6eygghoqh@bivouac.eciton.net> References: <20171115165637.31118-1-ard.biesheuvel@linaro.org> <20171125114739.bchqwtp6eygghoqh@bivouac.eciton.net> From: Ard Biesheuvel Date: Sun, 26 Nov 2017 11:04:21 +0000 Message-ID: To: Leif Lindholm Cc: "edk2-devel@lists.01.org" Subject: Re: [PATCH 00/19] ArmPlatformPkg: remove unused or migrated modules 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: Sun, 26 Nov 2017 11:00:02 -0000 Content-Type: text/plain; charset="UTF-8" On 25 November 2017 at 11:47, Leif Lindholm wrote: > On Wed, Nov 15, 2017 at 04:56:18PM +0000, Ard Biesheuvel wrote: >> Remove everything from ArmPlatformPkg that is either unused, or so highly >> specific to development platforms manufactured by ARM Ltd. that they really >> don't belong in the main branch. >> >> Note that the migration involves patches against edk2-platforms that add >> some of this code back under Platform/ARM. Most of it (but not all of it) >> has been posted as a separate series already. >> >> Ard Biesheuvel (19): >> ArmPkg: remove unused ArmGicSecLib library implementation >> ArmPlatformPkg: remove ArmPlatformSecLib definition and >> implementations >> ArmPlatformPkg: remove unused DebugSecExtraActionLib library >> ArmPlatformPkg: remove ArmTrustedMonitorLib definition and >> implementation >> ArmPlatformPkg: remove ArmGetCpuCountPerCluster () from ArmPlatformLib >> ArmPlatformPkg: remove NorFlashArmVExpressLib >> ArmPlatformPkg: remove ArmJunoPkg >> ArmPlatformPkg/PL031RealTimeClockLib: drop ArmPlatformSysConfigLib >> reference >> ArmPlatformPkg: remove ArmVExpressPkg > > Please hold off on this one (remove ArmVExpressPkg) until we hear back > from Evan. > Sure. Note that this affects a couple of other patches as well, please see below for what I was able to merge. >> ArmPlatformPkg: remove ArmPlatformSysConfigLib library class > > No separate comment on this one, just to break it away from the below. > >> ArmPlatformPkg: remove unused PL301Axi driver >> ArmPlatformPkg: remove unused PL35x driver >> ArmPlatformPkg: remove PL34xDmc driver >> ArmPlatformPkg: remove unused PL310 driver >> ArmPlatformPkg: remove unused ArmTrustZone driver >> ArmPlatformPkg: remove unused SP804 driver and TimerLib >> implementation > > Please separate the "delete used drivers" from the rest of this set. > Let's give potential out-of-tree users a few weeks to respond. > > My last working day before sabbatical is 15 Dec, so how about > Reviewed-by-if-no-comments-by-15-Dec: Leif Lindholm > ? > >> ArmPlatformPkg: remove unused L2X0CacheLibNull library >> ArmPlatformPkg: remove EblCmdLib implementation >> ArmPlatformPkg: remove BootMonFs and ArmShellCmdRunAxf > > For the remainder: > Reviewed-by: Leif Lindholm > Thanks. Pushed the following 142fa386eb90 ArmPkg: remove unused ArmGicSecLib library implementation 37fcab3e106f ArmPlatformPkg: remove ArmPlatformSecLib definition and implementations 7c0e497d6ea9 ArmPlatformPkg: remove unused DebugSecExtraActionLib library 0abce0329edf ArmPlatformPkg: remove ArmTrustedMonitorLib definition and implementation 4cebe0453fea ArmPlatformPkg: remove ArmGetCpuCountPerCluster () from ArmPlatformLib 4d472fb8ae15 ArmPlatformPkg: remove ArmJunoPkg 0d36a219c7bd ArmPlatformPkg/PL031RealTimeClockLib: drop ArmPlatformSysConfigLib reference 05129444f67e ArmPlatformPkg: remove unused L2X0CacheLibNull library d780544d3d1c ArmPlatformPkg: remove EblCmdLib implementation Once we've decided how to proceed, I will rebase and send out the remaining patches.