firewire: core: Open-code topology list walk

A later change will make list_for_each_entry() cache the next element
before entering the loop body. for_each_fw_node() intentionally appends
newly discovered child nodes to the temporary walk list while the list is
being traversed.

Keep the loop open-coded so the next node is looked up only after
children have been appended. This preserves the current breadth-first
traversal semantics and prepares the code for the list iterator update.

Signed-off-by: Kaitao Cheng <chengkaitao@kylinos.cn>
Link: https://lore.kernel.org/r/20260609061347.93688-3-kaitao.cheng@linux.dev
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
This commit is contained in:
Kaitao Cheng 2026-06-09 14:13:35 +08:00 committed by Takashi Sakamoto
parent 21e163988c
commit e954726de5

View File

@ -272,7 +272,9 @@ static void for_each_fw_node(struct fw_card *card, struct fw_node *root,
fw_node_get(root);
list_add_tail(&root->link, &list);
parent = NULL;
list_for_each_entry(node, &list, link) {
for (node = list_first_entry(&list, typeof(*node), link);
!list_entry_is_head(node, &list, link);
node = list_next_entry(node, link)) {
node->color = card->color;
for (i = 0; i < node->port_count; i++) {