Kyösti Mälkki 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:
(2 comments)
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)
You want it placed closer to the deprecated function? Or should I ammend the comments and say please […]
Remaining dev_find_slot() calls (all in AMDFAM10) cannot be replaced with pcidev_path_on_root() due to non-zero bus argument. Perhaps we don't want to deprecate this one when dev_find_slot() is removed?
https://review.coreboot.org/c/coreboot/+/35632/1/src/include/device/device.h File src/include/device/device.h:
https://review.coreboot.org/c/coreboot/+/35632/1/src/include/device/device.h... PS1, Line 310: DEVTREE_CONST struct device *pcidev_path_on_root_debug(pci_devfn_t devfn, const char *func); Maybe latter should move up if we intent to have it around after dev_find_slot() is gone?