From mboxrd@z Thu Jan 1 00:00:00 1970 From: "kefeng.wang" Subject: Re: [PATCH] ahci: disable ncq feature for hisilicon sata Date: Tue, 10 Jun 2014 18:07:05 +0800 Message-ID: <5396D8C9.6020607@linaro.org> References: <1398416351-30567-1-git-send-email-kefeng.wang@linaro.org> <20140514170757.GD15690@htj.dyndns.org> <5396D529.308@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from mail-pa0-f45.google.com ([209.85.220.45]:38771 "EHLO mail-pa0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750836AbaFJKHr (ORCPT ); Tue, 10 Jun 2014 06:07:47 -0400 Received: by mail-pa0-f45.google.com with SMTP id hz1so539173pad.18 for ; Tue, 10 Jun 2014 03:07:47 -0700 (PDT) In-Reply-To: <5396D529.308@linaro.org> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Tejun Heo Cc: Hans de Goede , Zhangfei Gao , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, wangkefeng.wang@huawei.com, xuwei5@hisilicon.com Hi Tejun, I have a question, the hip04 and hix5hd2 soc use the old version of IP, but they also have a new version of IP which support FBS and NCQ, and now the new one is used by a new soc. How can I distinguish them, use "hisilicon,hisi-ahci-vX" or just add "generic-ahci" string for the IP which is conformant with spec and has no special customization and issue? Any advices, thanks. Regards, Kefeng On 2014-6-10 17:51, kefeng.wang wrote: > NCQ feature is unsupported on hisilicon sata controller, so disable it. > This version of IP is used by hip04 and hix5hd2 soc. > > Signed-off-by: Kefeng Wang > --- > drivers/ata/ahci_platform.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/ata/ahci_platform.c b/drivers/ata/ahci_platform.c > index ebe505c..0cfda8b 100644 > --- a/drivers/ata/ahci_platform.c > +++ b/drivers/ata/ahci_platform.c > @@ -58,7 +58,7 @@ static int ahci_probe(struct platform_device *pdev) > } > > if (of_device_is_compatible(dev->of_node, "hisilicon,hisi-ahci")) > - hflags |= AHCI_HFLAG_NO_FBS; > + hflags = AHCI_HFLAG_NO_FBS | AHCI_HFLAG_NO_NCQ; > > rc = ahci_platform_init_host(pdev, hpriv, &ahci_port_info, > hflags, 0, 0); >