Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 97ad2bdc authored by Lorenzo Pieralisi's avatar Lorenzo Pieralisi Committed by Bjorn Helgaas
Browse files

ARM/PCI: Convert PCI scan API to pci_scan_root_bus_bridge() 



The introduction of pci_scan_root_bus_bridge() provides a PCI core API to
scan a PCI root bus backed by an already initialized struct pci_host_bridge
object, which simplifies the bus scan interface and makes the PCI scan root
bus interface easier to generalize as members are added to the struct
pci_host_bridge.

Convert ARM bios32 code to pci_scan_root_bus_bridge() to improve the PCI
root bus scanning interface.

Signed-off-by: default avatarLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
[bhelgaas: fold in warning fix from Arnd Bergmann <arnd@arndb.de>:
http://lkml.kernel.org/r/20170621215323.3921382-1-arnd@arndb.de]
[bhelgaas: set bridge->ops for mv78xx0]
[bhelgaas: fold in fixes from Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>:
http://lkml.kernel.org/r/20170701135457.GB8977@red-moon

]
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Russell King <linux@armlinux.org.uk>
Cc: Andrew Lunn <andrew@lunn.ch>
parent cea9bc0b

arch/arm/include/asm/mach/pci.h

+2 −1
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 
struct pci_sys_data;

struct pci_ops;

struct pci_bus;

struct pci_host_bridge;

struct device;



struct hw_pci {
@@ -25,7 +26,7 @@ struct hw_pci { 
	unsigned int	io_optional:1;

	void		**private_data;

	int		(*setup)(int nr, struct pci_sys_data *);

	struct pci_bus *(*scan)(int nr, struct pci_sys_data *);

	int		(*scan)(int nr, struct pci_host_bridge *);

	void		(*preinit)(void);

	void		(*postinit)(void);

	u8		(*swizzle)(struct pci_dev *dev, u8 *pin);

arch/arm/kernel/bios32.c

+25 −14
Original line number Diff line number Diff line
@@ -458,10 +458,14 @@ static void pcibios_init_hw(struct device *parent, struct hw_pci *hw, 
	int nr, busnr;



	for (nr = busnr = 0; nr < hw->nr_controllers; nr++) {

		sys = kzalloc(sizeof(struct pci_sys_data), GFP_KERNEL);

		if (WARN(!sys, "PCI: unable to allocate sys data!"))

		struct pci_host_bridge *bridge;



		bridge = pci_alloc_host_bridge(sizeof(struct pci_sys_data));

		if (WARN(!bridge, "PCI: unable to allocate bridge!"))

			break;



		sys = pci_host_bridge_priv(bridge);



		sys->busnr   = busnr;

		sys->swizzle = hw->swizzle;

		sys->map_irq = hw->map_irq;
@@ -473,7 +477,6 @@ static void pcibios_init_hw(struct device *parent, struct hw_pci *hw, 
		ret = hw->setup(nr, sys);



		if (ret > 0) {

			struct pci_host_bridge *host_bridge;



			ret = pcibios_init_resource(nr, sys, hw->io_optional);

			if (ret)  {
@@ -482,25 +485,33 @@ static void pcibios_init_hw(struct device *parent, struct hw_pci *hw, 
			}



			if (hw->scan)

				sys->bus = hw->scan(nr, sys);

			else

				sys->bus = pci_scan_root_bus_msi(parent,

					sys->busnr, hw->ops, sys,

					&sys->resources, hw->msi_ctrl);



			if (WARN(!sys->bus, "PCI: unable to scan bus!")) {

				kfree(sys);

				ret = hw->scan(nr, bridge);

			else {

				list_splice_init(&sys->resources,

						 &bridge->windows);

				bridge->dev.parent = parent;

				bridge->sysdata = sys;

				bridge->busnr = sys->busnr;

				bridge->ops = hw->ops;

				bridge->msi = hw->msi_ctrl;

				bridge->align_resource =

						hw->align_resource;



				ret = pci_scan_root_bus_bridge(bridge);

			}



			if (WARN(ret < 0, "PCI: unable to scan bus!")) {

				pci_free_host_bridge(bridge);

				break;

			}



			sys->bus = bridge->bus;



			busnr = sys->bus->busn_res.end + 1;



			list_add(&sys->node, head);



			host_bridge = pci_find_host_bridge(sys->bus);

			host_bridge->align_resource = hw->align_resource;

		} else {

			kfree(sys);

			pci_free_host_bridge(bridge);

			if (ret < 0)

				break;

		}

arch/arm/mach-dove/pcie.c

+12 −5
Original line number Diff line number Diff line
@@ -152,16 +152,23 @@ static void rc_pci_fixup(struct pci_dev *dev) 
}

DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL, PCI_ANY_ID, rc_pci_fixup);



static struct pci_bus __init *

dove_pcie_scan_bus(int nr, struct pci_sys_data *sys)

static int __init

dove_pcie_scan_bus(int nr, struct pci_host_bridge *bridge)

{

	struct pci_sys_data *sys = pci_host_bridge_priv(bridge);



	if (nr >= num_pcie_ports) {

		BUG();

		return NULL;

		return -EINVAL;

	}



	return pci_scan_root_bus(NULL, sys->busnr, &pcie_ops, sys,

				 &sys->resources);

	list_splice_init(&sys->resources, &bridge->windows);

	bridge->dev.parent = NULL;

	bridge->sysdata = sys;

	bridge->busnr = sys->busnr;

	bridge->ops = &pcie_ops;



	return pci_scan_root_bus_bridge(bridge);

}



static int __init dove_pcie_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)

arch/arm/mach-iop13xx/pci.c

+20 −11
Original line number Diff line number Diff line
@@ -504,10 +504,10 @@ iop13xx_pci_abort(unsigned long addr, unsigned int fsr, struct pt_regs *regs) 


/* Scan an IOP13XX PCI bus.  nr selects which ATU we use.

 */

struct pci_bus *iop13xx_scan_bus(int nr, struct pci_sys_data *sys)

int iop13xx_scan_bus(int nr, struct pci_host_bridge *bridge)

{

	int which_atu;

	struct pci_bus *bus = NULL;

	int which_atu, ret;

	struct pci_sys_data *sys = pci_host_bridge_priv(bridge);



	switch (init_atu) {

	case IOP13XX_INIT_ATU_ATUX:
@@ -525,9 +525,14 @@ struct pci_bus *iop13xx_scan_bus(int nr, struct pci_sys_data *sys) 


	if (!which_atu) {

		BUG();

		return NULL;

		return -ENODEV;

	}



	list_splice_init(&sys->resources, &bridge->windows);

	bridge->dev.parent = NULL;

	bridge->sysdata = sys;

	bridge->busnr = sys->busnr;



	switch (which_atu) {

	case IOP13XX_INIT_ATU_ATUX:

		if (time_after_eq(jiffies + msecs_to_jiffies(1000),
@@ -535,18 +540,22 @@ struct pci_bus *iop13xx_scan_bus(int nr, struct pci_sys_data *sys) 
			while(time_before(jiffies, atux_trhfa_timeout))

				udelay(100);



		bus = pci_bus_atux = pci_scan_root_bus(NULL, sys->busnr,

						       &iop13xx_atux_ops,

						       sys, &sys->resources);

		bridge->ops = &iop13xx_atux_ops;

		ret = pci_scan_root_bus_bridge(bridge);

		if (!ret)

			pci_bus_atux = bridge->bus;

		break;

	case IOP13XX_INIT_ATU_ATUE:

		bus = pci_bus_atue = pci_scan_root_bus(NULL, sys->busnr,

						       &iop13xx_atue_ops,

						       sys, &sys->resources);

		bridge->ops = &iop13xx_atue_ops;

		ret = pci_scan_root_bus_bridge(bridge);

		if (!ret)

			pci_bus_atue = bridge->bus;

		break;

	default:

		ret = -EINVAL;

	}



	return bus;

	return ret;

}



/* This function is called from iop13xx_pci_init() after assigning valid

arch/arm/mach-iop13xx/pci.h

+2 −1
Original line number Diff line number Diff line
@@ -11,9 +11,10 @@ extern size_t iop13xx_atue_mem_size; 
extern size_t iop13xx_atux_mem_size;



struct pci_sys_data;

struct pci_host_bridge;

struct hw_pci;

int iop13xx_pci_setup(int nr, struct pci_sys_data *sys);

struct pci_bus *iop13xx_scan_bus(int nr, struct pci_sys_data *);

int iop13xx_scan_bus(int nr, struct pci_host_bridge *bridge);

void iop13xx_atu_select(struct hw_pci *plat_pci);

void iop13xx_pci_init(void);

void iop13xx_map_pci_memory(void);