board: am57xx: Fix missing check for beagle_x15

PATCH New Nishanth Menon 3 participants 4 messages Sep 2, 2016 – Sep 7, 2016

lore

Share & tools

lore

Nishanth Menon

author

2016-09-02 18:51 UTC #0 lore raw

PATCH v2 1/1 board: am57xx: Fix missing check for beagle_x15

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 <[email protected]>

---
Changes since V1:
- move the x15 check higher up in the list.

V1: https://patchwork.ozlabs.org/patch/665144/

 board/ti/am57xx/board.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

board/ti/am57xx/board.c +3 -1

index 927d1364fe46..64de60254136 100644
@@ -338,7 +338,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";

Signature · 1 line

2.9.1.200.gb1ec08f

Tom Rini

2016-09-02 19:08 UTC #1 lore raw

PATCH v2 1/1 board: am57xx: Fix missing check for beagle_x15

Quoted 7 lines · Nishanth Menon

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 <[email protected]>

Reviewed-by Tom Rini <[email protected]>

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160902/15ad57f4/attachment.sig>

Lokesh Vutla

2016-09-03 16:54 UTC #2 lore raw

PATCH v2 1/1 board: am57xx: Fix missing check for beagle_x15

In reply to Nishanth Menon

Quoted 7 lines · Nishanth Menon

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 <[email protected]>

Reviewed-by Lokesh Vutla <[email protected]>

Thanks and regards,
Lokesh

Quoted 25 lines

---
Changes since V1:
- move the x15 check higher up in the list.

V1: https://patchwork.ozlabs.org/patch/665144/

 board/ti/am57xx/board.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

board/ti/am57xx/board.c +3 -1

index 927d1364fe46..64de60254136 100644
@@ -338,7 +338,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";

Tom Rini

2016-09-07 18:00 UTC #3 lore raw

PATCH v2 1/1 board: am57xx: Fix missing check for beagle_x15

In reply to Nishanth Menon

Quoted 9 lines · Nishanth Menon

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 <[email protected]>
Reviewed-by Tom Rini <[email protected]>
Reviewed-by Lokesh Vutla <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160907/8000e332/attachment.sig>