On Fri, Jun 15, 2012 at 6:04 PM, Sergei Shtylyov <sshtylyov@xxxxxxxxxx> wrote:
> Hello.
>
>
> On 15-06-2012 12:09, Huacai Chen wrote:
>
>> Signed-off-by: Huacai Chen<chenhc@xxxxxxxxxx>
>> Signed-off-by: Hongliang Tao<taohl@xxxxxxxxxx>
>> Signed-off-by: Hua Yan<yanh@xxxxxxxxxx>
>
>
> You should have CCed the 'linux-ide' mailing list.
>
>
>> diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
>> index ebaf67e..3e3cfd8 100644
>> --- a/drivers/ata/ahci.c
>> +++ b/drivers/ata/ahci.c
>> @@ -183,7 +183,12 @@ static const struct ata_port_info ahci_port_info[] =
>> {
>> },
>> [board_ahci_sb700] = /* for SB700 and SB800 */
>> {
>> +#ifndef CONFIG_CPU_LOONGSON3
>> AHCI_HFLAGS (AHCI_HFLAG_IGN_SERR_INTERNAL),
>> +#else
>> + AHCI_HFLAGS (AHCI_HFLAG_IGN_SERR_INTERNAL |
>> + AHCI_HFLAG_32BIT_ONLY),
>> +#endif
>
>
> No, this #ifdef'ery won't do. You should add a new board type.
All Loongson-3 based machines use AMD SB700 chipsets, add a new board
type is better than #ifdef?
>
> MBR, Sergei
