[edk2-devel] [PATCH v2] MdePkg: Define the DevicePath argument from LoadImage as optional

ManickamX Srinivasan posted 1 patch 4 months, 3 weeks ago
Failed in applying to current master (apply log)
MdePkg/Include/Uefi/UefiSpec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[edk2-devel] [PATCH v2] MdePkg: Define the DevicePath argument from LoadImage as optional
Posted by ManickamX Srinivasan 4 months, 3 weeks ago
Update the EFI LoadImage API in accordance with the
UEFI v2.10 specification.

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
---
 MdePkg/Include/Uefi/UefiSpec.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h
index 7dfe35b499..e83e14d347 100644
--- a/MdePkg/Include/Uefi/UefiSpec.h
+++ b/MdePkg/Include/Uefi/UefiSpec.h
@@ -898,7 +898,7 @@ EFI_STATUS
 (EFIAPI *EFI_IMAGE_LOAD)(
   IN  BOOLEAN                      BootPolicy,
   IN  EFI_HANDLE                   ParentImageHandle,
-  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath,
+  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath   OPTIONAL,
   IN  VOID                         *SourceBuffer OPTIONAL,
   IN  UINTN                        SourceSize,
   OUT EFI_HANDLE                   *ImageHandle
-- 
2.30.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112209): https://edk2.groups.io/g/devel/message/112209
Mute This Topic: https://groups.io/mt/103050250/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] MdePkg: Define the DevicePath argument from LoadImage as optional
Posted by Michael D Kinney 4 months, 3 weeks ago
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>
> Sent: Thursday, December 7, 2023 9:41 PM
> To: devel@edk2.groups.io
> Cc: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; T V,
> Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Subject: [PATCH v2] MdePkg: Define the DevicePath argument from
> LoadImage as optional
> 
> Update the EFI LoadImage API in accordance with the
> UEFI v2.10 specification.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> ---
>  MdePkg/Include/Uefi/UefiSpec.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> b/MdePkg/Include/Uefi/UefiSpec.h
> index 7dfe35b499..e83e14d347 100644
> --- a/MdePkg/Include/Uefi/UefiSpec.h
> +++ b/MdePkg/Include/Uefi/UefiSpec.h
> @@ -898,7 +898,7 @@ EFI_STATUS
>  (EFIAPI *EFI_IMAGE_LOAD)(
>    IN  BOOLEAN                      BootPolicy,
>    IN  EFI_HANDLE                   ParentImageHandle,
> -  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath,
> +  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath   OPTIONAL,
>    IN  VOID                         *SourceBuffer OPTIONAL,
>    IN  UINTN                        SourceSize,
>    OUT EFI_HANDLE                   *ImageHandle
> --
> 2.30.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112239): https://edk2.groups.io/g/devel/message/112239
Mute This Topic: https://groups.io/mt/103050250/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] MdePkg: Define the DevicePath argument from LoadImage as optional
Posted by Michael D Kinney 4 months, 3 weeks ago
Merged: https://github.com/tianocore/edk2/pull/5129


> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Friday, December 8, 2023 12:24 PM
> To: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>;
> devel@edk2.groups.io
> Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> <zhiguang.liu@intel.com>; T V, Krishnamoorthy
> <krishnamoorthy.t.v@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: RE: [PATCH v2] MdePkg: Define the DevicePath argument from
> LoadImage as optional
> 
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> > -----Original Message-----
> > From: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>
> > Sent: Thursday, December 7, 2023 9:41 PM
> > To: devel@edk2.groups.io
> > Cc: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>; Kinney,
> > Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; T
> V,
> > Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> > Subject: [PATCH v2] MdePkg: Define the DevicePath argument from
> > LoadImage as optional
> >
> > Update the EFI LoadImage API in accordance with the
> > UEFI v2.10 specification.
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> > Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> > ---
> >  MdePkg/Include/Uefi/UefiSpec.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> > b/MdePkg/Include/Uefi/UefiSpec.h
> > index 7dfe35b499..e83e14d347 100644
> > --- a/MdePkg/Include/Uefi/UefiSpec.h
> > +++ b/MdePkg/Include/Uefi/UefiSpec.h
> > @@ -898,7 +898,7 @@ EFI_STATUS
> >  (EFIAPI *EFI_IMAGE_LOAD)(
> >    IN  BOOLEAN                      BootPolicy,
> >    IN  EFI_HANDLE                   ParentImageHandle,
> > -  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath,
> > +  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath   OPTIONAL,
> >    IN  VOID                         *SourceBuffer OPTIONAL,
> >    IN  UINTN                        SourceSize,
> >    OUT EFI_HANDLE                   *ImageHandle
> > --
> > 2.30.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112240): https://edk2.groups.io/g/devel/message/112240
Mute This Topic: https://groups.io/mt/103050250/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] MdePkg: Define the DevicePath argument from LoadImage as optional
Posted by Zhiguang Liu 4 months, 3 weeks ago
Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>

> -----Original Message-----
> From: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>
> Sent: Friday, December 8, 2023 1:41 PM
> To: devel@edk2.groups.io
> Cc: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; T V,
> Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Subject: [PATCH v2] MdePkg: Define the DevicePath argument from
> LoadImage as optional
> 
> Update the EFI LoadImage API in accordance with the UEFI v2.10
> specification.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> ---
>  MdePkg/Include/Uefi/UefiSpec.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> b/MdePkg/Include/Uefi/UefiSpec.h index 7dfe35b499..e83e14d347 100644
> --- a/MdePkg/Include/Uefi/UefiSpec.h
> +++ b/MdePkg/Include/Uefi/UefiSpec.h
> @@ -898,7 +898,7 @@ EFI_STATUS
>  (EFIAPI *EFI_IMAGE_LOAD)(
>    IN  BOOLEAN                      BootPolicy,
>    IN  EFI_HANDLE                   ParentImageHandle,
> -  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath,
> +  IN  EFI_DEVICE_PATH_PROTOCOL     *DevicePath   OPTIONAL,
>    IN  VOID                         *SourceBuffer OPTIONAL,
>    IN  UINTN                        SourceSize,
>    OUT EFI_HANDLE                   *ImageHandle
> --
> 2.30.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112210): https://edk2.groups.io/g/devel/message/112210
Mute This Topic: https://groups.io/mt/103050250/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-