[PATCH] pci: designware: fix missing msi irqs

Jingoo Han jg1.han at samsung.com
Thu Dec 19 21:41:56 EST 2013


On Monday, December 16, 2013 3:49 PM, Mohit KUMAR DCG wrote:
> On Monday, December 16, 2013 6:24 AM, Jingoo Han wrote:
> > On Friday, December 13, 2013 3:29 AM, Harro Haan wrote:
> > >
> > > The interrupts were cleared after the irq handler was called.
> > > This means that new interrupts that occur after the handler handled
> > > the previous irq but before the interrupt is cleared will be missed.
> > >
> > > Signed-off-by: Harro Haan <hrhaan at gmail.com>
> > > Cc: Mohit Kumar <mohit.kumar at st.com>
> > > Cc: Jingoo Han <jg1.han at samsung.com>
> > > Cc: Bjorn Helgaas <bhelgaas at google.com>
> > > Cc: Marek Vasut <marex at denx.de>
> > > Cc: Richard Zhu <hong-xing.zhu at freescale.com>
> > > Cc: Shawn Guo <shawn.guo at linaro.org>
> > > Cc: Pratyush Anand <pratyush.anand at st.com>
> > > Cc: Tim Harvey <tharvey at gateworks.com>
> > > Cc: Juergen Beisert <jbe at pengutronix.de>
> > > Cc: Arnd Bergmann <arnd at arndb.de>
> > > Cc: Siva Reddy Kallam <siva.kallam at samsung.com>
> > > Cc: Srikanth T Shivanand <ts.srikanth at samsung.com>
> > > Cc: Sean Cross <xobs at kosagi.com>
> > > Cc: linux-pci at vger.kernel.org
> > > Cc: linux-arm-kernel at lists.infradead.org
> >
> > Acked-by: Jingoo Han <jg1.han at samsung.com>
> >
> > I was not able to reproduce the problem, however it looks good. Also, there
> > is no side effect on Exynos platform.
> >
> > Mohit,
> > If you have some comments, please let us know.
> 
> - Quickly tested on SPEAr1310, working fine without any side effect. So is,
> Acked-by: Mohit Kumar <mohit.kumar at st.com>

Hi Bjorn Helgaas,

Would you apply this patch to your 'pci/host-designware' branch
with the following Tested-by and Acked-by?

Tested-by: Marek Vasut <marex at denx.de>
Tested-by:  Matthias Mann <m.mann at arkona-technologies.de>
Acked-by: Jingoo Han <jg1.han at samsung.com>
Acked-by: Mohit Kumar <mohit.kumar at st.com>


> ..
> >
> > > ---
> > >  drivers/pci/host/pcie-designware.c |    4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)




More information about the linux-arm-kernel mailing list