[edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: Document EcLib return value

[Michael Kubacki]

Reviewed-by: Michael Kubacki <michael.kubacki at microsoft.com>

On 8/25/2021 11:55 PM, Nate DeSimone wrote:
> Added EFI_INVALID_PARAMETER to the EcRead()
> function's list of return values.
> 
> Cc: Chasel Chiu <chasel.chiu at intel.com>
> Cc: Michael Kubacki <michael.kubacki at microsoft.com>
> Cc: Benjamin Doron <benjamin.doron00 at gmail.com>
> Signed-off-by: Nate DeSimone <nathaniel.l.desimone at intel.com>
> ---
>   .../Intel/KabylakeOpenBoardPkg/Include/Library/EcLib.h   | 9 +++++----
>   .../KabylakeOpenBoardPkg/Library/BaseEcLib/EcCommands.c  | 9 +++++----
>   2 files changed, 10 insertions(+), 8 deletions(-)
> 
> diff --git a/Platform/Intel/KabylakeOpenBoardPkg/Include/Library/EcLib.h b/Platform/Intel/KabylakeOpenBoardPkg/Include/Library/EcLib.h
> index 7c58e592d9..e95accc465 100644
> --- a/Platform/Intel/KabylakeOpenBoardPkg/Include/Library/EcLib.h
> +++ b/Platform/Intel/KabylakeOpenBoardPkg/Include/Library/EcLib.h
> @@ -7,7 +7,7 @@
>     Make sure you meet the requirements for the library (protocol dependencies, use
>     restrictions, etc).
>   
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
>   SPDX-License-Identifier: BSD-2-Clause-Patent
>   
>   **/
> @@ -109,9 +109,10 @@ LpcEcInterface (
>     @param[in]  Address          Address to read
>     @param[out] Data             Data received
>   
> -  @retval    EFI_SUCCESS       Command success
> -  @retval    EFI_DEVICE_ERROR  Command error
> -  @retval    EFI_TIMEOUT       Command timeout
> +  @retval    EFI_SUCCESS            Command success
> +  @retval    EFI_INVALID_PARAMETER  Data is NULL
> +  @retval    EFI_DEVICE_ERROR       Command error
> +  @retval    EFI_TIMEOUT            Command timeout
>   **/
>   EFI_STATUS
>   EcRead (
> diff --git a/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseEcLib/EcCommands.c b/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseEcLib/EcCommands.c
> index d14edb75de..14a746172b 100644
> --- a/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseEcLib/EcCommands.c
> +++ b/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseEcLib/EcCommands.c
> @@ -1,7 +1,7 @@
>   /** @file
>     Common EC commands.
>   
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
>   SPDX-License-Identifier: BSD-2-Clause-Patent
>   
>   **/
> @@ -16,9 +16,10 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>     @param[in]  Address          Address to read
>     @param[out] Data             Data received
>   
> -  @retval    EFI_SUCCESS       Command success
> -  @retval    EFI_DEVICE_ERROR  Command error
> -  @retval    EFI_TIMEOUT       Command timeout
> +  @retval    EFI_SUCCESS            Command success
> +  @retval    EFI_INVALID_PARAMETER  Data is NULL
> +  @retval    EFI_DEVICE_ERROR       Command error
> +  @retval    EFI_TIMEOUT            Command timeout
>   **/
>   EFI_STATUS
>   EcRead (
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79847): https://edk2.groups.io/g/devel/message/79847
Mute This Topic: https://groups.io/mt/85153845/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-