Re: Fw: [PATCH] ahci: add ACard 8620 support (rough draft)

[Jeff Garzik]

Jim Yeh wrote:
> Dear Jeff,
>
> This is Jim,an IC PM from ACARD Technology. Thanks for your effort on 
> building Linux driver for ACARD ATP8620. Since ACARD already have a 
> verified AHCI driver for ATP8620, we'd like to submit this one for the 
> kernel built-in driver (please see attachment). This driver support 
> Linux Kernel 2.6.29 AHCI mode for ATP8620 (2ch SATA IC) and ATP8624 (4ch 
> SATA IC). It already passed our own stability and compatibility test 
> with SATA devices like HDD, ODD, or PortMultiplier. If you have any 
> question or need the hardware for testing, please don't hesitate to 
> contact me. Thanks again for your help.

Thank you for your email.

We understand you have verified your driver, but it is standard Linux 
driver policy to avoid code duplication.  Multiple implementations of 
AHCI drivers implies several negative factors:

* each bug fix must be reviewed across multiple AHCI drivers, to 
determine if the bug fix must be copied

* non-uniform user experience, depending on AHCI platform

* Shared AHCI code gets much more testing and verification.

* When a hardware vendor's AHCI chip reaches End Of Life, or the company 
goes out of business, users are not abandoned.

So, for the official Linux kernel drivers that are shipped by Red Hat, 
Novell/SuSE, Canonical and other vendors, we will need to modify the 
existing Linux AHCI driver to support ATP8620 and ATP8624.


And yes -- test hardware would be greatly appreciated!  If you could 
send these, that would be appreciated:

	To Tejun Heo
	------------
	1 x ATP8620
	1 x ATP8624

	To Jeff Garzik
	--------------
	1 x ATP8620
	1 x ATP8624

I assume this is PCI, PCI-X or PCI-Express form factor?

Regards and thanks,

	Jeff




--
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/