dm: pmic: ignore disabled node in pmic_bind_children
authorPatrick Delaunay <patrick.delaunay@foss.st.com>
Wed, 26 Oct 2022 13:05:10 +0000 (15:05 +0200)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Mon, 12 Dec 2022 10:25:28 +0000 (11:25 +0100)
Ignore the disabled children node in pmic_bind_children() so the
disabled regulators in device tree are not registered.

This patch is based on the dm_scan_fdt_node() code - only the
activated nodes are bound -  and it solves possible issue when a
deactivated regulator is bound, error for duplicated regulator name
for example.

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/power/pmic/pmic-uclass.c

index 5dcf6d8079d270e8b5c03f580316131e9137e6f6..0e2f5e1f41111eff5342d46810fbb9d30c617003 100644 (file)
@@ -39,6 +39,10 @@ int pmic_bind_children(struct udevice *pmic, ofnode parent,
                node_name = ofnode_get_name(node);
 
                debug("* Found child node: '%s'\n", node_name);
+               if (!ofnode_is_enabled(node)) {
+                       debug("  - ignoring disabled device\n");
+                       continue;
+               }
 
                child = NULL;
                for (info = child_info; info->prefix && info->driver; info++) {