On 10/12/18 4:01 AM, Pramod Kumar wrote:

ls1012afrwy and ls1012afrdm both boards are variant of ls1012a based SoC, by default board and board_name uboot env variables are set as ls1012afrdm. whenever board is detected as ls1012afrwy, uboot env variables are updated to ls1012afrwy if it set as other than ls1012afrwy

Signed-off-by: Pramod Kumar pramod.kumar_1@nxp.com

Changes in v2:

• fix subject and commit message

board/freescale/ls1012afrdm/ls1012afrdm.c | 9 +++++++++ 1 file changed, 9 insertions(+)

diff --git a/board/freescale/ls1012afrdm/ls1012afrdm.c b/board/freescale/ls1012afrdm/ls1012afrdm.c index 5db1027717..d149d489d5 100644 --- a/board/freescale/ls1012afrdm/ls1012afrdm.c +++ b/board/freescale/ls1012afrdm/ls1012afrdm.c @@ -42,6 +42,7 @@ int checkboard(void) puts("Board: LS1012AFRDM "); #else int rev;

• const char *b_name, *board;

  rev = get_board_version();

@@ -60,6 +61,14 @@ int checkboard(void) puts(": unknown"); break; }

• board = env_get("board");
• if (strncmp(board, "ls1012afrwy", strlen("ls1012afrwy")) != 0)

• env_set("board", "ls1012afrwy");
  

• b_name = env_get("board_name");
• if (strncmp(b_name, "ls1012afrwy", strlen("ls1012afrwy")) != 0)

• env_set("board_name", "ls1012afrwy");
  

#endif

return 0;

I don't see how you detect the board. You only check if the variables were set. By the way, where do you use "board_name"?

York