From: Pete Batard <pete@akeo.ie>
To: Kurt Kennett <Kurt.Kennett@microsoft.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [PATCH 0/4] BaseTools: Add VS2017 support, including ARM and AARCH64
Date: Tue, 14 Nov 2017 17:30:56 +0000 [thread overview]
Message-ID: <bbb463ec-4b6b-290d-5366-70cb0a8e74a7@akeo.ie> (raw)
In-Reply-To: <BN6PR21MB04684BB6BCE181F5BC5C74739C280@BN6PR21MB0468.namprd21.prod.outlook.com>
Hi Kurt,
On 2017.11.14 16:16, Kurt Kennett wrote:
> Does this build and boot a platform?
Not yet, but I believe I got pretty close to a full build for ARM.
However, the resources I can devote to finalizing ARM image generation,
and even more so, to add the missing resources for AARCH64 image
generation are limited, even more so as I am no ARM specialist.
I was actually hoping that, with the application of these patches, EDK2
contributors would step in to help with the effort of filling the
assembly gaps, as well as finalize image generation for ARM and AARCH64.
> This should not be committed until a platform is building and booting to UEFI shell, IMO.
Well, I see it a bit as a catch 22.
If we don't get the VS2017 ARM toolchains in, I don't think many people
will be willing to help with the image generation, because there will be
very little incentive for them to do so. And if we wait for image
generation to be sorted out before adding the toolchains, it may very
well be a couple more years before everything is finalized... which
would penalize the people who simply want the convenience of VS2017
support to build regular ARM and AARCH64 applications.
IMO, a gradual approach to introducing VS2017 ARM/AARCH64 support might
be preferable to an "all or nothing" one.
Regards,
/Pete
prev parent reply other threads:[~2017-11-14 17:26 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-14 12:32 [PATCH 0/4] BaseTools: Add VS2017 support, including ARM and AARCH64 Pete Batard
2017-11-14 12:32 ` [PATCH 1/4] MdeModulePkg, NetworkPkg: Fix VS2017 IA32 warnings Pete Batard
2017-11-14 12:32 ` [PATCH 2/4] BaseTools: Add VS2017 IA32 and X64 support Pete Batard
2017-11-14 12:32 ` [PATCH 3/4] BaseTools: Add VS2017 ARM support Pete Batard
2017-11-14 12:32 ` [PATCH 4/4] BaseTools: Add VS2017 AARCH64 support Pete Batard
2017-11-14 16:03 ` [PATCH 0/4] BaseTools: Add VS2017 support, including ARM and AARCH64 Gao, Liming
2017-11-14 17:07 ` Pete Batard
2017-11-14 16:16 ` Kurt Kennett
2017-11-14 17:30 ` Pete Batard [this message]
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=bbb463ec-4b6b-290d-5366-70cb0a8e74a7@akeo.ie \
--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