On 2/12/20 7:37 PM, Patrick Delaunay wrote:
> Lock the OTP used for board identification for the ST boards after
> programming.
>
> Signed-off-by: Patrick Delaunay <[email protected]>
> ---
>
>  board/st/common/cmd_stboard.c | 21 ++++++++++++++++++++-
>  1 file changed, 20 insertions(+), 1 deletion(-)
>
> diff --git a/board/st/common/cmd_stboard.c b/board/st/common/cmd_stboard.c
> index c7ca773b1c..1573e35410 100644
> --- a/board/st/common/cmd_stboard.c
> +++ b/board/st/common/cmd_stboard.c
> @@ -42,7 +42,7 @@ static int do_stboard(cmd_tbl_t *cmdtp, int flag, int argc,
>                     char * const argv[])
>  {
>       int ret;
> -     u32 otp;
> +     u32 otp, lock;
>       u8 revision;
>       unsigned long board, variant, bom;
>       struct udevice *dev;
> @@ -66,11 +66,20 @@ static int do_stboard(cmd_tbl_t *cmdtp, int flag, int 
> argc,
>               return CMD_RET_FAILURE;
>       }
>  
> +     ret = misc_read(dev, STM32_BSEC_LOCK(BSEC_OTP_BOARD),
> +                     &lock, sizeof(lock));
> +     if (ret < 0) {
> +             puts("LOCK read error");
> +             return CMD_RET_FAILURE;
> +     }
> +
>       if (argc == 0) {
>               if (!otp)
>                       puts("Board : OTP board FREE\n");
>               else
>                       display_stboard(otp);
> +             printf("      OTP %d %s locked !\n", BSEC_OTP_BOARD,
> +                    lock == 1 ? "" : "NOT");
>               return CMD_RET_SUCCESS;
>       }
>  
> @@ -129,6 +138,16 @@ static int do_stboard(cmd_tbl_t *cmdtp, int flag, int 
> argc,
>               puts("BOARD programming error\n");
>               return CMD_RET_FAILURE;
>       }
> +
> +     /* write persistent lock */
> +     otp = 1;
> +     ret = misc_write(dev, STM32_BSEC_LOCK(BSEC_OTP_BOARD),
> +                      &otp, sizeof(otp));
> +     if (ret < 0) {
> +             puts("BOARD lock error\n");
> +             return CMD_RET_FAILURE;
> +     }
> +
>       puts("BOARD programming done\n");
>  
>       return CMD_RET_SUCCESS;

Acked-by: Patrice Chotard <[email protected]>

Thanks

Patrice

Reply via email to