diff mbox series

[v3,05/28] dm: test: Check all devices have a sequence numbers

Message ID 20201217042034.411902-4-sjg@chromium.org
State Accepted
Commit 1c55b229232c87751fbe3d2b378b496352960816
Delegated to: Simon Glass
Headers show
Series dm: Change the way sequence numbers are implemented | expand

Commit Message

Simon Glass Dec. 17, 2020, 4:20 a.m. UTC
Add a test that the new sequence numbers work as expected. Every device
should get one.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

(no changes since v2)

Changes in v2:
- Drop the GD_FLG_DM_NO_SEQ flag

 test/dm/core.c | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

Comments

Simon Glass Dec. 19, 2020, 4:40 p.m. UTC | #1
Add a test that the new sequence numbers work as expected. Every device
should get one.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

(no changes since v2)

Changes in v2:
- Drop the GD_FLG_DM_NO_SEQ flag

 test/dm/core.c | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

Applied to u-boot-dm/next, thanks!
diff mbox series

Patch

diff --git a/test/dm/core.c b/test/dm/core.c
index 0e0696af987..a7c0f40b775 100644
--- a/test/dm/core.c
+++ b/test/dm/core.c
@@ -1066,3 +1066,22 @@  static int dm_test_inactive_child(struct unit_test_state *uts)
 	return 0;
 }
 DM_TEST(dm_test_inactive_child, UT_TESTF_SCAN_PDATA);
+
+/* Make sure all bound devices have a sequence number */
+static int dm_test_all_have_seq(struct unit_test_state *uts)
+{
+	struct udevice *dev;
+	struct uclass *uc;
+
+	list_for_each_entry(uc, &gd->uclass_root, sibling_node) {
+		list_for_each_entry(dev, &uc->dev_head, uclass_node) {
+			if (dev->sqq == -1)
+				printf("Device '%s' has no seq (%d)\n",
+				       dev->name, dev->sqq);
+			ut_assert(dev->sqq != -1);
+		}
+	}
+
+	return 0;
+}
+DM_TEST(dm_test_all_have_seq, UT_TESTF_SCAN_PDATA);