Hi Ray,
The function DetectAndPreparePlatformPciDevicePath is the second parameter of 
VisitAllInstancesOfProtocol.
It follows the below type:
typedef
EFI_STATUS
(EFIAPI *PROTOCOL_INSTANCE_CALLBACK)(
  IN EFI_HANDLE            Handle,
  IN VOID                 *Instance,
  IN VOID                 *Context
  );

The same function pointer type is also defined in OvmfPkg.
I didn't change the function pointer type to avoid same type having different 
definition in edk2 repo.
Do I need to consider that? What's your suggestion?

Thanks
Zhiguang


-----Original Message-----
From: Ni, Ray <ray...@intel.com> 
Sent: Tuesday, May 10, 2022 3:39 PM
To: Liu, Zhiguang <zhiguang....@intel.com>; devel@edk2.groups.io
Cc: Dong, Guo <guo.d...@intel.com>; Maurice Ma <maurice...@intel.com>; You, 
Benjamin <benjamin....@intel.com>; Rhodes, Sean <sean@starlabs.systems>
Subject: RE: [PATCH 1/3] UefiPayloadPkg: Simplify code logic

> 
> +DetectAndPreparePlatformPciDevicePath (
> 
>    IN EFI_HANDLE  Handle,
> 
>    IN VOID        *Instance,
> 
>    IN VOID        *Context

Is "Context" needed? Can you please remove it?



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89672): https://edk2.groups.io/g/devel/message/89672
Mute This Topic: https://groups.io/mt/91007797/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to