[edk2-devel] [PATCH v2 0/3] Fix warning message issues

Zhang, Shenglei posted 3 patches 1 week ago
Failed in applying to current master (apply log)
IntelFsp2Pkg/FspSecCore/FspSecCoreS.inf                  | 3 ++-
MdeModulePkg/Universal/CapsulePei/CapsulePei.inf         | 3 ++-
MdeModulePkg/Universal/CapsulePei/CapsuleX64.inf         | 1 +
MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf      | 9 ++++++++-
.../UefiShellAcpiViewCommandLib.inf                      | 2 ++
5 files changed, 15 insertions(+), 3 deletions(-)

[edk2-devel] [PATCH v2 0/3] Fix warning message issues

Posted by Zhang, Shenglei 1 week ago
There are some header files used but not included in INF
files. This causes warings are generated when building the
packages. So now add them into INF files.
https://bugzilla.tianocore.org/show_bug.cgi?id=2036

Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jaben Carsey <jaben.carsey@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Shenglei Zhang (3):
  MdeModulePkg: Add missing header files in INF files
  ShellPkg/UefiShellAcpiViewCommandLib: Add missing header files in INF
  IntelFsp2Pkg/FspSecCore: Add missing header file in INF file

 IntelFsp2Pkg/FspSecCore/FspSecCoreS.inf                  | 3 ++-
 MdeModulePkg/Universal/CapsulePei/CapsulePei.inf         | 3 ++-
 MdeModulePkg/Universal/CapsulePei/CapsuleX64.inf         | 1 +
 MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf      | 9 ++++++++-
 .../UefiShellAcpiViewCommandLib.inf                      | 2 ++
 5 files changed, 15 insertions(+), 3 deletions(-)

-- 
2.18.0.windows.1


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

