Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
The GalagoPro3 platform in KabylakeOpenBoardPkg is using the
instance of PlatformSecLib in MinPlatformPkg instead of the
instance in KabylakeOpenBoardPkg. The version in MinPlatformPkg
does not support FSP 2.1 Dispatch Mode, whearas the version in
KabylakeOpenBoardPkg does.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Michael Kubacki <michael.a.kubacki@intel.com>
Cc: Jeremy Soller <jeremy@system76.com>
Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
---
Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc
index d67e0cc000..f3dd2b0c91 100644
--- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc
+++ b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc
@@ -85,7 +85,7 @@
PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHookLib.inf
FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf
- PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf
+ PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFspWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf
FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/BaseFspWrapperApiLib.inf
FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTestLib/PeiFspWrapperApiTestLib.inf
--
2.23.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#48362): https://edk2.groups.io/g/devel/message/48362
Mute This Topic: https://groups.io/mt/34367442/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
On 10/2/19 9:40 AM, Nate DeSimone wrote: > The GalagoPro3 platform in KabylakeOpenBoardPkg is using the > instance of PlatformSecLib in MinPlatformPkg instead of the > instance in KabylakeOpenBoardPkg. The version in MinPlatformPkg > does not support FSP 2.1 Dispatch Mode, whearas the version in typo: "whereas" > KabylakeOpenBoardPkg does. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Cc: Michael Kubacki <michael.a.kubacki@intel.com> > Cc: Jeremy Soller <jeremy@system76.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > index d67e0cc000..f3dd2b0c91 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > +++ b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > @@ -85,7 +85,7 @@ > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHookLib.inf > > FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf > - PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > + PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFspWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > > FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/BaseFspWrapperApiLib.inf > FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTestLib/PeiFspWrapperApiTestLib.inf > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48411): https://edk2.groups.io/g/devel/message/48411 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Good catch, thank you! -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe Mathieu-Daudé Sent: Thursday, October 3, 2019 12:40 AM To: devel@edk2.groups.io; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kubacki, Michael A <michael.a.kubacki@intel.com>; Jeremy Soller <jeremy@system76.com> Subject: Re: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: Wrong instance of PlatformSecLib is used. On 10/2/19 9:40 AM, Nate DeSimone wrote: > The GalagoPro3 platform in KabylakeOpenBoardPkg is using the instance > of PlatformSecLib in MinPlatformPkg instead of the instance in > KabylakeOpenBoardPkg. The version in MinPlatformPkg does not support > FSP 2.1 Dispatch Mode, whearas the version in typo: "whereas" > KabylakeOpenBoardPkg does. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Cc: Michael Kubacki <michael.a.kubacki@intel.com> > Cc: Jeremy Soller <jeremy@system76.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > index d67e0cc000..f3dd2b0c91 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > +++ b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > @@ -85,7 +85,7 @@ > > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHoo > kLib.inf > > > FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/PeiFspW > rapperHobProcessLib/PeiFspWrapperHobProcessLib.inf > - > PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapperPla > tformSecLib/SecFspWrapperPlatformSecLib.inf > + > + PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFspWr > + apperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > > FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/BaseFspWrapperApiLib.inf > > FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTestL > ib/PeiFspWrapperApiTestLib.inf > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48414): https://edk2.groups.io/g/devel/message/48414 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
With this typo correction, Reviewed-by: Chasel Chiu <chasel.chiu@intel.com> > -----Original Message----- > From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> > Sent: Thursday, October 3, 2019 3:56 PM > To: devel@edk2.groups.io; philmd@redhat.com > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kubacki, Michael A > <michael.a.kubacki@intel.com>; Jeremy Soller <jeremy@system76.com> > Subject: RE: [edk2-devel] [edk2-platforms] [PATCH V1] > KabylakeOpenBoardPkg: Wrong instance of PlatformSecLib is used. > > Good catch, thank you! > > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe > Mathieu-Daudé > Sent: Thursday, October 3, 2019 12:40 AM > To: devel@edk2.groups.io; Desimone, Nathaniel L > <nathaniel.l.desimone@intel.com> > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kubacki, Michael A > <michael.a.kubacki@intel.com>; Jeremy Soller <jeremy@system76.com> > Subject: Re: [edk2-devel] [edk2-platforms] [PATCH V1] > KabylakeOpenBoardPkg: Wrong instance of PlatformSecLib is used. > > On 10/2/19 9:40 AM, Nate DeSimone wrote: > > The GalagoPro3 platform in KabylakeOpenBoardPkg is using the instance > > of PlatformSecLib in MinPlatformPkg instead of the instance in > > KabylakeOpenBoardPkg. The version in MinPlatformPkg does not support > > FSP 2.1 Dispatch Mode, whearas the version in > > typo: "whereas" > > > KabylakeOpenBoardPkg does. > > > > Cc: Chasel Chiu <chasel.chiu@intel.com> > > Cc: Michael Kubacki <michael.a.kubacki@intel.com> > > Cc: Jeremy Soller <jeremy@system76.com> > > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > > --- > > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | > 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git > > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > > index d67e0cc000..f3dd2b0c91 100644 > > --- > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > > @@ -85,7 +85,7 @@ > > > > > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHo > o > > kLib.inf > > > > > > > FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/PeiF > spW > > rapperHobProcessLib/PeiFspWrapperHobProcessLib.inf > > - > > > PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapper > Pla > > tformSecLib/SecFspWrapperPlatformSecLib.inf > > + > > + > PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFsp > Wr > > + apperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > > > > > > > FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/Bas > e > > FspWrapperApiLib.inf > > > > > FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTestL > > ib/PeiFspWrapperApiTestLib.inf > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48527): https://edk2.groups.io/g/devel/message/48527 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Reviewed-by: Michael Kubacki <michael.a.kubacki@intel.com> > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Nate > DeSimone > Sent: Wednesday, October 2, 2019 12:41 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kubacki, Michael A > <michael.a.kubacki@intel.com>; Jeremy Soller <jeremy@system76.com> > Subject: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: > Wrong instance of PlatformSecLib is used. > > The GalagoPro3 platform in KabylakeOpenBoardPkg is using the instance of > PlatformSecLib in MinPlatformPkg instead of the instance in > KabylakeOpenBoardPkg. The version in MinPlatformPkg does not support > FSP 2.1 Dispatch Mode, whearas the version in KabylakeOpenBoardPkg does. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Cc: Michael Kubacki <michael.a.kubacki@intel.com> > Cc: Jeremy Soller <jeremy@system76.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 > +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > index d67e0cc000..f3dd2b0c91 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > @@ -85,7 +85,7 @@ > > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHo > okLib.inf > FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/P > eiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf- > PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/SecFspWrapperPlatformSecLib.inf+ > PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFs > pWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/Ba > seFspWrapperApiLib.inf > FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTest > Lib/PeiFspWrapperApiTestLib.inf-- > 2.23.0.windows.1 > > > -=-=-=-=-=-= > Groups.io Links: You receive all messages sent to this group. > > View/Reply Online (#48362): https://edk2.groups.io/g/devel/message/48362 > Mute This Topic: https://groups.io/mt/34367442/1772268 > Group Owner: devel+owner@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub > [michael.a.kubacki@intel.com] -=-=-=-=-=-= -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48409): https://edk2.groups.io/g/devel/message/48409 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Thanks Michael. Another thing we should consider doing a some point in the future is moving the PlatformSecLib implementation that supports dispatch mode from KabylakeOpenBoardPkg to MinPlatformPkg. -----Original Message----- From: Kubacki, Michael A <michael.a.kubacki@intel.com> Sent: Wednesday, October 2, 2019 6:35 PM To: devel@edk2.groups.io; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Jeremy Soller <jeremy@system76.com> Subject: RE: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: Wrong instance of PlatformSecLib is used. Reviewed-by: Michael Kubacki <michael.a.kubacki@intel.com> > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Nate > DeSimone > Sent: Wednesday, October 2, 2019 12:41 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kubacki, Michael A > <michael.a.kubacki@intel.com>; Jeremy Soller <jeremy@system76.com> > Subject: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: > Wrong instance of PlatformSecLib is used. > > The GalagoPro3 platform in KabylakeOpenBoardPkg is using the instance > of PlatformSecLib in MinPlatformPkg instead of the instance in > KabylakeOpenBoardPkg. The version in MinPlatformPkg does not support > FSP 2.1 Dispatch Mode, whearas the version in KabylakeOpenBoardPkg does. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Cc: Michael Kubacki <michael.a.kubacki@intel.com> > Cc: Jeremy Soller <jeremy@system76.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc | 2 > +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > index d67e0cc000..f3dd2b0c91 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > @@ -85,7 +85,7 @@ > > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHo > okLib.inf > FspWrapperHobProcessLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/P > eiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf- > PlatformSecLib|$(PLATFORM_PACKAGE)/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/SecFspWrapperPlatformSecLib.inf+ > PlatformSecLib|$(PLATFORM_BOARD_PACKAGE)/FspWrapper/Library/SecFs > pWrapperPlatformSecLib/SecFspWrapperPlatformSecLib.inf > FspWrapperApiLib|IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/Ba > seFspWrapperApiLib.inf > FspWrapperApiTestLib|IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTest > Lib/PeiFspWrapperApiTestLib.inf-- > 2.23.0.windows.1 > > > -=-=-=-=-=-= > Groups.io Links: You receive all messages sent to this group. > > View/Reply Online (#48362): > https://edk2.groups.io/g/devel/message/48362 > Mute This Topic: https://groups.io/mt/34367442/1772268 > Group Owner: devel+owner@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub > [michael.a.kubacki@intel.com] -=-=-=-=-=-= -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48413): https://edk2.groups.io/g/devel/message/48413 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Reviewed-by: Jeremy Soller <jeremy@system76.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48398): https://edk2.groups.io/g/devel/message/48398 Mute This Topic: https://groups.io/mt/34367442/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.