[edk2-devel] Reviewer for the possible duplicated CSM components in OvmfPkg

[Wu, Hao A]

> -----Original Message-----
> From: David Woodhouse [mailto:dwmw2 at infradead.org]
> Sent: Monday, May 20, 2019 1:11 PM
> To: Wu, Hao A; devel at edk2.groups.io
> Cc: Laszlo Ersek; Ni, Ray; Justen, Jordan L; Ard Biesheuvel
> Subject: Re: Reviewer for the possible duplicated CSM components in OvmfPkg
> 
> On Mon, 2019-05-20 at 05:08 +0000, Wu, Hao A wrote:
> > Hello David,
> >
> > According to the request in a previous mail discussion thread
> > 'Drop CSM support in OvmfPkg?':
> >
> > https://edk2.groups.io/g/devel/topic/30916686#35723
> >
> > We plan to duplicate the required CSM modules into the OvmfPkg/Csm folder,
> > so that those modules will be kept within the edk2 repo.
> >
> > Meanwhile, Laszlo suggested me to first figure out the ownership (review
> > duty) for these CSM components before such duplication:
> >
> > https://edk2.groups.io/g/devel/topic/31627718#40659
> >
> > Would you like to take such role to be the reviewer for the CSM components
> > that will be copied into the OvmfPkg?
> >
> > Thanks in advance.
> 
> Yes, I believe I have sufficient motivation to spend time on that am I
> am willing to do so. My only caveat is that I haven't managed to

Thanks.

> reproduce the working CSM situation that I had when I first implemented
> this all; even by going back to those versions of OVMF + SeaBIOS.
> 
> Lazslo has pointed me at some potential reasons for that, but I haven't
> yet managed to chase them up.

My thought on this is the CSM components in the framework packages are
stable for a period of time. So my guess is that the issue is not directly
related to these components.

Also, if it turns out that there are some missing components left
uncopied, we are able to get them back (from the repo history) and put
them into OvmfPkg then. I can help on that.

Does this sound good to you?

Best Regards,
Hao Wu

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#40995): https://edk2.groups.io/g/devel/message/40995
Mute This Topic: https://groups.io/mt/31682287/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-