Hi Sunil
I did not find any RiscV64QemuVirt related PR there. (I searched 
RiscV64QemuVirt)

May I know if you have tried the patch series in EDK2 CI?

Thank you
Yao, Jiewen

> -----Original Message-----
> From: Yao, Jiewen
> Sent: Thursday, February 9, 2023 1:07 PM
> To: Sunil V L <suni...@ventanamicro.com>; devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kin...@intel.com>; abner.ch...@amd.com;
> Andrew Fish <af...@apple.com>; Leif Lindholm <quic_llind...@quicinc.com>
> Subject: RE: [edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 20/20]
> Maintainers.txt: Add entry for OvmfPkg/RiscVVirt
> 
> Thanks.
> 
> I agree to add Sunil V L <suni...@ventanamicro.com> to be a reviewer.
> Reviewed-by: Jiewen Yao <jiewen....@intel.com>
> 
> > -----Original Message-----
> > From: Sunil V L <suni...@ventanamicro.com>
> > Sent: Thursday, February 9, 2023 12:34 PM
> > To: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com>
> > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; abner.ch...@amd.com;
> > Andrew Fish <af...@apple.com>; Leif Lindholm <quic_llind...@quicinc.com>
> > Subject: Re: [edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 20/20]
> > Maintainers.txt: Add entry for OvmfPkg/RiscVVirt
> >
> > Hi Jiewen,
> >
> > On Thu, Feb 09, 2023 at 03:32:20AM +0000, Yao, Jiewen wrote:
> > > Hello
> > > I have not tracked all discussion. Sorry for that.
> > > May I know if we have discussed the option to create RiscVVirtPkg ? 
> > > similar
> to
> > https://github.com/tianocore/edk2/tree/master/ArmVirtPkg . To me, that is
> > more natural place.
> > >
> >
> > Thanks for the feedback. Yes, this was discussed and recommendation was
> > to add in OvmfPkg instead of creating RiscVVirtPkg..
> > https://edk2.groups.io/g/devel/message/85932
> >
> > >
> > > For the new content on RiscVVirt, I recommend to add one more
> > reviewer/maintainer.
> > > I notice the patches are Acked-by: Abner Chang. Can we get his help on 
> > > that?
> > >
> >
> > Agree, that was in my plan to find out some one. Abner had given up
> > RISC-V mandatory roles. He was helping me here since he was the original
> > to start this. So, I didn't add him but I will be happy if Abner agrees.
> >
> > Will find out and add one more reviewer when I revise the series.
> >
> > Thanks!
> > Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99897): https://edk2.groups.io/g/devel/message/99897
Mute This Topic: https://groups.io/mt/96593524/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to