Re: [PATCH] ahci.c: walkaround for SB600 SATA internal error issue

[Tejun Heo <htejun@gmail.com>]

Conke Hu wrote:
>   There is a HW issue in ATI SB600 SATA that PxSERR.E should not be
> set on some conditions, for example, when there is no media in SATA
> CD/DVD drive or media is not ready, AHCI controller fails to execute
> ATAPI commands and reports PORT_IRQ_TF_ERR, but ATI SB600 SATA
> controller sets PxSERR.E at the
> same time, which is not necessary.
>    This patch is just to ignore the INTERNAL ERROR in such case.
> Without this patch, ahci error handler will report many errors as
> below:
>    ----------- cut from dmesg -----------
> ata9: soft resetting port
> ata9: SATA link up 1.5 Gbps (SStatus 113 SControl 300)
> ata9.00: configured for UDMA/33
> ata9: EH complete
> ata9.00: exception Emask 0x40 SAct 0x0 SErr 0x800 action 0x2
> ata9.00: (irq_stat 0x40000001)
> ata9.00: cmd a0/00:00:00:00:20/00:00:00:00:00/a0 tag 0 cdb 0x0 data 0
>        res 51/24:03:00:00:20/00:00:00:00:00/a0 Emask 0x40 (internal error)
> ata9: soft resetting port
> ata9: SATA link up 1.5 Gbps (SStatus 113 SControl 300)
> ata9.00: configured for UDMA/33
> ata9: EH complete
> ata9.00: exception Emask 0x40 SAct 0x0 SErr 0x800 action 0x2
> ata9.00: (irq_stat 0x40000001)
> ata9.00: cmd a0/01:00:00:00:00/00:00:00:00:00/a0 tag 0 cdb 0x43 data 12 in
>        res 51/24:03:00:00:00/00:00:00:00:00/a0 Emask 0x40 (internal error)
>    -------- end cut ---------
> 
> Signed-off-by: Conke Hu <conke.hu@amd.com>

Acked-by: Tejun Heo <htejun@gmail.com>

-- 
tejun