Re: [patch 4/4] MMC-over-SPI host driver

[Pierre Ossman <drzeus-mmc-p3sGCRWkH8CeZLLa646FqQ@public.gmane.org>]

David Brownell wrote:
>>> Right now it looks like the main reason to keep including mmc.h is
>>> to handle one aspect of mapping the data error token. ...
>> Don't map at all. Just give the core what you got from the card.
> 
> Tried that, it fails for the obvious (in retrospect) reason:  after each
> command that's issued, something needs to ensure that the return code won't
> be MMC_ERR_NONE after errors.  That means looking at those R1 bits which
> report that hardware status.

The error codes were never supposed to report card status, only controller
status. So MMC_ERR_NONE is perfectly valid for a status response filled with
error bits.

> And it's got to do that pretty much immediately, since when an error shows
> up in the command, the data stage must not be performed.  It's impractical
> to consider delegating such triage to the core.
> 

This is an architectural limitation in the MMC layer that affects the native
protocol aswell. If you need it fixed, I'd say we have to do it in a proper,
general manner.

Why can't you do what native controllers do, just fling the data out there and
break when the card doesn't respond properly to it?

>>> Having a simple flag in mmc_host would be much better.
>> Perhaps. But there's a lot of value in consistency.
> 
> Unless you strongly object, I'll go the flag route though.  The thing is,
> the IOS thing ("kitchen sink") will never vanish if it keeps growning at
> every opportunity.
> 

But if we keep inventing new systems ad-hoc, we'll have a much bigger mess to
deal with.

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/