On Mon, 25 Jan 2010 12:38:49 -0800 Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote: > On Mon, 25 Jan 2010 11:42:29 -0200 > Breno Leitao <leitao@xxxxxxxxxxxxxxxxxx> wrote: > > > Hello, > > > > I found that qlge is broken on PPC, and it got broken after commit > > 06a1cbafb253c4c60d6a54a994887f5fbceabcc0. It happens because > > dev->pcie is not set on PPC, because the function > > set_pcie_port_type(), who sets dev->pcie, is not being called on PPC > > PCI code. > > You mean dev->is_pcie? > > Why isn't pci_scan_device calling pci_setup_device for you? That > should do the proper PCIe init depending on the device, along with > extracting other device info... Cc'ing Ben for PPC. Ben, should PPC use pci_scan_device when probing its root busses? Sounds like it just uses pci_device_add for each one it finds instead? If you don't actually need scanning (though what about hotplug?) we can move the call to device_add instead... -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html