Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966586AbbDWQhf (ORCPT ); Thu, 23 Apr 2015 12:37:35 -0400 Received: from v094114.home.net.pl ([79.96.170.134]:60706 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S966393AbbDWQhd (ORCPT ); Thu, 23 Apr 2015 12:37:33 -0400 From: "Rafael J. Wysocki" To: "Suthikulpanit, Suravee" Cc: "mika.westerberg@linux.intel.com" , "robert.moore@intel.com" , "lv.zheng@intel.com" , "hanjun.guo@linaro.org" , "lenb@kernel.org" , "hdegoede@redhat.com" , "tj@kernel.org" , "mjg59@srcf.ucam.org" , "gregkh@linuxfoundation.org" , "al.stone@linaro.org" , "graeme.gregory@linaro.org" , "Duran, Leo" , "linux-ide@vger.kernel.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linaro-acpi@lists.linaro.org" Subject: Re: [V8 PATCH 0/3] Introduce ACPI support for ahci_platform driver Date: Thu, 23 Apr 2015 19:02:16 +0200 Message-ID: <1478598.GRG3QyNdDN@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/4.0.0+; KDE/4.11.5; x86_64; ; ) In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5641 Lines: 129 On Thursday, April 23, 2015 04:32:25 PM Suthikulpanit, Suravee wrote: > Mika/Rafael, > > If there are no other concerns about this patch series, do you think this > ready to be pushed to linux-next tree along with the rest of the ARM64 > ACPI patch series from Linaro? There are concerns regarding patch [1/3] from Bob and Lv (please look at the last message from Bob in this thread for details). > >On 3/30/2015 4:56 PM, Suravee Suthikulpanit wrote: > >> This patch series introduce ACPI support for AHCI platform driver. > >> Existing ACPI support for AHCI assumes the device controller is a PCI > >>device. > >> Since there is no ACPI _CID for generic AHCI controller, the driver > >> could not use it for matching devices. Therefore, this patch introduces > >> a mechanism for drivers to match devices using ACPI _CLS method. > >> _CLS contains PCI-defined class-code. > >> > >> This patch series also modifies ACPI modalias to add class-code to the > >> exisiting format, which currently only uses _HID and _CIDs. This is > >>required > >> to support loadable modules w/ _CLS. > >> > >> This is rebased from and tested with: > >> > >> http://git.linaro.org/leg/acpi/acpi.git acpi-5.1-v11 > >> > >> This topic was discussed earlier here (as part of introducing support > >>for > >> AMD Seattle SATA controller): > >> > >> http://marc.info/?l=linux-arm-kernel&m=141083492521584&w=2 > >> > >> Changes from V7 (https://lkml.org/lkml/2015/3/26/592) > >> * [1/3] Return AE_AML_OPERAND_TYPE when _CLS package containing > >> invalid type (per Robert Moore suggestion). > >> * [2/3] Fixed build error due missing ACPI_DEVICE_CLASS definition > >> when disabling ACPI. > >> > >> Changes from V6 (https://lkml.org/lkml/2015/3/25/797) > >> * Adding Acked-by Mika, and Reviewed-by Hanjun > >> * Minor clen up to use lower case 0xffffff for cls_msk > >> (per Mika suggestions). > >> * Modify the ACPI_DEVICE_CLASS macro to use designated initializer > >> (per Mika suggestions). > >> > >> Changes from V5 (https://lkml.org/lkml/2015/3/6/24) > >> * Rebased and tested with acpi-5.1-v11 > >> * Splitting up the ACPICA changes into a separate patch [1/3]. > >> (per Mika suggestion) > >> * Adding class-code mask support (per Mika suggestion) > >> * Use macro to define struct acpi_device_id entry (per Mika suggestion) > >> * Note: Mika also recommend reordering the member of struct > >>acpi_device_id > >> and define a macro to be used for declaring each table entry. This > >>is a > >> large amount of changes, and will be done separtely from this patch > >>series. > >> > >> Changes from V4 (https://lkml.org/lkml/2015/3/2/56) > >> * [1/2] Bug fixed: Reorder the declaration of > >> struct acpi_pnp_device_id cls in the struct acpi_device_info > >> (include/acpi/actypes.h) since compatible_id_list must be last one. > >> * [2/2] Added Acked-by: Tejun Heo > >> > >> Changes from V3 (https://lkml.org/lkml/2015/2/8/106) > >> * Instead of introducing new structure acpi_device_cls, add cls into > >> the acpi_device_id, and modify the __acpi_match_device > >> to also match for cls. (per Mika suggestion.) > >> * Add loadable module support, which requires changes in ACPI > >> modalias. (per Mika suggestion.) > >> * Rebased and tested with acpi-5.1-v9 > >> > >> Changes from V2 (https://lkml.org/lkml/2015/1/5/662) > >> * Update with review comment from Rafael in patch 1/2 > >> * Rebased and tested with acpi-5.1-v8 > >> > >> Changes from V1 (https://lkml.org/lkml/2014/12/19/345) > >> * Rebased to 3.19.0-rc2 > >> * Change from acpi_cls in device_driver to acpi_match_cls (Hanjun > >>comment) > >> * Change the matching logic in acpi_driver_match_device() due to the > >>new > >> special PRP0001 _HID. > >> * Simplify the return type of acpi_match_device_cls() to boolean. > >> > >> Changes from RFC (https://lkml.org/lkml/2014/12/17/446) > >> * Remove #ifdef and make non-ACPI version of the acpi_match_device_cls > >> as inline. (per Arnd) > >> * Simplify logic to retrieve and evaluate _CLS handle. (per Hanjun) > >> > >> Suravee Suthikulpanit (3): > >> ACPICA: Add ACPI _CLS processing > >> ACPI / scan: Add support for ACPI _CLS device matching > >> ata: ahci_platform: Add ACPI _CLS matching > >> > >> drivers/acpi/acpica/acutils.h | 3 ++ > >> drivers/acpi/acpica/nsxfname.c | 21 +++++++++-- > >> drivers/acpi/acpica/utids.c | 73 > >>+++++++++++++++++++++++++++++++++++++++ > >> drivers/acpi/scan.c | 36 ++++++++++++++++--- > >> drivers/ata/Kconfig | 2 +- > >> drivers/ata/ahci_platform.c | 9 +++++ > >> include/acpi/acnames.h | 1 + > >> include/acpi/actypes.h | 4 ++- > >> include/linux/acpi.h | 14 ++++++++ > >> include/linux/mod_devicetable.h | 2 ++ > >> scripts/mod/devicetable-offsets.c | 2 ++ > >> scripts/mod/file2alias.c | 32 +++++++++++++++-- > >> 12 files changed, 189 insertions(+), 10 deletions(-) > >> > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/