From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) by mx.groups.io with SMTP id smtpd.web08.26204.1613942517118847493 for ; Sun, 21 Feb 2021 13:21:57 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=hdhDXbZA; spf=pass (domain: nuviainc.com, ip: 209.85.221.49, mailfrom: leif@nuviainc.com) Received: by mail-wr1-f49.google.com with SMTP id l12so17039640wry.2 for ; Sun, 21 Feb 2021 13:21:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nuviainc-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=ZIxPxmaEmSzPZ6JACq4Dd6bmkXQRxfiWiEsi4ljlljE=; b=hdhDXbZAx+Pze/1yZVUIv7N8msR+EAKMGzqbN6Xf1yIUAYeQnqevmHSihcgi2PunOq VcFR1SylpjC2OuhRa/JKYFat9uCbuCXq1Oib3RFMl3EAuxt8/hIdjdKykv2jBwyofh56 w5JzQxezaK8VLr93b0Tl4xBV1T9kWMHd7qrt0AQGpGZuR6yw1VRkxFkjW8GYuVNRXs+6 n0/NvzNf37Fc38hj9Pnfy042gbnZlziW4Zz+i09wBV+G5KP74gyGZQwKjdwb72cM0XkE YtigSo+curLzFcz+/bA0t98nZzyVzXBPI0kccP7qSGvEWI/OJWW40K1zEtNP6e6WS780 aKIw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=ZIxPxmaEmSzPZ6JACq4Dd6bmkXQRxfiWiEsi4ljlljE=; b=MEjrYmnoNCJPQdoCbee5NgBX0tfAhfn0H6Uqx1ImYJYKbyOqcF2TbMufALU7IWbekP Dx9QA2o4n5QDONwyLF36HmxhbAk+ZA8TwT/jntbYyGAYlDYBysitFqT2rfN94a2P9uuB S9ncfYWpdBs4lpvEZfcnf7tjSam5NoPOypzWEf7BMG+Llt0Ywo9hi98XDqCljpsYjL3C 5OioR3lzENWvgTAfmSTLyl3RHPncim2/Yc3nKkJllq3iTMS0Lv69S9Nv5Tv8nhI3cKxg XTWj/02SiYY9a/5HCjU/RstSHkU05GwsOseRX9gzGBuOMHJVBfdpq5TI7wdT7hZhd/Yz 1GHg== X-Gm-Message-State: AOAM532jLgDhZ+X9s5mYP4Gc223naT5KoFhmFiP2INUApR8ps4+395Ee MA4Q8tENVkHJ6BRb9a70sokVHQ== X-Google-Smtp-Source: ABdhPJwW4AJOoI0OunjHaVAX9gMlSsS6rA3K+PtHOo2h1X5uGbP7bGSh9S6v5xp+hJ15Iz4SoYT0hg== X-Received: by 2002:a5d:60c4:: with SMTP id x4mr8313836wrt.384.1613942515426; Sun, 21 Feb 2021 13:21:55 -0800 (PST) Return-Path: Received: from vanye (cpc1-cmbg19-2-0-cust915.5-4.cable.virginm.net. [82.27.183.148]) by smtp.gmail.com with ESMTPSA id o129sm23968374wme.21.2021.02.21.13.21.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 21 Feb 2021 13:21:55 -0800 (PST) Date: Sun, 21 Feb 2021 21:21:53 +0000 From: "Leif Lindholm" To: Rebecca Cran Cc: devel@edk2.groups.io, Ard Biesheuvel Subject: Re: [PATCH v2 2/3] ArmPkg: Rename some functions and parameters in OemMiscLib Message-ID: <20210221212153.GO1664@vanye> References: <20210221032900.6485-1-rebecca@nuviainc.com> <20210221032900.6485-3-rebecca@nuviainc.com> MIME-Version: 1.0 In-Reply-To: <20210221032900.6485-3-rebecca@nuviainc.com> User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sat, Feb 20, 2021 at 20:28:59 -0700, Rebecca Cran wrote: > o Rename 'Offset' parameter in OemUpdateSmbiosInfo to 'Field'. > o Rename OemGetProcessorMaxSockets to OemGetMaxProcessors. > o Rename OemIsSocketPresent to OemIsProcessorPresent. > o Update Universal/Smbios to follow the changes to OemMiscLib. > > Signed-off-by: Rebecca Cran > --- > ArmPkg/Include/Library/OemMiscLib.h | 12 +++--- > ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c | 13 +++---- > ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c | 40 ++++++++++---------- > ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c | 28 +------------- > 4 files changed, 33 insertions(+), 60 deletions(-) > > diff --git a/ArmPkg/Include/Library/OemMiscLib.h b/ArmPkg/Include/Library/OemMiscLib.h > index ad0e77685dbe..31dfe7dac2a6 100644 > --- a/ArmPkg/Include/Library/OemMiscLib.h > +++ b/ArmPkg/Include/Library/OemMiscLib.h > @@ -116,13 +116,13 @@ OemGetCacheInformation ( > IN OUT SMBIOS_TABLE_TYPE7 *SmbiosCacheTable > ); > > -/** Gets the maximum number of sockets supported by the platform. > +/** Gets the maximum number of processors supported by the platform. > > - @return The maximum number of sockets. > + @return The maximum number of processors. > **/ > UINT8 > EFIAPI > -OemGetProcessorMaxSockets ( > +OemGetMaxProcessors ( > VOID > ); > > @@ -146,7 +146,7 @@ OemGetChassisType ( > **/ > BOOLEAN > EFIAPI > -OemIsSocketPresent ( > +OemIsProcessorPresent ( > IN UINTN ProcessorIndex > ); > > @@ -154,14 +154,14 @@ OemIsSocketPresent ( > > @param mHiiHandle The HII handle. > @param TokenToUpdate The string to update. > - @param Offset The field to get information about. > + @param Field The field to get information about. > **/ > VOID > EFIAPI > OemUpdateSmbiosInfo ( > IN EFI_HII_HANDLE HiiHandle, > IN EFI_STRING_ID TokenToUpdate, > - IN OEM_MISC_SMBIOS_HII_STRING_FIELD Offset > + IN OEM_MISC_SMBIOS_HII_STRING_FIELD Field > ); > > #endif // OEM_MISC_LIB_H_ > diff --git a/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c b/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c > index 6b179941e414..b02a568426dd 100644 > --- a/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c > +++ b/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c > @@ -13,7 +13,6 @@ > #include > #include > #include > - > #include > > > @@ -80,13 +79,13 @@ OemGetCacheInformation ( > return TRUE; > } > > -/** Gets the maximum number of sockets supported by the platform. > +/** Gets the maximum number of processors supported by the platform. > > - @return The maximum number of sockets. > + @return The maximum number of processors. > **/ > UINT8 > EFIAPI > -OemGetProcessorMaxSockets ( > +OemGetMaxProcessors ( > VOID > ) > { > @@ -119,7 +118,7 @@ OemGetChassisType ( > **/ > BOOLEAN > EFIAPI > -OemIsSocketPresent ( > +OemIsProcessorPresent ( > IN UINTN ProcessorIndex > ) > { > @@ -131,14 +130,14 @@ OemIsSocketPresent ( > > @param mHiiHandle The HII handle. > @param TokenToUpdate The string to update. > - @param Offset The field to get information about. > + @param Field The field to get information about. > **/ > VOID > EFIAPI > OemUpdateSmbiosInfo ( > IN EFI_HII_HANDLE mHiiHandle, > IN EFI_STRING_ID TokenToUpdate, > - IN OEM_MISC_SMBIOS_HII_STRING_FIELD Offset > + IN OEM_MISC_SMBIOS_HII_STRING_FIELD Field > ) > { > ASSERT (FALSE); > diff --git a/ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c b/ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c > index d03de12a820e..0cb56c53975e 100644 > --- a/ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c > +++ b/ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c > @@ -439,8 +439,8 @@ AddSmbiosCacheTypeTable ( > strings following the data fields. > > @param[out] Type4Record The Type 4 structure to allocate and initialize > - @param[in] ProcessorIndex The index of the processor socket > - @param[in] Populated Whether the specified processor socket is > + @param[in] ProcessorIndex The index of the processor > + @param[in] Populated Whether the specified processor is > populated. > > @retval EFI_SUCCESS The Type 4 structure was successfully > @@ -460,7 +460,7 @@ AllocateType4AndSetProcessorInformationStrings ( > EFI_STRING_ID SerialNumber; > EFI_STRING_ID AssetTag; > EFI_STRING_ID PartNumber; > - EFI_STRING ProcessorSocketStr; > + EFI_STRING ProcessorStr; > EFI_STRING ProcessorManuStr; > EFI_STRING ProcessorVersionStr; > EFI_STRING SerialNumberStr; > @@ -468,7 +468,7 @@ AllocateType4AndSetProcessorInformationStrings ( > EFI_STRING PartNumberStr; > CHAR8 *OptionalStrStart; > CHAR8 *StrStart; > - UINTN ProcessorSocketStrLen; > + UINTN ProcessorStrLen; > UINTN ProcessorManuStrLen; > UINTN ProcessorVersionStrLen; > UINTN SerialNumberStrLen; > @@ -497,14 +497,14 @@ AllocateType4AndSetProcessorInformationStrings ( > SET_HII_STRING_IF_PCD_NOT_EMPTY (PcdProcessorAssetTag, AssetTag); > SET_HII_STRING_IF_PCD_NOT_EMPTY (PcdProcessorPartNumber, PartNumber); > > - // Processor Socket Designation > + // Processor Designation > StringBufferSize = sizeof (CHAR16) * SMBIOS_STRING_MAX_LENGTH; > - ProcessorSocketStr = AllocateZeroPool (StringBufferSize); > - if (ProcessorSocketStr == NULL) { > + ProcessorStr = AllocateZeroPool (StringBufferSize); > + if (ProcessorStr == NULL) { > return EFI_OUT_OF_RESOURCES; > } > > - ProcessorSocketStrLen = UnicodeSPrint (ProcessorSocketStr, StringBufferSize, > + ProcessorStrLen = UnicodeSPrint (ProcessorStr, StringBufferSize, > L"CPU%02d", ProcessorIndex + 1); > > // Processor Manufacture > @@ -528,7 +528,7 @@ AllocateType4AndSetProcessorInformationStrings ( > PartNumberStrLen = StrLen (PartNumberStr); > > TotalSize = sizeof (SMBIOS_TABLE_TYPE4) + > - ProcessorSocketStrLen + 1 + > + ProcessorStrLen + 1 + > ProcessorManuStrLen + 1 + > ProcessorVersionStrLen + 1 + > SerialNumberStrLen + 1 + > @@ -545,12 +545,12 @@ AllocateType4AndSetProcessorInformationStrings ( > > OptionalStrStart = (CHAR8 *)(*Type4Record + 1); > UnicodeStrToAsciiStrS ( > - ProcessorSocketStr, > + ProcessorStr, > OptionalStrStart, > - ProcessorSocketStrLen + 1 > + ProcessorStrLen + 1 > ); > > - StrStart = OptionalStrStart + ProcessorSocketStrLen + 1; > + StrStart = OptionalStrStart + ProcessorStrLen + 1; > UnicodeStrToAsciiStrS ( > ProcessorManuStr, > StrStart, > @@ -586,7 +586,7 @@ AllocateType4AndSetProcessorInformationStrings ( > ); > > Exit: > - FreePool (ProcessorSocketStr); > + FreePool (ProcessorStr); > FreePool (ProcessorManuStr); > FreePool (ProcessorVersionStr); > FreePool (SerialNumberStr); > @@ -618,7 +618,7 @@ AddSmbiosProcessorTypeTable ( > UINT64 *ProcessorId; > PROCESSOR_CHARACTERISTIC_FLAGS ProcessorCharacteristics; > OEM_MISC_PROCESSOR_DATA MiscProcessorData; > - BOOLEAN SocketPopulated; > + BOOLEAN ProcessorPopulated; > > Type4Record = NULL; > > @@ -632,12 +632,12 @@ AddSmbiosProcessorTypeTable ( > L2CacheHandle = 0xFFFF; > L3CacheHandle = 0xFFFF; > > - SocketPopulated = OemIsSocketPresent(ProcessorIndex); > + ProcessorPopulated = OemIsProcessorPresent (ProcessorIndex); > > Status = AllocateType4AndSetProcessorInformationStrings ( > &Type4Record, > ProcessorIndex, > - SocketPopulated > + ProcessorPopulated > ); > if (EFI_ERROR (Status)) { > return Status; > @@ -649,7 +649,7 @@ AddSmbiosProcessorTypeTable ( > &Type4Record->ProcessorCharacteristics, > &MiscProcessorData); > > - if (SocketPopulated) { > + if (ProcessorPopulated) { > AddSmbiosCacheTypeTable (ProcessorIndex, &L1CacheHandle, > &L2CacheHandle, &L3CacheHandle); > } > @@ -713,7 +713,7 @@ ProcessorSubClassEntryPoint( > ) > { > EFI_STATUS Status; > - UINT32 SocketIndex; > + UINT32 ProcessorIndex; > > // > // Locate dependent protocols > @@ -740,8 +740,8 @@ ProcessorSubClassEntryPoint( > // > // Add SMBIOS tables for populated sockets. > // > - for (SocketIndex = 0; SocketIndex < OemGetProcessorMaxSockets(); SocketIndex++) { > - Status = AddSmbiosProcessorTypeTable (SocketIndex); > + for (ProcessorIndex = 0; ProcessorIndex < OemGetMaxProcessors (); ProcessorIndex++) { > + Status = AddSmbiosProcessorTypeTable (ProcessorIndex); > if (EFI_ERROR (Status)) { > DEBUG ((DEBUG_ERROR, "Add Processor Type Table Failed! %r.\n", Status)); > return Status; > diff --git a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c > index e6adbceba2d5..fc4dba319aad 100644 > --- a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c > +++ b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c > @@ -23,27 +23,6 @@ > > #include "SmbiosMisc.h" > > -/** > - * Returns the chassis type in SMBIOS format. > - * > - * @return Chassis type > -**/ > -UINT8 > -GetChassisType ( > - VOID > - ) > -{ > - EFI_STATUS Status; > - UINT8 ChassisType; > - > - Status = OemGetChassisType (&ChassisType); > - if (EFI_ERROR (Status)) { > - return 0; > - } > - > - return ChassisType; > -} > - This function is outright deleted, not renamed. Does this belong in another patch? / Leif > /** > This function makes boot time changes to the contents of the > MiscChassisManufacturer (Type 3) record. > @@ -80,8 +59,6 @@ SMBIOS_MISC_TABLE_FUNCTION(MiscChassisManufacturer) > CONTAINED_ELEMENT ContainedElements; > UINT8 ExtendLength; > > - UINT8 ChassisType; > - > ExtendLength = 0; > > // > @@ -165,10 +142,7 @@ SMBIOS_MISC_TABLE_FUNCTION(MiscChassisManufacturer) > > SmbiosRecord->Hdr.Length = sizeof (SMBIOS_TABLE_TYPE3) + ExtendLength + 1; > > - ChassisType = GetChassisType (); > - if (ChassisType != 0) { > - SmbiosRecord->Type = ChassisType; > - } > + SmbiosRecord->Type = OemGetChassisType (); > > //ContainedElements > ASSERT (ContainedElementCount < 2); > -- > 2.26.2 >