diff options
| author | Suneel Garapati | 2019-10-19 16:02:48 -0700 |
|---|---|---|
| committer | Stefan Roese | 2020-08-25 05:41:09 +0200 |
| commit | bc30140d20d041a6ed5d52175e9953c36f8f5473 (patch) | |
| tree | 8bd475dc362c27d31cb74750af76a18b51bdac47 | |
| parent | b3699a13a675e06f2b27569cfc94f0a3ad8505cc (diff) | |
pci: pci-uclass: Make DT subnode parse optional
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 <sgarapati@marvell.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
| -rw-r--r-- | drivers/pci/pci-uclass.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c index fde5b83adf4..cf1bbccbba3 100644 --- a/drivers/pci/pci-uclass.c +++ b/drivers/pci/pci-uclass.c @@ -696,7 +696,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); if (ofnode_valid(node) && !ofnode_is_available(node)) { debug("%s: Ignoring disabled device\n", __func__); |