[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20201008113614.GA1226@e121166-lin.cambridge.arm.com>
Date: Thu, 8 Oct 2020 12:36:14 +0100
From: Lorenzo Pieralisi <lorenzo.pieralisi@....com>
To: Jisheng Zhang <Jisheng.Zhang@...aptics.com>
Cc: Kishon Vijay Abraham I <kishon@...com>,
Rob Herring <robh@...nel.org>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Jingoo Han <jingoohan1@...il.com>,
Gustavo Pimentel <gustavo.pimentel@...opsys.com>,
linux-omap@...r.kernel.org, linux-pci@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 0/2] PCI: dwc: fix two MSI issues
On Wed, Sep 30, 2020 at 09:12:05AM +0800, Jisheng Zhang wrote:
> Fix two MSI issues. One to skip PCIE_MSI_INTR0* programming if MSI is
> disabled, another to use an address in the driver data for MSI address,
> to fix the MSI page leakage during suspend/resume.
>
> Since v4:
> - fix pci-dra7xx.c
>
> Since v3:
> - add Acked-by tag
> - change patch2 commit msg to make it clear
> - map the MSI msg with dma_map_single_attrs() for some platforms
> which either has seperate addrs for dma and phy or has mem access
> limitation for the PCIe.
>
> Since v2:
> - add Acked-by tag
> - use an address in the driver data for MSI address. Thank Ard and Rob
> for pointing out this correct direction.
> - Since the MSI page has gone, the leak issue doesn't exist anymore,
> remove unnecessary patches.
> - Remove dw_pcie_free_msi rename and the last patch. They could be
> targeted to next. So will send out patches in a separate series.
>
> Since v1:
> - add proper error handling patches.
> - solve the msi page leakage by moving dw_pcie_msi_init() from each
> users to designware host
>
>
> Jisheng Zhang (2):
> PCI: dwc: Skip PCIE_MSI_INTR0* programming if MSI is disabled
> PCI: dwc: Fix MSI page leakage in suspend/resume
>
> drivers/pci/controller/dwc/pci-dra7xx.c | 18 +++++++++-
> .../pci/controller/dwc/pcie-designware-host.c | 34 +++++++++----------
> drivers/pci/controller/dwc/pcie-designware.h | 2 +-
> 3 files changed, 35 insertions(+), 19 deletions(-)
Hi,
can you rebase this series against my pci/dwc branch please ?
Thanks,
Lorenzo
Powered by blists - more mailing lists