From: Suneel Garapati <sgarap...@marvell.com>

Parse subnode DT properties only if parent node is valid.
Otherwise, assert is triggered on ofnode_valid in ofnode_first_subnode
from dev_for_each_subnode.

Signed-off-by: Suneel Garapati <sgarap...@marvell.com>
---
 drivers/pci/pci-uclass.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c
index 848ac5a65e..007b7472c6 100644
--- a/drivers/pci/pci-uclass.c
+++ b/drivers/pci/pci-uclass.c
@@ -675,7 +675,8 @@ static int pci_find_and_bind_driver(struct udevice *parent,
              find_id->vendor, find_id->device);
 
        /* Determine optional OF node */
-       pci_dev_find_ofnode(parent, bdf, &node);
+       if (ofnode_valid(dev_ofnode(parent)))
+               pci_dev_find_ofnode(parent, bdf, &node);
 
        start = ll_entry_start(struct pci_driver_entry, pci_driver_entry);
        n_ents = ll_entry_count(struct pci_driver_entry, pci_driver_entry);
-- 
2.23.0

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to