Messages in this thread | | | Date | Mon, 16 Oct 2023 16:09:58 +0200 | From | Robert Richter <> | Subject | Re: [PATCH v11 07/20] cxl/hdm: Use stored Component Register mappings to map HDM decoder capability |
| |
On 02.10.23 15:43:51, Jonathan Cameron wrote: > On Wed, 27 Sep 2023 17:43:26 +0200 > Robert Richter <rrichter@amd.com> wrote: > > > Now, that the Component Register mappings are stored, use them to > > enable and map the HDM decoder capabilities. The Component Registers > > do not need to be probed again for this, remove probing code. > > > > The HDM capability applies to Endpoints, USPs and VH Host Bridges. The > > Endpoint's component register mappings are located in the cxlds and > > else in the port's structure. Duplicate the cxlds->reg_map in > > port->reg_map for endpoint ports. > > > > Signed-off-by: Terry Bowman <terry.bowman@amd.com> > > Signed-off-by: Robert Richter <rrichter@amd.com> > > Reviewed-by: Dave Jiang <dave.jiang@intel.com> > > [rework to drop cxl_port_get_comp_map()] > > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > A few comments inline. > > Also, Dan's SoB doesn't make sense if you are the Author and he's not > the one sending the email. The fun of patches bounced back and forwards > is sometimes you have to tweak this stuff on each posting... :(
Yeah, there was some back and forth with the code going on: Terry changed a little, Dan was taking it and reposting, Terry was also submitting the previous series, Dave reviewed it. So I decided here to just swap Terry and my SOBs as I was posting the series now and left the remaining to Dan the way as it will look like when he will apply the patch, which means updating the SOB chain again. This keeps the comment for the change he made to the patch.
-Robert
> > > --- > > drivers/cxl/core/hdm.c | 48 ++++++++++++++++------------------------- > > drivers/cxl/core/port.c | 29 +++++++++++++++++++------ > > drivers/cxl/mem.c | 5 ++--- > > 3 files changed, 43 insertions(+), 39 deletions(-)
| |