RE: [PATCH 03/20] cxl: Move CXL spec defines to public header

[Manish Honap]

> -----Original Message-----
> From: Jonathan Cameron <jonathan.cameron@xxxxxxxxxx>
> Sent: 18 March 2026 20:27
> To: Dave Jiang <dave.jiang@xxxxxxxxx>
> Cc: Manish Honap <mhonap@xxxxxxxxxx>; Aniket Agashe <aniketa@xxxxxxxxxx>;
> Ankit Agrawal <ankita@xxxxxxxxxx>; Alex Williamson
> <alwilliamson@xxxxxxxxxx>; Vikram Sethi <vsethi@xxxxxxxxxx>; Jason
> Gunthorpe <jgg@xxxxxxxxxx>; Matt Ochs <mochs@xxxxxxxxxx>; Shameer Kolothum
> Thodi <skolothumtho@xxxxxxxxxx>; alejandro.lucero-palau@xxxxxxx;
> dave@xxxxxxxxxxxx; alison.schofield@xxxxxxxxx; vishal.l.verma@xxxxxxxxx;
> ira.weiny@xxxxxxxxx; dan.j.williams@xxxxxxxxx; jgg@xxxxxxxx; Yishai Hadas
> <yishaih@xxxxxxxxxx>; kevin.tian@xxxxxxxxx; Neo Jia <cjia@xxxxxxxxxx>;
> Tarun Gupta (SW-GPU) <targupta@xxxxxxxxxx>; Zhi Wang <zhiw@xxxxxxxxxx>;
> Krishnakant Jaju <kjaju@xxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; linux-
> cxl@xxxxxxxxxxxxxxx; kvm@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 03/20] cxl: Move CXL spec defines to public header
> 
> External email: Use caution opening links or attachments
> 
> 
> On Fri, 13 Mar 2026 09:56:10 -0700
> Dave Jiang <dave.jiang@xxxxxxxxx> wrote:
> 
> > On 3/13/26 5:18 AM, Jonathan Cameron wrote:
> > > On Thu, 12 Mar 2026 02:04:23 +0530
> > > mhonap@xxxxxxxxxx wrote:
> > >
> > >> From: Manish Honap <mhonap@xxxxxxxxxx>
> > >>
> > >> HDM decoder capability structure and component reg block size needs
> > >> to be used by VFIO subsystem. Move the macros from private CXL
> > >> header to public one.
> > >>
> > >> Signed-off-by: Manish Honap <mhonap@xxxxxxxxxx>
> > >
> > > Not really related to this patch, but...
> > >
> > > Maybe this is the time to think about doing a PCI like
> > > uapi/linux/cxl_regs.h header?
> >
> > Sounds reasonable. Maybe makes sense to be in uapi/cxl/? There's already
> a features.h header in there.
> 
> Sure. I'd forgotten about that directory.
> Jonathan

Okay, I have moved the CXL r4.0 spec defined register bits and flags in
include/uapi/cxl/cxl_regs.h but for the bits/flags which are required
for emulation handling, I have kept in vfio_cxl_priv.h.