public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Sami Mujawar" <sami.mujawar@arm.com>
To: devel@edk2.groups.io, AbdulLateef.Attar@amd.com,
	"nd@arm.com" <nd@arm.com>,
	Pierre Gondois <pierre.gondois@arm.com>
Subject: Re: [edk2-devel] [PATCH v2 0/2] DynamicTablesPkg: Fix IA32 and X64 compilation errors
Date: Wed, 3 Jan 2024 14:54:00 +0000	[thread overview]
Message-ID: <b457d25e-603c-46d0-8bb3-62061ba15032@arm.com> (raw)
In-Reply-To: <cover.1704265673.git.AbdulLateef.Attar@amd.com>

Hi Abdul,

I have merged this series as d7d4f09ff815..2576cbafec98.

Now that support for X64 and IA32 is enabled in DynamicTablesPkg, it
would be good to enable that in the edk2 CI. That way we do not
accidently break the builds.

To enable edk2 CI for AARCH64 using GCC, I had made the change at
https://github.com/tianocore/edk2/blob/master/.azurepipelines/templates/pr-gate-build-job.yml#L50

I am not sure what is required to support X64 and IA32 builds using
VS2019. Can you provide a patch for that, please?

Also, by any chance have you tried building DynamicTablesPkg with GCC
for X64 and IA32?

Regards,

Sami Mujawar

On 03/01/2024 09:52 am, Abdul Lateef Attar via groups.io wrote:
> From: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
>
> PR: https://github.com/tianocore/edk2/pull/5209
> v2: Fix the IA32 compilation errors.
> v1: Fix the X64 compilation errors.
>
> Cc: Pierre Gondois pierre.gondois@arm.com
> Cc: Sami Mujawar sami.mujawar@arm.com
> Cc: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
>
> Abdul Lateef Attar (2):
>    DynamicTablesPkg: Fix X64 compilation errors
>    DynamicTablesPkg: Fix IA32 compilation errors
>
>   DynamicTablesPkg/DynamicTables.dsc.inc          | 17 ++++++++++++-----
>   DynamicTablesPkg/DynamicTablesPkg.dsc           |  7 +++++--
>   .../Library/Common/AmlLib/AmlLib.inf            |  2 ++
>   .../Library/Common/AmlLib/CodeGen/AmlCodeGen.c  |  4 ++--
>   .../Common/DynamicPlatRepoLib/DynamicPlatRepo.c |  7 ++++---
>   .../ConfigurationManagerObjectParser.c          |  2 +-
>   .../Library/Common/TableHelperLib/TableHelper.c |  6 ++++--
>   .../Common/TableHelperLib/TableHelperLib.inf    |  2 ++
>   8 files changed, 32 insertions(+), 15 deletions(-)
>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113098): https://edk2.groups.io/g/devel/message/113098
Mute This Topic: https://groups.io/mt/103499168/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  parent reply	other threads:[~2024-01-03 14:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-03  9:52 [edk2-devel] [PATCH v2 0/2] DynamicTablesPkg: Fix IA32 and X64 compilation errors Abdul Lateef Attar via groups.io
2024-01-03  9:52 ` [edk2-devel] [PATCH v2 1/2] DynamicTablesPkg: Fix " Abdul Lateef Attar via groups.io
2024-01-03 11:33   ` Sami Mujawar
2024-01-03  9:52 ` [edk2-devel] [PATCH v2 2/2] DynamicTablesPkg: Fix IA32 " Abdul Lateef Attar via groups.io
2024-01-03 11:33   ` Sami Mujawar
2024-01-03 14:54 ` Sami Mujawar [this message]
2024-01-04  2:11   ` [edk2-devel] [PATCH v2 0/2] DynamicTablesPkg: Fix IA32 and X64 " Abdul Lateef Attar via groups.io
     [not found]   ` <17A702FE941215EA.10454@groups.io>
2024-01-04  4:04     ` Abdul Lateef Attar via groups.io
2024-01-04 10:21       ` Sami Mujawar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b457d25e-603c-46d0-8bb3-62061ba15032@arm.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox