From: Kishon Vijay Abraham I <kishon@ti.com> To: Niklas Cassel <niklas.cassel@axis.com>, Bjorn Helgaas <bhelgaas@google.com>, Jingoo Han <jingoohan1@gmail.com>, Joao Pinto <Joao.Pinto@synopsys.com> Cc: Niklas Cassel <niklass@axis.com>, <linux-omap@vger.kernel.org>, <linux-pci@vger.kernel.org>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v2 05/17] PCI: designware-ep: Remove static keyword from dw_pcie_ep_reset_bar() Date: Tue, 31 Oct 2017 10:39:30 +0530 [thread overview] Message-ID: <7ad6dbed-f799-11b3-8d7d-b7d0cad35f4d@ti.com> (raw) In-Reply-To: <20171030124221.20690-6-niklas.cassel@axis.com> On Monday 30 October 2017 06:12 PM, Niklas Cassel wrote: > This way pci-dra7xx.c does not need its own copy of dw_pcie_ep_reset_bar(). > > Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> Acked-by: Kishon Vijay Abraham I <kishon@ti.com> > --- > V2: > * New patch in this series. > > drivers/pci/dwc/pci-dra7xx.c | 9 --------- > drivers/pci/dwc/pcie-designware-ep.c | 2 +- > drivers/pci/dwc/pcie-designware.h | 5 +++++ > 3 files changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/pci/dwc/pci-dra7xx.c b/drivers/pci/dwc/pci-dra7xx.c > index d0848006945a..a743545da4d4 100644 > --- a/drivers/pci/dwc/pci-dra7xx.c > +++ b/drivers/pci/dwc/pci-dra7xx.c > @@ -336,15 +336,6 @@ static irqreturn_t dra7xx_pcie_irq_handler(int irq, void *arg) > return IRQ_HANDLED; > } > > -static void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > -{ > - u32 reg; > - > - reg = PCI_BASE_ADDRESS_0 + (4 * bar); > - dw_pcie_writel_dbi2(pci, reg, 0x0); > - dw_pcie_writel_dbi(pci, reg, 0x0); > -} > - > static void dra7xx_pcie_ep_init(struct dw_pcie_ep *ep) > { > struct dw_pcie *pci = to_dw_pcie_from_ep(ep); > diff --git a/drivers/pci/dwc/pcie-designware-ep.c b/drivers/pci/dwc/pcie-designware-ep.c > index c291da2a10ba..47134a85a342 100644 > --- a/drivers/pci/dwc/pcie-designware-ep.c > +++ b/drivers/pci/dwc/pcie-designware-ep.c > @@ -30,7 +30,7 @@ void dw_pcie_ep_linkup(struct dw_pcie_ep *ep) > pci_epc_linkup(epc); > } > > -static void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > +void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > { > u32 reg; > > diff --git a/drivers/pci/dwc/pcie-designware.h b/drivers/pci/dwc/pcie-designware.h > index 5a1da459eda5..37dfad8d003f 100644 > --- a/drivers/pci/dwc/pcie-designware.h > +++ b/drivers/pci/dwc/pcie-designware.h > @@ -338,6 +338,7 @@ static inline int dw_pcie_host_init(struct pcie_port *pp) > void dw_pcie_ep_linkup(struct dw_pcie_ep *ep); > int dw_pcie_ep_init(struct dw_pcie_ep *ep); > void dw_pcie_ep_exit(struct dw_pcie_ep *ep); > +void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar); > #else > static inline void dw_pcie_ep_linkup(struct dw_pcie_ep *ep) > { > @@ -351,5 +352,9 @@ static inline int dw_pcie_ep_init(struct dw_pcie_ep *ep) > static inline void dw_pcie_ep_exit(struct dw_pcie_ep *ep) > { > } > + > +static inline void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > +{ > +} > #endif > #endif /* _PCIE_DESIGNWARE_H */ >
WARNING: multiple messages have this Message-ID (diff)
From: Kishon Vijay Abraham I <kishon@ti.com> To: Niklas Cassel <niklas.cassel@axis.com>, Bjorn Helgaas <bhelgaas@google.com>, Jingoo Han <jingoohan1@gmail.com>, Joao Pinto <Joao.Pinto@synopsys.com> Cc: Niklas Cassel <niklass@axis.com>, linux-omap@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 05/17] PCI: designware-ep: Remove static keyword from dw_pcie_ep_reset_bar() Date: Tue, 31 Oct 2017 10:39:30 +0530 [thread overview] Message-ID: <7ad6dbed-f799-11b3-8d7d-b7d0cad35f4d@ti.com> (raw) In-Reply-To: <20171030124221.20690-6-niklas.cassel@axis.com> On Monday 30 October 2017 06:12 PM, Niklas Cassel wrote: > This way pci-dra7xx.c does not need its own copy of dw_pcie_ep_reset_bar(). > > Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> Acked-by: Kishon Vijay Abraham I <kishon@ti.com> > --- > V2: > * New patch in this series. > > drivers/pci/dwc/pci-dra7xx.c | 9 --------- > drivers/pci/dwc/pcie-designware-ep.c | 2 +- > drivers/pci/dwc/pcie-designware.h | 5 +++++ > 3 files changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/pci/dwc/pci-dra7xx.c b/drivers/pci/dwc/pci-dra7xx.c > index d0848006945a..a743545da4d4 100644 > --- a/drivers/pci/dwc/pci-dra7xx.c > +++ b/drivers/pci/dwc/pci-dra7xx.c > @@ -336,15 +336,6 @@ static irqreturn_t dra7xx_pcie_irq_handler(int irq, void *arg) > return IRQ_HANDLED; > } > > -static void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > -{ > - u32 reg; > - > - reg = PCI_BASE_ADDRESS_0 + (4 * bar); > - dw_pcie_writel_dbi2(pci, reg, 0x0); > - dw_pcie_writel_dbi(pci, reg, 0x0); > -} > - > static void dra7xx_pcie_ep_init(struct dw_pcie_ep *ep) > { > struct dw_pcie *pci = to_dw_pcie_from_ep(ep); > diff --git a/drivers/pci/dwc/pcie-designware-ep.c b/drivers/pci/dwc/pcie-designware-ep.c > index c291da2a10ba..47134a85a342 100644 > --- a/drivers/pci/dwc/pcie-designware-ep.c > +++ b/drivers/pci/dwc/pcie-designware-ep.c > @@ -30,7 +30,7 @@ void dw_pcie_ep_linkup(struct dw_pcie_ep *ep) > pci_epc_linkup(epc); > } > > -static void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > +void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > { > u32 reg; > > diff --git a/drivers/pci/dwc/pcie-designware.h b/drivers/pci/dwc/pcie-designware.h > index 5a1da459eda5..37dfad8d003f 100644 > --- a/drivers/pci/dwc/pcie-designware.h > +++ b/drivers/pci/dwc/pcie-designware.h > @@ -338,6 +338,7 @@ static inline int dw_pcie_host_init(struct pcie_port *pp) > void dw_pcie_ep_linkup(struct dw_pcie_ep *ep); > int dw_pcie_ep_init(struct dw_pcie_ep *ep); > void dw_pcie_ep_exit(struct dw_pcie_ep *ep); > +void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar); > #else > static inline void dw_pcie_ep_linkup(struct dw_pcie_ep *ep) > { > @@ -351,5 +352,9 @@ static inline int dw_pcie_ep_init(struct dw_pcie_ep *ep) > static inline void dw_pcie_ep_exit(struct dw_pcie_ep *ep) > { > } > + > +static inline void dw_pcie_ep_reset_bar(struct dw_pcie *pci, enum pci_barno bar) > +{ > +} > #endif > #endif /* _PCIE_DESIGNWARE_H */ >
next prev parent reply other threads:[~2017-10-31 5:10 UTC|newest] Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-10-30 12:42 [PATCH v2 00/17] dwc MSI fixes, ARTPEC-6 EP mode support, ARTPEC-7 SoC support Niklas Cassel 2017-10-30 12:42 ` Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 01/17] PCI: dwc: Use DMA-API for allocating MSI data Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 02/17] PCI: designware-ep: dw_pcie_ep_set_msi() should only set MMC bits Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 03/17] PCI: designware-ep: Read-only registers need DBI_RO_WR_EN to be writable Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 04/17] PCI: designware-ep: Pre-allocate memory for MSI in dw_pcie_ep_init Niklas Cassel 2017-10-31 6:01 ` Kishon Vijay Abraham I 2017-10-31 20:57 ` Niklas Cassel 2017-11-16 17:16 ` Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 05/17] PCI: designware-ep: Remove static keyword from dw_pcie_ep_reset_bar() Niklas Cassel 2017-10-31 5:09 ` Kishon Vijay Abraham I [this message] 2017-10-31 5:09 ` Kishon Vijay Abraham I 2017-10-30 12:42 ` [PATCH v2 06/17] PCI: designware-ep: Add generic function for raising MSI irq Niklas Cassel 2017-10-31 6:22 ` Kishon Vijay Abraham I 2017-10-31 21:06 ` Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 07/17] PCI: dwc: dra7xx: Refactor Kconfig and Makefile handling for host/ep mode Niklas Cassel 2017-10-31 6:23 ` Kishon Vijay Abraham I 2017-10-30 12:42 ` [PATCH v2 08/17] PCI: dwc: dra7xx: Assign pp->ops in dra7xx_add_pcie_port() rather than in probe Niklas Cassel 2017-10-31 8:14 ` Kishon Vijay Abraham I 2017-10-31 8:14 ` Kishon Vijay Abraham I 2017-10-30 12:42 ` [PATCH v2 09/17] PCI: dwc: dra7xx: Add ifdefs for host/ep specific code Niklas Cassel 2017-10-31 8:29 ` Kishon Vijay Abraham I 2017-10-31 8:29 ` Kishon Vijay Abraham I 2017-10-31 21:27 ` Niklas Cassel 2017-10-31 21:27 ` Niklas Cassel 2017-10-31 21:38 ` Niklas Cassel 2017-10-31 21:38 ` Niklas Cassel 2017-10-31 22:51 ` Niklas Cassel 2017-10-31 22:51 ` Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 10/17] PCI: dwc: artpec6: Remove unused defines Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 11/17] PCI: dwc: artpec6: Use BIT and GENMASK macros Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 12/17] PCI: dwc: artpec6: Split artpec6_pcie_establish_link to smaller functions Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 13/17] bindings: PCI: artpec: Add support for endpoint mode Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 14/17] PCI: dwc: artpec6: " Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 15/17] PCI: dwc: Make cpu_addr_fixup take struct dw_pcie as argument Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 16/17] bindings: PCI: artpec: Add support for the ARTPEC-7 SoC Niklas Cassel 2017-10-30 12:42 ` [PATCH v2 17/17] PCI: dwc: artpec6: " Niklas Cassel
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=7ad6dbed-f799-11b3-8d7d-b7d0cad35f4d@ti.com \ --to=kishon@ti.com \ --cc=Joao.Pinto@synopsys.com \ --cc=bhelgaas@google.com \ --cc=jingoohan1@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=niklas.cassel@axis.com \ --cc=niklass@axis.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.