Hi, On Thu, 06 Aug 2026 11:18:18 -0700, Scott Moser wrote: > pci_get_devfn() reads addr.phys_hi and returns the low bits of it as the > requested devfn even on the -ENOENT branch, where ofnode_read_pci_addr() > has NOT written *addr. The C standard leaves that read undefined; in > practice the returned value depends on the compilers stack layout and on > -ftrivial-auto-var-init. That value ends up in pplat->devfn (set by > pci_uclass_child_post_bind()) and is later compared for equality in > pci_bus_find_devfn() during PCI enumeration, so an under-determined > value produces an under-determined driver-binding outcome. > > [...]
Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-ufs (u-boot-pci-fixes) [1/1] dm: pci: fix uninitialized fdt_pci_addr fall-through in pci_get_devfn https://source.denx.de/u-boot/custodians/u-boot-ufs/-/commit/85a7290d4bc976a57c7ecbcfcc3d6aa67665ffaf -- Neil
