Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965480AbbD0WSo (ORCPT ); Mon, 27 Apr 2015 18:18:44 -0400 Received: from mail-bn1bbn0103.outbound.protection.outlook.com ([157.56.111.103]:14707 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S965422AbbD0WSj convert rfc822-to-8bit (ORCPT ); Mon, 27 Apr 2015 18:18:39 -0400 Authentication-Results: spf=none (sender IP is 165.204.84.221) smtp.mailfrom=amd.com; intel.com; dkim=none (message not signed) header.d=none; X-WSS-ID: 0NNHKMX-07-ZX2-02 X-M-MSG: Message-ID: <553EB5B4.4030901@amd.com> Date: Mon, 27 Apr 2015 17:18:28 -0500 From: Suravee Suthikulpanit User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: "Rafael J. Wysocki" CC: "Zheng, Lv" , "mika.westerberg@linux.intel.com" , "Moore, Robert" , "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 1/3] ACPICA: Add ACPI _CLS processing References: <3419939.l8boJE7mrB@vostro.rjw.lan> In-Reply-To: <3419939.l8boJE7mrB@vostro.rjw.lan> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8BIT X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:165.204.84.221;CTRY:US;IPV:NLI;EFV:NLI;BMV:1;SFV:NSPM;SFS:(10019020)(6009001)(428002)(51704005)(24454002)(377454003)(164054003)(199003)(479174004)(189002)(110136001)(50986999)(59896002)(86362001)(4001350100001)(83506001)(19580395003)(46102003)(92566002)(80316001)(101416001)(19580405001)(50466002)(106466001)(36756003)(65816999)(105586002)(2950100001)(76176999)(47776003)(65806001)(33656002)(77096005)(87266999)(87936001)(77156002)(65956001)(62966003)(23676002)(54356999);DIR:OUT;SFP:1102;SCL:1;SRVR:BN1PR02MB199;H:atltwp01.amd.com;FPR:;SPF:None;MLV:sfv;A:1;MX:1;LANG:en; X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN1PR02MB199; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5002010)(5005006)(3002001);SRVR:BN1PR02MB199;BCL:0;PCL:0;RULEID:;SRVR:BN1PR02MB199; X-Forefront-PRVS: 0559FB9674 X-OriginatorOrg: amd4.onmicrosoft.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Apr 2015 22:18:35.4631 (UTC) X-MS-Exchange-CrossTenant-Id: fde4dada-be84-483f-92cc-e026cbee8e96 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=fde4dada-be84-483f-92cc-e026cbee8e96;Ip=[165.204.84.221];Helo=[atltwp01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN1PR02MB199 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2881 Lines: 66 On 04/27/2015 03:38 PM, Rafael J. Wysocki wrote: > On Sunday, April 26, 2015 10:45:29 PM Suthikulpanit, Suravee wrote: >> On 4/24/15, 21:28, "Rafael J. Wysocki" wrote: >> >>> On Friday, April 24, 2015 04:08:31 PM Suravee Suthikulpanit wrote: >>>> On 4/16/15 20:45, Zheng, Lv wrote: >>>>> Before back porting this to ACPICA, let me ask one simple question. >>>>> According to the spec, the _CLS is optional and PCI specific. >>>>> So why should we implement it in ACPICA core not OSPM specific >>>> modules? >>>>> If this need to be implemented in ACPICA, then what about the >>>> following device identification objects? >>>>> _DDN, _HRV, _MLS, _PLD, _STR, _SUN >>>>> >>>>> Thanks and best regards >>>>> -Lv >>>> >>>> Hi, >>>> >>>> Sorry for late reply. As for the justification for introducing the _CLS >>>> support in the ACPICA, this is mainly because ACPI does not currently >>>> define _CID for certain device classes, which used to mostly be PCI >>>> devices. Instead, ACPI spec mentioned that _CLS can be used for loading >>>> generic drivers on hardware that is compatible with PCI-defined device >>>> classes, but that is not implemented on the PCI bus (and is therefore >>>> enumerated by ACPI.) >>> >>> I think it would be good to point to the particular part of the spec >>> making that provision. In what section is that mentioned, exactly? >> >> Here is the copied from section 6.1.3 _CLS (Class Code) from ACPI 5.1 spec: >> "This object is used to supply OSPM with the PCI-defined class, subclass >> and programming interface for a device. This object is optional but may be >> useful for generic drivers written for PCI devices that move off of PCI >> and are enumerated by ACPI." > > OK, so the "move off of PCI" part should be understood as "something that > used to be on the PCI bus, but now may be included into an SoC directly > in which case it won't be a PCI device any more", right? That's right. For example, the SATA controller is a good example for this case. On most x86 platforms, they are often enumerated as PCI devices. However, in the ARM64 SOC (e.g. on AMD Seattle), it could be enumerated as non-PCI device. Thanks, Suravee > >> Otherwise, if the community think it¹s better to not putting the _CLS the >> _CLS parsing code in ACPICA since, I can try looking into pulling the code >> out of ACPICA. I also noticed a little issue in the patch series where the >> ACPI_VALID_CLS is used in the patch 1 but defined in patch 2. I can send >> out v9 with the fix once we agree on the _CLS parsing. > > To me that pretty much depends on the answer to the above question. > > -- 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/