From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933020AbcFBVpN (ORCPT ); Thu, 2 Jun 2016 17:45:13 -0400 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:52277 "HELO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S932462AbcFBVpJ (ORCPT ); Thu, 2 Jun 2016 17:45:09 -0400 From: "Rafael J. Wysocki" To: Borislav Petkov , Dave Hansen Cc: linux-kernel@vger.kernel.org, x86@kernel.org, dave.hansen@linux.intel.com, adrian.hunter@intel.com, ak@linux.intel.com, luto@kernel.org, dvhart@infradead.org, dougthompson@xmission.com, edubezval@gmail.com, hpa@zytor.com, mingo@redhat.com, jacob.jun.pan@linux.intel.com, kan.liang@intel.com, lenb@kernel.org, linux-acpi@vger.kernel.org, linux-edac@vger.kernel.org, linux-mmc@vger.kernel.org, linux-pm@vger.kernel.org, mchehab@osg.samsung.com, peterz@infradead.org, platform-driver-x86@vger.kernel.org, rafael.j.wysocki@intel.com, rajneesh.bhardwaj@intel.com, souvik.k.chakravarty@intel.com, srinivas.pandruvada@linux.intel.com, eranian@google.com, tglx@linutronix.de, tony.luck@intel.com, ulf.hansson@linaro.org, viresh.kumar@linaro.org, vishwanath.somayaji@intel.com, zheng.z.yan@intel.com, rui.zhang@intel.com Subject: Re: [PATCH 01/20] x86, intel: Introduce macros for Intel family numbers Date: Thu, 02 Jun 2016 23:49 +0200 Message-ID: <2025489.OPKq8WOssX@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/4.5.0-rc1+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20160602065715.GA10803@pd.tnic> References: <20160602001157.D8BE59D7@viggo.jf.intel.com> <20160602065715.GA10803@pd.tnic> 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 On Thursday, June 02, 2016 08:57:15 AM Borislav Petkov wrote: > On Wed, Jun 01, 2016 at 05:11:57PM -0700, Dave Hansen wrote: > > +#define INTEL_FAM6_MODEL_CORE_YONAH 0x0E > > +#define INTEL_FAM6_MODEL_CORE2_MEROM 0x0F > > That "MODEL_" part looks redundant too IMO - you could simply do > > INTEL_FAM6_NEHALEM and > INTEL_FAM6_SKYLAKE_DESKTOP > ... > > and so on and it is still clear what it is. I'm with you on that, but that's rather minor and the changes in this series are generally fine by me, so Dave please feel free to add Acked-by: Rafael J. Wysocki to all of it. Thanks, Rafael