From nobody Sat Nov 2 12:17:09 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zoho.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1489455565509280.1168728870954; Mon, 13 Mar 2017 18:39:25 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id C3B2A803D3; Mon, 13 Mar 2017 18:39:23 -0700 (PDT) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 4EAB6803D2 for ; Mon, 13 Mar 2017 18:39:22 -0700 (PDT) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Mar 2017 18:39:21 -0700 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by fmsmga006.fm.intel.com with ESMTP; 13 Mar 2017 18:39:20 -0700 X-Original-To: edk2-devel@lists.01.org X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,161,1486454400"; d="scan'208";a="76275037" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 14 Mar 2017 09:39:13 +0800 Message-Id: <1489455554-94000-1-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 Subject: [edk2] [patch] MdePkg/HiiImageEx: Fix incorrect comments of functions X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Ruiyu Ni , Eric Dong , Liming Gao MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Cc: Liming Gao Cc: Ruiyu Ni Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Ruiyu Ni --- MdePkg/Include/Protocol/HiiImageEx.h | 80 +++++++++++++++++++-------------= ---- 1 file changed, 43 insertions(+), 37 deletions(-) diff --git a/MdePkg/Include/Protocol/HiiImageEx.h b/MdePkg/Include/Protocol= /HiiImageEx.h index b92fc3a..5a48380 100644 --- a/MdePkg/Include/Protocol/HiiImageEx.h +++ b/MdePkg/Include/Protocol/HiiImageEx.h @@ -26,12 +26,11 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. =20 typedef struct _EFI_HII_IMAGE_EX_PROTOCOL EFI_HII_IMAGE_EX_PROTOCOL; =20 /** The prototype of this extension function is the same with EFI_HII_IMAGE_= PROTOCOL.NewImage(). - Same with EFI_HII_IMAGE_PROTOCOL.NewImage().This protocol invokes -EFI_HII_IMAGE_PROTOCOL.NewImage() implicitly. + This protocol invokes EFI_HII_IMAGE_PROTOCOL.NewImage() implicitly. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. @param PackageList Handle of the package list where this ima= ge will be added. @param ImageId On return, contains the new image id, whi= ch is @@ -55,26 +54,28 @@ EFI_STATUS ); =20 /** Return the information about the image, associated with the package list. The prototype of this extension function is the same with EFI_HII_IMAGE_= PROTOCOL.GetImage(). - Same with EFI_HII_IMAGE_PROTOCOL.SetImage(),this protocol invokes EFI_HI= I_IMAGE_PROTOCOL.SetImage() implicitly. + + This function is similar to EFI_HII_IMAGE_PROTOCOL.GetImage().The differ= ence is that + this function will locate all EFI_HII_IMAGE_DECODER_PROTOCOL instances i= nstalled in the + system if the decoder of the certain image type is not supported by the + EFI_HII_IMAGE_EX_PROTOCOL. The function will attempt to decode the image= to the + EFI_IMAGE_INPUT using the first EFI_HII_IMAGE_DECODER_PROTOCOL instance = that + supports the requested image type. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. - @param PackageList Handle of the package list where this ima= ge will - be searched. - @param ImageId The image's id,, which is unique within - PackageList. + @param PackageList The package list in the HII database to s= earch for the + specified image. + @param ImageId The image's id, which is unique within Pa= ckageList. @param Image Points to the image. =20 @retval EFI_SUCCESS The new image was returned successfully. - @retval EFI_NOT_FOUND The image specified by ImageId is not in = the - database. The specified PackageList is no= t in - the database. - @retval EFI_BUFFER_TOO_SMALL The buffer specified by ImageSize is too = small to - hold the image. - @retval EFI_INVALID_PARAMETER The Image or ImageSize was NULL. + @retval EFI_NOT_FOUND The image specified by ImageId is not ava= ilable. The specified + PackageList is not in the Database. + @retval EFI_INVALID_PARAMETER Image was NULL or ImageId was 0. @retval EFI_OUT_OF_RESOURCES The bitmap could not be retrieved because= there was not enough memory. =20 **/ typedef @@ -85,25 +86,26 @@ EFI_STATUS IN EFI_IMAGE_ID ImageId, OUT EFI_IMAGE_INPUT *Image ); =20 /** - Change the information about the image. The prototype of this extension - function is the same with EFI_HII_IMAGE_PROTOCOL.SetImage(). Same with - EFI_HII_IMAGE_PROTOCOL.DrawImageId(),this protocol invokes EFI_HII_IMAGE= _PROTOCOL.DrawImageId() implicitly. + Change the information about the image. + + Same with EFI_HII_IMAGE_PROTOCOL.SetImage(),this protocol invokes + EFI_HII_IMAGE_PROTOCOL.SetImage()implicitly. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. @param PackageList The package list containing the images. - @param ImageId The image's id,, which is unique within - PackageList. + @param ImageId The image's id, which is unique within Pa= ckageList. @param Image Points to the image. =20 - @retval EFI_SUCCESS The new image was updated successfully. + @retval EFI_SUCCESS The new image was successfully updated. @retval EFI_NOT_FOUND The image specified by ImageId is not in = the database. The specified PackageList is no= t in the database. - @retval EFI_INVALID_PARAMETER The Image was NULL. + @retval EFI_INVALID_PARAMETER The Image was NULL, the ImageId was 0 or + the Image->Bitmap was NULL. =20 **/ typedef EFI_STATUS (EFIAPI *EFI_HII_SET_IMAGE_EX)( @@ -112,34 +114,35 @@ EFI_STATUS IN EFI_IMAGE_ID ImageId, IN CONST EFI_IMAGE_INPUT *Image ); =20 /** - Renders an image to a bitmap or to the display. The prototype of this ex= tension - function is the same with EFI_HII_IMAGE_PROTOCOL.DrawImage(). - Same with EFI_HII_IMAGE_PROTOCOL.SetImage(),this protocol invokes EFI_HI= I_IMAGE_PROTOCOL.SetImage() implicitly. + Renders an image to a bitmap or to the display. + + The prototype of this extension function is the same with + EFI_HII_IMAGE_PROTOCOL.DrawImage(). This protocol invokes + EFI_HII_IMAGE_PROTOCOL.DrawImage() implicitly. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. @param Flags Describes how the image is to be drawn. @param Image Points to the image to be displayed. @param Blt If this points to a non-NULL on entry, th= is points to the image, which is Width pixels wide = and Height pixels high. The image will be dr= awn onto this image and EFI_HII_DRAW_FLAG_CLIP is= implied. - If this points to a NULL on entry, then = a buffer - will be allocated to hold the generated = image and + If this points to a NULL on entry, then a= buffer + will be allocated to hold the generated i= mage and the pointer updated on exit. It is the ca= ller's responsibility to free this buffer. @param BltX Specifies the offset from the left and to= p edge of - the output image of the first pixel in t= he image. + the output image of the first pixel in th= e image. @param BltY Specifies the offset from the left and to= p edge of - the output image of the first pixel in t= he image. + the output image of the first pixel in th= e image. =20 @retval EFI_SUCCESS The image was successfully drawn. @retval EFI_OUT_OF_RESOURCES Unable to allocate an output buffer for B= lt. @retval EFI_INVALID_PARAMETER The Image or Blt was NULL. - Any combination of Flags is invalid. =20 **/ typedef EFI_STATUS (EFIAPI *EFI_HII_DRAW_IMAGE_EX)( @@ -151,21 +154,24 @@ EFI_STATUS IN UINTN BltY ); =20 /** Renders an image to a bitmap or the screen containing the contents of th= e specified - image. The prototype of this extension function is the same with E - FI_HII_IMAGE_PROTOCOL.DrawImageId(). - Same with EFI_HII_IMAGE_PROTOCOL.DrawImageId(),this protocol invokes -EFI_HII_IMAGE_PROTOCOL.DrawImageId() implicitly. + image. + + This function is similar to EFI_HII_IMAGE_PROTOCOL.DrawImageId(). The di= fference is that + this function will locate all EFI_HII_IMAGE_DECODER_PROTOCOL instances i= nstalled in the + system if the decoder of the certain image type is not supported by the + EFI_HII_IMAGE_EX_PROTOCOL. The function will attempt to decode the image= to the + EFI_IMAGE_INPUT using the first EFI_HII_IMAGE_DECODER_PROTOCOL instance = that + supports the requested image type. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. @param Flags Describes how the image is to be drawn. @param PackageList The package list in the HII database to s= earch for the specified image. - @param ImageId The image's id, which is unique within - PackageList. + @param ImageId The image's id, which is unique within Pa= ckageList. @param Blt If this points to a non-NULL on entry, th= is points to the image, which is Width pixels wide = and Height pixels high. The image will be dra= wn onto this image and EFI_HII_DRAW_FLAG_CLIP is = implied. If this points to a NULL on entry, then a= buffer @@ -177,11 +183,11 @@ EFI_HII_IMAGE_PROTOCOL.DrawImageId() implicitly. @param BltY Specifies the offset from the left and to= p edge of the output image of the first pixel in th= e image. =20 @retval EFI_SUCCESS The image was successfully drawn. @retval EFI_OUT_OF_RESOURCES Unable to allocate an output buffer for B= lt. - @retval EFI_INVALID_PARAMETER The Blt was NULL. + @retval EFI_INVALID_PARAMETER The Blt was NULL or ImageId was 0. @retval EFI_NOT_FOUND The image specified by ImageId is not in = the database. The specified PackageList is not in the d= atabase. =20 **/ typedef @@ -204,19 +210,19 @@ EFI_STATUS system if the decoder of image type is not supported by the EFI_HII_IMAG= E_EX_PROTOCOL. =20 @param This A pointer to the EFI_HII_IMAGE_EX_PROTOCO= L instance. @param PackageList Handle of the package list where this ima= ge will be searched. - @param ImageId The image's id,, which is unique within P= ackageList. + @param ImageId The image's id, which is unique within Pa= ckageList. @param Image Points to the image. =20 @retval EFI_SUCCESS The new image was returned successfully. @retval EFI_NOT_FOUND The image specified by ImageId is not in = the database. The specified PackageList is no= t in the database. @retval EFI_BUFFER_TOO_SMALL The buffer specified by ImageSize is too = small to hold the image. - @retval EFI_INVALID_PARAMETER The Image or ImageSize was NULL. + @retval EFI_INVALID_PARAMETER The Image was NULL or the ImageId was 0. @retval EFI_OUT_OF_RESOURCES The bitmap could not be retrieved because= there was not enough memory. =20 **/ typedef --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel