Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754924AbdGUXCu (ORCPT ); Fri, 21 Jul 2017 19:02:50 -0400 Received: from bombadil.infradead.org ([65.50.211.133]:44308 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753904AbdGUXCs (ORCPT ); Fri, 21 Jul 2017 19:02:48 -0400 Date: Fri, 21 Jul 2017 16:02:43 -0700 From: Darren Hart To: "Chakravarty, Souvik K" Cc: "Bhardwaj, Rajneesh" , "platform-driver-x86@vger.kernel.org" , "andy@infradead.org" , "linux-kernel@vger.kernel.org" , "Murthy, Shanth" Subject: Re: [PATCH v2 2/2] Telemetry: remove redundant macro definition Message-ID: <20170721230243.GA7888@fury> References: <1500560474-24268-1-git-send-email-rajneesh.bhardwaj@intel.com> <1500560474-24268-2-git-send-email-rajneesh.bhardwaj@intel.com> <5F7315E704FA0841B5DFCE90329B2BB462E90E8A@BGSMSX102.gar.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5F7315E704FA0841B5DFCE90329B2BB462E90E8A@BGSMSX102.gar.corp.intel.com> User-Agent: Mutt/1.8.0 (2017-02-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2671 Lines: 73 On Fri, Jul 21, 2017 at 03:29:39AM +0000, Chakravarty, Souvik K wrote: > Just missed the email from Darren. :-) > Reviewed-by: Souvik K Chakravarty Thank you Souvik. While these one liners are OK for trivial changes like this, please note that Andy and I depend on individual driver maintainers to provide thorough reviews, and it helps to document that review to help us understand what sorts of things you considered before approving the patch. Please keep this in mind in the future for more complex changes. This set is on Andy's queue, so the final review/follow-up will come from him. Thanks, > > > -----Original Message----- > > From: Chakravarty, Souvik K > > Sent: Friday, July 21, 2017 8:45 AM > > To: 'Rajneesh Bhardwaj' ; platform-driver- > > x86@vger.kernel.org > > Cc: dvhart@infradead.org; andy@infradead.org; linux- > > kernel@vger.kernel.org; Murthy, Shanth ; > > Bhardwaj, Rajneesh > > Subject: RE: [PATCH v2 2/2] Telemetry: remove redundant macro definition > > > > Both set of two looks good. +1. > > > > > -----Original Message----- > > > From: platform-driver-x86-owner@vger.kernel.org > > > [mailto:platform-driver- x86-owner@vger.kernel.org] On Behalf Of > > > Rajneesh Bhardwaj > > > Sent: Thursday, July 20, 2017 7:51 PM > > > To: platform-driver-x86@vger.kernel.org > > > Cc: dvhart@infradead.org; andy@infradead.org; linux- > > > kernel@vger.kernel.org; Murthy, Shanth ; > > > Chakravarty, Souvik K ; Bhardwaj, > > > Rajneesh > > > Subject: [PATCH v2 2/2] Telemetry: remove redundant macro definition > > > > > > Telemetry driver includes intel_telemetry.h which defines > > > TELEM_MAX_OS_ALLOCATED_EVENTS already. > > > > > > Signed-off-by: Rajneesh Bhardwaj > > > --- > > > drivers/platform/x86/intel_telemetry_pltdrv.c | 1 - > > > 1 file changed, 1 deletion(-) > > > > > > diff --git a/drivers/platform/x86/intel_telemetry_pltdrv.c > > > b/drivers/platform/x86/intel_telemetry_pltdrv.c > > > index 6393b3b1d5a6..e0424d5a795a 100644 > > > --- a/drivers/platform/x86/intel_telemetry_pltdrv.c > > > +++ b/drivers/platform/x86/intel_telemetry_pltdrv.c > > > @@ -46,7 +46,6 @@ > > > #define TELEM_SAMPLING_DEFAULT_PERIOD 0xD > > > > > > #define TELEM_MAX_EVENTS_SRAM 28 > > > -#define TELEM_MAX_OS_ALLOCATED_EVENTS 20 > > > #define TELEM_SSRAM_STARTTIME_OFFSET 8 > > > #define TELEM_SSRAM_EVTLOG_OFFSET 16 > > > > > > -- > > > 2.7.4 > > -- Darren Hart VMware Open Source Technology Center