From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755071AbaDWMn1 (ORCPT ); Wed, 23 Apr 2014 08:43:27 -0400 Received: from mx1.redhat.com ([209.132.183.28]:62010 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752562AbaDWMnZ (ORCPT ); Wed, 23 Apr 2014 08:43:25 -0400 Message-ID: <5357B563.2060009@redhat.com> Date: Wed, 23 Apr 2014 14:43:15 +0200 From: Hans de Goede User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Kefeng Wang , Tejun Heo , Maxime Ripard , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org CC: wangkefeng.wang@huawei.com Subject: Re: [PATCH v5 0/3] ahci: add support for hisilicon sata References: <1398256603-20160-1-git-send-email-kefeng.wang@linaro.org> In-Reply-To: <1398256603-20160-1-git-send-email-kefeng.wang@linaro.org> X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 04/23/2014 02:36 PM, Kefeng Wang wrote: > channge in v5: > > 1) drop AHCI_HFLAGS(...) from ahci_xgene.c and do use host_flags > in ahci_platform_init_host, it's my carelessness. > Thanks, looks good now, this series is: Reviewed-by: Hans de Goede Regards, Hans > changes in v4: > > 1) add a dynamic host_flags parameter in ahci_platform_init_host > to make it more flexible, modify some driver to use new > way to pass host flags, suggested by Hans > > changes in v3: > > 1) move NO_FBS chunk after YES_FBS, Tejun's suggestion > 2) use ata_port_info pointer to avoid struct ata_port_info > on the stack twice > > changes in v2: > > 1) make code more concise according to Hans's advice > > Kefeng Wang (3): > ata: ahci: append new hflag AHCI_HFLAG_NO_FBS > libahci_platform: add host_flags parameter in > ahci_platform_init_host() > ahci: add support for Hisilicon sata > > .../devicetree/bindings/ata/ahci-platform.txt | 3 ++- > drivers/ata/ahci.h | 1 + > drivers/ata/ahci_da850.c | 3 ++- > drivers/ata/ahci_imx.c | 3 ++- > drivers/ata/ahci_platform.c | 8 +++++++- > drivers/ata/ahci_st.c | 2 +- > drivers/ata/ahci_sunxi.c | 9 ++++++--- > drivers/ata/ahci_xgene.c | 7 +++++-- > drivers/ata/libahci.c | 5 +++++ > drivers/ata/libahci_platform.c | 5 ++++- > include/linux/ahci_platform.h | 1 + > 11 files changed, 36 insertions(+), 11 deletions(-) >