[coreboot-gerrit] Change in coreboot[master]: soc/intel: Expand SA_DEV_ROOT for ramstage

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34411 )

Change subject: soc/intel: Expand SA_DEV_ROOT for ramstage
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34411/2/src/soc/intel/skylake/inclu... 
File src/soc/intel/skylake/include/soc/pci_devs.h:

https://review.coreboot.org/c/coreboot/+/34411/2/src/soc/intel/skylake/inclu... 
PS2, Line 35: SA_DEV_ROOT
> TBH: I would rather remove devicetree from smm entirely but did not check why we have it.

I know at least 1 place using it: CNL for reading heci disable config for finalize operation. Since we already have the devicetree in smm, I was planning on moving Chrome EC related configs from macros to device tree configs some day. Not sure about what other places are using it.