Re: [2.6 patch] select ATA_SFF

From: Mark Lord
Date: Tue Apr 22 2008 - 10:44:28 EST


Tejun Heo wrote:
Mark Lord wrote:
Tejun Heo wrote:
Sergei Shtylyov wrote:
Jeff, Tejun, what "sff" in the file name actually means? Isn't it strange that the drivers lacking DMA support or not really compliant with SFF-8038i have to link with this file?

Maybe it should be libata-tf and libata-bmdma, but sff (sans bmdma)
and bmdma is acceptable, hopefully, right?

What's sff sans bmdma?

Supposed to be TF interface. IIRC, the SFF term was first from Alan although it's entirely possible that I misunderstood it and used it in the wrong way. Alan, can you please clear up the confusion?
..

SFF stands for "Small Form Factor", as in the "SFF Committee Information
Specification for Bus Master Programming Interface for IDE ATA Controllers Rev 1.0"
from May 16, 1994.

That document is basically a committee rubber-stamp of the earlier Intel
"PCI IDE Controller Specification Revision 1.0" dated 3/4/94.

Ah.. that means we have the whole naming thing wrong. Yewwww....
..

I suppose. If we were needlessly pedantic, then perhaps
the libata-bmdma.c should be renamed to libata-sff.c,
and the libata-sff.c should be renamed libata-tf.c

But there's probably been a document or two since then,
where the SFF folks have documented the TF interfaces, too.

So.. whatever. :)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/