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
