[PATCH 1/2] ARM: meson: isolate loading of socinfo

[Neil Armstrong]

On 27/11/2020 17:28, Stefan Agner wrote:
> Move loading of socinfo into a separate function so the value can be
> reused later.
> 
> Signed-off-by: Stefan Agner <stefan at agner.ch>
> ---
> 
>  arch/arm/mach-meson/board-info.c | 14 +++++++++++++-
>  1 file changed, 13 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-meson/board-info.c b/arch/arm/mach-meson/board-info.c
> index e61d1adb10..a0bcf38f29 100644
> --- a/arch/arm/mach-meson/board-info.c
> +++ b/arch/arm/mach-meson/board-info.c
> @@ -131,7 +131,7 @@ static void print_board_model(void)
>  	printf("Model: %s\n", model ? model : "Unknown");
>  }
>  
> -int show_board_info(void)
> +static unsigned int get_socinfo(void)
>  {
>  	struct regmap *regmap;
>  	int nodeoffset, ret;
> @@ -163,8 +163,20 @@ int show_board_info(void)
>  		return 0;
>  	}
>  
> +	return socinfo;
> +}
> +
> +int show_board_info(void)
> +{
> +	unsigned int socinfo;
> +
>  	/* print board information */
>  	print_board_model();
> +
> +	socinfo = get_socinfo();
> +	if (!socinfo)
> +		return 0;
> +
>  	printf("SoC:   Amlogic Meson %s (%s) Revision %x:%x (%x:%x)\n",
>  	       socinfo_to_soc_id(socinfo),
>  	       socinfo_to_package_id(socinfo),
> 

Reviewed-by: Neil Armstrong <narmstrong at baylibre.com>

Applying it u-boot-amlogic-next

Thanks !

Neil