Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752776AbcDKGv5 (ORCPT ); Mon, 11 Apr 2016 02:51:57 -0400 Received: from [198.137.202.9] ([198.137.202.9]:39264 "EHLO bombadil.infradead.org" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751960AbcDKGv4 (ORCPT ); Mon, 11 Apr 2016 02:51:56 -0400 Date: Sun, 10 Apr 2016 23:48:21 -0700 From: Darren Hart To: "Chakravarty, Souvik K" Cc: "Zha, Qipeng" , Aubrey Li , "platform-driver-x86@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] platform:x86 decouple telemetry driver from the optional IPC resources Message-ID: <20160411064821.GA14283@dvhart-mobl5.amr.corp.intel.com> References: <1459452489-46827-1-git-send-email-aubrey.li@linux.intel.com> <20160410024513.GA18689@dvhart-mobl5.amr.corp.intel.com> <5F7315E704FA0841B5DFCE90329B2BB42CAC7024@BGSMSX102.gar.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5F7315E704FA0841B5DFCE90329B2BB42CAC7024@BGSMSX102.gar.corp.intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1493 Lines: 41 On Mon, Apr 11, 2016 at 04:04:55AM +0000, Chakravarty, Souvik K wrote: > > > > -----Original Message----- > > From: Zha, Qipeng > > Sent: Monday, April 11, 2016 7:34 AM > > 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 > > > > >> 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. > > We went over this internally once. So OK from my POV. Qipeng and Souvik, I appreciate the responses. As a point of process, I won't include a "Reviewed-by:" unless you explicitly include it in your response. -- Darren Hart Intel Open Source Technology Center