Received-SPF: pass (google.com: domain of linux-kernel+bounces-160879-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Fri, 26 Apr 2024 20:31:26 -0700
From: Ira Weiny <ira.weiny@intel.com>
To: Dan Williams <dan.j.williams@intel.com>, Ira Weiny <ira.weiny@intel.com>,
	Dave Jiang <dave.jiang@intel.com>, Jonathan Cameron
	<jonathan.cameron@huawei.com>, Smita Koralahalli
	<Smita.KoralahalliChannabasappa@amd.com>, Shiju Jose <shiju.jose@huawei.com>
CC: Dan Carpenter <dan.carpenter@linaro.org>, Yazen Ghannam
	<yazen.ghannam@amd.com>, Davidlohr Bueso <dave@stgolabs.net>, "Alison
 Schofield" <alison.schofield@intel.com>, Vishal Verma
	<vishal.l.verma@intel.com>, Ard Biesheuvel <ardb@kernel.org>,
	<linux-efi@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-cxl@vger.kernel.org>, Ira Weiny <ira.weiny@intel.com>, "Rafael J.
 Wysocki" <rafael@kernel.org>, Tony Luck <tony.luck@intel.com>, "Borislav
 Petkov" <bp@alien8.de>
Subject: Re: [PATCH v3 1/2] acpi/ghes: Process CXL Component Events
Message-ID: <662c718e590de_15addb294d1@iweiny-mobl.notmuch>
References: <20240426-cxl-cper3-v3-0-6ade7dfc849e@intel.com>
 <20240426-cxl-cper3-v3-1-6ade7dfc849e@intel.com>
 <662c3db180909_b6e02942e@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <662c3db180909_b6e02942e@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?j4Yh9jEdCNmNTicg69TbG/Q2B7AocKqMPsc9xPPgEZGbTg6+5adMv66K4hng?=
 =?us-ascii?Q?Rh8wRts9ZNepaM4/bSTcIx9gUm6pJc4/nlYzNFgUmuA/EbI3ApaxW4BGgroj?=
 =?us-ascii?Q?jpkPfrE1eQsRLZNk1R0bbCQlLPQjx9VUYSC+TzL17eYVeFpKJDbv4U0Obrug?=
 =?us-ascii?Q?4juA4y+nvxtY5gSLSOPJauH3nfvzNdi8IQzRenKx0b2r+pnGF/GSlmj4AfQR?=
 =?us-ascii?Q?BI5KbOLDFHrHkphWxZYWJkwEOq95fVcZe8Ndt3za5lMbPkjlxL8e99hTDw92?=
 =?us-ascii?Q?4HxFw1Sh3Ara7FXqIKDOXDcpnCoNuZEVmb0pQxWURCcz5LDTKVTkAlDgfkB0?=
 =?us-ascii?Q?Jne1TWG9CQH39PwcIXi+zvYRwpHQbS/4RIRGgvs4AnzsvZV+cFkgT8t69DPs?=
 =?us-ascii?Q?pVzrQCgvs1UgL5Mxydjbxl0UQWunksohQXtbKOC+mbrqRg7nj2d9ttKb+l95?=
 =?us-ascii?Q?QllZrbofak9ONtoXb2uY+86Ru7Kh58sWhVcdVLPH3V8+/dDGTppaukGkRjTv?=
 =?us-ascii?Q?X7pprXJZbt7D4gg/PQsdPeRtqhr9nKYQICAsF9ZvpB7T+P/7avG35Q83TotJ?=
 =?us-ascii?Q?8Jprbo4cluovZjre6nD/I8FrHV0r4vfdIttc4KEzZs+a4eC41+Xt+OiwWoOQ?=
 =?us-ascii?Q?kd0oNPhSlKzkagrIrhtxQCs4fIdF72KCSfHY68YCUx1seiGI6rcWrt6JoZE6?=
 =?us-ascii?Q?6IdQIF6I0XmLZy7mvn6anYvBKRVA5GIwcd0sr7XS5S6Zo5iqRyCVwhPulWwr?=
 =?us-ascii?Q?RW/GwYcNeE2AQ5kcfsOAVcT3htYeMCb/3zwmMsL342J5tCgiQoApKzXHRfMY?=
 =?us-ascii?Q?iTC5+g5KGLtSjL9n/vaVCa2VE1UmHb9SdTxH654GV1Rkd6QyfiUoW5yoTHyA?=
 =?us-ascii?Q?j6SflkhgACEhgXpMEy8uDnjzRe2utYRIenyobQ92P93jIIh3JS2A8olqTrD+?=
 =?us-ascii?Q?rdYv1ho+QbQN5Z86B0Bla4AlFjR2nyqMX8luR/m9QuShf/YZr7MfVOebN3hD?=
 =?us-ascii?Q?7ENbZeerSZeN3A6sHlseZOFVLCKzbxmEHj9cRx38iLdQ4Z0Aqn3U2zsJiy0o?=
 =?us-ascii?Q?Wbr35s5ccwu58bLJS3QWcjEf7mnYafva/AKgDZt+lBA3VTSVx/SsQVbykWWG?=
 =?us-ascii?Q?V+gELauWkoPQFP0zqAmRr8NMhUCk7+viHikZIh6VQ2eW2RCZ0DbkHTkHiADw?=
 =?us-ascii?Q?o5DC+tgf+JioeQbF/3/taXSG7tlhL+Bv77DKYr47baV/KvfpkcG5uFFRdbjd?=
 =?us-ascii?Q?dt0pk8e3jwLF3uOArudShi5mS49zc23AWkUgsdp12pY7Yn3P6iLkHi8047Wy?=
 =?us-ascii?Q?z8u+WjLHbQxOhca02cf2f1g2rYhGuNPBm0exco0m9NHAIay5RObjAePoWPK4?=
 =?us-ascii?Q?MavE3bAP4Wgo4N+BLfHPEOaYwY0aE5GBz5zSd5ClPzLGliIrAovUOG9jRETW?=
 =?us-ascii?Q?iDxex6d+lz08w3rvTKZO/MMpHSseoA9zzRtZLcmg2pjF37G2b/V8lOyMKaJP?=
 =?us-ascii?Q?tIwrqPM2pxz5wP09wo8dCbpIQapp5NoTz0lFfCJPJKc5g5BzZup19wyJIbN3?=
 =?us-ascii?Q?U9w72cd4JLj17AXQ+WJFmlmbWS5Q21V+nsRTyIx0?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 1e3b3f89-14ed-4270-c2fb-08dc666a8747
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Apr 2024 03:31:30.3641
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: tbuEk6RJ4t9yfMxRbS7OGBDJ8SSBQhcpwgMQNVISM2Hc4AWFgLI1g6YWp22OnYZ8S5RpbEHy4HnjZ8OFjjZ41g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7687
X-OriginatorOrg: intel.com

Dan Williams wrote:
> Ira Weiny wrote:

[snip]

> > 
> > [0]
> > Link: https://lore.kernel.org/all/cover.1711598777.git.alison.schofield@intel.com/
> > [1]
> > Link: https://lore.kernel.org/all/65d111eb87115_6c745294ac@dwillia2-xfh.jf.intel.com.notmuch/
> > [2]
> > Link: https://lore.kernel.org/all/b963c490-2c13-4b79-bbe7-34c6568423c7@moroto.mountain/
> 
> Like I mentioned last time, perhaps Dave can reformat these lines on
> applying to:

Yes you did and I rushed these out today.

> 
> Link: http://lore.kernel.org/r/cover.1711598777.git.alison.schofield@intel.com [0]
> Link: http://lore.kernel.org/r/65d111eb87115_6c745294ac@dwillia2-xfh.jf.intel.com.notmuch [1]
> Link: http://lore.kernel.org/r/b963c490-2c13-4b79-bbe7-34c6568423c7@moroto.mountain [2]
> 
> Other than that, this looks good to me:
> 
> Reviewed-by: Dan Williams <dan.j.williams@intel.com>

Wow!  B4 actually picked these up and added them for me!

I'll respin v4 is like 5 min.

Thanks for the review,
Ira