From patchwork Thu Aug 24 16:21:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Leif Lindholm X-Patchwork-Id: 110935 Delivered-To: patch@linaro.org Received: by 10.37.128.210 with SMTP id c18csp8716148ybm; Thu, 24 Aug 2017 09:21:58 -0700 (PDT) X-Received: by 10.84.231.198 with SMTP id g6mr7516622pln.36.1503591718854; Thu, 24 Aug 2017 09:21:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1503591718; cv=none; d=google.com; s=arc-20160816; b=QVUNotp967crRkwuD2jRK/b+4KlGy7v0Qf9Bb1TeoFrVgFHXzmF0dDUd6d7bAPzcnc fWFcreSlJQ4OrHrGdT+RjWptPkKcTKNrziHe8U4IsVpw6JoSqq56x53F9jzk2iw0gwPJ CzkLWahSxOKil9prAezBv1XGDs6Ff4Az+gm6guwiG+QMIlKUUHHBxOBPEhGq3XcELKQp 2psKFj6t7/eIyKD3HA4A6MJH+fshLZOdhMrOTkg2cj+zGVdgMarQZPQ/zzR2YyLi23OH UcWpt0jRxibpljLofjjujO2xQGbTaDxJn0yJCmy338nP8zDOfmQpD0m2tGMV6gJXiaRX Ggew== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:cc :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:message-id:date:to:from:dkim-signature :delivered-to:arc-authentication-results; bh=LyWO8n5t0pJeqZ7EYDz6glyIikFBFIAniSFWv+tZFLY=; b=Osadg/DIXcSQliwUoXThmBvv+4IT4hrGyJ4LWPUbByo6HvvQ18ABP44qggHImVd48X jbOFHJulKGjTpY2VFXeLWuqRIRwG6O3B8sqMWPLXw/3JsxJSaKILyYDEO/W8O0PDrsD+ mQBLye9JCUFrvhjysmMvGcPR5aWgARs1Gwcuu2PnYIlr429nFWa6dU2UYCKfvsIDa59y WeyRCT+RAwpUPf//DqnG7mlepJA6+Vh5204N1i35+LTk25zEc1AkH+A5FgVmqPzUZBm5 xUkGPXubSMQdRa4pFXbwWE548CvbPpt9fIZ+gKEsdAZ7Ynvx/4qOHw8sriq1MJcRueam goDw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=PHtZss1c; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from ml01.01.org (ml01.01.org. [2001:19d0:306:5::1]) by mx.google.com with ESMTPS id q22si1975139pfg.244.2017.08.24.09.21.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 24 Aug 2017 09:21:58 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) client-ip=2001:19d0:306:5::1; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=PHtZss1c; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 1348621E49BD4; Thu, 24 Aug 2017 09:19:23 -0700 (PDT) X-Original-To: edk2-devel@lists.01.org Delivered-To: edk2-devel@lists.01.org Received: from mail-wr0-x22e.google.com (mail-wr0-x22e.google.com [IPv6:2a00:1450:400c:c0c::22e]) (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 310962095B9D1 for ; Thu, 24 Aug 2017 09:19:21 -0700 (PDT) Received: by mail-wr0-x22e.google.com with SMTP id j41so4060268wre.4 for ; Thu, 24 Aug 2017 09:21:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=+QISQIRcgF1EqBunuxSplLTYDZU2gFqUzy39R7w6ZiY=; b=PHtZss1cGjKdgUjAcSv958rWsEaPAa3GmzlcYaYW1LLRWZIyL411Ws1Hz1e4UPmT9K cD7Nf8GeF8cI5Xh3R7lDDDIqy+1p3DBvG8tlPJqgM5sk+d4Kx38NiEgWIWAdXc1BfV+r kdp/2ySWRgcL3IV2lIozAqkGo8ANrBBeJpPF8= 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=+QISQIRcgF1EqBunuxSplLTYDZU2gFqUzy39R7w6ZiY=; b=FmYSFfL1NnMYtljGboj1VQsKRH2+6RU8ByFynI7zaQhUGGJPQUPM/I9JMbJliYKbmz 4wGVzTraUtRvK682NUqk18vLROuBHG6BI7QY3fKpDvUgmgXNaIuvwdAKxSyZ0YO0KN/j fOCNT3NknChX5zK9GDyDLSvybfxJpoQIp/y/YGJMuaTgO9AIR0msbkmA3pH2w/4a357J 6MtrWVD/IIgCjudcHq7lk0YCKaoqEhfa18Z9jGAOUKf7EDuPfc18YBQKo0EfsKsiukaD gItNbmXrthjZPNNtySJQt7VartpywKcJJFuUuCZ6IosNACzKe83QSNC5SGwzCNdPHL1N SbPg== X-Gm-Message-State: AHYfb5ia9lUpHetpJEyopZCfgyVG882lDGBRn6wctv/4nuDx0QnjAo3Y XDj6WSjtLD5U5WyfIYgGwA== X-Received: by 10.223.165.129 with SMTP id g1mr3928115wrc.108.1503591714446; Thu, 24 Aug 2017 09:21:54 -0700 (PDT) Received: from vanye.hemma.eciton.net (cpc92316-cmbg19-2-0-cust118.5-4.cable.virginm.net. [82.12.0.119]) by smtp.gmail.com with ESMTPSA id p33sm4576087wrc.81.2017.08.24.09.21.53 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 24 Aug 2017 09:21:53 -0700 (PDT) From: Leif Lindholm To: edk2-devel@lists.01.org Date: Thu, 24 Aug 2017 17:21:53 +0100 Message-Id: <20170824162153.23494-1-leif.lindholm@linaro.org> X-Mailer: git-send-email 2.11.0 Subject: [edk2] [PATCH] Platform/Silicon: clean up BeagleBoard cargo culting 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: , Cc: Michael D Kinney , Ard Biesheuvel MIME-Version: 1.0 Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" A group of Pcd settings have been copied around ARM platforms from the original BeagleBoardPkg, which 1) Used Ebl. 2) Was designed to minimise image size. None of the affected platforms _use_ Ebl, and all of them could benefit from keeping ComponentName*Protocol information around, so remove these overrides and build with default settings. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Leif Lindholm --- Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 22 ---------------------- Platform/Marvell/Armada/Armada.dsc.inc | 22 ---------------------- Silicon/Hisilicon/Hisilicon.dsc.inc | 23 ----------------------- 3 files changed, 67 deletions(-) -- 2.11.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel Reviewed-by: Ard Biesheuvel diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc index f2a05e8e6..12ce5ad96 100644 --- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc +++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc @@ -280,23 +280,6 @@ ################################################################################ [PcdsFeatureFlag.common] - gEfiMdePkgTokenSpaceGuid.PcdComponentNameDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnosticsDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdComponentName2Disable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnostics2Disable|TRUE - - # - # Control what commands are supported from the UI - # Turn these on and off to add features or save size - # - gEmbeddedTokenSpaceGuid.PcdEmbeddedMacBoot|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedDirCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHobCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHwDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedPciDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedIoEnable|FALSE - gEmbeddedTokenSpaceGuid.PcdEmbeddedScriptCmd|FALSE - gEmbeddedTokenSpaceGuid.PcdCacheEnable|TRUE # Use the Vector Table location in CpuDxe. We will not copy the Vector Table at PcdCpuVectorBaseAddress @@ -312,7 +295,6 @@ gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwareVersionString|L"$(FIRMWARE_VER)" !endif - gEmbeddedTokenSpaceGuid.PcdEmbeddedPrompt|"VExpress" gEfiMdePkgTokenSpaceGuid.PcdMaximumUnicodeStringLength|1000000 gEfiMdePkgTokenSpaceGuid.PcdMaximumAsciiStringLength|1000000 gEfiMdePkgTokenSpaceGuid.PcdMaximumLinkedListLength|1000000 @@ -356,10 +338,6 @@ # DEBUG_ERROR 0x80000000 // Error gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8000000F - gEmbeddedTokenSpaceGuid.PcdEmbeddedAutomaticBootCommand|"" - gEmbeddedTokenSpaceGuid.PcdEmbeddedDefaultTextColor|0x07 - gEmbeddedTokenSpaceGuid.PcdEmbeddedMemVariableStoreSize|0x10000 - # # Optional feature to help prevent EFI memory map fragments # Turned on and off via: PcdPrePiProduceMemoryTypeInformationHob diff --git a/Platform/Marvell/Armada/Armada.dsc.inc b/Platform/Marvell/Armada/Armada.dsc.inc index 4e8f289fc..7e5c55c05 100644 --- a/Platform/Marvell/Armada/Armada.dsc.inc +++ b/Platform/Marvell/Armada/Armada.dsc.inc @@ -208,23 +208,6 @@ ################################################################################ [PcdsFeatureFlag.common] - gEfiMdePkgTokenSpaceGuid.PcdComponentNameDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnosticsDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdComponentName2Disable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnostics2Disable|TRUE - - # - # Control what commands are supported from the UI - # Turn these on and off to add features or save size - # - gEmbeddedTokenSpaceGuid.PcdEmbeddedMacBoot|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedDirCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHobCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHwDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedPciDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedIoEnable|FALSE - gEmbeddedTokenSpaceGuid.PcdEmbeddedScriptCmd|FALSE - gEmbeddedTokenSpaceGuid.PcdCacheEnable|TRUE # Use the Vector Table location in CpuDxe. @@ -250,7 +233,6 @@ gArmPlatformTokenSpaceGuid.PcdDefaultConOutPaths|L"VenHw(D3987D4B-971A-435F-8CAF-4967EB627241)/Uart(115200,8,N,1)/VenPcAnsi()" gArmPlatformTokenSpaceGuid.PcdDefaultConInPaths|L"VenHw(D3987D4B-971A-435F-8CAF-4967EB627241)/Uart(115200,8,N,1)/VenPcAnsi()" - gEmbeddedTokenSpaceGuid.PcdEmbeddedPrompt|"Marvell>> " gEfiMdePkgTokenSpaceGuid.PcdMaximumUnicodeStringLength|1000000 gEfiMdePkgTokenSpaceGuid.PcdMaximumAsciiStringLength|1000000 gEfiMdePkgTokenSpaceGuid.PcdMaximumLinkedListLength|1000000 @@ -323,10 +305,6 @@ gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8000000F !endif - gEmbeddedTokenSpaceGuid.PcdEmbeddedAutomaticBootCommand|"" - gEmbeddedTokenSpaceGuid.PcdEmbeddedDefaultTextColor|0x07 - gEmbeddedTokenSpaceGuid.PcdEmbeddedMemVariableStoreSize|0x10000 - # # Optional feature to help prevent EFI memory map fragments # Turned on and off via: PcdPrePiProduceMemoryTypeInformationHob diff --git a/Silicon/Hisilicon/Hisilicon.dsc.inc b/Silicon/Hisilicon/Hisilicon.dsc.inc index a48c4773a..dd7fa6a3d 100644 --- a/Silicon/Hisilicon/Hisilicon.dsc.inc +++ b/Silicon/Hisilicon/Hisilicon.dsc.inc @@ -238,25 +238,6 @@ ################################################################################ [PcdsFeatureFlag.common] - gEfiMdePkgTokenSpaceGuid.PcdComponentNameDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnosticsDisable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdComponentName2Disable|TRUE - gEfiMdePkgTokenSpaceGuid.PcdDriverDiagnostics2Disable|TRUE - - # - # Control what commands are supported from the UI - # Turn these on and off to add features or save size - # - ## Set PcdEmbeddedMacBoot to FALSE, or console mode will be changed when - ## entering EBL and not restored when exiting. - gEmbeddedTokenSpaceGuid.PcdEmbeddedMacBoot|FALSE - gEmbeddedTokenSpaceGuid.PcdEmbeddedDirCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHobCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedHwDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedPciDebugCmd|TRUE - gEmbeddedTokenSpaceGuid.PcdEmbeddedIoEnable|FALSE - gEmbeddedTokenSpaceGuid.PcdEmbeddedScriptCmd|FALSE - gEmbeddedTokenSpaceGuid.PcdCacheEnable|TRUE # Use the Vector Table location in CpuDxe. We will not copy the Vector Table at PcdCpuVectorBaseAddress @@ -273,7 +254,6 @@ [PcdsFixedAtBuild.common] gArmPlatformTokenSpaceGuid.PcdFirmwareVendor|"ARM Versatile Express" - gEmbeddedTokenSpaceGuid.PcdEmbeddedPrompt|"VExpress" gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize|44 gEmbeddedTokenSpaceGuid.PcdPrePiCpuIoSize|0 gEfiMdePkgTokenSpaceGuid.PcdMaximumUnicodeStringLength|1000000 @@ -316,9 +296,6 @@ gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x80000004 gEfiMdePkgTokenSpaceGuid.PcdReportStatusCodePropertyMask|0x06 - gEmbeddedTokenSpaceGuid.PcdEmbeddedAutomaticBootCommand|"" - gEmbeddedTokenSpaceGuid.PcdEmbeddedDefaultTextColor|0x07 - gEmbeddedTokenSpaceGuid.PcdEmbeddedMemVariableStoreSize|0x10000 # # Optional feature to help prevent EFI memory map fragments