From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web12.6943.1589177565702954198 for ; Sun, 10 May 2020 23:12:45 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=P+WNnEAe; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: zhiguang.liu@intel.com) IronPort-SDR: RTEACrRxK9fKVnGDC/ph55KRfo4NpO1+iWDfQLI3UW3+Z1YVFHG0cvu2leFuYHfxjN7gdP/jQv sVxRVemlWNyA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2020 23:12:44 -0700 IronPort-SDR: 31vyd1d4BsfIE3b3t23RCQzE7SG6TjR8204ME7pUYXoY4JTP0AUicT2MFUaGVCIxSf7D3xeZOZ 2Lvap+RBrjYg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,378,1583222400"; d="scan'208";a="261662990" Received: from orsmsx105.amr.corp.intel.com ([10.22.225.132]) by orsmga003.jf.intel.com with ESMTP; 10 May 2020 23:12:44 -0700 Received: from orsmsx114.amr.corp.intel.com (10.22.240.10) by ORSMSX105.amr.corp.intel.com (10.22.225.132) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 10 May 2020 23:12:44 -0700 Received: from ORSEDG001.ED.cps.intel.com (10.7.248.4) by ORSMSX114.amr.corp.intel.com (10.22.240.10) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 10 May 2020 23:12:44 -0700 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.176) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 10 May 2020 23:12:44 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Zp67jB9Phsvw+oWOvf+IRBVCRMXcKli7Yth7S4JmN+bokyOfv74dM0yB5cJTrv+bCyvHM0wgNPBcFL9EVqieQh1A9HN2gcgxB9sOMaiNF9YRhxUZRJxgYso5lsa9TW6L1uru/QwEBmFmvsmGs/gY/FVw5zqQiqldhwJc+X6Uu1qu3ShqORC8NNM+UYiwTY1e0utZ2k1iDCIzVsyqrWh/F3uuoKtuV1qmEFLqNuGMTxbTDoLHKso9vkit6bs4yA5sCGoNBDGzSowDU+8w9ZZWVuwYGnIcJCgVx0fUivfC8ADwbIh8Bmp1cfcQCk1UPCzAtRf+K7s27LaBq5nQQ52ifA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=flX7mgkA/i8kZwoDEjLH4z7Y5T5QhiCuHeNP1YmlFa4=; b=iJ7+RXMUOHLnFjzhv0B2vuDGhMHaUExsbREl3Bwx/13dIXyKMV0Fm+V5lLL2gNnYllk1UhQfkcgqzFI33l9EY2QJkuDUhy1Nnvnu/Y8VVpIazUMeMNsLHADgcxWu0xQLBu9DwfGY5BPATxl5+nT8fzzcaSuInCs8+PIlcwu8ychNp0B2M1hYKRMBq8IaCFLA8fTsKs8evFjQNavGxDw+/6yBtO0rW9m54bjcaSFCmj47WLVVgUxiX71V4pEFLQV8Tl0j61WW1kyNrSU3bgr1UI8GpM7y8KCIftcD29RlgZolV2hENkEO6Dc+0KyflKlzleIjnR0ebFZPMRbwBP12ZQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=flX7mgkA/i8kZwoDEjLH4z7Y5T5QhiCuHeNP1YmlFa4=; b=P+WNnEAemt8j3M0g9x6bm3PceV/SDBwnDNj8GrtvHb8AwiG0887GFTuUZ98nQnOQCdDMjlV4ezJLwU5hWka9xLdwo8Dbdeay9VBLCgsnpBCUYwZOP5pU1r5QN3ttkfGP7juA8T92CNxh7Ju+5n6Fj6WsWuJzUSLm5KhEsHi73FQ= Received: from BN7PR11MB2804.namprd11.prod.outlook.com (2603:10b6:406:ac::12) by BN7PR11MB2628.namprd11.prod.outlook.com (2603:10b6:406:b2::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2958.29; Mon, 11 May 2020 06:12:42 +0000 Received: from BN7PR11MB2804.namprd11.prod.outlook.com ([fe80::79e2:ed1f:4d6b:532c]) by BN7PR11MB2804.namprd11.prod.outlook.com ([fe80::79e2:ed1f:4d6b:532c%7]) with mapi id 15.20.2979.033; Mon, 11 May 2020 06:12:42 +0000 From: "Zhiguang Liu" To: "devel@edk2.groups.io" , "oleksiyy@ami.com" CC: "Gao, Liming" , "Kinney, Michael D" , "Felixp@ami.com" Subject: Re: [edk2-devel] [PATCH v2 2/4] MdePkg: Fix OUT parameters marked as IN OUT Thread-Topic: [edk2-devel] [PATCH v2 2/4] MdePkg: Fix OUT parameters marked as IN OUT Thread-Index: AQHWJKbYrOpLKuAw10SuoaQu8DfiSqiibQfw Date: Mon, 11 May 2020 06:12:42 +0000 Message-ID: References: <20200507193443.76728-1-oleksiyy@ami.com> <20200507193443.76728-3-oleksiyy@ami.com> In-Reply-To: <20200507193443.76728-3-oleksiyy@ami.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.2.0.6 dlp-product: dlpe-windows authentication-results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.102.204.45] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 26f391bf-6515-4ed8-1279-08d7f57250cd x-ms-traffictypediagnostic: BN7PR11MB2628: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:2399; x-forefront-prvs: 04004D94E2 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: S6b5AWhqkRuuPalcz/7X3f+wYgTV+p0yvB1WqFmPfIPKzo8oJ7942F8Zk9egWrJjRMa4cGKtpREjpHEftIRDUmcuTAqbwoBOLc4qbpGN00vYfvfE8ptBJ5NYslmRWE01eh5xBd6CqQL072gBrzLEymkCYz4ZRKtllNS984/EooRynGciJIl+cw5Dkj91gxj/76BMcAaQaNDoaSpbNW3Q3wpRJhPhareBSRtM7wJojmr7vTwcC2KGz0EvAWQWX45QuPwJ/XrszoIIskAnej1IUDJxOIgXQQAPyV0Dvf3ACtwnJVs6xnQlby7IpAztFEGw8lwJZVhapunx8OzGB2krmEcW4gL0qVfJQdTHFjHGIDazPtxcEFN5vbwlPllpB5VR/CCEflbfU03Uc1I/csUT6h5sPpd8lLBjxRHh2PlV0Jon4Drn5U6QVx2pkX2VzDvnBuMSEmUzn+loOx2UUJDLWC2+/p3unFkrxp4dokNeW9yvU1HBIGwEh1JAN1+B4sbXI7aUjx5zYHQG1Pjv55h7I0lb3bybhpnkCjseNIzzWfAvxz3GjXfovXqWfwdgSdXqGBPAeHTQ21XZ4kkhGPXb/GvZIIjD4BPXQUa+TQ7GgWk= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN7PR11MB2804.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(366004)(136003)(39860400002)(376002)(346002)(396003)(33430700001)(66556008)(52536014)(9686003)(5660300002)(55016002)(30864003)(478600001)(316002)(7696005)(186003)(6506007)(66446008)(33440700001)(64756008)(66476007)(966005)(53546011)(66946007)(26005)(8936002)(8676002)(76116006)(33656002)(4326008)(2906002)(54906003)(110136005)(86362001)(71200400001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: sJ+rG2gwWdnRisYcGNEXDIbc0W6yk2wtzWtl5KzTzsL21N/bCEwFKleGbQy0lEAjEfNo+LSECYhCM85GGne9Yufgvrbr8ezPHBioWuhatbsIcbIfpQtIlLgQaMr2TJ9DcW3QS89TtzK51yk4CkxdK7IQ4z64W7N4Wm62r7nA7qI6RcqIuP1ufO/s2MdzIUG6jixL5XweWkyMf0MfxdhcoEkbHXJN5amoocxtC+dBIqfXtnFWx8BA7eeiFFk1Dgq6YQ5vp1311ley2Na4QK8CHTEgqNZ6H6j0BUjqyL86I2a+D0rzXOJG4UgZV+dlMlRyB/Ok9pUOgBN0UApwiFawUJNWN+y+/vAqDOedFj3WmIngBriyGPbIv9btAlGw2nNqnYyt45mb0Si9OAiQpyC+sOlgxK+HmDldlcKWjCwDBX73ijcUYTn3v4xLEjPpscdSSYUkQMOH86ZRl9IFMhNwmamkUY2At8tRvLGsA47rXHZmNwGYCvysMDJM6XQVUiIl MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 26f391bf-6515-4ed8-1279-08d7f57250cd X-MS-Exchange-CrossTenant-originalarrivaltime: 11 May 2020 06:12:42.4509 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: XXzm1ehj3oiL8YQasTHk/nSkm1iAEAEBf+Wt7CY0ybpMEu5rwFB6haU2vX8GraVXG3mdDphxP8b7X9THx2oCow== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR11MB2628 Return-Path: zhiguang.liu@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Zhiguang Liu > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Oleksiy > Yakovlev > Sent: Friday, May 8, 2020 3:35 AM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Kinney, Michael D > ; Felixp@ami.com; oleksiyy@ami.com > Subject: [edk2-devel] [PATCH v2 2/4] MdePkg: Fix OUT parameters marked > as IN OUT >=20 > Some output parameters throughout the specification were mistakenly > marked as IN OUT. > "IN OUT" replaced with "OUT" in the following interfaces >=20 > EFI_BOOT_SERVICES.GetMemoryMap():MemoryMap > EFI_BOOT_SERVICES.LocateHandleBuffer():NoHandles > EFI_SIMPLE_POINTER_PROTOCOL.GetState():State > EFI_ABSOLUTE_POINTER_PROTOCOL.GetState():State > EFI_EDID_OVERRIDE_PROTOCOL.GetEdid():EdidSize and Edid > EFI_ATA_PASS_THRU_PROTOCOL.BuildDevicePath():DevicePath > EFI_NVM_EXPRESS_PASS_THRU_PROTOCOL.BuildDevicePath():DevicePath > EFI_SD_MMC_PASS_THRU_PROTOCOL.BuildDevicePath():DevicePath > EFI_EXT_SCSI_PASS_THRU_PROTOCOL.BuildDevicePath():DevicePath > EFI_FIRMWARE_MANAGEMENT_PROTOCOL.GetImage():Image > (UEFI 2.8 errata a, mantis 2035) >=20 > Signed-off-by: Oleksiy Yakovlev > --- > MdePkg/Include/Protocol/AbsolutePointer.h | 2 +- > MdePkg/Include/Protocol/AtaPassThru.h | 4 ++-- > MdePkg/Include/Protocol/EdidOverride.h | 4 ++-- > MdePkg/Include/Protocol/FirmwareManagement.h | 4 ++-- > MdePkg/Include/Protocol/NvmExpressPassthru.h | 4 ++-- > MdePkg/Include/Protocol/ScsiPassThruExt.h | 2 +- > MdePkg/Include/Protocol/SdMmcPassThru.h | 4 ++-- > MdePkg/Include/Protocol/SimplePointer.h | 2 +- > MdePkg/Include/Uefi/UefiSpec.h | 8 ++++---- > 9 files changed, 17 insertions(+), 17 deletions(-) >=20 > diff --git a/MdePkg/Include/Protocol/AbsolutePointer.h > b/MdePkg/Include/Protocol/AbsolutePointer.h > index 77d34ae..d59ac97 100644 > --- a/MdePkg/Include/Protocol/AbsolutePointer.h > +++ b/MdePkg/Include/Protocol/AbsolutePointer.h > @@ -169,7 +169,7 @@ typedef > EFI_STATUS > (EFIAPI *EFI_ABSOLUTE_POINTER_GET_STATE)( > IN EFI_ABSOLUTE_POINTER_PROTOCOL *This, > - IN OUT EFI_ABSOLUTE_POINTER_STATE *State > + OUT EFI_ABSOLUTE_POINTER_STATE *State > ); >=20 >=20 > diff --git a/MdePkg/Include/Protocol/AtaPassThru.h > b/MdePkg/Include/Protocol/AtaPassThru.h > index ce6f57f..888dff2 100644 > --- a/MdePkg/Include/Protocol/AtaPassThru.h > +++ b/MdePkg/Include/Protocol/AtaPassThru.h > @@ -315,7 +315,7 @@ EFI_STATUS > @param[in] PortMultiplierPort The port multiplier port number of= the > ATA device for which a > device path node is to be allocate= d and built. If there is no > port multiplier, then specify 0xFF= FF. > - @param[in,out] DevicePath A pointer to a single device path = node that > describes the ATA > + @param[out] DevicePath A pointer to a single device path = node that > describes the ATA > device specified by Port and PortM= ultiplierPort. This > function > is responsible for allocating the = buffer DevicePath with the > boot service AllocatePool(). It is= the caller's responsibility > @@ -334,7 +334,7 @@ EFI_STATUS > IN EFI_ATA_PASS_THRU_PROTOCOL *This, > IN UINT16 Port, > IN UINT16 PortMultiplierPort, > - IN OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > + OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > ); >=20 > /** > diff --git a/MdePkg/Include/Protocol/EdidOverride.h > b/MdePkg/Include/Protocol/EdidOverride.h > index 9036f8b..709ee48 100644 > --- a/MdePkg/Include/Protocol/EdidOverride.h > +++ b/MdePkg/Include/Protocol/EdidOverride.h > @@ -44,8 +44,8 @@ EFI_STATUS > IN EFI_EDID_OVERRIDE_PROTOCOL *This, > IN EFI_HANDLE *ChildHandle, > OUT UINT32 *Attributes, > - IN OUT UINTN *EdidSize, > - IN OUT UINT8 **Edid > + OUT UINTN *EdidSize, > + OUT UINT8 **Edid > ); >=20 > /// > diff --git a/MdePkg/Include/Protocol/FirmwareManagement.h > b/MdePkg/Include/Protocol/FirmwareManagement.h > index 297bb5f..b501261 100644 > --- a/MdePkg/Include/Protocol/FirmwareManagement.h > +++ b/MdePkg/Include/Protocol/FirmwareManagement.h > @@ -345,7 +345,7 @@ EFI_STATUS > @param[in] This A pointer to the > EFI_FIRMWARE_MANAGEMENT_PROTOCOL instance. > @param[in] ImageIndex A unique number identifying the firmwa= re > image(s) within the device. > The number is between 1 and Descriptor= Count. > - @param[in, out] Image Points to the buffer where the current= image is > copied to. > + @param[out] Image Points to the buffer where the current= image is > copied to. > @param[in, out] ImageSize On entry, points to the size of the bu= ffer > pointed to by Image, in bytes. > On return, points to the length of the= image, in bytes. >=20 > @@ -364,7 +364,7 @@ EFI_STATUS > (EFIAPI *EFI_FIRMWARE_MANAGEMENT_PROTOCOL_GET_IMAGE)( > IN EFI_FIRMWARE_MANAGEMENT_PROTOCOL *This, > IN UINT8 ImageIndex, > - IN OUT VOID *Image, > + OUT VOID *Image, > IN OUT UINTN *ImageSize > ); >=20 > diff --git a/MdePkg/Include/Protocol/NvmExpressPassthru.h > b/MdePkg/Include/Protocol/NvmExpressPassthru.h > index 7420924..870b9b5 100644 > --- a/MdePkg/Include/Protocol/NvmExpressPassthru.h > +++ b/MdePkg/Include/Protocol/NvmExpressPassthru.h > @@ -214,7 +214,7 @@ EFI_STATUS > @param[in] NamespaceId The NVM Express namespace ID for = which > a device path node is to be > allocated and built. Caller must s= et the NamespaceId to > zero if the > device path node will contain a va= lid UUID. > - @param[in,out] DevicePath A pointer to a single device path = node that > describes the NVM Express > + @param[out] DevicePath A pointer to a single device path = node that > describes the NVM Express > namespace specified by NamespaceId= . This function is > responsible for > allocating the buffer DevicePath w= ith the boot service > AllocatePool(). > It is the caller's responsibility = to free DevicePath when the > caller @@ -231,7 +231,7 @@ EFI_STATUS (EFIAPI > *EFI_NVM_EXPRESS_PASS_THRU_BUILD_DEVICE_PATH)( > IN EFI_NVM_EXPRESS_PASS_THRU_PROTOCOL *This, > IN UINT32 NamespaceId, > - IN OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > + OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > ); >=20 > /** > diff --git a/MdePkg/Include/Protocol/ScsiPassThruExt.h > b/MdePkg/Include/Protocol/ScsiPassThruExt.h > index c83da26..994fbba 100644 > --- a/MdePkg/Include/Protocol/ScsiPassThruExt.h > +++ b/MdePkg/Include/Protocol/ScsiPassThruExt.h > @@ -259,7 +259,7 @@ EFI_STATUS > IN EFI_EXT_SCSI_PASS_THRU_PROTOCOL *This, > IN UINT8 *Target, > IN UINT64 Lun, > - IN OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > + OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > ); >=20 > /** > diff --git a/MdePkg/Include/Protocol/SdMmcPassThru.h > b/MdePkg/Include/Protocol/SdMmcPassThru.h > index 3db6a3e..ac5470b 100644 > --- a/MdePkg/Include/Protocol/SdMmcPassThru.h > +++ b/MdePkg/Include/Protocol/SdMmcPassThru.h > @@ -164,7 +164,7 @@ EFI_STATUS > @param[in] This A pointer to the > EFI_SD_MMMC_PASS_THRU_PROTOCOL instance. > @param[in] Slot Specifies the slot number of the SD car= d for which a > device > path node is to be allocated and built. > - @param[in,out] DevicePath A pointer to a single device path node = that > describes the SD > + @param[out] DevicePath A pointer to a single device path node = that > describes the SD > card specified by Slot. This function i= s responsible for > allocating the buffer DevicePath with t= he boot service > AllocatePool(). It is the caller's resp= onsibility to free @@ - > 182,7 +182,7 @@ EFI_STATUS (EFIAPI > *EFI_SD_MMC_PASS_THRU_BUILD_DEVICE_PATH) ( > IN EFI_SD_MMC_PASS_THRU_PROTOCOL *This, > IN UINT8 Slot, > - IN OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > + OUT EFI_DEVICE_PATH_PROTOCOL **DevicePath > ); >=20 > /** > diff --git a/MdePkg/Include/Protocol/SimplePointer.h > b/MdePkg/Include/Protocol/SimplePointer.h > index 2f5a940..c631414 100644 > --- a/MdePkg/Include/Protocol/SimplePointer.h > +++ b/MdePkg/Include/Protocol/SimplePointer.h > @@ -109,7 +109,7 @@ typedef > EFI_STATUS > (EFIAPI *EFI_SIMPLE_POINTER_GET_STATE)( > IN EFI_SIMPLE_POINTER_PROTOCOL *This, > - IN OUT EFI_SIMPLE_POINTER_STATE *State > + OUT EFI_SIMPLE_POINTER_STATE *State > ); >=20 > /// > diff --git a/MdePkg/Include/Uefi/UefiSpec.h > b/MdePkg/Include/Uefi/UefiSpec.h index 444aa35..29a033c 100644 > --- a/MdePkg/Include/Uefi/UefiSpec.h > +++ b/MdePkg/Include/Uefi/UefiSpec.h > @@ -209,7 +209,7 @@ EFI_STATUS > On output, it is the size of t= he buffer returned by the > firmware if > the buffer was large enough, o= r the size of the buffer > needed to contain > the map if the buffer was too = small. > - @param[in, out] MemoryMap A pointer to the buffer in whi= ch > firmware places the current memory > + @param[out] MemoryMap A pointer to the buffer in whi= ch > firmware places the current memory > map. > @param[out] MapKey A pointer to the location in w= hich firmware > returns the key for the > current memory map. > @@ -230,7 +230,7 @@ typedef > EFI_STATUS > (EFIAPI *EFI_GET_MEMORY_MAP)( > IN OUT UINTN *MemoryMapSize, > - IN OUT EFI_MEMORY_DESCRIPTOR *MemoryMap, > + OUT EFI_MEMORY_DESCRIPTOR *MemoryMap, > OUT UINTN *MapKey, > OUT UINTN *DescriptorSize, > OUT UINT32 *DescriptorVersion > @@ -1554,7 +1554,7 @@ EFI_STATUS > @param[in] Protocol Provides the protocol to search by. > This parameter is only valid for a Sear= chType of ByProtocol. > @param[in] SearchKey Supplies the search key depending on th= e > SearchType. > - @param[in, out] NoHandles The number of handles returned in Buffe= r. > + @param[out] NoHandles The number of handles returned in Buffe= r. > @param[out] Buffer A pointer to the buffer to return the r= equested > array of handles that > support Protocol. >=20 > @@ -1572,7 +1572,7 @@ EFI_STATUS > IN EFI_LOCATE_SEARCH_TYPE SearchType, > IN EFI_GUID *Protocol, OPTIONAL > IN VOID *SearchKey, OPTIONAL > - IN OUT UINTN *NoHandles, > + OUT UINTN *NoHandles, > OUT EFI_HANDLE **Buffer > ); >=20 > -- > 2.9.0.windows.1 >=20 >=20 > Please consider the environment before printing this email. >=20 > The information contained in this message may be confidential and > proprietary to American Megatrends (AMI). This communication is intende= d > to be read only by the individual or entity to whom it is addressed or b= y their > designee. If the reader of this message is not the intended recipient, y= ou are > on notice that any distribution of this message, in any form, is strictl= y > prohibited. Please promptly notify the sender by reply e-mail or by > telephone at 770-246-8600, and then delete or destroy all copies of the > transmission. >=20 >=20