[PATCH 3/8] sunxi: board: Simplify Pine A64 DT selection logic

[André Przywara]

On 03/09/2020 06:07, Samuel Holland wrote:
> Instead of using an entirely separate matching algorithm, simply update
> the name of the DT we want to match. Enabling this logic does not depend
> on the FIT config name, only on the initial guess of the board name.

Yeah, clever solution. The original code was indeed quite confusing,
partly because it tried to desperately save on code size. Turns out your
solution is much smaller now ;-)

> Importantly, the initial guess must be "sun50i-a64-pine64-plus", because
> otherwise the logic would trigger when "sun50i-a64-pine64-lts" was
> written to the SPL header.

Ah, good catch.

> 
> Signed-off-by: Samuel Holland <samuel at sholland.org>

One nit below, with that:

Reviewed-by: Andre Przywara <andre.przywara at arm.com>

> ---
>  board/sunxi/board.c | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
> 
> diff --git a/board/sunxi/board.c b/board/sunxi/board.c
> index 5c9b811f27a..fb0d5bf4743 100644
> --- a/board/sunxi/board.c
> +++ b/board/sunxi/board.c
> @@ -914,14 +914,10 @@ int board_fit_config_name_match(const char *name)
>  		return 0;
>  	}
>  #ifdef CONFIG_PINE64_DT_SELECTION
> -/* Differentiate the two Pine64 board DTs by their DRAM size. */
> -	if (strstr(name, "-pine64") && strstr(best_dt_name, "-pine64")) {
> -		if ((gd->ram_size > 512 * 1024 * 1024))
> -			return !strstr(name, "plus");
> -		else
> -			return !!strstr(name, "plus");
> -	} else {
> -		return strcmp(name, best_dt_name);
> +	else if (strstr(best_dt_name, "-pine64-plus")) {

I think you don't need this "else" here, as the "if" part returns
already. Makes this even easier to read.

Cheers,
Andre.

> +		/* Differentiate the Pine A64 boards by their DRAM size. */
> +		if ((gd->ram_size == 512 * 1024 * 1024))
> +			best_dt_name = "sun50i-a64-pine64";
>  	}
>  #endif
>  	return strcmp(name, best_dt_name);
>