[Uboot-stm32] [PATCH 08/10] board: stm32mp1: display reference only for STMicroelectronics board

[Patrice CHOTARD]

On 2/12/20 7:37 PM, Patrick Delaunay wrote:
> Display the reference MBxxxx found in OTP49
> only for STMicroelectronics boards when CONFIG_CMD_STBOARD
> is activated.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
>
>  board/st/stm32mp1/stm32mp1.c | 27 ++++++++++++++-------------
>  1 file changed, 14 insertions(+), 13 deletions(-)
>
> diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c
> index 76399e2d62..463248bcd7 100644
> --- a/board/st/stm32mp1/stm32mp1.c
> +++ b/board/st/stm32mp1/stm32mp1.c
> @@ -101,20 +101,21 @@ int checkboard(void)
>  		printf(" (%s)", fdt_compat);
>  	puts("\n");
>  
> -	ret = uclass_get_device_by_driver(UCLASS_MISC,
> -					  DM_GET_DRIVER(stm32mp_bsec),
> -					  &dev);
> +	if (CONFIG_IS_ENABLED(CMD_STBOARD)) {
> +		ret = uclass_get_device_by_driver(UCLASS_MISC,
> +						  DM_GET_DRIVER(stm32mp_bsec),
> +						  &dev);
>  
> -	if (!ret)
> -		ret = misc_read(dev, STM32_BSEC_SHADOW(BSEC_OTP_BOARD),
> -				&otp, sizeof(otp));
> -	if (ret > 0 && otp) {
> -		printf("Board: MB%04x Var%d.%d Rev.%c-%02d\n",
> -		       otp >> 16,
> -		       (otp >> 12) & 0xF,
> -		       (otp >> 4) & 0xF,
> -		       ((otp >> 8) & 0xF) - 1 + 'A',
> -		       otp & 0xF);
> +		if (!ret)
> +			ret = misc_read(dev, STM32_BSEC_SHADOW(BSEC_OTP_BOARD),
> +					&otp, sizeof(otp));
> +		if (ret > 0 && otp)
> +			printf("Board: MB%04x Var%d.%d Rev.%c-%02d\n",
> +			       otp >> 16,
> +			       (otp >> 12) & 0xF,
> +			       (otp >> 4) & 0xF,
> +			       ((otp >> 8) & 0xF) - 1 + 'A',
> +			       otp & 0xF);
>  	}
>  
>  	return 0;

Acked-by: Patrice Chotard <patrice.chotard at st.com>

Thanks

Patrice