From nobody Sat Nov 2 12:32:25 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 1490583734966535.3860043927272; Sun, 26 Mar 2017 20:02:14 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 7699D20D2C3B9; Sun, 26 Mar 2017 20:02:13 -0700 (PDT) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (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 C15C921BC6ADB for ; Sun, 26 Mar 2017 20:02:09 -0700 (PDT) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Mar 2017 20:02:09 -0700 Received: from jfan12-desk.ccr.corp.intel.com ([10.239.158.57]) by fmsmga001.fm.intel.com with ESMTP; 26 Mar 2017 20:02:08 -0700 X-Original-To: edk2-devel@lists.01.org DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1490583729; x=1522119729; h=from:to:cc:subject:date:message-id:in-reply-to: references; bh=ZS2I0U3bD0bYqXxrcZvz7bz+/obEfhyAPnJ0wzcL9+Q=; b=lTpKMzJMmiKjVd1Aq1vL7YMDUvppHwiVzdRtWZja6+Mayxgv9TIEHhLc al7QxMmBHgZ70ZRhXtYO3o5dj3J7jQ==; X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,229,1486454400"; d="scan'208";a="1127411317" From: Jeff Fan To: edk2-devel@lists.01.org Date: Mon, 27 Mar 2017 11:02:00 +0800 Message-Id: <20170327030204.13572-2-jeff.fan@intel.com> X-Mailer: git-send-email 2.9.3.windows.2 In-Reply-To: <20170327030204.13572-1-jeff.fan@intel.com> References: <20170327030204.13572-1-jeff.fan@intel.com> Subject: [edk2] [PATCH 1/5] UefiCpuPkg/RegisterCpuFeaturesLib: Fix the function header issues X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Michael Kinney , Feng Tian 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: Feng Tian Cc: Michael Kinney Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan --- .../Include/Library/RegisterCpuFeaturesLib.h | 60 +++++++++++-------= --- .../RegisterCpuFeaturesLib/CpuFeaturesInitialize.c | 2 +- .../DxeRegisterCpuFeaturesLib.c | 2 +- .../PeiRegisterCpuFeaturesLib.c | 2 +- .../RegisterCpuFeaturesLib/RegisterCpuFeatures.h | 2 +- .../RegisterCpuFeaturesLib.c | 62 +++++++++++-------= ---- 6 files changed, 67 insertions(+), 63 deletions(-) diff --git a/UefiCpuPkg/Include/Library/RegisterCpuFeaturesLib.h b/UefiCpuP= kg/Include/Library/RegisterCpuFeaturesLib.h index 3fb8209..81a1f4b 100644 --- a/UefiCpuPkg/Include/Library/RegisterCpuFeaturesLib.h +++ b/UefiCpuPkg/Include/Library/RegisterCpuFeaturesLib.h @@ -257,35 +257,37 @@ RETURN_STATUS /** Registers a CPU Feature. =20 - @param GetConfigDataFunc CPU feature get configuration data function. = This - is an optional parameter that may be NULL. I= f NULL, - then the most recently registered function fo= r the - CPU feature is used. If no functions are reg= istered - for a CPU feature, then the CPU configuration= data - for the registered feature is NULL. - @param SupportFunc CPU feature support function. This is an opt= ional - parameter that may be NULL. If NULL, then th= e most - recently registered function for the CPU feat= ure is - used. If no functions are registered for a CPU - feature, then the CPU feature is assumed to be - supported by all CPUs. - @param InitializeFunc CPU feature initialize function. This is an = optional - parameter that may be NULL. If NULL, then th= e most - recently registered function for the CPU feat= ure is - used. If no functions are registered for a CPU - feature, then the CPU feature initialization = is - skipped. - @param ... Variable argument list of UINT32 CPU feature = value. - Values with no modifiers are the features pro= vided - by the registered functions. - Values with CPU_FEATURE_BEFORE modifier are f= eatures - that must be initialized after the features p= rovided - by the registered functions are used. - Values with CPU_FEATURE_AFTER modifier are fe= atures - that must be initialized before the features = provided - by the registered functions are used. - The last argument in this variable argument l= ist must - always be CPU_FEATURE_END. + @param[in] FeatureName A Null-terminated Ascii string indicates = CPU feature + name. + @param[in] GetConfigDataFunc CPU feature get configuration data functi= on. This + is an optional parameter that may be NULL= . If NULL, + then the most recently registered functio= n for the + CPU feature is used. If no functions are= registered + for a CPU feature, then the CPU configura= tion data + for the registered feature is NULL. + @param[in] SupportFunc CPU feature support function. This is an= optional + parameter that may be NULL. If NULL, the= n the most + recently registered function for the CPU = feature is + used. If no functions are registered for = a CPU + feature, then the CPU feature is assumed = to be + supported by all CPUs. + @param[in] InitializeFunc CPU feature initialize function. This is= an optional + parameter that may be NULL. If NULL, the= n the most + recently registered function for the CPU = feature is + used. If no functions are registered for = a CPU + feature, then the CPU feature initializat= ion is + skipped. + @param[in] ... Variable argument list of UINT32 CPU feat= ure value. + Values with no modifiers are the features= provided + by the registered functions. + Values with CPU_FEATURE_BEFORE modifier a= re features + that must be initialized after the featur= es provided + by the registered functions are used. + Values with CPU_FEATURE_AFTER modifier ar= e features + that must be initialized before the featu= res provided + by the registered functions are used. + The last argument in this variable argume= nt list must + always be CPU_FEATURE_END. =20 @retval RETURN_SUCCESS The CPU feature was successfully regis= tered. @retval RETURN_OUT_OF_RESOURCES There are not enough resources to regi= ster diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/CpuFeaturesInitializ= e.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/CpuFeaturesInitialize.c index cd689af..e0fe38c 100644 --- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/CpuFeaturesInitialize.c +++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/CpuFeaturesInitialize.c @@ -18,7 +18,7 @@ Worker function to save PcdCpuFeaturesCapability. =20 @param[in] SupportedFeatureMask The pointer to CPU feature bits mask b= uffer -*/ +**/ VOID SetCapabilityPcd ( IN UINT8 *SupportedFeatureMask diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/DxeRegisterCpuFeatur= esLib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/DxeRegisterCpuFeaturesL= ib.c index b5a1de7..c49f556 100644 --- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/DxeRegisterCpuFeaturesLib.c +++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/DxeRegisterCpuFeaturesLib.c @@ -168,7 +168,7 @@ SwitchNewBsp ( /** Worker function to retrieve the number of logical processor in the platf= orm. =20 - @param[out] NumberOfProcessors Pointer to the total number of l= ogical + @param[out] NumberOfCpus Pointer to the total number of l= ogical processors in the system, includ= ing the BSP and disabled APs. @param[out] NumberOfEnabledProcessors Pointer to the number of enabled= logical diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeatur= esLib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesL= ib.c index 12a516a..8fde742 100644 --- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c +++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c @@ -219,7 +219,7 @@ SwitchNewBsp ( /** Worker function to retrieve the number of logical processor in the platf= orm. =20 - @param[out] NumberOfProcessors Pointer to the total number of l= ogical + @param[out] NumberOfCpus Pointer to the total number of l= ogical processors in the system, includ= ing the BSP and disabled APs. @param[out] NumberOfEnabledProcessors Pointer to the number of enabled= logical diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.= h b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h index 2b136b6..11f8eb7 100644 --- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h +++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h @@ -147,7 +147,7 @@ StartupAPsWorker ( /** Worker function to retrieve the number of logical processor in the platf= orm. =20 - @param[out] NumberOfProcessors Pointer to the total number of l= ogical + @param[out] NumberOfCpus Pointer to the total number of l= ogical processors in the system, includ= ing the BSP and disabled APs. @param[out] NumberOfEnabledProcessors Pointer to the number of enabled= logical diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesL= ib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c index 3fec2e6..7ef0155 100644 --- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c +++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c @@ -240,7 +240,7 @@ CheckCpuFeaturesDependency ( /** Worker function to register CPU Feature. =20 - @param[in] CpuFeature Pointer to CPU feature entry + @param[in] CpuFeature Pointer to CPU feature entry =20 @retval RETURN_SUCCESS The CPU feature was successfully regis= tered. @retval RETURN_OUT_OF_RESOURCES There are not enough resources to regi= ster @@ -372,35 +372,37 @@ SetCpuFeaturesBitMask ( /** Registers a CPU Feature. =20 - @param GetConfigDataFunc CPU feature get configuration data function. = This - is an optional parameter that may be NULL. I= f NULL, - then the most recently registered function fo= r the - CPU feature is used. If no functions are reg= istered - for a CPU feature, then the CPU configuration= data - for the registered feature is NULL. - @param SupportFunc CPU feature support function. This is an opt= ional - parameter that may be NULL. If NULL, then th= e most - recently registered function for the CPU feat= ure is - used. If no functions are registered for a CPU - feature, then the CPU feature is assumed to be - supported by all CPUs. - @param InitializeFunc CPU feature initialize function. This is an = optional - parameter that may be NULL. If NULL, then th= e most - recently registered function for the CPU feat= ure is - used. If no functions are registered for a CPU - feature, then the CPU feature initialization = is - skipped. - @param ... Variable argument list of UINT32 CPU feature = value. - Values with no modifiers are the features pro= vided - by the registered functions. - Values with CPU_FEATURE_BEFORE modifier are f= eatures - that must be initialized after the features p= rovided - by the registered functions are used. - Values with CPU_FEATURE_AFTER modifier are fe= atures - that must be initialized before the features = provided - by the registered functions are used. - The last argument in this variable argument l= ist must - always be CPU_FEATURE_END. + @param[in] FeatureName A Null-terminated Ascii string indicates = CPU feature + name. + @param[in] GetConfigDataFunc CPU feature get configuration data functi= on. This + is an optional parameter that may be NULL= . If NULL, + then the most recently registered functio= n for the + CPU feature is used. If no functions are= registered + for a CPU feature, then the CPU configura= tion data + for the registered feature is NULL. + @param[in] SupportFunc CPU feature support function. This is an= optional + parameter that may be NULL. If NULL, the= n the most + recently registered function for the CPU = feature is + used. If no functions are registered for = a CPU + feature, then the CPU feature is assumed = to be + supported by all CPUs. + @param[in] InitializeFunc CPU feature initialize function. This is= an optional + parameter that may be NULL. If NULL, the= n the most + recently registered function for the CPU = feature is + used. If no functions are registered for = a CPU + feature, then the CPU feature initializat= ion is + skipped. + @param[in] ... Variable argument list of UINT32 CPU feat= ure value. + Values with no modifiers are the features= provided + by the registered functions. + Values with CPU_FEATURE_BEFORE modifier a= re features + that must be initialized after the featur= es provided + by the registered functions are used. + Values with CPU_FEATURE_AFTER modifier ar= e features + that must be initialized before the featu= res provided + by the registered functions are used. + The last argument in this variable argume= nt list must + always be CPU_FEATURE_END. =20 @retval RETURN_SUCCESS The CPU feature was successfully regis= tered. @retval RETURN_OUT_OF_RESOURCES There are not enough resources to regi= ster --=20 2.9.3.windows.2 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel