From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web09.2422.1574678996278389725 for ; Mon, 25 Nov 2019 02:49:56 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=aYzBTa43; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: philmd@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1574678995; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=SVLHNVCoTConHr55uQR2KcJN2n2dplN3EPEVL6eM57M=; b=aYzBTa43/oHScvSLCFdDxNaUPb4z2syM34VGnwe72LMxUBd+bdU+s4UvXDZMK6gCiUdrly +rHTl9PM5nuP3n5a1o1VJisYAjvvs3mmCh019OA++8+6yzIBgeY5nFA25VisS2tQpJ7WEI 4357gQZlQmESSpoMn1GFLVjmb2X+7lI= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-237-iPYlrlDFPriSJouuum1IhQ-1; Mon, 25 Nov 2019 05:49:51 -0500 Received: by mail-wm1-f72.google.com with SMTP id z3so4417454wmk.1 for ; Mon, 25 Nov 2019 02:49:51 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=Eh0TqHDKGbk+4I0XbkSssSxt3a2gL/a4Y7fHoqgYU0U=; b=kq2qVjapx8zcy7bLwewaUnkYDeC+VEc/yZYYUHCk4AzMPDEcGoQ2+4YaOgQHYSMzBD 8e52RCtHjS1ZNT9F0SQnF45OI8J+IW5tjnbyUTP4a5yJbbp8ZJ3Gz4MDnU9njdizWd9S fUpY8Xz0z/ruEHjBA0LUH9JdaMzQJX8xJu15pnjLO+UDFODVZc1uVJdYVu69T4O+u3/2 Rtibro6UQUofO+/rmCok9nh3k89HPOTsyHrFAvHJvxS7ISXpnAsa9YKFnEZBsh23VknV HqRXCU46k2RX+Fc1b13fL52b+M6/OWilQhH3YcrLSxVFrS65FmuVbcFILARd9cE2dZUd 5Rvw== X-Gm-Message-State: APjAAAWwDYF4Nz5QoDwS2a2MbcatXKPYkAV4VO9mAz4/Gt4k0k7jFOar KxAg7CFuDUeDw3zwv/f0OKdLgacmiWmU8oB6nic0OhQNuoW9sxbDYerpcsSvFfR+no4DW5dIyRu vJS2cjYXN3imSXQ== X-Received: by 2002:a5d:55c5:: with SMTP id i5mr32865899wrw.385.1574678990664; Mon, 25 Nov 2019 02:49:50 -0800 (PST) X-Google-Smtp-Source: APXvYqxCvxMfe8f7rBSFCVB6UGMH5rrmLQUxcEbYaSMWGri969Py5vpbjI/TXe7qtAa1lRP+zGsxiw== X-Received: by 2002:a5d:55c5:: with SMTP id i5mr32865874wrw.385.1574678990457; Mon, 25 Nov 2019 02:49:50 -0800 (PST) Return-Path: Received: from [192.168.1.35] (182.red-88-21-103.staticip.rima-tde.net. [88.21.103.182]) by smtp.gmail.com with ESMTPSA id r63sm11682486wma.2.2019.11.25.02.49.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 25 Nov 2019 02:49:49 -0800 (PST) Subject: Re: [edk2-devel] [PATCH v2 1/1] DynamicTablesPkg: Fix 'missing header' warnings To: devel@edk2.groups.io, sami.mujawar@arm.com Cc: Alexei.Fedorov@arm.com, Matteo.Carlini@arm.com, Laura.Moretta@arm.com, nd@arm.com References: <20191125092247.18080-1-sami.mujawar@arm.com> From: =?UTF-8?B?UGhpbGlwcGUgTWF0aGlldS1EYXVkw6k=?= Message-ID: Date: Mon, 25 Nov 2019 11:49:49 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 MIME-Version: 1.0 In-Reply-To: <20191125092247.18080-1-sami.mujawar@arm.com> X-MC-Unique: iPYlrlDFPriSJouuum1IhQ-1 X-Mimecast-Spam-Score: 0 Content-Language: en-US Content-Type: text/plain; charset=WINDOWS-1252; format=flowed Content-Transfer-Encoding: quoted-printable On 11/25/19 10:22 AM, Sami Mujawar via Groups.Io wrote: > The Basetools scripts have been updated to report > warnings if header files are not listed in the > [Sources] section of the inf files. >=20 > Update the inf files to include the respective > header files to fix the reported warnings. >=20 > Cc: Alexei Fedorov > Signed-off-by: Sami Mujawar > --- >=20 > The changes can be seen at: > https://github.com/samimujawar/edk2/tree/686_dynamic_tables_fix_include_w= arning_v1 >=20 > Notes: > v2: > - Fixed patch check failure in commit message, [SAM= I] > changed 'CC' to 'Cc'. No code change. Again, already reviewed: https://www.mail-archive.com/devel@edk2.groups.io/msg11965.html Reviewed-by: Philippe Mathieu-Daude > v1: > - Fix missing header warnings [SAM= I] >=20 > DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.= inf | 1 + > DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLibArm.inf = | 3 ++- > DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/AcpiPpttLibArm.inf = | 1 + > 3 files changed, 4 insertions(+), 1 deletion(-) >=20 > diff --git a/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTable= FactoryDxe.inf b/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTab= leFactoryDxe.inf > index b47a690dd72702a2d7157cbb21249ba1d01d08cd..920aa81e31b402cae645d919e= e6559d8c9bde136 100644 > --- a/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactory= Dxe.inf > +++ b/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactory= Dxe.inf > @@ -23,6 +23,7 @@ [Defines] > [Sources] > AcpiTableFactory/AcpiTableFactory.c > DeviceTreeTableFactory/DeviceTreeTableFactory.c > + DynamicTableFactory.h > DynamicTableFactoryDxe.c > SmbiosTableFactory/SmbiosTableFactory.c > =20 > diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLib= Arm.inf b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLibArm.i= nf > index 3bbd0e7818c6172f9178b12a3d0960e2d298bca1..2f987e3b6d339d4b5f342cb52= b8dc712cf4e585c 100644 > --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLibArm.inf > +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLibArm.inf > @@ -1,7 +1,7 @@ > ## @file > # IORT Table Generator > # > -# Copyright (c) 2017 - 2018, ARM Limited. All rights reserved. > +# Copyright (c) 2017 - 2019, ARM Limited. All rights reserved. > # > # SPDX-License-Identifier: BSD-2-Clause-Patent > ## > @@ -18,6 +18,7 @@ [Defines] > =20 > [Sources] > IortGenerator.c > + IortGenerator.h > =20 > [Packages] > MdePkg/MdePkg.dec > diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/AcpiPpttLib= Arm.inf b/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/AcpiPpttLibArm.i= nf > index 3cb13d7d8fd3df04f3e16cc6161d9736f7579a80..2c7d19513d6ceea29fd9f271f= 983a6cf6d2c1046 100644 > --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/AcpiPpttLibArm.inf > +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/AcpiPpttLibArm.inf > @@ -18,6 +18,7 @@ [Defines] > =20 > [Sources] > PpttGenerator.c > + PpttGenerator.h > =20 > [Packages] > EmbeddedPkg/EmbeddedPkg.dec >=20