Date: Sun, 7 Oct 2007 02:21:29 +0200 (CEST) From: Yann Dirson Subject: firewire: in-code doc updates. Signed-off-by: Yann Dirson Signed-off-by: Stefan Richter (update) --- drivers/firewire/fw-topology.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) Index: linux/drivers/firewire/fw-topology.c =================================================================== --- linux.orig/drivers/firewire/fw-topology.c +++ linux/drivers/firewire/fw-topology.c @@ -166,7 +166,7 @@ static inline struct fw_node *fw_node(st * This function builds the tree representation of the topology given * by the self IDs from the latest bus reset. During the construction * of the tree, the function checks that the self IDs are valid and - * internally consistent. On succcess this funtions returns the + * internally consistent. On succcess this function returns the * fw_node corresponding to the local card otherwise NULL. */ static struct fw_node *build_tree(struct fw_card *card, @@ -215,6 +215,10 @@ static struct fw_node *build_tree(struct */ for (i = 0, h = &stack; i < child_port_count; i++) h = h->prev; + /* + * When the stack is empty, this yields an invalid value, + * but that pointer will never be dereferenced. + */ child = fw_node(h); node = fw_node_create(q, port_count, card->color);