On Fri, 3 Sep 2021 19:20:50 -0700
Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
> From: Li Qiang (Johnny Li) <johnny.li@xxxxxxxxxxxxxxxx>
>
> Indicator string for mbox and memdev register set to status
> incorrectly in error message.
>
> Cc: <stable@xxxxxxxxxxxxxxx>
> Fixes: 30af97296f48 ("cxl/pci: Map registers based on capabilities")
> Signed-off-by: Li Qiang (Johnny Li) <johnny.li@xxxxxxxxxxxxxxxx>
> Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx>
fwiw obviously correct.
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> ---
> drivers/cxl/pci.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
> index 37903259ee79..8e45aa07d662 100644
> --- a/drivers/cxl/pci.c
> +++ b/drivers/cxl/pci.c
> @@ -1041,8 +1041,8 @@ static int cxl_probe_regs(struct cxl_mem *cxlm, void __iomem *base,
> !dev_map->memdev.valid) {
> dev_err(dev, "registers not found: %s%s%s\n",
> !dev_map->status.valid ? "status " : "",
> - !dev_map->mbox.valid ? "status " : "",
> - !dev_map->memdev.valid ? "status " : "");
> + !dev_map->mbox.valid ? "mbox " : "",
> + !dev_map->memdev.valid ? "memdev " : "");
> return -ENXIO;
> }
>
>
