Re: [PATCH v2 0/6] Ovmf: Drop IntelFramework[Module]Pkg dependency

["Wu, Hao A" <hao.a.wu@intel.com>]

> -----Original Message-----
> From: Ni, Ray
> Sent: Wednesday, June 12, 2019 10:04 AM
> To: Wu, Hao A; David Woodhouse; Justen, Jordan L; devel@edk2.groups.io
> Cc: Laszlo Ersek; Ard Biesheuvel; Phillips, D Scott
> Subject: RE: [PATCH v2 0/6] Ovmf: Drop IntelFramework[Module]Pkg
> dependency
> 
> Hao,
> Will the CSM duplication cause any code change that may impact CSM
> functionality?

Hello Ray,

I think there should be no functional impact for the duplication.
There is no change to the .C/.H files.

Best Regards,
Hao Wu

> If no, how about firstly duplicate them first?
> 
> David,
> Will this approach work for you?
> 
> Thanks,
> Ray
> 
> > -----Original Message-----
> > From: Wu, Hao A <hao.a.wu@intel.com>
> > Sent: Wednesday, June 12, 2019 9:19 AM
> > To: David Woodhouse <dwmw2@infradead.org>; Justen, Jordan L
> > <jordan.l.justen@intel.com>; devel@edk2.groups.io
> > Cc: Ni, Ray <ray.ni@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ard
> > Biesheuvel <ard.biesheuvel@linaro.org>; Phillips, D Scott
> > <d.scott.phillips@intel.com>
> > Subject: RE: [PATCH v2 0/6] Ovmf: Drop IntelFramework[Module]Pkg
> > dependency
> >
> > > -----Original Message-----
> > > From: Wu, Hao A
> > > Sent: Tuesday, June 11, 2019 4:06 PM
> > > To: 'David Woodhouse'; Justen, Jordan L; devel@edk2.groups.io
> > > Cc: Ni, Ray; Laszlo Ersek; Ard Biesheuvel; Phillips, D Scott
> > > Subject: RE: [PATCH v2 0/6] Ovmf: Drop IntelFramework[Module]Pkg
> > > dependency
> > >
> > > > -----Original Message-----
> > > > From: David Woodhouse [mailto:dwmw2@infradead.org]
> > > > Sent: Tuesday, June 11, 2019 4:01 PM
> > > > To: Wu, Hao A; Justen, Jordan L; devel@edk2.groups.io
> > > > Cc: Ni, Ray; Laszlo Ersek; Ard Biesheuvel; Phillips, D Scott
> > > > Subject: Re: [PATCH v2 0/6] Ovmf: Drop IntelFramework[Module]Pkg
> > > > dependency
> > > >
> > > > On Tue, 2019-06-11 at 07:49 +0000, Wu, Hao A wrote:
> > > > > Hello all,
> > > > >
> > > > > For the series:
> > > > > [PATCH v2 00/10] Duplicate required CSM components for OVMF
> > > > > https://edk2.groups.io/g/devel/message/41385
> > > > >
> > > > > I sent it out on May 27th, and did not receive a confirmation from
> > > > > David for about 2 weeks. I think maybe the CSM support on OVMF is
> > > > > no longer cared, and thus I sent another series today to drop such
> > support:
> > > > > [PATCH v1 0/2] OVMF: Drop CSM support
> > > > > https://edk2.groups.io/g/devel/message/42142
> > > > >
> > > > > Please help to provide some feedbacks/confirmation on what should
> > > > > I do
> > > > for
> > > > > the next step. (This task actually blocks our proposal to remove
> > > > > IntelFramework[Module]Pkg from edk2.)
> > > >
> > > >
> > > > Apologies for the delay. It wasn't clear that the first series was
> > > > actually tested by booting with a CSM, and I was reluctant to ack it
> > > > without knowing that it didn't cause a regression.
> > >
> > > Agree, I understand that.
> > >
> > > >
> > > > This is, of course, complicated by the fact that I can't actually
> > > > get a CSM boot to complete at the moment. But it gets a long way
> > > > into the boot process before something goes wrong, so confirmation
> > > > that it still gets at least that far with your 'Duplicate required CSM
> > components'
> > > > series would be good.
> > > >
> > > > Is there a git tree I can pull from? I seem to recall that last time
> > > > I spent serious time working on OVMF, life was horribly complicated
> > > > by CRLF line endings, and it was extremely painful to just apply
> > > > patches from email... is that still the case?
> > >
> > > Yes, you can get the first series at:
> > > https://github.com/hwu25/edk2/commits/ovmf_csm_v2
> >
> > Hello David,
> >
> > May I know how long will it take for your verification and acknowledging
> the
> > series that duplicate the required CSM modules under OvmfPkg?
> >
> > Thanks in advance.
> >
> > Best Regards,
> > Hao Wu
> >
> > >
> > > Best Regards,
> > > Hao Wu
> > >
> > > >