[PATCH 01/14] trace: Use notrace for short

[Stefan Herbrechtsmeier]

Hi Simon,

Am 22.12.2022 um 00:08 schrieb Simon Glass:
> The attribute syntax is quite verbose. Use the macro provided for this
> purpose.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>   arch/arm/cpu/armv7/s5p-common/timer.c   |  2 +-
>   arch/arm/mach-exynos/include/mach/cpu.h |  6 +++---
>   arch/x86/include/asm/global_data.h      |  2 +-
>   arch/x86/include/asm/msr.h              |  2 +-
>   arch/x86/include/asm/u-boot-x86.h       |  2 +-
>   common/spl/spl_fit.c                    |  1 +
>   doc/develop/trace.rst                   |  2 +-
>   lib/efi_loader/efi_freestanding.c       |  4 ++--
>   lib/trace.c                             | 26 +++++++++++--------------
>   9 files changed, 22 insertions(+), 25 deletions(-)
>

[snip]

> diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c
> index 08da7fed88e..0e026bb3d8e 100644
> --- a/common/spl/spl_fit.c
> +++ b/common/spl/spl_fit.c
> @@ -599,6 +599,7 @@ static int spl_fit_upload_fpga(struct spl_fit_info *ctx, int node,
>   			debug("Ignoring compatible = %s property\n",
>   			      compatible);
>   	}
> +	return 0;
>   
>   	ret = fpga_load(devnum, (void *)fpga_image->load_addr,
>   			fpga_image->size, BIT_FULL, flags);


This return makes the spl_fit_upload_fpga function useless. I assume 
this change was by accident.

Regards
   Stefan