Re: [PATCH v4 2/3] cxl: Move mailbox related bits to the same context

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

On Mon, 14 Oct 2024 11:36:31 -0700
Alison Schofield <alison.schofield@intel.com> wrote:

> On Mon, Oct 14, 2024 at 03:51:54PM +0100, Jonathan Cameron wrote:
> > On Thu, 5 Sep 2024 15:35:46 -0700
> > Dave Jiang <dave.jiang@intel.com> wrote:
> >   
> > > Create a new 'struct cxl_mailbox' and move all mailbox related bits to
> > > it. This allows isolation of all CXL mailbox data in order to export
> > > some of the calls to external kernel callers and avoid exporting of CXL
> > > driver specific bits such has device states. The allocation of
> > > 'struct cxl_mailbox' is also split out with cxl_mailbox_create() so the
> > > mailbox can be created independently.
> > > 
> > > Reviewed-by: Fan Ni <fan.ni@samsung.com>
> > > Reviewed-by: Alejandro Lucero <alucerop@amd.com>
> > > Reviewed-by: Alison Schofield <alison.schofield@intel.com>
> > > Link: https://patch.msgid.link/20240724185649.2574627-2-dave.jiang@intel.com
> > > Signed-off-by: Dave Jiang <dave.jiang@intel.com>  
> > 
> > LGTM.
> > Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > 
> > Note this is currently being carried in Shiju's RAS features set
> > as well.  What is the merge plan?  My preference is an immutable branch
> > that Borislav could pull into the EDAC tree and on which we can base
> > other stuff in CXL 
> > That is a branch based on 6.12-rc1 directly with an appropriate tag
> > that Shiju's patch set can refer to.  
> 
> 
> This set was merged in 6.12.
Ah. Shiju, seems you have an old base.  Fix that for next version of the
RAS features

> 
> > 
> > Thanks,
> > 
> > Jonathan  
> 
>