On Wed, Jan 03, 2018 at 10:15:40PM +0100, Cyrille Pitchen wrote: > Remove empty lines, useless comments and sort rules by alphabetical > order. > > Signed-off-by: Cyrille Pitchen <cyrille.pitchen@xxxxxxxxxxxxxxxxxx> > --- > Hi Bjorn, > > This is the kernel oops I get when I test this patch. > I've applied it on top of the series for the Cadence PCIe controller. > I didn't have time to investigate more yet, sorry! > > kernel BUG at drivers/base/driver.c:153! > Internal error: Oops - BUG: 0 [#1] PREEMPT SMP > Modules linked in: > CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.15.0-rc4+ #151 > Hardware name: Cadence CSP (DT) > pstate: 40000005 (nZcv daif -PAN -UAO) > pc : driver_register+0xc0/0xf8 > lr : pcie_port_service_register+0x4c/0x60 > sp : ffffffc006457d90 > x29: ffffffc006457d90 x28: 0000000000000000 > x27: ffffff80086e2028 x26: ffffff800884a000 > x25: ffffff8008690474 x24: ffffff80086d7068 > x23: ffffff800868b0b8 x22: ffffff80086d7078 > x21: 0000000000000000 x20: ffffff80088692d4 > x19: ffffff80086af380 x18: 0000000000000000 > x17: 000000000000037f x16: 0000000000000000 > x15: 0000000000000000 x14: 0000000000000400 > x13: 0000000000000001 x12: 0000000000000000 > x11: 0000000000000001 x10: 0000000000000980 > x9 : ffffffc006457b60 x8 : ffffffc0064589e0 > x7 : ffffffc006458100 x6 : 000000000000a6e8 > x5 : 000000000000a6e8 x4 : 0000000000000000 > x1 : ffffff800881d720 x0 : ffffff800881c820 > Process swapper/0 (pid: 1, stack limit = 0x00000000d680c8f0) > Call trace: > driver_register+0xc0/0xf8 > pcied_init+0x24/0x7c > do_one_initcall+0x38/0x120 > kernel_init_freeable+0x134/0x1d0 > kernel_init+0x10/0x100 > ret_from_fork+0x10/0x18 > Code: 2a1403e0 a94153f3 a8c27bfd d65f03c0 (d4210000) Thanks, I think this happens because we have device_initcall(pcied_init); device_initcall(pcie_portdrv_init); and pcied_init() depends on something done by pcie_portdrv_init(). I think this is a mess but it's a lot of work to clean up. So I guess we'll just rely on the link ordering for now. > drivers/pci/Makefile | 84 +++++++++++++++++++++------------------------------- > 1 file changed, 33 insertions(+), 51 deletions(-) > > diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile > index 3b6c22afb2aa..fd4401cf1b4a 100644 > --- a/drivers/pci/Makefile > +++ b/drivers/pci/Makefile > @@ -3,57 +3,39 @@ > # Makefile for the PCI bus specific drivers. > # > > -obj-$(CONFIG_PCI) += access.o bus.o probe.o host-bridge.o remove.o pci.o \ > - pci-driver.o search.o pci-sysfs.o rom.o setup-res.o \ > - irq.o vpd.o setup-bus.o vc.o mmap.o setup-irq.o > +ccflags-$(CONFIG_PCI_DEBUG) := -DDEBUG > > +# PCI core > +obj-$(CONFIG_ACPI) += pci-acpi.o > +obj-$(CONFIG_HOTPLUG_PCI) += hotplug/ > +obj-$(CONFIG_PCI) += access.o bus.o probe.o host-bridge.o \ > + remove.o pci.o pci-driver.o search.o \ > + pci-sysfs.o rom.o setup-res.o irq.o \ > + vpd.o setup-bus.o vc.o mmap.o \ > + setup-irq.o > ifdef CONFIG_PCI > -obj-$(CONFIG_PROC_FS) += proc.o > -obj-$(CONFIG_SYSFS) += slot.o > -obj-$(CONFIG_OF) += of.o > +obj-$(CONFIG_OF) += of.o > +obj-$(CONFIG_PROC_FS) += proc.o > +obj-$(CONFIG_SYSFS) += slot.o > endif > - > -obj-$(CONFIG_PCI_QUIRKS) += quirks.o > - > -# Build PCI Express stuff if needed > -obj-$(CONFIG_PCIEPORTBUS) += pcie/ > - > -# Build the PCI Hotplug drivers if we were asked to > -obj-$(CONFIG_HOTPLUG_PCI) += hotplug/ > - > -# Build the PCI MSI interrupt support > -obj-$(CONFIG_PCI_MSI) += msi.o > - > -obj-$(CONFIG_PCI_ATS) += ats.o > -obj-$(CONFIG_PCI_IOV) += iov.o > - > -# > -# ACPI Related PCI FW Functions > -# ACPI _DSM provided firmware instance and string name > -# > -obj-$(CONFIG_ACPI) += pci-acpi.o > - > -# SMBIOS provided firmware instance and labels > -obj-$(CONFIG_PCI_LABEL) += pci-label.o > - > -# Intel MID platform PM support > -obj-$(CONFIG_X86_INTEL_MID) += pci-mid.o > - > -obj-$(CONFIG_PCI_SYSCALL) += syscall.o > - > -obj-$(CONFIG_PCI_STUB) += pci-stub.o > - > -obj-$(CONFIG_PCI_ECAM) += ecam.o > - > -obj-$(CONFIG_XEN_PCIDEV_FRONTEND) += xen-pcifront.o > - > -ccflags-$(CONFIG_PCI_DEBUG) := -DDEBUG > - > -# PCI host controller drivers > -obj-y += host/ > -obj-y += switch/ > - > -obj-$(CONFIG_PCI_ENDPOINT) += endpoint/ > -obj-$(CONFIG_PCIE_CADENCE) += cadence/ > -# PCI dwc controller drivers > -obj-y += dwc/ > +obj-$(CONFIG_PCIEPORTBUS) += pcie/ > +obj-$(CONFIG_PCI_ATS) += ats.o > +obj-$(CONFIG_PCI_ECAM) += ecam.o > +obj-$(CONFIG_PCI_IOV) += iov.o > +obj-$(CONFIG_PCI_LABEL) += pci-label.o > +obj-$(CONFIG_PCI_MSI) += msi.o > +obj-$(CONFIG_PCI_QUIRKS) += quirks.o > +obj-$(CONFIG_PCI_STUB) += pci-stub.o > +obj-$(CONFIG_PCI_SYSCALL) += syscall.o > +obj-$(CONFIG_X86_INTEL_MID) += pci-mid.o > +obj-$(CONFIG_XEN_PCIDEV_FRONTEND) += xen-pcifront.o > + > +# PCI controllers > +obj-y += host/ > +obj-y += switch/ > +obj-$(CONFIG_PCI_ENDPOINT) += endpoint/ > + > +# PCI controllers supporting both host and endpoint modes: > +# their associated drivers must currently be linked after the EPC/EPF libraries > +obj-$(CONFIG_PCIE_CADENCE) += cadence/ > +obj-y += dwc/ > -- > 2.11.0 >
