Re: [PATCH v4 10/14] cxl: Enable AMD Zen5 address translation using ACPI PRMT

From: Robert Richter

Date: Thu Nov 13 2025 - 06:24:21 EST


On 11.11.25 15:30:49, Jonathan Cameron wrote:
> On Mon, 3 Nov 2025 19:47:51 +0100
> Robert Richter <rrichter@xxxxxxx> wrote:

> > +void cxl_setup_prm_address_translation(struct cxl_root *cxl_root)
> > +{
> > + struct device *host = cxl_root->port.uport_dev;
> > + u64 spa;
> > + struct prm_cxl_dpa_spa_data data = { .spa = &spa, };
>
> Trailing comma is a tiny bit pointless as any change will have to replace
> the line anyway. So I'd drop it. (trivial so ignore if you like ;)

I have chosen the MIT comma here, but dropped it in v5. Are you
Cambridge or Oxford? ;)

https://xkcd.com/2995/

Thanks for your reviews.

-Robert