Re: [edk2-devel] [PATCH v4 1/7] Silicon/ARM/NeoverseN1Soc: Fix missing function documentation

["Khasim Mohammed" <khasim.mohammed@arm.com>]

[-- Attachment #1: Type: text/plain, Size: 3844 bytes --]

On Wed, Oct 27, 2021 at 08:28 AM, PierreGondois wrote:

> 
> Hi Khasim,
> 
> + Sami
> 
> Thanks for the new patch-set, everything looks good to me:
> 
> Reviewed-by: Pierre Gondois <pierre.gondois@arm.com>

Hi all,

As we have entered the soft freeze phase, I would like to provide additional info to the edk2 community about the following series (7 Patches version 5) of the patches for N1SDP. These patches were already reviewed a few weeks ago, with reviewed-by as shown above, these patches are important for N1SDP platform as they provide all the functionality required to get the N1SDP platform booting various OS distributions like busybox, poky and ubuntu with ACPI tables.

These patches are independent of any other platform and specific to only N1SDP hardware thereby it would not break any other platform or existing features in the edk2 software.

I request you to consider these patches and the merge request and merge the patches to appropriate repository.

Thanks.

Regards,
Khasim

> 
> Just a reminder for Sami about the _OSC method:
> https://edk2.groups.io/g/devel/message/82724
> 
> Regards,
> 
> Pierre
> 
> 
> On 10/26/21 18:39, Khasim Mohammed via groups.io wrote:
> 
>> This patch adds missing documentation for few of the functions
>> and fixes few formatting changes.
>> 
>> Signed-off-by: Khasim Syed Mohammed <khasim.mohammed@arm.com>
>> ---
>> .../Library/PlatformLib/PlatformLib.c | 46 +++++++++++++++++--
>> 1 file changed, 41 insertions(+), 5 deletions(-)
>> 
>> diff --git a/Silicon/ARM/NeoverseN1Soc/Library/PlatformLib/PlatformLib.c
>> b/Silicon/ARM/NeoverseN1Soc/Library/PlatformLib/PlatformLib.c
>> index f722080e56..c0effd37f3 100644
>> --- a/Silicon/ARM/NeoverseN1Soc/Library/PlatformLib/PlatformLib.c
>> +++ b/Silicon/ARM/NeoverseN1Soc/Library/PlatformLib/PlatformLib.c
>> @@ -1,9 +1,9 @@
>> /** @file
>> -*
>> -* Copyright (c) 2018-2020, ARM Limited. All rights reserved.
>> -*
>> -* SPDX-License-Identifier: BSD-2-Clause-Patent
>> -*
>> +
>> + Copyright (c) 2018-2021, ARM Limited. All rights reserved.<BR>
>> +
>> + SPDX-License-Identifier: BSD-2-Clause-Patent
>> +
>> **/
>> 
>> #include <Library/ArmPlatformLib.h>
>> @@ -17,6 +17,12 @@ STATIC ARM_CORE_INFO mCoreInfoTable[] = {
>> { 0x1, 0x1 } // Cluster 1, Core 1
>> };
>> 
>> +/**
>> + Return the current Boot Mode.
>> +
>> + @return The boot reason on the platform.
>> +
>> +**/
>> EFI_BOOT_MODE
>> ArmPlatformGetBootMode (
>> VOID
>> @@ -25,6 +31,16 @@ ArmPlatformGetBootMode (
>> return BOOT_WITH_FULL_CONFIGURATION;
>> }
>> 
>> +/**
>> + Initialize controllers that must be setup in the normal world.
>> +
>> + This function is called by the ArmPlatformPkg/Pei or
>> + ArmPlatformPkg/Pei/PlatformPeim in the PEI phase.
>> +
>> + @param[in] MpId Processor ID
>> + @retval RETURN_SUCCESS
>> +
>> +**/
>> RETURN_STATUS
>> ArmPlatformInitialize (
>> IN UINTN MpId
>> @@ -33,6 +49,17 @@ ArmPlatformInitialize (
>> return RETURN_SUCCESS;
>> }
>> 
>> +/**
>> + Populate the Platform core information.
>> +
>> + This function populates the ARM_MP_CORE_INFO_PPI with information
>> + about the cores.
>> +
>> + @param[out] CoreCount Number of cores
>> + @param[out] ArmCoreTable Table containing information about the cores
>> + @retval EFI_SUCCESS
>> +
>> +**/
>> EFI_STATUS
>> PrePeiCoreGetMpCoreInfo (
>> OUT UINTN *CoreCount,
>> @@ -56,6 +83,15 @@ EFI_PEI_PPI_DESCRIPTOR gPlatformPpiTable[] = {
>> }
>> };
>> 
>> +/**
>> + Return the Platform specific PPIs
>> +
>> + This function exposes the N1Sdp Specific PPIs.
>> +
>> + @param[out] PpiListSize Size in Bytes of the Platform PPI List
>> + @param[out] PpiList Platform PPI List
>> +
>> +**/
>> VOID
>> ArmPlatformGetPlatformPpiList (
>> OUT UINTN *PpiListSize,
> 
>

[-- Attachment #2: Type: text/html, Size: 4191 bytes --]