public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Sunil V L" <sunilvl@ventanamicro.com>
To: "Chang, Abner" <Abner.Chang@amd.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"ray.ni@intel.com" <ray.ni@intel.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>,
	lichao <lichao@loongson.cn>,
	"Kirkendall, Garrett" <Garrett.Kirkendall@amd.com>,
	"Grimes, Paul" <Paul.Grimes@amd.com>,
	"He, Jiangang" <Jiangang.He@amd.com>,
	"Attar, AbdulLateef (Abdul Lateef)" <AbdulLateef.Attar@amd.com>,
	Leif Lindholm <quic_llindhol@quicinc.com>,
	Andrew Fish <afish@apple.com>
Subject: Re: [edk2-devel] The principles of EDK2 module reconstruction for archs
Date: Thu, 29 Sep 2022 20:52:04 +0530	[thread overview]
Message-ID: <YzW4HB3GC59h+kUk@sunil-laptop> (raw)
In-Reply-To: <MN2PR12MB3966624AEAFDB0C6660BB6A2EA579@MN2PR12MB3966.namprd12.prod.outlook.com>

On Thu, Sep 29, 2022 at 02:54:05PM +0000, Chang, Abner wrote:
> [AMD Official Use Only - General]
> 
> Hi Sunil,
> One more thing other than the module reconstruction for archs before you sending patch to edk2:
> Not sure how would you do on migrating the RISC-V code from edk2-platforms to edk2.  Did you make some other changes to the RISC-V CpuDxe on edk2-platform?
> Please keep the files history and send the patch for the migration first.  Then have the follow up patches for your changes if any and also add the Ventana license.
> 
> Below branches could be the reference for this migration,
> https://github.com/changab/edk2/commits/RISC-V-MIGRATION-EDK2-PR
> https://github.com/changab/edk2-platforms/commits/RISC-V-MIGRATION-EDK2-PLATFORM
> 
> Thanks Sunil.
> Abner
> 
Thanks Abner. Let me take a look at your branch.

We have some changes and are not migrating everything from
edk2-platforms. But I am not sure whether we can maintain the commit
history when we migrate from a different repo. I think this should be like a 
new review and all old RB tags which were for edk2-platforms need to be removed.

For ex:
https://github.com/changab/edk2/commit/eca5ff6bea66be94fd58421ba98cb54d1f4181a6

IMO, RB tag should be removed and should be reviewed fresh when it is
being added to edk2 repo.

Thanks
Sunil

  reply	other threads:[~2022-09-29 15:22 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-23 15:38 The principles of EDK2 module reconstruction for archs Chang, Abner
2022-09-23 18:00 ` Michael D Kinney
2022-09-26  7:33   ` Chang, Abner
2022-09-26 15:44     ` [edk2-devel] " Michael D Kinney
2022-09-27  9:25       ` Chang, Abner
2022-09-28  3:56   ` Ni, Ray
2022-09-28  4:35     ` Chang, Abner
2022-09-28  3:33 ` Ni, Ray
2022-09-28  4:30   ` [edk2-devel] " Chang, Abner
2022-09-28  5:42     ` Ni, Ray
2022-09-28  5:54       ` Chang, Abner
2022-09-28  7:34   ` Sunil V L
2022-09-28 15:00     ` Chang, Abner
2022-09-29  7:10       ` Attar, AbdulLateef (Abdul Lateef)
2022-09-29  7:42         ` Ni, Ray
2022-09-30  7:10           ` Chang, Abner
2022-09-30 15:28             ` Michael D Kinney
2022-09-30 16:08               ` Leif Lindholm
2022-09-30 18:52                 ` Michael D Kinney
2022-10-03  3:13                   ` Chang, Abner
2022-10-03  5:17                     ` Michael D Kinney
2022-10-03  5:36                       ` Chang, Abner
2022-10-03 16:54                         ` Michael D Kinney
2022-10-04 14:05                           ` Chang, Abner
2022-10-04 14:17                             ` Michael D Kinney
2022-10-05  5:38                               ` Chang, Abner
2022-10-06  8:37                                 ` Chang, Abner
2022-10-11  1:51                                   ` Ni, Ray
2022-10-11  2:20                                     ` Chang, Abner
2022-10-11 20:16                                       ` Michael D Kinney
2022-10-12  0:52                                         ` Chang, Abner
2022-10-12  1:09                                           ` Michael D Kinney
2022-10-12  1:26                                             ` Chang, Abner
2022-09-30  9:15       ` The principles of EDK2 module reconstruction for archs (wiki page on tianocore.github.io) Chang, Abner
2022-09-29  7:47     ` [edk2-devel] The principles of EDK2 module reconstruction for archs Ni, Ray
2022-09-29 14:54       ` Chang, Abner
2022-09-29 15:22         ` Sunil V L [this message]
2022-09-30  0:20           ` Chang, Abner

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=YzW4HB3GC59h+kUk@sunil-laptop \
    --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