[edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[Ni, Ray]

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1819

Since there are V2 FSP packages (IntelFsp2Pkg, IntelFsp2WrapperPkg),
this patch removes IntelFspPkg, IntelFspWrapperPkg to remove
obsolete code in edk2 repo.

Signed-off-by: Ray Ni <ray.ni@intel.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
--------------
NOTE: This patch is to completely remove IntelFspPkg and
IntelFspWrapperPkg folder in edk2 repo.
I don't want to flood people's inbox with a big patch just
removing every line of code in this two packages.

The patch that modifies Maintainers.txt will be sent out later
after IntelFspPkg and IntelFspWrapperPkg are removed.

Re: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[Chiu, Chasel]

Hi Nate,

Last time we talked, IntelFspPkg can be removed but IntelFspWrapperPkg might be still needed, would you please check if this is still true?
With Nate's confirmation, Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>

Thanks!
Chasel

> -----Original Message-----
> From: Ni, Ray
> Sent: Thursday, May 16, 2019 4:38 PM
> To: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Cc: devel@edk2.groups.io
> Subject: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1819
> 
> Since there are V2 FSP packages (IntelFsp2Pkg, IntelFsp2WrapperPkg), this patch
> removes IntelFspPkg, IntelFspWrapperPkg to remove obsolete code in edk2
> repo.
> 
> Signed-off-by: Ray Ni <ray.ni@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> --------------
> NOTE: This patch is to completely remove IntelFspPkg and IntelFspWrapperPkg
> folder in edk2 repo.
> I don't want to flood people's inbox with a big patch just removing every line of
> code in this two packages.
> 
> The patch that modifies Maintainers.txt will be sent out later after IntelFspPkg
> and IntelFspWrapperPkg are removed.

Re: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[Nate DeSimone]

Please commit Chasel's fix for https://edk2.groups.io/g/devel/topic/31860753#41659

Once Chasel's fix is committed...

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>

-----Original Message-----
From: Ni, Ray 
Sent: Thursday, May 16, 2019 1:38 AM
To: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
Cc: devel@edk2.groups.io
Subject: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1819

Since there are V2 FSP packages (IntelFsp2Pkg, IntelFsp2WrapperPkg), this patch removes IntelFspPkg, IntelFspWrapperPkg to remove obsolete code in edk2 repo.

Signed-off-by: Ray Ni <ray.ni@intel.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
--------------
NOTE: This patch is to completely remove IntelFspPkg and IntelFspWrapperPkg folder in edk2 repo.
I don't want to flood people's inbox with a big patch just removing every line of code in this two packages.

The patch that modifies Maintainers.txt will be sent out later after IntelFspPkg and IntelFspWrapperPkg are removed.

Re: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[Chiu, Chasel]

Just committed this fix.
https://edk2.groups.io/g/devel/topic/31860753#41659

Thanks!
Chasel


> -----Original Message-----
> From: Desimone, Nathaniel L
> Sent: Thursday, June 6, 2019 1:59 PM
> To: Ni, Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Zeng, Star
> <star.zeng@intel.com>
> Cc: devel@edk2.groups.io
> Subject: RE: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove
> them
> 
> Please commit Chasel's fix for
> https://edk2.groups.io/g/devel/topic/31860753#41659
> 
> Once Chasel's fix is committed...
> 
> Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
> 
> -----Original Message-----
> From: Ni, Ray
> Sent: Thursday, May 16, 2019 1:38 AM
> To: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Cc: devel@edk2.groups.io
> Subject: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1819
> 
> Since there are V2 FSP packages (IntelFsp2Pkg, IntelFsp2WrapperPkg), this patch
> removes IntelFspPkg, IntelFspWrapperPkg to remove obsolete code in edk2
> repo.
> 
> Signed-off-by: Ray Ni <ray.ni@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> --------------
> NOTE: This patch is to completely remove IntelFspPkg and IntelFspWrapperPkg
> folder in edk2 repo.
> I don't want to flood people's inbox with a big patch just removing every line of
> code in this two packages.
> 
> The patch that modifies Maintainers.txt will be sent out later after IntelFspPkg
> and IntelFspWrapperPkg are removed.

Re: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove them

[Ni, Ray]

Chasel, Nate,
Thanks. I will remove the two packages ASAP.

> -----Original Message-----
> From: Chiu, Chasel <chasel.chiu@intel.com>
> Sent: Tuesday, June 11, 2019 3:19 PM
> To: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Ni, Ray
> <ray.ni@intel.com>; Zeng, Star <star.zeng@intel.com>
> Cc: devel@edk2.groups.io
> Subject: RE: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove
> them
> 
> 
> Just committed this fix.
> https://edk2.groups.io/g/devel/topic/31860753#41659
> 
> Thanks!
> Chasel
> 
> 
> > -----Original Message-----
> > From: Desimone, Nathaniel L
> > Sent: Thursday, June 6, 2019 1:59 PM
> > To: Ni, Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>;
> > Zeng, Star <star.zeng@intel.com>
> > Cc: devel@edk2.groups.io
> > Subject: RE: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg:
> > Remove them
> >
> > Please commit Chasel's fix for
> > https://edk2.groups.io/g/devel/topic/31860753#41659
> >
> > Once Chasel's fix is committed...
> >
> > Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
> >
> > -----Original Message-----
> > From: Ni, Ray
> > Sent: Thursday, May 16, 2019 1:38 AM
> > To: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> > Cc: devel@edk2.groups.io
> > Subject: [edk2-devel][Patch] IntelFspPkg&IntelFspWrapperPkg: Remove
> > them
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1819
> >
> > Since there are V2 FSP packages (IntelFsp2Pkg, IntelFsp2WrapperPkg),
> > this patch removes IntelFspPkg, IntelFspWrapperPkg to remove obsolete
> > code in edk2 repo.
> >
> > Signed-off-by: Ray Ni <ray.ni@intel.com>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Star Zeng <star.zeng@intel.com>
> > --------------
> > NOTE: This patch is to completely remove IntelFspPkg and
> > IntelFspWrapperPkg folder in edk2 repo.
> > I don't want to flood people's inbox with a big patch just removing
> > every line of code in this two packages.
> >
> > The patch that modifies Maintainers.txt will be sent out later after
> > IntelFspPkg and IntelFspWrapperPkg are removed.