Re: [PATCH 4/8] arm_ffa: sandbox: Replace the emulator error log with debug log

[Ilias Apalodimas <ilias.apalodimas@linaro.org>]

On Fri, Nov 01, 2024 at 02:20:13PM +0000, abdellatif.elkhlifi@arm.com wrote:
> From: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
>
> Set the log to a debug log and reformulate the message
>
> The message is about showing what the emulated FF-A ABI decided
> based on the user arguments. The log is just for information purposes
> and helpful when debugging.
>
> Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
> ---
>  drivers/firmware/arm-ffa/ffa-emul-uclass.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/firmware/arm-ffa/ffa-emul-uclass.c b/drivers/firmware/arm-ffa/ffa-emul-uclass.c
> index 1521d9b66ac..3e8b5288d3b 100644
> --- a/drivers/firmware/arm-ffa/ffa-emul-uclass.c
> +++ b/drivers/firmware/arm-ffa/ffa-emul-uclass.c
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0+
>  /*
> - * Copyright 2022-2023 Arm Limited and/or its affiliates <open-source-office@arm.com>
> + * Copyright 2022-2024 Arm Limited and/or its affiliates <open-source-office@arm.com>
>   *
>   * Authors:
>   *   Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
> @@ -658,8 +658,7 @@ void sandbox_arm_ffa_smccc_smc(ffa_value_t *args, ffa_value_t *res)
>  			args->a0);
>  	}
>
> -	if (ret != 0)
> -		log_err("FF-A ABI internal failure (%d)\n", ret);
> +	log_debug("Emulated FF-A ABI feedback (%d)\n", ret);
>  }
>
>  /**
> --
> 2.25.1
>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>