firmwaremgr_find_by_node() matches the device node pointers to find the
firmware manager associated to a node. This function is called by the
of_overlay code when it finds a firmware-name property to find a firmware
manager for this node. This works when the overlay is applied to the
live tree, but not when it's applied to the tree we are going to load
the kernel with. To overcome this limitation match by the nodes
reproducible name instead of pointers.

Signed-off-by: Sascha Hauer <[email protected]>
---
 common/firmware.c | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/common/firmware.c b/common/firmware.c
index fce1f4b596..fc8ccd768c 100644
--- a/common/firmware.c
+++ b/common/firmware.c
@@ -66,10 +66,22 @@ struct firmware_mgr *firmwaremgr_find(const char *id)
 struct firmware_mgr *firmwaremgr_find_by_node(struct device_node *np)
 {
        struct firmware_mgr *mgr;
+       char *na, *nb;
 
-       list_for_each_entry(mgr, &firmwaremgr_list, list)
-               if (mgr->handler->device_node == np)
+       na = of_get_reproducible_name(np);
+
+       list_for_each_entry(mgr, &firmwaremgr_list, list) {
+               nb = of_get_reproducible_name(mgr->handler->device_node);
+               if (!strcmp(na, nb)) {
+                       free(na);
+                       free(nb);
                        return mgr;
+               }
+
+               free(nb);
+       }
+
+       free(na);
 
        return NULL;
 }
-- 
2.29.2


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to