From: Ray Jui <rjui@broadcom.com> To: Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Arnd Bergmann <arnd@arndb.de>, Mark Rutland <mark.rutland@arm.com>, Bjorn Helgaas <bhelgaas@google.com>, Hauke Mehrtens <hauke@hauke-m.de> Cc: <linux-kernel@vger.kernel.org>, <bcm-kernel-feedback-list@broadcom.com>, <linux-arm-kernel@lists.infradead.org>, <linux-pci@vger.kernel.org>, Ray Jui <rjui@broadcom.com> Subject: [PATCH v4 1/4] PCI: iproc: enable arm64 support for iProc PCIe Date: Tue, 21 Jul 2015 18:29:40 -0700 [thread overview] Message-ID: <1437528583-855-2-git-send-email-rjui@broadcom.com> (raw) In-Reply-To: <1437528583-855-1-git-send-email-rjui@broadcom.com> This patch enables arm64 support to the iProc PCIe driver Note struct pci_sys_data is arm32 specific and will eventually be removed. This change is done in such a way that when struct pci_sys_data is removed from arm32, one only needs to also remove it from pcie-iproc.h, no other change in the iProc PCIe core driver is needed In addition, arm64 based PCI driver does not require call to pci_fixup_irqs, as it implements OF based irq parsing and mapping in pcibios_add_device Signed-off-by: Ray Jui <rjui@broadcom.com> Reviewed-by: Scott Branden <sbranden@broadcom.com> --- drivers/pci/host/pcie-iproc.c | 15 ++++----------- drivers/pci/host/pcie-iproc.h | 8 ++++++-- 2 files changed, 10 insertions(+), 13 deletions(-) diff --git a/drivers/pci/host/pcie-iproc.c b/drivers/pci/host/pcie-iproc.c index d77481e..8a556d5 100644 --- a/drivers/pci/host/pcie-iproc.c +++ b/drivers/pci/host/pcie-iproc.c @@ -58,11 +58,6 @@ #define SYS_RC_INTX_EN 0x330 #define SYS_RC_INTX_MASK 0xf -static inline struct iproc_pcie *sys_to_pcie(struct pci_sys_data *sys) -{ - return sys->private_data; -} - /** * Note access to the configuration registers are protected at the higher layer * by 'pci_lock' in drivers/pci/access.c @@ -71,8 +66,7 @@ static void __iomem *iproc_pcie_map_cfg_bus(struct pci_bus *bus, unsigned int devfn, int where) { - struct pci_sys_data *sys = bus->sysdata; - struct iproc_pcie *pcie = sys_to_pcie(sys); + struct iproc_pcie *pcie = bus->sysdata; unsigned slot = PCI_SLOT(devfn); unsigned fn = PCI_FUNC(devfn); unsigned busno = bus->number; @@ -208,10 +202,7 @@ int iproc_pcie_setup(struct iproc_pcie *pcie, struct list_head *res) iproc_pcie_reset(pcie); - pcie->sysdata.private_data = pcie; - - bus = pci_create_root_bus(pcie->dev, 0, &iproc_pcie_ops, - &pcie->sysdata, res); + bus = pci_create_root_bus(pcie->dev, 0, &iproc_pcie_ops, pcie, res); if (!bus) { dev_err(pcie->dev, "unable to create PCI root bus\n"); ret = -ENOMEM; @@ -229,7 +220,9 @@ int iproc_pcie_setup(struct iproc_pcie *pcie, struct list_head *res) pci_scan_child_bus(bus); pci_assign_unassigned_bus_resources(bus); +#ifdef CONFIG_ARM pci_fixup_irqs(pci_common_swizzle, pcie->map_irq); +#endif pci_bus_add_devices(bus); return 0; diff --git a/drivers/pci/host/pcie-iproc.h b/drivers/pci/host/pcie-iproc.h index ba0a108..0ee9673 100644 --- a/drivers/pci/host/pcie-iproc.h +++ b/drivers/pci/host/pcie-iproc.h @@ -18,18 +18,22 @@ /** * iProc PCIe device + * @sysdata: Per PCI controller data. This needs to be kept at the beginning of + * struct iproc_pcie, to enable support of both ARM32 and ARM64 platforms with + * minimal changes in the iProc PCIe core driver * @dev: pointer to device data structure * @base: PCIe host controller I/O register base * @resources: linked list of all PCI resources - * @sysdata: Per PCI controller data * @root_bus: pointer to root bus * @phy: optional PHY device that controls the Serdes * @irqs: interrupt IDs */ struct iproc_pcie { +#ifdef CONFIG_ARM + struct pci_sys_data sysdata; +#endif struct device *dev; void __iomem *base; - struct pci_sys_data sysdata; struct pci_bus *root_bus; struct phy *phy; int irqs[IPROC_PCIE_MAX_NUM_IRQS]; -- 1.7.9.5
WARNING: multiple messages have this Message-ID (diff)
From: rjui@broadcom.com (Ray Jui) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 1/4] PCI: iproc: enable arm64 support for iProc PCIe Date: Tue, 21 Jul 2015 18:29:40 -0700 [thread overview] Message-ID: <1437528583-855-2-git-send-email-rjui@broadcom.com> (raw) In-Reply-To: <1437528583-855-1-git-send-email-rjui@broadcom.com> This patch enables arm64 support to the iProc PCIe driver Note struct pci_sys_data is arm32 specific and will eventually be removed. This change is done in such a way that when struct pci_sys_data is removed from arm32, one only needs to also remove it from pcie-iproc.h, no other change in the iProc PCIe core driver is needed In addition, arm64 based PCI driver does not require call to pci_fixup_irqs, as it implements OF based irq parsing and mapping in pcibios_add_device Signed-off-by: Ray Jui <rjui@broadcom.com> Reviewed-by: Scott Branden <sbranden@broadcom.com> --- drivers/pci/host/pcie-iproc.c | 15 ++++----------- drivers/pci/host/pcie-iproc.h | 8 ++++++-- 2 files changed, 10 insertions(+), 13 deletions(-) diff --git a/drivers/pci/host/pcie-iproc.c b/drivers/pci/host/pcie-iproc.c index d77481e..8a556d5 100644 --- a/drivers/pci/host/pcie-iproc.c +++ b/drivers/pci/host/pcie-iproc.c @@ -58,11 +58,6 @@ #define SYS_RC_INTX_EN 0x330 #define SYS_RC_INTX_MASK 0xf -static inline struct iproc_pcie *sys_to_pcie(struct pci_sys_data *sys) -{ - return sys->private_data; -} - /** * Note access to the configuration registers are protected at the higher layer * by 'pci_lock' in drivers/pci/access.c @@ -71,8 +66,7 @@ static void __iomem *iproc_pcie_map_cfg_bus(struct pci_bus *bus, unsigned int devfn, int where) { - struct pci_sys_data *sys = bus->sysdata; - struct iproc_pcie *pcie = sys_to_pcie(sys); + struct iproc_pcie *pcie = bus->sysdata; unsigned slot = PCI_SLOT(devfn); unsigned fn = PCI_FUNC(devfn); unsigned busno = bus->number; @@ -208,10 +202,7 @@ int iproc_pcie_setup(struct iproc_pcie *pcie, struct list_head *res) iproc_pcie_reset(pcie); - pcie->sysdata.private_data = pcie; - - bus = pci_create_root_bus(pcie->dev, 0, &iproc_pcie_ops, - &pcie->sysdata, res); + bus = pci_create_root_bus(pcie->dev, 0, &iproc_pcie_ops, pcie, res); if (!bus) { dev_err(pcie->dev, "unable to create PCI root bus\n"); ret = -ENOMEM; @@ -229,7 +220,9 @@ int iproc_pcie_setup(struct iproc_pcie *pcie, struct list_head *res) pci_scan_child_bus(bus); pci_assign_unassigned_bus_resources(bus); +#ifdef CONFIG_ARM pci_fixup_irqs(pci_common_swizzle, pcie->map_irq); +#endif pci_bus_add_devices(bus); return 0; diff --git a/drivers/pci/host/pcie-iproc.h b/drivers/pci/host/pcie-iproc.h index ba0a108..0ee9673 100644 --- a/drivers/pci/host/pcie-iproc.h +++ b/drivers/pci/host/pcie-iproc.h @@ -18,18 +18,22 @@ /** * iProc PCIe device + * @sysdata: Per PCI controller data. This needs to be kept at the beginning of + * struct iproc_pcie, to enable support of both ARM32 and ARM64 platforms with + * minimal changes in the iProc PCIe core driver * @dev: pointer to device data structure * @base: PCIe host controller I/O register base * @resources: linked list of all PCI resources - * @sysdata: Per PCI controller data * @root_bus: pointer to root bus * @phy: optional PHY device that controls the Serdes * @irqs: interrupt IDs */ struct iproc_pcie { +#ifdef CONFIG_ARM + struct pci_sys_data sysdata; +#endif struct device *dev; void __iomem *base; - struct pci_sys_data sysdata; struct pci_bus *root_bus; struct phy *phy; int irqs[IPROC_PCIE_MAX_NUM_IRQS]; -- 1.7.9.5
next prev parent reply other threads:[~2015-07-22 1:29 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-07-22 1:29 [PATCH v4 0/4] Add Broadcom North Star 2 support Ray Jui 2015-07-22 1:29 ` Ray Jui 2015-07-22 1:29 ` Ray Jui [this message] 2015-07-22 1:29 ` [PATCH v4 1/4] PCI: iproc: enable arm64 support for iProc PCIe Ray Jui 2015-07-22 20:53 ` Bjorn Helgaas 2015-07-22 20:53 ` Bjorn Helgaas 2015-07-22 21:09 ` Ray Jui 2015-07-22 21:09 ` Ray Jui 2015-07-22 21:40 ` Bjorn Helgaas 2015-07-22 21:40 ` Bjorn Helgaas 2015-07-22 1:29 ` [PATCH v4 2/4] PCI: iproc: Fix ARM64 dependency in Kconfig Ray Jui 2015-07-22 1:29 ` Ray Jui 2015-07-22 1:29 ` [PATCH v4 3/4] arm64: Add Broadcom iProc family support Ray Jui 2015-07-22 1:29 ` Ray Jui 2015-07-24 8:51 ` Will Deacon 2015-07-24 8:51 ` Will Deacon 2015-07-24 8:51 ` Will Deacon 2015-07-24 15:42 ` Ray Jui 2015-07-24 15:42 ` Ray Jui 2015-07-24 15:42 ` Ray Jui 2015-07-27 18:35 ` Olof Johansson 2015-07-27 18:35 ` Olof Johansson 2015-07-27 18:35 ` Olof Johansson 2015-07-27 18:44 ` Ray Jui 2015-07-27 18:44 ` Ray Jui 2015-07-27 18:44 ` Ray Jui 2015-07-22 1:29 ` [PATCH v4 4/4] arm64: dts: Add Broadcom North Star 2 support Ray Jui 2015-07-22 1:29 ` Ray Jui
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1437528583-855-2-git-send-email-rjui@broadcom.com \ --to=rjui@broadcom.com \ --cc=arnd@arndb.de \ --cc=bcm-kernel-feedback-list@broadcom.com \ --cc=bhelgaas@google.com \ --cc=catalin.marinas@arm.com \ --cc=hauke@hauke-m.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=will.deacon@arm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.