[coreboot-gerrit] Change in ...coreboot[master]: util/sconfig: Throw an error if override tree has no devices

[Furquan Shaikh (Code Review)]

Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32289

Change subject: util/sconfig: Throw an error if override tree has no devices
......................................................................

util/sconfig: Throw an error if override tree has no devices

If override tree does not have any device, then the chip info
structure in it cannot be associated with the correct device and ends
up being added as a standalone chip info structure without any device
actually using it. This change prevents this condition by throwing an
error during compilation.

BUG=b:130342895

Change-Id: I7b8bb6b3228030a465976ca32ce8ef63f41365dd
Signed-off-by: Furquan Shaikh 
---
M util/sconfig/main.c
1 file changed, 5 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/32289/1

diff --git a/util/sconfig/main.c b/util/sconfig/main.c
index 4ac935e..5382f47 100644
--- a/util/sconfig/main.c
+++ b/util/sconfig/main.c
@@ -1316,6 +1316,11 @@
    	override_devtree = argv[OVERRIDE_DEVICEFILE_ARG];
    	parse_devicetree(override_devtree, &override_root_bus);

+		if (!dev_has_children(&override_root_dev)) {
+			fprintf(stderr, "ERROR: Override tree needs at least one device!\n");
+			exit(1);
+		}
+
    	override_devicetree(&base_root_bus, &override_root_bus);
    }