svn commit: r188621 - head/sys/dev/ata/chipsets

[Sam Leffler]

Alexander Motin wrote:
> Author: mav
> Date: Sat Feb 14 23:02:59 2009
> New Revision: 188621
> URL: http://svn.freebsd.org/changeset/base/188621
>
> Log:
>   Tunes to AHCI reset sequences:
>   - specification claims that 1 second is just a maximum controller reset time;
>   implement controller reset properly to save almost 1 second of boot, and
>   about half second of resume time;
>   - enable channel interrupts only after channel status reset to fix duplicate
>   device creation on resume due to unwanted device connection event;
>   - as described in specification, wait for disk ready status after channel
>   power-up; it is not so important when disk already touched by BIOS, but
>   solves device not ready problems on resume and probably some other cases.
>   - uncomment channel stop/start on soft-reset as it is declared mandatory by
>   specification; it was commented due to some random drive detection problems
>   on VIA and JMicron controllers, but I hope it is fixed by previous point.
>
>   

These sounds like unrelated changes.  Given the fragility of ata interop 
you should do these as separate commits in case folks need to bisect 
changes on breakage.

    Sam