Re: [PATCH spi for-5.4 13/14] spi: spi-fsl-dspi: Reduce indentation level in dspi_interrupt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 20 Aug 2019 at 16:02, Mark Brown <broonie@xxxxxxxxxx> wrote:
>
> On Sun, Aug 18, 2019 at 09:01:14PM +0300, Vladimir Oltean wrote:
> > There is no point in checking which interrupt source was triggered in
> > SPI_SR, since only EOQ and TCFQ modes trigger interrupts anyway (see the
> > writes to SPI_RSER). In DMA mode, the RSER is configured for RFDF_DIRS=1
> > and TFFF_DIRS=1, aka FIFO events generate eDMA requests and not CPU
> > interrupts.
>
> It's also good to check interrupt sources in case the interrupt line
> might be shared, that means that it's possible that the interrupt
> handler will be called when there's no interrupt at all from the IP.  It
> also helps with robustness in case there's some system error though
> that's (hopefully!) a lot less common.  This driver does set IRQF_SHARED
> so it does support that, I don't know how many systems could do it but
> it seems a shame to remove the support from the driver.

Ok, I hadn't thought of that, I'll add the check back.



[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux