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.delau...@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.chot...@st.com>
Thanks
Patrice
