Re: [PATCH] Add IDE mode support for SB600 SATA

[Tejun Heo <htejun@gmail.com>]

Tejun Heo wrote:
> Conke Hu wrote:
>> ATI SB600 SATA controller supports 4 modes: Legacy IDE, Native IDE, 
>> AHCI and RAID. Legacy/Native IDE mode is designed for compatibility 
>> with some old OS without AHCI driver but looses SATAII/AHCI features 
>> such as NCQ. This patch will make SB600 SATA run in AHCI mode even if 
>> it was set as IDE mode by system BIOS.
[--snip--]
> Other than that, Acked-by: Tejun Heo <htejun@gmail.com>

At the second thought, I think this should be done in 
ahci_init_controller().

* Unlike Jmicron's case, this doesn't affect PCI bus scan.  Actually, it 
does change class code but that's not as disruptive as Jmicron's case 
and as long as ahci ignores class code, it doesn't really matter. 
Driver can be chosen by changing loading order - this is both plus and 
minus.

* As Arjan pointed out, that unlock-modify-lock sequence should be done 
on resume too.  ahci_init_controller() is the right place for such 
stuff.  This chip is going into notebooks, right?

-- 
tejun