| Message ID | 20200924190650.24c0ff8c@xhacker.debian (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | PCI: dwc: improve msi handling | expand |
On Thu, Sep 24, 2020 at 12:6:50, Jisheng Zhang <Jisheng.Zhang@synaptics.com> wrote: > The dw_pcie_free_msi() does more things than freeing the msi page, > for example, remove irq domain etc., rename it as dw_pcie_msi_deinit. > > Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com> > --- > drivers/pci/controller/dwc/pcie-designware-host.c | 6 +++--- > drivers/pci/controller/dwc/pcie-designware.h | 4 ++-- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pci/controller/dwc/pcie-designware-host.c > index 9e04e8ef3aa4..d2de8bc5db91 100644 > --- a/drivers/pci/controller/dwc/pcie-designware-host.c > +++ b/drivers/pci/controller/dwc/pcie-designware-host.c > @@ -278,7 +278,7 @@ int dw_pcie_allocate_domains(struct pcie_port *pp) > return 0; > } > > -void dw_pcie_free_msi(struct pcie_port *pp) > +void dw_pcie_msi_deinit(struct pcie_port *pp) > { > if (pp->msi_irq) { > irq_set_chained_handler(pp->msi_irq, NULL); > @@ -500,7 +500,7 @@ int dw_pcie_host_init(struct pcie_port *pp) > > err_free_msi: > if (pci_msi_enabled() && !pp->ops->msi_host_init) > - dw_pcie_free_msi(pp); > + dw_pcie_msi_deinit(pp); > return ret; > } > EXPORT_SYMBOL_GPL(dw_pcie_host_init); > @@ -510,7 +510,7 @@ void dw_pcie_host_deinit(struct pcie_port *pp) > pci_stop_root_bus(pp->root_bus); > pci_remove_root_bus(pp->root_bus); > if (pci_msi_enabled() && !pp->ops->msi_host_init) > - dw_pcie_free_msi(pp); > + dw_pcie_msi_deinit(pp); > } > EXPORT_SYMBOL_GPL(dw_pcie_host_deinit); > > diff --git a/drivers/pci/controller/dwc/pcie-designware.h b/drivers/pci/controller/dwc/pcie-designware.h > index f911760dcc69..43b8061e1bec 100644 > --- a/drivers/pci/controller/dwc/pcie-designware.h > +++ b/drivers/pci/controller/dwc/pcie-designware.h > @@ -371,7 +371,7 @@ static inline void dw_pcie_dbi_ro_wr_dis(struct dw_pcie *pci) > #ifdef CONFIG_PCIE_DW_HOST > irqreturn_t dw_handle_msi_irq(struct pcie_port *pp); > void dw_pcie_msi_init(struct pcie_port *pp); > -void dw_pcie_free_msi(struct pcie_port *pp); > +void dw_pcie_msi_deinit(struct pcie_port *pp); > void dw_pcie_setup_rc(struct pcie_port *pp); > int dw_pcie_host_init(struct pcie_port *pp); > void dw_pcie_host_deinit(struct pcie_port *pp); > @@ -386,7 +386,7 @@ static inline void dw_pcie_msi_init(struct pcie_port *pp) > { > } > > -static inline void dw_pcie_free_msi(struct pcie_port *pp) > +static inline void dw_pcie_msi_deinit(struct pcie_port *pp) > { > } > > -- > 2.28.0 Acked-by: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pci/controller/dwc/pcie-designware-host.c index 9e04e8ef3aa4..d2de8bc5db91 100644 --- a/drivers/pci/controller/dwc/pcie-designware-host.c +++ b/drivers/pci/controller/dwc/pcie-designware-host.c @@ -278,7 +278,7 @@ int dw_pcie_allocate_domains(struct pcie_port *pp) return 0; } -void dw_pcie_free_msi(struct pcie_port *pp) +void dw_pcie_msi_deinit(struct pcie_port *pp) { if (pp->msi_irq) { irq_set_chained_handler(pp->msi_irq, NULL); @@ -500,7 +500,7 @@ int dw_pcie_host_init(struct pcie_port *pp) err_free_msi: if (pci_msi_enabled() && !pp->ops->msi_host_init) - dw_pcie_free_msi(pp); + dw_pcie_msi_deinit(pp); return ret; } EXPORT_SYMBOL_GPL(dw_pcie_host_init); @@ -510,7 +510,7 @@ void dw_pcie_host_deinit(struct pcie_port *pp) pci_stop_root_bus(pp->root_bus); pci_remove_root_bus(pp->root_bus); if (pci_msi_enabled() && !pp->ops->msi_host_init) - dw_pcie_free_msi(pp); + dw_pcie_msi_deinit(pp); } EXPORT_SYMBOL_GPL(dw_pcie_host_deinit); diff --git a/drivers/pci/controller/dwc/pcie-designware.h b/drivers/pci/controller/dwc/pcie-designware.h index f911760dcc69..43b8061e1bec 100644 --- a/drivers/pci/controller/dwc/pcie-designware.h +++ b/drivers/pci/controller/dwc/pcie-designware.h @@ -371,7 +371,7 @@ static inline void dw_pcie_dbi_ro_wr_dis(struct dw_pcie *pci) #ifdef CONFIG_PCIE_DW_HOST irqreturn_t dw_handle_msi_irq(struct pcie_port *pp); void dw_pcie_msi_init(struct pcie_port *pp); -void dw_pcie_free_msi(struct pcie_port *pp); +void dw_pcie_msi_deinit(struct pcie_port *pp); void dw_pcie_setup_rc(struct pcie_port *pp); int dw_pcie_host_init(struct pcie_port *pp); void dw_pcie_host_deinit(struct pcie_port *pp); @@ -386,7 +386,7 @@ static inline void dw_pcie_msi_init(struct pcie_port *pp) { } -static inline void dw_pcie_free_msi(struct pcie_port *pp) +static inline void dw_pcie_msi_deinit(struct pcie_port *pp) { }
The dw_pcie_free_msi() does more things than freeing the msi page, for example, remove irq domain etc., rename it as dw_pcie_msi_deinit. Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com> --- drivers/pci/controller/dwc/pcie-designware-host.c | 6 +++--- drivers/pci/controller/dwc/pcie-designware.h | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-)