Re:Acard ATP8620 2SATA / 1 IDE driver - AHCI.C Nov082007

["jameshsu" <jameshsu@acard.com>]

FYI
----- Original Message -----
From: LaurenceWu
To: jameshsu
Cc: Jason Wu ; DerweiChen (陳德威)
Sent: Thursday, November 08, 2007 4:34 PM
Subject: Re: Should be Acard ATP8620 2SATA / 1 IDE driver


Hi James,

We didn't study about ata/ahci.c, but it should be based on AHCI1.0 or 1.1
spec. That is, NO P.M. FIS base switching, but supports both non-NCQ and NCQ
protocols.

For NCQ or nonNCQ, 8620 is very AHCI-like, although not fully compatible,
programmer can easily modify standard ahci.c
for 8620. The main differences between 8620 and AHCI are :

1. PRD table format changed, (please compare AHCI 1.x section 4.2.3.3 and
8620 datasheet section 7.3), 'I' bit in 8620 is defined as 'EOT' and NO
PRDTL value are available in the
    Command List Structure.

2. For NCQ transfer, PxIS bit 3(SDBS) is changed. ATP8620 add the Reg_144h
to accumulate 32 Sactive bits in each SDB FIS.
    The Reg_144h is RWC and all its 32 bits are 'ORed'  to form the PxIS
bit3 and interrupt, if PxIE bit 3 enabled.


Yes.  Modifying the ata/ahci.c is OK to support atp8620.

Regards,
Laurence
----- Original Message -----
From: jameshsu
To: laurence@mail.acard.com
Cc: Jason Wu
Sent: Thursday, November 08, 2007 10:36 AM
Subject: Re: Should be Acard ATP8620 2SATA / 1 IDE driver


FYI

ata/ahci.c driver structure should support our 8620 AHCI-compatible
hardware, right?!
Any restriction and special condition we should inform them??
If no, I will tell them nothing.
Please advise! Thanks!

James
----- Original Message -----
From: Jeff Garzik
To: jameshsu
Cc: Andrew Morton ; IDE/ATA development list ; tytso@us.ibm.com ; Matti
Aarnio ; James Bottomley ; 'David Miller' ; Daniel Weng ; Jason Wu ;
laurence@mail.acard.com
Sent: Thursday, November 08, 2007 6:13 AM
Subject: Re: Should be Acard ATP8620 2SATA / 1 IDE driver


On Tue, Nov 06, 2007 at 07:25:46PM +0800, jameshsu wrote:
> Hi Jeff,
>
> Please help Acard to add this chip spec on the web site in your earlier
> conveniance.
> http://gkernel.sourceforge.net/specs/
> http://linux-ata.org/driver-status.html#open_chipsets

Updated, thanks much!


> By the way, once you complete the SATA sample driver , please inform us ,
so
> we could modify, test and submit in the near future.
> If any chip info still missing or need us to involve, please let me know.

I began working on a sample driver, but looking at your document, it
appears you are AHCI-compatible?

If so, we would prefer to modify drivers/ata/ahci.c to support your
hardware.  This already supports AHCI variants from Intel, NVIDIA, ATI,
VIA, JMicron and Marvell.

Jeff