Re: [PATCH] SPI: Add driver for Cadence SPI controller

[Mark Brown]

On Tue, Mar 18, 2014 at 12:13:45PM +0000, Harini Katakam wrote:

> I'm going to remove the bits_per_word check anyway.
> But the clock configuration still needs to be done.
> Where should it be done spi_setup() or transfer?

It needs to be done on the transfer - that is required anyway since the
clock rate is specified per transfer.

> > > In this case both possible interrupt conditions are handled.

> > Are you sure that's the case, and even if you are that's still not
> > handling the case where the device isn't flagging an interrupt at all.

> The IXR_ALL mask is only used to disable all the interrupts in the beginning.
> These two are the only interrupts enabled.
> And RXNEMPTY status is just polled. That interrupt is not enabled either

This is all going to be fragile in the face of bugs or changes in the
code though and like I keep saying it doesn't handle interrupt sharing.

Attachment: signature.asc
Description: Digital signature