On Wed, 2 Sep 2020 11:36:57 +1000, Oliver O'Halloran wrote: > The pnv_phb->initialized flag is an odd beast. It was added back in 2012 in > commit db1266c85261 ("powerpc/powernv: Skip check on PE if necessary") to > allow devices to be enabled even if the device had not yet been assigned to > a PE. Allowing the device to be enabled before the PE is configured may > cause spurious EEH events since none of the IOMMU context has been setup. > > I'm not entirely sure why this was ever necessary. My best guess is that it > was an workaround for a bug or some other undesireable behaviour from the > PCI core. Either way, it's unnecessary now since as of commit dc3d8f85bb57 > ("powerpc/powernv/pci: Re-work bus PE configuration") we can guarantee that > the PE will be configured before the PCI core will allow drivers to bind to > the device. > > [...]
Applied to powerpc/next. [1/1] powerpc/powernv/pci: Drop pnv_phb->initialized https://git.kernel.org/powerpc/c/24b4c6b1a7fc79fe8142d50cb439944b81b659ff cheers