View/Reply Online (#45397): https://edk2.groups.io/g/devel/message/45397
Mute This Topic: https://groups.io/mt/32838102/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/3] Fix warning message issues

Posted by Carsey, Jaben 1 week ago
 What changed in v2?

Thanks
-Jaben


> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Sunday, August 11, 2019 11:23 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>; Carsey, Jaben <jaben.carsey@intel.com>; Ni, Ray
> <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Chiu, Chasel
> <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [PATCH v2 0/3] Fix warning message issues
> 
> There are some header files used but not included in INF
> files. This causes warings are generated when building the
> packages. So now add them into INF files.
> https://bugzilla.tianocore.org/show_bug.cgi?id=2036
> 
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Jaben Carsey <jaben.carsey@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Shenglei Zhang (3):
>   MdeModulePkg: Add missing header files in INF files
>   ShellPkg/UefiShellAcpiViewCommandLib: Add missing header files in INF
>   IntelFsp2Pkg/FspSecCore: Add missing header file in INF file
> 
>  IntelFsp2Pkg/FspSecCore/FspSecCoreS.inf                  | 3 ++-
>  MdeModulePkg/Universal/CapsulePei/CapsulePei.inf         | 3 ++-
>  MdeModulePkg/Universal/CapsulePei/CapsuleX64.inf         | 1 +
>  MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf      | 9 ++++++++-
>  .../UefiShellAcpiViewCommandLib.inf                      | 2 ++
>  5 files changed, 15 insertions(+), 3 deletions(-)
> 
> --
> 2.18.0.windows.1


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

View/Reply Online (#45455): https://edk2.groups.io/g/devel/message/45455
Mute This Topic: https://groups.io/mt/32838102/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/3] Fix warning message issues

Posted by Zhang, Shenglei 1 week ago
Jaben:

I add the v2 message below "---". And here is the link for v2 patch.
https://edk2.groups.io/g/devel/message/45398?p=,,,20,0,0,0::Created,,MdeModulePkg%3A+Add+missing+header+files+in+INF+files,20,2,0,32838103

Thanks,
Shenglei

> -----Original Message-----
> From: Carsey, Jaben
> Sent: Monday, August 12, 2019 10:49 PM
> To: Zhang, Shenglei <shenglei.zhang@intel.com>; devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>;
> Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Chiu,
> Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: RE: [PATCH v2 0/3] Fix warning message issues
> 
>  What changed in v2?
> 
> Thanks
> -Jaben
> 
> 
> > -----Original Message-----
> > From: Zhang, Shenglei
> > Sent: Sunday, August 11, 2019 11:23 PM
> > To: devel@edk2.groups.io
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> > <hao.a.wu@intel.com>; Carsey, Jaben <jaben.carsey@intel.com>; Ni, Ray
> > <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Chiu, Chasel
> > <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> > Subject: [PATCH v2 0/3] Fix warning message issues
> >
> > There are some header files used but not included in INF
> > files. This causes warings are generated when building the
> > packages. So now add them into INF files.
> > https://bugzilla.tianocore.org/show_bug.cgi?id=2036
> >
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Hao A Wu <hao.a.wu@intel.com>
> > Cc: Jaben Carsey <jaben.carsey@intel.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Zhichao Gao <zhichao.gao@intel.com>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Star Zeng <star.zeng@intel.com>
> > Shenglei Zhang (3):
> >   MdeModulePkg: Add missing header files in INF files
> >   ShellPkg/UefiShellAcpiViewCommandLib: Add missing header files in INF
> >   IntelFsp2Pkg/FspSecCore: Add missing header file in INF file
> >
> >  IntelFsp2Pkg/FspSecCore/FspSecCoreS.inf                  | 3 ++-
> >  MdeModulePkg/Universal/CapsulePei/CapsulePei.inf         | 3 ++-
> >  MdeModulePkg/Universal/CapsulePei/CapsuleX64.inf         | 1 +
> >  MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf      | 9
> ++++++++-
> >  .../UefiShellAcpiViewCommandLib.inf                      | 2 ++
> >  5 files changed, 15 insertions(+), 3 deletions(-)
> >
> > --
> > 2.18.0.windows.1


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

View/Reply Online (#45469): https://edk2.groups.io/g/devel/message/45469
Mute This Topic: https://groups.io/mt/32838102/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/3] Fix warning message issues

Posted by Carsey, Jaben 1 week ago
Thanks. I had not checked each of the patches in the set, only the one CC'd to me and this one.

Thanks
-Jaben


> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Monday, August 12, 2019 5:53 PM
> To: Carsey, Jaben <jaben.carsey@intel.com>; devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>; Ni, Ray <ray.ni@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>
> Subject: RE: [PATCH v2 0/3] Fix warning message issues
> 
> Jaben:
> 
> I add the v2 message below "---". And here is the link for v2 patch.
> https://edk2.groups.io/g/devel/message/45398?p=,,,20,0,0,0::Created,,Mde
> ModulePkg%3A+Add+missing+header+files+in+INF+files,20,2,0,32838103
> 
> Thanks,
> Shenglei
> 
> > -----Original Message-----
> > From: Carsey, Jaben
> > Sent: Monday, August 12, 2019 10:49 PM
> > To: Zhang, Shenglei <shenglei.zhang@intel.com>; devel@edk2.groups.io
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>;
> > Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Chiu,
> > Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> > Subject: RE: [PATCH v2 0/3] Fix warning message issues
> >
> >  What changed in v2?
> >
> > Thanks
> > -Jaben
> >
> >
> > > -----Original Message-----
> > > From: Zhang, Shenglei
> > > Sent: Sunday, August 11, 2019 11:23 PM
> > > To: devel@edk2.groups.io
> > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> > > <hao.a.wu@intel.com>; Carsey, Jaben <jaben.carsey@intel.com>; Ni,
> Ray
> > > <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Chiu, Chasel
> > > <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> > > Subject: [PATCH v2 0/3] Fix warning message issues
> > >
> > > There are some header files used but not included in INF
> > > files. This causes warings are generated when building the
> > > packages. So now add them into INF files.
> > > https://bugzilla.tianocore.org/show_bug.cgi?id=2036
> > >
> > > Cc: Jian J Wang <jian.j.wang@intel.com>
> > > Cc: Hao A Wu <hao.a.wu@intel.com>
> > > Cc: Jaben Carsey <jaben.carsey@intel.com>
> > > Cc: Ray Ni <ray.ni@intel.com>
> > > Cc: Zhichao Gao <zhichao.gao@intel.com>
> > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > Cc: Star Zeng <star.zeng@intel.com>
> > > Shenglei Zhang (3):
> > >   MdeModulePkg: Add missing header files in INF files
> > >   ShellPkg/UefiShellAcpiViewCommandLib: Add missing header files in INF
> > >   IntelFsp2Pkg/FspSecCore: Add missing header file in INF file
> > >
> > >  IntelFsp2Pkg/FspSecCore/FspSecCoreS.inf                  | 3 ++-
> > >  MdeModulePkg/Universal/CapsulePei/CapsulePei.inf         | 3 ++-
> > >  MdeModulePkg/Universal/CapsulePei/CapsuleX64.inf         | 1 +
> > >  MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf      | 9
> > ++++++++-
> > >  .../UefiShellAcpiViewCommandLib.inf                      | 2 ++
> > >  5 files changed, 15 insertions(+), 3 deletions(-)
> > >
> > > --
> > > 2.18.0.windows.1


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

View/Reply Online (#45474): https://edk2.groups.io/g/devel/message/45474
Mute This Topic: https://groups.io/mt/32838102/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-