Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35632 )
Change subject: device: add commentary to dev_find_slot() ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35632/1/src/device/device_const.c File src/device/device_const.c:
https://review.coreboot.org/c/coreboot/+/35632/1/src/device/device_const.c@2... PS1, Line 236: DEVTREE_CONST struct device *pcidev_path_on_root_debug(pci_devfn_t devfn, const char *func)
Remaining dev_find_slot() calls (all in AMDFAM10) cannot be replaced with pcidev_path_on_root() due […]
Can't we provide a variant that supplies bus number? Go through dev_root->link_list to find those devices/buses?
Or am I misunderstanding the AMDFAM10 issue? If I am could you elaborate on it further for me, please?