From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [edk2-devel] [edk2-platforms][PATCH 0/3] Introduce Ampere Mt. Jade support to edk2-platforms To: Leif Lindholm ,devel@edk2.groups.io From: "Nhi Pham" X-Originating-Location: Ho Chi Minh City, Ho Chi Minh, VN (42.119.193.214) X-Originating-Platform: Mac Chrome 87 User-Agent: GROUPS.IO Web Poster MIME-Version: 1.0 Date: Thu, 03 Dec 2020 04:15:27 -0800 References: <20201203114602.GQ1664@vanye> In-Reply-To: <20201203114602.GQ1664@vanye> Message-ID: <6521.1606997727161006207@groups.io> Content-Type: multipart/alternative; boundary="NBHjtwtLzSMOeODSkJsF" --NBHjtwtLzSMOeODSkJsF Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hi Leif, Please find my response inline. On Thu, Dec 3, 2020 at 06:46 PM, Leif Lindholm wrote: >=20 > On Thu, Dec 03, 2020 at 11:03:59 +0700, Nhi Pham wrote: >=20 >> This short patch series introduces the support for Ampere Computing's >> Arm64 hardware platforms, >=20 > Well, it doesn't really, does it? > It adds a Maintainers.txt entry, a Readme.md describing a fork > somewhere, and links that Readme into the top-level one. >=20 > I think it's great you've posted it here for people to see, but can > you clarify your intent somewhat? > Is this a precursor to a full upstreaming effort? Yes, this patch series works as bootstrap that we will push all Mt. Jade c= ode in one shot (about 35 commits) to the edk2-platforms for review after t= his is approved and merged. This series provides Maintainers and Reviewers = in the Maintainer.txt who are responsible for giving code review feedback (= Reviewed-by tag) if stewards can not review a large patch series. Thanks, Nhi >=20 >=20 > I don't see a point in merging just this, although I can (and will) > provide some feedback on the documentation. >=20 > Best Regards, >=20 > Leif >=20 >=20 >> specifically the Mt. Jade platform based on >> Ampere's Altra Family processor at this moment. >>=20 >> A public fork of EDK2 for supporting Mt. Jade is currently available at >> https://github.com/AmpereComputing/edk2-platforms >>=20 >> Cc: Leif Lindholm >> Cc: Michael D Kinney >> Cc: Ard Biesheuvel >> Cc: Nate DeSimone >>=20 >> Nhi Pham (1): >> Maintainers.txt: Update for Ampere platforms >>=20 >> Phong Vo (2): >> Platform/Ampere: Add Readme.md >> Update Readme.md >>=20 >> Maintainers.txt | 6 + >> Platform/Ampere/Readme.md | 167 ++++++++++++++++++++ >> Readme.md | 3 + >> 3 files changed, 176 insertions(+) >> create mode 100644 Platform/Ampere/Readme.md >>=20 >> -- >> 2.17.1 >=20 > --NBHjtwtLzSMOeODSkJsF Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hi Leif,

Please find my response inline.

On Thu, Dec = 3, 2020 at 06:46 PM, Leif Lindholm wrote:
On Thu, Dec 03, 2020 at 11:03:59 +0700, Nhi Pham wrote:
This short patch series introduces the support for Ampere Comp= uting's
Arm64 hardware platforms,
Well, it doesn't really, does it?
It adds a Maintainers.txt entry, a = Readme.md describing a fork
somewhere, and links that Readme into the = top-level one.

I think it's great you've posted it here for peop= le to see, but can
you clarify your intent somewhat?
Is this a pr= ecursor to a full upstreaming effort?
Yes, this patch series works as bootstrap that we will push all Mt. Jade c= ode in one shot (about 35 commits) to the edk2-platforms for review after t= his is approved and merged. This series provides Maintainers and Reviewers = in the Maintainer.txt who are responsible for giving code review feedback (= Reviewed-by tag) if stewards can not review a large patch series.

Thanks,
Nhi

I don't see a point in merging just this, although I can= (and will)
provide some feedback on the documentation.

Bes= t Regards,

Leif

specifically the Mt. Jade platform based on
Ampere's Altr= a Family processor at this moment.

A public fork of EDK2 for sup= porting Mt. Jade is currently available at
https= ://github.com/AmpereComputing/edk2-platforms

Cc: Leif Lindho= lm <leif@nuviainc.com>
Cc: Michael D Kinney <michael.d.kinney= @intel.com>
Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
= Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>

Nhi Pha= m (1):
Maintainers.txt: Update for Ampere platforms

Phong V= o (2):
Platform/Ampere: Add Readme.md
Update Readme.md

Maintainers.txt | 6 +
Platform/Ampere/Readme.md | 167 +++++++++++++++= +++++
Readme.md | 3 +
3 files changed, 176 insertions(+)
cre= ate mode 100644 Platform/Ampere/Readme.md

--
2.17.1
--NBHjtwtLzSMOeODSkJsF--