Pierre Ossman <drzeus-list@xxxxxxxxx> wrote: > On Sat, 28 Jun 2008 14:43:13 +0200 > Haavard Skinnemoen <haavard.skinnemoen@xxxxxxxxx> wrote: > > > Tests 15 and 17 return -EILSEQ instead of -ETIMEDOUT. The at91_mci > > driver has the same problem, and I think it's a hardware issue -- the > > controller wrongly flags a CRC error instead of a data timeout error > > if the card doesn't respond with any CRC status after a write. I don't > > know how to work around that problem. > > If that's how the hardware behaves, then EILSEQ will have to do. The > test is more about forcing people to do proper error management in the > driver than anything else. Have a check that you don't report a bad > bytes_xfered though. bytes_xfered is 0 if any block failed. If I understand correctly, this is good enough, but not optimal. I want to improve this later, but I might need some more feedback from the DMA engine subsystem (e.g. adding "actual" and "status" fields to the descriptor.) The DMA slave interface isn't perfect yet, but I think the current incarnation is actually useful and performs well even though it's very basic. We can make incremental improvements later to improve error reporting, offer more advanced control over the transfers, and support other use cases better (e.g. audio.) Haavard -- To unsubscribe from this list: send the line "unsubscribe linux-embedded" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
