From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ot0-f175.google.com (mail-ot0-f175.google.com [74.125.82.175]) (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 A974781DB3 for ; Mon, 16 Jan 2017 07:20:06 -0800 (PST) Received: by mail-ot0-f175.google.com with SMTP id 104so45830382otd.3 for ; Mon, 16 Jan 2017 07:20:07 -0800 (PST) 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=RpvFutILS/GGvi4uEXTxUK3xsp1bVW8eUWb659ifIy0=; b=GJueYMOX0p0XKYjK7k1j9yQuNRCB/vZ5SMozDwabclGaAwI9zyM8kjYpMoNwnWi5gQ IZaQZLmuowAAhsD1EPETXNlW33LqEMXV1I36rUzHBUWG82GyxjAPX66zklv2KpkdlBCK Q2MoHXE324exXPSe8ug+ZeAa33UNk03NWsWuO5GjLy/OgNqobDopLbThNmACz7bMQAlQ B7xibgBVrPc1MRk3gIBuIm217UHdfeBDuj0zSpbNlpfcbto21Bj8Zzmp3zMGTwDhkCHi u2AK3TpiyarxzrqDrUhCHDNiT05zHTJMcMsEKhgRlDy8sR/77Dwur8IHONLLq+RwTGoN Yeqg== X-Gm-Message-State: AIkVDXKOCUH0S3QIpY7uPP912A7ocX3XKwywT9TnI9oDDrwX4UiUlOtQMTy2DcxIeuG4G9JiCNpOPMO08WjKPEzV X-Received: by 10.157.13.22 with SMTP id 22mr17678044oti.16.1484580006409; Mon, 16 Jan 2017 07:20:06 -0800 (PST) MIME-Version: 1.0 Received: by 10.74.76.136 with HTTP; Mon, 16 Jan 2017 07:20:06 -0800 (PST) In-Reply-To: References: <1484294219-3320-1-git-send-email-bhsharma@redhat.com> From: Bhupesh Sharma Date: Mon, 16 Jan 2017 20:50:06 +0530 Message-ID: To: Andrew Fish Cc: edk2-devel , Ni Ruiyu , Yao Jiewen Subject: Re: [PATCH 1/1] AcpiTool: Fix minor casing and file path issues 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: Mon, 16 Jan 2017 15:20:06 -0000 Content-Type: text/plain; charset=UTF-8 Hi Andrew, On Sat, Jan 14, 2017 at 7:33 AM, Andrew Fish wrote: > >> On Jan 12, 2017, at 11:56 PM, Bhupesh Sharma wrote: >> >> This patch fixes some minor casing and file path issues > > Should we add a Linux builder with a case sensitive file system? > > macOS defaults to case insensitive but our build machines always use a case sensitive filesystem so I've seen my share of case bugs.... > > One of the tricks we use on macOS is to create a DMG (disk image file) that is case sensitive and then you mount it as /Volumes/Case and you don't have to repartition your disk. I'm guessing you can do something similar in Linux. I think that is doable but normal grep searches on linux (for identifiers) without the -i parameter (search irrespective of the case) do not return function names being used if the casing is not handled properly. In the case of AcpiTool the file names and paths are not too messed up and the minor issues can be fixed by this patch. But ofcourse if I am running a BoardPkg created using Windows tool on Linux, I would be tempted to use a Mac like approach to get the stuff to properly compile. Regards, Bhupesh > > Thanks, > > Andrew Fish > >> with the AcpiToolPkg from Yao Jiewen >> (https://github.com/jyao1/EdkiiShellTool). >> >> This patch also adds AARCH64 as one of the supported architectures >> on which AcpiTool can run. >> >> Compiled and tested this patch under the following environment: >> >> - Compiled for AARCH64 architecture using GCC4.9 toolchain from Linaro. >> - Exercised the 'DumpACPI.efi' thus generated on Qemu for AARCH64 as an >> application. >> - Able to dump all relevant ACPI tables. >> >> This patch is also available here for easy access: >> https://github.com/bhupesh-sharma/EdkiiShellTool >> >> Cc: Ni Ruiyu >> Cc: Evan Lloyd >> Cc: Sami Mujawar >> Cc: Yao Jiewen >> Signed-off-by: Bhupesh Sharma >> Contributed-under: TianoCore Contribution Agreement 1.0 >> --- >> AcpiToolPkg/AcpiToolPkg.dsc | 2 +- >> AcpiToolPkg/DumpAcpi/DumpACPI.c | 2 +- >> AcpiToolPkg/DumpAcpi/DumpACPI.h | 2 +- >> AcpiToolPkg/DumpAcpi/DumpAcpiDSDT.c | 2 +- >> AcpiToolPkg/DumpAcpi/DumpAcpiPSDT.c | 2 +- >> AcpiToolPkg/DumpAcpi/DumpAcpiSSDT.c | 2 +- >> AcpiToolPkg/Library/DumpAcpi/DumpAcpiDMARLib/DumpAcpiDMARLib.c | 2 +- >> .../Library/DumpAcpi/DumpAcpiTableFuncLib/DumpAcpiTableFuncLib.c | 6 +++--- >> 8 files changed, 10 insertions(+), 10 deletions(-) >> >> diff --git a/AcpiToolPkg/AcpiToolPkg.dsc b/AcpiToolPkg/AcpiToolPkg.dsc >> index 9943f7610315..c0fcec5b2518 100644 >> --- a/AcpiToolPkg/AcpiToolPkg.dsc >> +++ b/AcpiToolPkg/AcpiToolPkg.dsc >> @@ -17,7 +17,7 @@ >> PLATFORM_VERSION = 0.1 >> DSC_SPECIFICATION = 0x00010005 >> OUTPUT_DIRECTORY = Build/AcpiToolPkg >> - SUPPORTED_ARCHITECTURES = IA32|X64 >> + SUPPORTED_ARCHITECTURES = IA32|X64|AARCH64 >> BUILD_TARGETS = DEBUG|RELEASE >> SKUID_IDENTIFIER = DEFAULT >> >> diff --git a/AcpiToolPkg/DumpAcpi/DumpACPI.c b/AcpiToolPkg/DumpAcpi/DumpACPI.c >> index 38cc804a72e9..5ea51e67a260 100644 >> --- a/AcpiToolPkg/DumpAcpi/DumpACPI.c >> +++ b/AcpiToolPkg/DumpAcpi/DumpACPI.c >> @@ -23,7 +23,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> >> #include >> >> -#include "DumpAcpi.h" >> +#include "DumpACPI.h" >> >> #define RSDP_SIGN_DEFINITION SIGNATURE_32 ('R', 'S', 'D', 'P') >> >> diff --git a/AcpiToolPkg/DumpAcpi/DumpACPI.h b/AcpiToolPkg/DumpAcpi/DumpACPI.h >> index 40c11e70f443..8307f50ec805 100644 >> --- a/AcpiToolPkg/DumpAcpi/DumpACPI.h >> +++ b/AcpiToolPkg/DumpAcpi/DumpACPI.h >> @@ -16,7 +16,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> >> #include >> #include >> -#include >> +#include >> >> #ifndef MAX_FILE_NAME_LEN >> #define MAX_FILE_NAME_LEN 0x1000 >> diff --git a/AcpiToolPkg/DumpAcpi/DumpAcpiDSDT.c b/AcpiToolPkg/DumpAcpi/DumpAcpiDSDT.c >> index 42541be92081..764f4b6470ba 100644 >> --- a/AcpiToolPkg/DumpAcpi/DumpAcpiDSDT.c >> +++ b/AcpiToolPkg/DumpAcpi/DumpAcpiDSDT.c >> @@ -24,7 +24,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> #include >> #include >> >> -#include "DumpAcpi.h" >> +#include "DumpACPI.h" >> >> VOID >> EFIAPI >> diff --git a/AcpiToolPkg/DumpAcpi/DumpAcpiPSDT.c b/AcpiToolPkg/DumpAcpi/DumpAcpiPSDT.c >> index 8878d3f6b594..b28e90f18172 100644 >> --- a/AcpiToolPkg/DumpAcpi/DumpAcpiPSDT.c >> +++ b/AcpiToolPkg/DumpAcpi/DumpAcpiPSDT.c >> @@ -24,7 +24,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> #include >> #include >> >> -#include "DumpAcpi.h" >> +#include "DumpACPI.h" >> >> VOID >> EFIAPI >> diff --git a/AcpiToolPkg/DumpAcpi/DumpAcpiSSDT.c b/AcpiToolPkg/DumpAcpi/DumpAcpiSSDT.c >> index 0a268eb136ab..c49b84c18ee0 100644 >> --- a/AcpiToolPkg/DumpAcpi/DumpAcpiSSDT.c >> +++ b/AcpiToolPkg/DumpAcpi/DumpAcpiSSDT.c >> @@ -24,7 +24,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> #include >> #include >> >> -#include "DumpAcpi.h" >> +#include "DumpACPI.h" >> >> VOID >> EFIAPI >> diff --git a/AcpiToolPkg/Library/DumpAcpi/DumpAcpiDMARLib/DumpAcpiDMARLib.c b/AcpiToolPkg/Library/DumpAcpi/DumpAcpiDMARLib/DumpAcpiDMARLib.c >> index 865a213fa706..7d6d8e0b0194 100644 >> --- a/AcpiToolPkg/Library/DumpAcpi/DumpAcpiDMARLib/DumpAcpiDMARLib.c >> +++ b/AcpiToolPkg/Library/DumpAcpi/DumpAcpiDMARLib/DumpAcpiDMARLib.c >> @@ -18,7 +18,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> #include >> #include >> #include >> -#include >> +#include >> >> VOID >> DumpDmarDeviceScopeEntry ( >> diff --git a/AcpiToolPkg/Library/DumpAcpi/DumpAcpiTableFuncLib/DumpAcpiTableFuncLib.c b/AcpiToolPkg/Library/DumpAcpi/DumpAcpiTableFuncLib/DumpAcpiTableFuncLib.c >> index a52a718bfa40..9036926065cb 100644 >> --- a/AcpiToolPkg/Library/DumpAcpi/DumpAcpiTableFuncLib/DumpAcpiTableFuncLib.c >> +++ b/AcpiToolPkg/Library/DumpAcpi/DumpAcpiTableFuncLib/DumpAcpiTableFuncLib.c >> @@ -12,9 +12,9 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> **/ >> >> #include >> -#include >> -#include >> -#include >> +#include >> +#include >> +#include >> >> typedef struct { >> UINT32 Signature; >> -- >> 2.7.4 >> >> _______________________________________________ >> edk2-devel mailing list >> edk2-devel@lists.01.org >> https://lists.01.org/mailman/listinfo/edk2-devel >