of: overlay: Correct symbol path fixups
authorPhil Elwell <phil@raspberrypi.com>
Thu, 6 Feb 2020 12:23:15 +0000 (12:23 +0000)
committerDom Cobley <popcornmix@gmail.com>
Mon, 21 Mar 2022 16:03:59 +0000 (16:03 +0000)
When symbols from overlays are added to the live tree their paths must
be rebased. The translated symbol is normally the result of joining
the fragment-relative path (with a leading "/") to the target path
(either copied directly from the "target-path" property or resolved
from the phandle). This translation fails when the target is the root
node (a common case for Raspberry Pi overlays) because the resulting
path starts with a double slash. For example, if target-path is "/" and
the fragment adds a node called "newnode", the label associated with
that node will be assigned the path "//newnode", which can't be found
in the tree.

Fix the failure case by explicitly replacing a target path of "/" with
an empty string.

Fixes: d1651b03c2df ("of: overlay: add overlay symbols to live device tree")

Signed-off-by: Phil Elwell <phil@raspberrypi.com>
drivers/of/overlay.c

index d80160c..2100d27 100644 (file)
@@ -245,6 +245,8 @@ static struct property *dup_and_fixup_symbol_prop(
        if (!target_path)
                return NULL;
        target_path_len = strlen(target_path);
+       if (!strcmp(target_path, "/"))
+               target_path_len = 0;
 
        new_prop = kzalloc(sizeof(*new_prop), GFP_KERNEL);
        if (!new_prop)