Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756296AbdLOUxl (ORCPT ); Fri, 15 Dec 2017 15:53:41 -0500 Received: from mga06.intel.com ([134.134.136.31]:34677 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755650AbdLOUxj (ORCPT ); Fri, 15 Dec 2017 15:53:39 -0500 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.45,406,1508828400"; d="scan'208";a="3182967" Date: Fri, 15 Dec 2017 13:53:36 -0700 From: Ross Zwisler To: "Rafael J. Wysocki" Cc: Ross Zwisler , Linux Kernel Mailing List , "Anaczkowski, Lukasz" , "Box, David E" , "Kogut, Jaroslaw" , "Koss, Marcin" , "Koziej, Artur" , "Lahtinen, Joonas" , "Moore, Robert" , "Nachimuthu, Murugasamy" , "Odzioba, Lukasz" , "Rafael J. Wysocki" , "Rafael J. Wysocki" , "Schmauss, Erik" , "Verma, Vishal L" , "Zheng, Lv" , Andrew Morton , Balbir Singh , Brice Goglin , Dan Williams , Dave Hansen , Jerome Glisse , John Hubbard , Len Brown , Tim Chen , devel@acpica.org, ACPI Devel Maling List , Linux Memory Management List , "linux-nvdimm@lists.01.org" Subject: Re: [PATCH v3 2/3] hmat: add heterogeneous memory sysfs support Message-ID: <20171215205336.GB5454@linux.intel.com> References: <20171214021019.13579-1-ross.zwisler@linux.intel.com> <20171214021019.13579-3-ross.zwisler@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1102 Lines: 28 On Fri, Dec 15, 2017 at 01:52:03AM +0100, Rafael J. Wysocki wrote: <> > > diff --git a/drivers/acpi/hmat/core.c b/drivers/acpi/hmat/core.c > > new file mode 100644 > > index 000000000000..61b90dadf84b > > --- /dev/null > > +++ b/drivers/acpi/hmat/core.c > > @@ -0,0 +1,536 @@ > > +/* > > + * Heterogeneous Memory Attributes Table (HMAT) representation in sysfs > > + * > > + * Copyright (c) 2017, Intel Corporation. > > + * > > + * This program is free software; you can redistribute it and/or modify it > > + * under the terms and conditions of the GNU General Public License, > > + * version 2, as published by the Free Software Foundation. > > + * > > + * This program is distributed in the hope it will be useful, but WITHOUT > > + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or > > + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > > + * more details. > > + */ > > Minor nit for starters: you should use SPDX license indentifiers in > new files and if you do so, the license boilerplace is not necessary > any more. Okay, I'll fix that up.