Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752296AbcDKCEd (ORCPT ); Sun, 10 Apr 2016 22:04:33 -0400 Received: from mga03.intel.com ([134.134.136.65]:62811 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751336AbcDKCEc convert rfc822-to-8bit (ORCPT ); Sun, 10 Apr 2016 22:04:32 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,462,1455004800"; d="scan'208";a="929600137" From: "Zha, Qipeng" To: Darren Hart , Aubrey Li , "Chakravarty, Souvik K" CC: "platform-driver-x86@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH] platform:x86 decouple telemetry driver from the optional IPC resources Thread-Topic: [PATCH] platform:x86 decouple telemetry driver from the optional IPC resources Thread-Index: AQHRixazLbAgizpGXUCCtbeMEgRqf5+CCbmAgAILw/A= Date: Mon, 11 Apr 2016 02:04:28 +0000 Message-ID: References: <1459452489-46827-1-git-send-email-aubrey.li@linux.intel.com> <20160410024513.GA18689@dvhart-mobl5.amr.corp.intel.com> In-Reply-To: <20160410024513.GA18689@dvhart-mobl5.amr.corp.intel.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_IC x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYmM2YjQ0OTktOGMwOC00YjhiLWFkZmEtNWJmY2ZhMjc2ZTk2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IjdZQmhqMkVrNTRXV2tCb2xZU1pRT3JsY2tjbFcwRm9HRFA3SUVMYnAzXC9rPSJ9 x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 665 Lines: 15 >> Currently the optional IPC resources prevent telemetry driver from >> probing if these resources are not in ACPI table. This patch decouples >> telemetry driver from these optional resources, so that telemetry >> driver has dependency only on the necessary ACPI resources. >> >> Signed-off-by: Aubrey Li >Given the impact to their recent contributions, I'm looking for reviews from Qipeng and Souvik before I merge this. >Qipeng, as the listed maintainer for these two files, I particularly need to hear from you. >Thanks, Hi Darren, I think this is a reasonable solution for such issue. Thanks Aubrey for this fixing patch.