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

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@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@st.com

Thanks

Patrice