Messages in this thread | | | Date | Tue, 18 Dec 2007 09:04:24 +0100 | From | Pierre Ossman <> | Subject | Re: [PATCH][MMC] Fix wrong EXT_CSD_REV handling |
| |
On Fri, 14 Dec 2007 11:01:03 +0900 "Kyungmin Park" <kmpark@infradead.org> wrote:
> > Yes, reserved word should or must be zero, then it should check "if > (ext_csd_struct <= 2)" instead of ">= 2". > In the Spec. 4.2, it can have three value 0, 1, or 2. There's no other > restriction.
As I said, the spec doesn't say "must", and I've seen cards with bogus data in there so I'm afraid we can't rely on the fields being zero. The version field is the only decent way of determining what to expect in the rest of the structure.
Rgds -- -- Pierre Ossman
Linux kernel, MMC maintainer http://www.kernel.org PulseAudio, core developer http://pulseaudio.org rdesktop, core developer http://www.rdesktop.org
| |