[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Z8nK5AdjeaNYjSbl@rric.localdomain>
Date: Thu, 6 Mar 2025 17:18:44 +0100
From: Robert Richter <rrichter@....com>
To: Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc: Alison Schofield <alison.schofield@...el.com>,
Vishal Verma <vishal.l.verma@...el.com>,
Ira Weiny <ira.weiny@...el.com>,
Dan Williams <dan.j.williams@...el.com>,
Dave Jiang <dave.jiang@...el.com>,
Davidlohr Bueso <dave@...olabs.net>, linux-cxl@...r.kernel.org,
linux-kernel@...r.kernel.org, Gregory Price <gourry@...rry.net>,
"Fabio M. De Francesco" <fabio.m.de.francesco@...ux.intel.com>,
Terry Bowman <terry.bowman@....com>
Subject: Re: [PATCH v3 14/18] cxl/region: Unfold cxl_find_root_decoder() into
cxl_endpoint_decoder_initialize()
On 14.02.25 16:33:46, Jonathan Cameron wrote:
> On Tue, 11 Feb 2025 10:53:44 +0100
> Robert Richter <rrichter@....com> wrote:
>
> > To determine other endpoint parameters such as interleaving parameters
> > during endpoint initialization, the iterator function in
> > cxl_find_root_decoder() can be used. Unfold this function into
> > cxl_endpoint_decoder_initialize() and make the iterator available
> > there.
> I'm not following this description at all. Perhaps this needs
> to wait until you have code that is reusing this to find those
> interleave parameters and similar.
As suggested by Dave too, I will move that along with some other
patches out of this rework and cleanup series. The motivation of the
changes is better seen then and are part then where they are actually
needed.
-Robert
Powered by blists - more mailing lists