Received-SPF: pass (google.com: domain of linux-kernel+bounces-89165-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Fri, 1 Mar 2024 13:53:21 -0800
From: Ira Weiny <ira.weiny@intel.com>
To: Dan Williams <dan.j.williams@intel.com>, Ira Weiny <ira.weiny@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>, Dave Jiang
	<dave.jiang@intel.com>, 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>
Subject: Re: [PATCH 1/4] cxl/event: Add missing include files
Message-ID: <65e24e516d330_314b33294f@iweiny-mobl.notmuch>
References: <20240228-cxl-cper3-v1-0-6aa3f1343c6c@intel.com>
 <20240228-cxl-cper3-v1-1-6aa3f1343c6c@intel.com>
 <65e238631fd64_3651e294c0@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <65e238631fd64_3651e294c0@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: =?utf-8?B?cHBnNms2Rm1MYUw0S21nbXZRUG15cWpOTXFEYTBlMkZnbmgycUIzZUhHYnQ2?=
 =?utf-8?B?QlFDRmdKZlhuMDkzWFBTRytEVE9mZlB0UXUvM3RiRFViR1hFTS9qamJIYkU1?=
 =?utf-8?B?RUMxVzN4d3NYUk9iYUhXdlBkRjI1OFVUY3crMFlwTDBya1NVeHFqZkF2azdw?=
 =?utf-8?B?TjJJWmMyMWUxUHhjcDVweFhnSUhoKytNUTM5c3F3RDNUbHBaNzRwdHQ5RlRP?=
 =?utf-8?B?bER0eU15SFBvK3lRSG1WOVdqVjJzUENBMmVPMWhUUkdpYzBDalBaUmZoWlE5?=
 =?utf-8?B?M0RrWVNxWUZqNnRicXRiVDZBK0VUOVVIdTFPUE9EMFFPbkhLRy9ZQlhmQ1Nu?=
 =?utf-8?B?azVpaDFpWXpNeUdUdXVnc2RvUnQxNTdVSlVuT3JLODNXU1pCb0I3aXFpdXVx?=
 =?utf-8?B?cG5NMTRZbStQUGc3cjgydGphWlRYdTRzeGJhRm5QYWVrZlcxd2VxcTdoNm1O?=
 =?utf-8?B?K1BlSUlhcDkxSTBWZ2tkY2tacmdKWkd1TS9zL2VJeHg3U1duYU42MDZ4UnFZ?=
 =?utf-8?B?UzlkWm5oSmNtMUNxS1lPSWY4UENtSytaK2JGbFVZVnNESmplcDQ3R3N2NTd4?=
 =?utf-8?B?UWVMV0VUaDFvcjFwYU96S2ZpakNOZ2E4eVdFYzlRVjI4eGtKczdkYkdIc1Bz?=
 =?utf-8?B?eFZhSEFQZWlBeUF2ajRseXpybVI4N3ZkSlhXY3N1NFI2cHkwNXl6VzBxbTF6?=
 =?utf-8?B?VjNGWDcwYmYycTlTWElhTEV3Yjk1TGJzbVlwbXpyTjhnUi9Rd3RmekJVZm1i?=
 =?utf-8?B?QnJReUFLNG1Ld3RDUlczaXpEdEVNRHBNZksxeVExZ3FNcjM4K2Jud0tEdlRz?=
 =?utf-8?B?M0lRNHptVWxaNHc1M3lMbWU1OCtud2pXcHBTWjlPY2RHdldMWi8xbFRyRXdS?=
 =?utf-8?B?cmtBSVJETkZDa3hTNnB1N01OdzQxdGdqbS9sN016dWtNVEdsbi9rcys3L29m?=
 =?utf-8?B?U3JIZUdlUnREQkV4ck1Sdi85aVV6dGhPQ0F1OUJJck8ybVRtNDlPU29EbWxW?=
 =?utf-8?B?NDVZMlNXOEtIODQ1OXVwaHlWUzVtRmlPOEsvT2JCVGpvUGNYcU4xbHluRFVO?=
 =?utf-8?B?SGsrYUJpNmt0Rkh3aUJuUFdVbTZTUkl4cFc0eGFjTkdXSnd6RFBLUjVNdlNX?=
 =?utf-8?B?QkYzWGhsL2pXRVFSaUhzV3Bva2s5MWVoL3ROOG9ZWml2OXNkLzROUDZFckJC?=
 =?utf-8?B?YkNqczU5SW5UbE5aMHhYanpIZG9lQ0lDeXVudDVneXZPSVllcXBzS0xYaU9M?=
 =?utf-8?B?OHBpMHE3a3FrakEzdHZ5a0Z2dWtXVGsvQy85azlvdVBTd0FEQ1pSd1JaUExY?=
 =?utf-8?B?SVJGVEE2MnRNWjJoeUJTUzdqTTRLV0IrUTFjMW9GMWFZQnNPemxqM3J3ZHRW?=
 =?utf-8?B?TUt3ODkycytPdGZVUldEL05NR0p4SW5KTEM2aUd6OTh6WDFZbU8ydUxEeFJp?=
 =?utf-8?B?M3NPMHYyYlVXWGdlS1FiN3B2QjBXTVdTZ3hNZXF6K2phYnhIeXdQQ3ZkVzly?=
 =?utf-8?B?eFNWRi9wMjRhK3VWQ2J3Q0l3YjJTOFV5VWZxT2poditqMUQwNVFaQVBtdnJT?=
 =?utf-8?B?TDc2T1NKcXdwMG1OMFhmTWtvNmtQVDNSbjBpdThZSlRCN2xBU0dCRDZueGxO?=
 =?utf-8?B?Z3ZxS3NTTW5nRmo5ZlRYK0o4RTNJeGg1SWk4QTJuVmpzVVlRMFVpZVpPMVhG?=
 =?utf-8?B?VW9PbXlWYTh3UWxyQ1FhUTQ1STJaTFFZMURPUkl5MTFQdWVtQkFWZEFJdTBl?=
 =?utf-8?B?RWRmN01wdHBvV1lwMDFNdllMcXJ4NlUrY0RabHBsTUJZNVF6MkxEUmt6RmpV?=
 =?utf-8?B?NVNwRWREQTRiTE9NOGFHei8yUGtpOU5qK1RlaVIxeGd5NUpPUkVYc1dlc0F1?=
 =?utf-8?B?MXBmSUJZNVd0OWgxMy9BRG0yMEVvQW1iMzhTZCtOcCtaNXN2WGVicGF3U3A4?=
 =?utf-8?B?eFI1MXB6S2p5YnR4QkVxOTkwVTlUZTJ2Z256ZWlxZHE5RXpMUGNjVHJvMDZD?=
 =?utf-8?B?dXNKZHBpa1hEQ0RJYkRNU2JpL0RwUHc3dUZ3eUM4ZnY3M1RENWlMb1NsLzA2?=
 =?utf-8?B?S1ZYQ3NKUlZtZVU1ZDIrT3p5UW82eFlDQ3JuamZZcGEwUDljVkVXakk4dVZG?=
 =?utf-8?Q?1xjEWhfsnUYyIvBx/8FUlkqUB?=
X-MS-Exchange-CrossTenant-Network-Message-Id: f95ed3f1-1283-4f98-6d44-08dc3a3a055a
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Mar 2024 21:53:25.3191
 (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: /qRMEIEw1PeBUnfcmhatIfT7DH4j/JdH9SRdeZhP7wJmggZmunCOoYZ8DQXQCQ5G2s9nKbAfHtzvM0kNqzHfmw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR11MB6781
X-OriginatorOrg: intel.com

Dan Williams wrote:
> Ira Weiny wrote:
> > Additional event testing using the cxl-event.h header revealed that it
> > did not include the correct headers for the types used.  Compile errors
> > such as:
> > 
> > 	include/linux/cxl-event.h|11 col 9| error: unknown type name ‘u8’
> > 
> > ... were seen.
> 
> Were seen where? Should this have the trio of Reported-by: Closes: and
> Fixes tags?

As I said after this in the commit message:

	"Omit the fixes tag because this does not cause any issues until
	the header is used again in other code."

I'll clarify it was seen when I used cxl-event.h in the testing code and
this happened.

Ira