[coreboot-gerrit] Change in coreboot[master]: sconfig: Allow chipset to provide a base devicetree

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44037 )

Change subject: sconfig: Allow chipset to provide a base devicetree
......................................................................

Patch Set 10:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44037/9/util/sconfig/main.c 
File util/sconfig/main.c:

https://review.coreboot.org/c/coreboot/+/44037/9/util/sconfig/main.c@713 
PS9, Line 713: base_root_dev
> > We might need that if devices are added/defined only in the overridetree and also referenced there […]

Oops meant to add it doesn't have to be that way though, I'm not sold on whether we want that or not. The root of the problem is we want to set a register value differently depending on which option was successfully probed by fw_config. The way I was expressing that currently is via two devices with the same alias; this would require a little more code in sconfig to provide that information so fw_config could do the enabling/disabling and fixing up of 'use' pointers.