From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c09::241; helo=mail-wm0-x241.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wm0-x241.google.com (mail-wm0-x241.google.com [IPv6:2a00:1450:400c:c09::241]) (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 D214D221523A5 for ; Mon, 4 Dec 2017 09:09:01 -0800 (PST) Received: by mail-wm0-x241.google.com with SMTP id f9so15716848wmh.0 for ; Mon, 04 Dec 2017 09:13:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=PTiFYQgSAYNQ49zxdMEHrmaN4myPwJuGiWNbQaNJEgU=; b=fBnnEcqHuLbLLppYaCuj+XAIuOVDSu85dvqLqKLKI3nwnScY8iA35BidJPAyVb6dkF GcmPFZKbyopqBwGucxKBNg4pJm8MPyoLpM+QyP6cxmr8WAQmj1pBVNpOCNS1sZPHuMLN mup7xM1KF63sSDOczTEMe4fPopDuUUcKZGFOA= 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:in-reply-to :references; bh=PTiFYQgSAYNQ49zxdMEHrmaN4myPwJuGiWNbQaNJEgU=; b=sWbDlixT5TI7U6Rkhu6KFs3sKHxTJpPpDx10ofAgRIj3RcZnYwYrNPfyRfcmNCHz5v vpGAdJzAUkLKjlqObO5MrAYyykRUi5hx7X8zVWKr2IZzljelSbAr6De3HIxCRJoXnDR6 WU3hTxSP3a8QbmdfhPjjrNpEgUvp0LThtR4YrE11uIIUY/qwtIzS2wL3JYougUtSq3AF OrGMhwUY8dpvYg7H60DQKkigk6ngp0eBbBcY8jknqoiknOJlnBxDAqtmdFsJk13sku0J ZSMd+Zn+aGPbtFbAJwrewio5VgYdb31AyQQxwet8aZMBFqmYPnWYlN9Wuh7N6dOMjnVa pVTw== X-Gm-Message-State: AKGB3mJgjYmQswZGzJmz8xuKjT6rb0jL2vtTAd/6JwyULMeXfJf2ygKy r1CP0wy/q30EaiRwkMA2iQpmls8/zn0= X-Google-Smtp-Source: AGs4zMad0ruX7nib4Ip6SYxPTUfk4mH2todQfu1JyiMtof8ippDUZLpmd3x/PzpH9PB8QXofv1dhgg== X-Received: by 10.28.66.25 with SMTP id p25mr3417847wma.154.1512407609803; Mon, 04 Dec 2017 09:13:29 -0800 (PST) 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 i3sm11514586wre.33.2017.12.04.09.13.28 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 04 Dec 2017 09:13:28 -0800 (PST) From: Leif Lindholm To: edk2-devel@lists.01.org Cc: ard.biesheuvel@linaro.org Date: Mon, 4 Dec 2017 17:13:25 +0000 Message-Id: <20171204171325.3359-2-leif.lindholm@linaro.org> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20171204171325.3359-1-leif.lindholm@linaro.org> References: <20171204171325.3359-1-leif.lindholm@linaro.org> Subject: [PATCH edk2-platforms 2/2] Platform/Hisilicon: fix conditional inclusion of tftp shell command 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: Mon, 04 Dec 2017 17:09:02 -0000 D02, D03 and D05 all have nice conditional clauses to only include the non-standard tftp shell command ifdef(INCLUDE_TFTP_COMMAND). Unfortunately, they all explicitly set DEFINE INCLUDE_TFTP_COMMAND=1 in their platform description files, meaning there is no way to exclude the command without an editor. So remove the DEFINE and let people specify on the command line whether to include it, as originally intended. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Leif Lindholm --- Platform/Hisilicon/D02/Pv660D02.dsc | 1 - Platform/Hisilicon/D03/D03.dsc | 1 - Platform/Hisilicon/D05/D05.dsc | 1 - 3 files changed, 3 deletions(-) diff --git a/Platform/Hisilicon/D02/Pv660D02.dsc b/Platform/Hisilicon/D02/Pv660D02.dsc index ab7c7ba7b7..a2a94945e5 100644 --- a/Platform/Hisilicon/D02/Pv660D02.dsc +++ b/Platform/Hisilicon/D02/Pv660D02.dsc @@ -28,7 +28,6 @@ [Defines] BUILD_TARGETS = DEBUG|RELEASE SKUID_IDENTIFIER = DEFAULT FLASH_DEFINITION = Platform/Hisilicon/D02/$(PLATFORM_NAME).fdf - DEFINE INCLUDE_TFTP_COMMAND=1 !include Silicon/Hisilicon/Hisilicon.dsc.inc diff --git a/Platform/Hisilicon/D03/D03.dsc b/Platform/Hisilicon/D03/D03.dsc index 00b459ab35..d37c370648 100644 --- a/Platform/Hisilicon/D03/D03.dsc +++ b/Platform/Hisilicon/D03/D03.dsc @@ -28,7 +28,6 @@ [Defines] BUILD_TARGETS = DEBUG|RELEASE SKUID_IDENTIFIER = DEFAULT FLASH_DEFINITION = Platform/Hisilicon/$(PLATFORM_NAME)/$(PLATFORM_NAME).fdf - DEFINE INCLUDE_TFTP_COMMAND=1 !include Silicon/Hisilicon/Hisilicon.dsc.inc diff --git a/Platform/Hisilicon/D05/D05.dsc b/Platform/Hisilicon/D05/D05.dsc index 60eca3c6fc..1574c3feef 100644 --- a/Platform/Hisilicon/D05/D05.dsc +++ b/Platform/Hisilicon/D05/D05.dsc @@ -29,7 +29,6 @@ [Defines] SKUID_IDENTIFIER = DEFAULT FLASH_DEFINITION = Platform/Hisilicon/$(PLATFORM_NAME)/$(PLATFORM_NAME).fdf DEFINE EDK2_SKIP_PEICORE=0 - DEFINE INCLUDE_TFTP_COMMAND=1 DEFINE NETWORK_IP6_ENABLE = FALSE DEFINE HTTP_BOOT_ENABLE = FALSE -- 2.11.0