diff mbox

board: am57xx: Fix missing check for beagle_x15

Message ID 20160902075354.25578-1-nm@ti.com
State New
Headers show

Commit Message

Nishanth Menon Sept. 2, 2016, 7:53 a.m. UTC
When beagleboard-X15 is booted, we see the following log:
Unidentified board claims BBRDX15_ in eeprom header

This is because of the missing check for x15 (the default) and reports
an error for a valid board configuration. fix the same.

Signed-off-by: Nishanth Menon <nm@ti.com>

---

based on v2016.09-rc2

 board/ti/am57xx/board.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.9.1.200.gb1ec08f

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Comments

Nishanth Menon Sept. 2, 2016, 5:12 p.m. UTC | #1
On Fri, Sep 2, 2016 at 6:04 AM, Lokesh Vutla <lokeshvutla@ti.com> wrote:
>

>

> On Friday 02 September 2016 01:23 PM, Nishanth Menon wrote:

[...]
> Instead an entry can be added for x15 like below? So, that future rev of

> x15 can be updated

>

> diff --git a/board/ti/am57xx/board.c b/board/ti/am57xx/board.c

> index bc5ff6e..cf095ce 100644

> --- a/board/ti/am57xx/board.c

> +++ b/board/ti/am57xx/board.c

> @@ -308,7 +308,9 @@ static void setup_board_eeprom_env(void)

>         if (rc)

>                 goto invalid_eeprom;

>

> -       if (board_is_am572x_evm())

> +       if (board_is_x15())

> +               name = "beagle_x15";

> +       else if (board_is_am572x_evm())

>                 name = "am57xx_evm";

>         else if (board_is_am572x_idk())

>                 name = "am572x_idk";


Yeah, makes sense to prepare for what we know is incoming anyways...
-- 
---
Regards,
Nishanth Menon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
diff mbox

Patch

diff --git a/board/ti/am57xx/board.c b/board/ti/am57xx/board.c
index 927d1364fe46..3c3ec8a74e78 100644
--- a/board/ti/am57xx/board.c
+++ b/board/ti/am57xx/board.c
@@ -342,7 +342,7 @@  static void setup_board_eeprom_env(void)
 		name = "am57xx_evm";
 	else if (board_is_am572x_idk())
 		name = "am572x_idk";
-	else
+	else if (!board_is_x15())
 		printf("Unidentified board claims %s in eeprom header\n",
 		       board_ti_get_name());