[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1456850284.15454.23.camel@hpe.com>
Date: Tue, 01 Mar 2016 09:38:04 -0700
From: Toshi Kani <toshi.kani@....com>
To: "Moore, Robert" <robert.moore@...el.com>,
"rjw@...ysocki.net" <rjw@...ysocki.net>,
"Williams, Dan J" <dan.j.williams@...el.com>
Cc: "Zheng, Lv" <lv.zheng@...el.com>,
"elliott@....com" <elliott@....com>,
"linux-nvdimm@...ts.01.org" <linux-nvdimm@...ts.01.org>,
"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"devel@...ica.org" <devel@...ica.org>
Subject: Re: [PATCH v2 1/3] ACPI/NFIT: Update Control Region Structure to
comply ACPI 6.1
On Tue, 2016-03-01 at 15:13 +0000, Moore, Robert wrote:
>
> > -----Original Message-----
> > From: Toshi Kani [mailto:toshi.kani@....com]
> > Sent: Monday, February 22, 2016 1:55 PM
> > To: rjw@...ysocki.net; Williams, Dan J
> > Cc: Moore, Robert; Zheng, Lv; elliott@....com; linux-nvdimm@...ts.01.or
> > g;
> > linux-acpi@...r.kernel.org; linux-kernel@...r.kernel.org;
> > devel@...ica.org; Toshi Kani
> > Subject: [PATCH v2 1/3] ACPI/NFIT: Update Control Region Structure to
> > comply ACPI 6.1
> >
> > ACPI 6.1, Table 5-133, updates NVDIMM Control Region Structure as
> > follows.
> > - Valid Fields, Manufacturing Location, and Manufacturing Date
> > are added from reserved range. No change in the structure size.
> > - IDs defined as SPD values are arrays of bytes. The spec
> > clarified that they need to be represented as arrays of bytes
> > as well.
> >
> > This patch makes the following changes to support this update.
> > - Change 'struct acpi_nfit_control_region' to reflect the update.
> > SPD IDs are defined as arrays of bytes, so that they can be
> > treated in the same way regardless of CPU endianness and are
> > not miss-treated as little-endian numeric values.
>
>
> I don't think we are going to start changing the ACPI tables defined in
> the ACPICA headers because of this. We do in fact have macros for this
> purpose.
Can you elaborate what macros you suggest to use for this purpose?
Thanks,
-Toshi
Powered by blists - more mailing lists